Skip to content

[Gotprgmer] week 1 #624

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Dec 10, 2024
Merged

[Gotprgmer] week 1 #624

merged 7 commits into from
Dec 10, 2024

Conversation

GotPrgmer
Copy link
Contributor

@GotPrgmer GotPrgmer commented Dec 3, 2024

답안 제출 문제

체크 리스트

  • PR을 프로젝트에 추가하고 Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 Status를 In Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@GotPrgmer GotPrgmer requested a review from a team as a code owner December 3, 2024 09:02
@github-actions github-actions bot added the java label Dec 3, 2024
@TonyKim9401
Copy link
Contributor

TonyKim9401 commented Dec 3, 2024

첫 PR 감사드립니다!
PR 등록시 몇가지 확인 사항에 대해 전달해드리고자 합니다.
먼저 Labels에 사용하신 언어인 java설정 감사드립니다.

바로 아래 Projects를 눌러 리트코드 스터디 3기 로 설정 후
image

문제 풀이 현황에 따라
풀고 계신 도중이라면 Solving
문제를 풀고 리뷰 요청시에는 In Review
리뷰가 끝나 Merge 직전이라면 Completed 로 설정 부탁드립니다.

Week의 경우 현재 주차를 선택해주시면 됩니다.
15주 동안 힘내시길 바랍니다.
감사합니다!
image

@GotPrgmer
Copy link
Contributor Author

아! status 설정하라는 말씀봤는데 어디서 설정하는 지 몰랐습니다! ㅎㅎ
디테일한 말씀 감사드립니다!

@DaleSeo
Copy link
Member

DaleSeo commented Dec 3, 2024

@GotPrgmer 3기 첫 번째 PR의 주인공이 되신 것을 축하드립니다! 🥇🎊

@GotPrgmer
Copy link
Contributor Author

GotPrgmer commented Dec 3, 2024

감사합니다!!
@DaleSeo

// 공간복잡도 : O(n)

class Solution_Gotprgmer {
static StringBuilder sb;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

얘를 굳이 함수 밖에서 static으로 선언하신 이유가 있을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

흠... 크게 없습니다. 그런 기준이 있어야 할거같은데 자바로 알고 푼지 2주밖에 안돼서 정립이 안되어있는거 같아요.
혹시 해당 코드에서 단점은 어떤게 있을까요?
또한 추천하시는 전역 변수 선언에 대한 기준이 있을까요?

Copy link
Member

@DaleSeo DaleSeo Dec 3, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

자바의 전역 변수 선언의 장단점에 대해서는 검색하시면 많이 나올 것 같아서, 저는 코딩 테스트 관점에서만 피드백을 드리겠습니다. 온라인 시험이면 크게 상관이 없지만, On-site 기술 면접에서는 이러한 질문을 받으시는 상황 자체가 지원자에게 불리하게 작용할 수 있습니다. 면접관과 프로그래밍 언어의 문법이나 기능에 대해서 질의응답을 하게 되면, 그만큼 알고리즘이나 자료구조에 대해서 대화를 나눌 수 있는 시간을 잃게 됩니다. 게다가 지원자가 명확한 답변을 주지 못한다면 아무래도 면접관은 안 좋은 인상을 받을 수 밖에 없습니다. 면접관이 애초에 이런 의구심을 갖지 않도록 전략적으로 코드를 짜는 연습을 해보시면 좋을 것 같습니다.

Copy link
Contributor Author

@GotPrgmer GotPrgmer Dec 4, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

전역 변수에 대해서 의식적으로 작성해보겠습니다.
왜 이렇게 썼는지에 대해 명확한 코드를 짜야겠네용. 감사합니다!

Copy link
Member

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

@GotPrgmer GotPrgmer merged commit 73a1c9d into DaleStudy:main Dec 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

3 participants