[riveroverflows] WEEK 03 Solutions

[riveroverflows]

답안 제출 문제

• Valid Palindrome #220
• Number of 1 Bits #232
• Combination Sum #254
• Decode Ways #268
• Maximum Subarray #275

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

[jylee2033]

Valid Palindrome 두번째 풀이에서 한 줄로 깔끔하게 처리하신 부분이 인상적이었어요! Number of 1 Bits 문제에서는 나누기/모듈러 대신 Brian Kernighan의 비트 트릭을 사용하신 점이 신기했고 덕분에 효율적인 풀이 배울 수 있었습니다. 수고하셨어요 😊

[jylee2033]

@DaleStudy 리뷰해줘

[dalestudy]

전체적으로 잘 구성된 코드와 상세한 설명이 인상적입니다! 특히 백트래킹과 비트 조작, 투포인터 방법 등 다양한 기법을 깔끔하게 구현하셨네요. 몇 가지 개선 제안을 드리자면:

1. 시간/공간 복잡도 명시
   각 솔루션에 TC, SC를 명확히 표기하셨지만, combinationSum의 경우 TC: O(N^(T/M))과 SC: O((T/M) * N^(T/M))은 최악의 경우 매우 크고, 이해를 돕기 위해 좀 더 구체적으로 설명하거나 참고자료 링크를 추가하는 것도 좋겠습니다.
   hammingWeight는 TC: O(log n)과 SC: O(1)이 적절하며, 정확하게 표기하셨습니다.
   isPalindrome의 경우 두 방법 모두 TC: O(n)과 SC는 방법에 따라 다르니 명확히 표기하셨습니다.

2. 접근법 제안

   • combinationSum: 백트래킹은 적절하며, 깔끔하게 구현하셨습니다. 혹시 더 최적화가 필요하다면, 후보를 정렬 후 가지치기를 통해 불필요한 경로를 더 일찍 끊는 방법도 고려할 수 있습니다.
   • hammingWeight: 이미 Brian Kernighan 기법을 잘 활용하셨고, 최적입니다.
   • isPalindrome: 투포인터와 리스트 역순 비교 두 방법 모두 적절하며, 각각의 장단점을 고려하여 선택할 수 있습니다.

3. 가독성 및 스타일

   • docstring에 TC, SC를 명확히 표기하셨고, 알고리즘 설명도 명료합니다.
   • 변수명 start, path, path_sum은 직관적이며, 일관성 있게 사용하셨습니다.
   • 다만, for i in range(start, len(candidates)): 부분에서 candidate를 변수로 두기보다 바로 path.append(candidates[i])와 같이 하셔도 깔끔할 수 있습니다.

4. 추가 고려사항

   • 예를 들어 combinationSum에서 candidates를 정렬 후, 중복 제거 또는 가지치기하여 속도를 높일 수 있습니다.
   • isPalindrome2에서 s = [c for c in s.lower() if c.isalnum()]는 리스트 생성이 비용이 크니, 필요시 filter 또는 generator를 활용하는 것도 고려 가능합니다.

전체적으로 뛰어난 솔루션들이며, 작은 최적화와 가독성 향상 가능성만 제안드립니다. 앞으로도 더 좋은 문제 해결 기대할게요!