Skip to content
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

[HaJunYoo][WEEK 01] 리트코드 문제 풀이 #303

Merged
merged 7 commits into from
Aug 20, 2024
Merged

Conversation

@YeonguChoe
Copy link
Contributor

3줄로 google coding style에 맞춰서 잘 푸셨네요.

@obzva obzva self-requested a review August 12, 2024 01:20
@YeonguChoe YeonguChoe marked this pull request as ready for review August 12, 2024 03:59
@yolophg
Copy link
Contributor

yolophg commented Aug 12, 2024

안녕하세요, 코치 Helena 입니다.
아직 풀이가 다 완료되신 것 같지 않은데, draft로 변경해주시면 감사드리겠습니다 :)
자세한 사항은 위키를 확인해주세요!
=> https://github.com/DaleStudy/leetcode-study/wiki/%EB%8B%B5%EC%95%88-%EC%A0%9C%EC%B6%9C-%EA%B0%80%EC%9D%B4%EB%93%9C

@HaJunYoo HaJunYoo marked this pull request as draft August 12, 2024 14:12
@yolophg
Copy link
Contributor

yolophg commented Aug 12, 2024

추가로, 아래 사진과 같이 Projects 에서 project와 iteration 설정도 같이 부탁드립니다!
이번엔 제가 해두었는데, 원할한 스터디 그룹 진도 관리와 통계를 위해서 중요한 부분이니 다음 이터레이션부터는 해주시면 감사드리겠습니다 :)
Screenshot 2024-08-12 at 11 48 15
Screenshot 2024-08-12 at 11 41 08

@HaJunYoo
Copy link
Contributor Author

확인했습니다!

@HaJunYoo
Copy link
Contributor Author

@yolophg 제가 OT 때, 한국 시간 저녁 10시로 잘못 알아서 참여를 못했습니다 ..!
https://github.com/orgs/DaleStudy/teams/leetcode

팀 페이지로 들어가면 아래와 같이 뜨는데요..!
혹시 inivitation은 어디서 확인할 수 있을까요?

스크린샷 2024-08-16 오전 12 29 34

@yolophg
Copy link
Contributor

yolophg commented Aug 16, 2024

@yolophg 제가 OT 때, 한국 시간 저녁 10시로 잘못 알아서 참여를 못했습니다 ..! https://github.com/orgs/DaleStudy/teams/leetcode

팀 페이지로 들어가면 아래와 같이 뜨는데요..! 혹시 inivitation은 어디서 확인할 수 있을까요?

스크린샷 2024-08-16 오전 12 29 34

아, 그렇군요. 아래 코멘트와 공지에서 확인하실 수 있듯이, 온보딩을 참석하신 분들을 최종 멤버로 확정하여 스터디 2기를 진행한다고 공지를 드렸었답니다! 그래서, 참석을 못하셔서 아마 최종 멤버로 확정되지 않아, invitation 대상에서 제외되신 듯 한데 이와 관련해서는 일단 달레님과도 논의를 해보고 공유드릴 수 있을 것 같아요.
#321 (comment)
#292

@HaJunYoo
Copy link
Contributor Author

@yolophg
해당 건은 제가 깃헙 공지를 제대로 팔로우업 하지 못한 불찰이라 🥲
논의 후 참여가 어려울 경우, 3기에 재도전해보겠습니다!

@DaleSeo
Copy link
Contributor

DaleSeo commented Aug 16, 2024

@HaJunYoo 님, 제가 디스코드로 DM 드렸는데 좀 확인 해주시겠습니까?

@HaJunYoo
Copy link
Contributor Author

넵 답변드렸습니다! @DaleSeo

@taekwon-dev taekwon-dev added the py label Aug 17, 2024
@HaJunYoo HaJunYoo marked this pull request as ready for review August 17, 2024 06:39
Copy link
Contributor

@obzva obzva left a comment

Choose a reason for hiding this comment

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

Big-O notation에 대한 코멘트 추가해주시면 될 것 같습니다 ㅎㅎ

@@ -0,0 +1,9 @@
class Solution:
def hammingWeight(self, n: int) -> int:
n = int(n)
Copy link
Contributor

Choose a reason for hiding this comment

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

안녕하세요 HajunYoo님
n = int(n) 코드를 작성하신 이유가 궁금합니다 :)
제 생각엔 def hammingWeight(self, n: int) -> int:에서 parameter nint로 받고 있기 때문에 굳이 적지 않아도 될 코드 같아요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

이 부분은 저도 의아했는데요
n이 string으로 인식이 되는 에러가 발생했어서, int로 type casting을 해보았습니다
나중에 한번 다시 시도해보겠습니다 ㅎㅎ

class Solution:
def kthSmallest(self, root: Optional[TreeNode], k: int) -> int:

self.count = 0
Copy link
Contributor

Choose a reason for hiding this comment

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

질문) 그냥 count로 선언하는 것과 self.count로 선언하는 것에는 어떤 차이가 있나요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

리트코드에서 count를 그냥 선언하면 지역 변수라서 읽어오지를 못했습니다!
global로 선언해도 동일해서, 인스턴수 변수로 관리해보았습니다


class Solution:
def topKFrequent(self, nums: List[int], k: int) -> List[int]:
counter_dict = defaultdict(int)
Copy link
Contributor

Choose a reason for hiding this comment

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

collections의 Counter를 쓰는 건 어떨지 조심스레 제안드립니다 :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

리뷰 감사드립니다 ㅎㅎ
사실 Counter 써보려고 했는데, 코딩 테스트에서 중요한 순간에 혹시 해당 라이브러리가 안 떠오를 수도 있고, Counter를 쓰지 않고 라이브 코딩테스트를 요청 받을 수 있을 것 같아 위와 같이 구성해보았습니다.

@HaJunYoo
Copy link
Contributor Author

Big-O notation에 대한 코멘트 추가해주시면 될 것 같습니다 ㅎㅎ

추가했습니다 ㅎㅎ

@HaJunYoo HaJunYoo requested a review from obzva August 17, 2024 13:17
@DaleSeo
Copy link
Contributor

DaleSeo commented Aug 17, 2024

@obzva 님, 남겨주셨던 피드백이 원하셨던데로 반영이 되었다면 requested changes 상태 좀 풀어주실 수 있으실까요? PR 병합이 불가능하네요! 모임 전까지 가급적 모든 PR을 병합하고 2주차를 시작하려고 합니다.

@HaJunYoo
Copy link
Contributor Author

스크린샷 2024-08-19 오후 11 22 20
스크린샷 2024-08-19 오후 11 22 44

@DaleSeo 님, 아직 제가 초대가 되지 않아, merge와 iteration을 조작할 권한이 없는 것 같습니다..!
확인 부탁드립니다

@dev-jonghoonpark
Copy link
Contributor

@HaJunYoo 안녕하세요~

깃헙과 연결된 메일로 초대장이 갔을 것 같은데
혹시 메일로 초대장 온게 따로 없으실까요?

@DaleSeo
Copy link
Contributor

DaleSeo commented Aug 20, 2024

@HaJunYoo 제가 모임 끝나고 보내드린 Invitation이 아직 Pending 상태로 확인되시는데 한 번 메일을 확인해보시겠어요?

Shot 2024-08-19 at 20 00 06@2x

본 PR은 제가 대신 병합해드리도록 하겠습니다.

@DaleSeo DaleSeo merged commit 4eda421 into DaleStudy:main Aug 20, 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.

8 participants