[chjung99] WEEK 03 solutions

[chjung99]

답안 제출 문제

• Contains Duplicate #217
• Invert Binary Tree #226
• Number of 1 Bits #232
• Reverse Linked List #223
• Valid Anagram #218

작성자 체크 리스트

• Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
• 문제를 모두 푸시면 프로젝트에서 Status를 In Review로 설정해주세요.
• 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

검토자 체크 리스트

Important

본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!

• 바로 이전에 올라온 PR에 본인을 코드 리뷰어로 추가해주세요.
• 본인이 검토해야하는 PR의 답안 코드에 피드백을 주세요.
• 토요일 전까지 PR을 병합할 수 있도록 승인해주세요.

[chjung99]

@DaleStudy 리뷰 해줘

[dalestudy]

좋은 작업입니다! 여러 문제에 대해 깔끔하고 효율적인 솔루션을 제시하셨네요. 특히, 시간과 공간 복잡도에 대한 주석도 명확하게 달아주셔서 이해하기 쉽습니다. 몇 가지 피드백 드리자면:

1. 복잡도 분석: 모든 답안에 TC와 SC를 명확히 표기하셨고, 분석도 적절합니다. 특히 hammingWeight에서 while 문이 O(log n)임을 잘 설명하셨습니다.

2. 가독성 및 스타일: 변수 이름과 구조가 명확하고, 메서드 이름도 직관적입니다. breadthFirstSearch라는 이름이 구체적이어서 이해하기 쉽고, 큐를 사용하는 BFS 방식도 적합합니다.

3. 더 나은 접근법 제안:

   • hammingWeight에서 n & 1을 활용하는 비트 연산으로 구현하면 더 효율적입니다. 예를 들어, while (n != 0) { cnt += n & 1; n >>>= 1; }로 변경 가능하며, 이 방법이 더 빠릅니다.
   • reverseList에서 재귀 대신 반복문을 사용하는 것도 추천합니다. 재귀는 호출 스택을 사용하므로 공간이 조금 더 들 수 있습니다.

4. 추가 제안:

   • containsDuplicate는 매우 깔끔하게 작성하셨고, 해시셋을 활용한 방법이 최적입니다.
   • isAnagram에서 배열 대신 HashMap도 고려할 수 있지만, 주어진 방식이 충분히 효율적입니다.

전반적으로 매우 깔끔하고, 효율적이며, 이해하기 쉽습니다. 앞으로도 이와 같이 명확한 주석과 구조를 유지한다면 훌륭한 코드가 될 겁니다. 계속 파이팅하세요!

[chjung99]

@DaleStudy 승인

[dalestudy]

승인되었습니다! 👍