[ymir0804] WEEK 08 Solutions

[ymir0804]

답안 제출 문제

• Combination Sum #254

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

[ymir0804]

@DaleStudy 리뷰해줘

[dalestudy]

이 코드 훌륭하게 작성되었어요! 깔끔한 재귀 백트래킹으로 문제를 해결했고, 정렬을 통해 가지치기도 적절히 활용하고 있습니다. 몇 가지 피드백 드리자면:

1. 시간/공간 복잡도 분석이 포함되지 않았어요. 이 문제는 일반적으로 TC: O(2^n) 또는 더 복잡할 수 있으며, SC는 재귀 호출 깊이와 결과 저장 용량에 따라 다릅니다. 이 정보를 추가하면 가독성과 이해를 돕습니다.

2. Arrays.sort()를 사용하여 가지치기를 잘 했어요. 더 나은 성능을 위해 이 부분은 적절하며, 현재 접근법이 최적에 가까워 보입니다.

3. 가독성 측면에서 변수명이나 구조는 명확하고 일관적입니다. 다만, remain 대신 target으로 명명하여 의미를 더 직관적으로 전달할 수도 있습니다.

4. 재귀 호출에서 start 인덱스를 그대로 넘겨서 중복 조합을 방지하는 점도 매우 좋습니다. 이 방법은 중복을 피하는 최선의 방법입니다.

5. 전체적으로 베스트 프랙티스에 부합하며, 깔끔하게 구현되어 있습니다. 작은 개선으로는, 주석을 추가하여 각 부분이 어떤 역할을 하는지 설명하면 이해도를 높일 수 있습니다.

계속해서 좋은 코드 유지하세요! 이 문제 해결 접근법은 매우 효율적이고 명확합니다.

[ymir0804]

@DaleStudy approve

[dalestudy]

승인되었습니다! 👍