[chjung99] WEEK 10 solutions

[chjung99]

답안 제출 문제

• Design Add And Search Words Data Structure #257
• House Robber #264
• House Robber II #265
• Implement Trie Prefix Tree #256
• Number of Islands #258

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

[chjung99]

@DaleStudy 리뷰해줘

[dalestudy]

전체적으로 잘 구성된 코드입니다. 특히, 각 문제별로 명확한 구조와 깔끔한 구현이 돋보입니다. 몇 가지 개선 포인트를 제안드리자면:

1. 시간/공간 복잡도 명시: 모든 코드에 시간과 공간 복잡도 분석이 빠져 있는데, 예를 들어 WordDictionary의 검색은 O(n) (단어 길이), houseRobber의 DP는 O(n)이고, NumberOfIslands는 BFS로 O(n*m)입니다. 이를 주석 또는 코드 내에 명시하면 가독성과 이해도가 높아집니다.

2. 중복 코드 최적화: rob 함수에서 Arrays.copyOfRange를 호출하는 대신, 인덱스를 활용하는 방법도 고려해보세요. 이는 불필요한 배열 복사를 방지하여 성능 향상에 도움이 될 수 있습니다.

3. 가독성 향상: 일부 변수명(node, n, m, dp)은 적절하지만, dfs와 같은 재귀 호출 함수명은 명확하게 유지되고 있어 좋습니다. 다만, Point 클래스에서 x, y 대신 row, col로 명명하면 더 직관적일 수 있습니다.

4. 일관된 스타일: 들여쓰기와 공백은 일관적이지만, 예를 들어 for 루프의 괄호 사이에 공백을 넣거나, if 조건문 뒤에 한 줄 띄우는 등 스타일 일관성을 유지하면 더 깔끔합니다.

5. 추가 최적화 가능성: houseRobber는 현재 DP 배열을 사용하지만, 공간을 O(1)로 줄일 수 있습니다. 예를 들어, 두 변수(prev, curr)만 유지하는 방법도 고려해보세요.

전체적으로 이해하기 쉽고 깔끔한 구현입니다. 이러한 작은 개선으로 성능과 가독성을 더 높일 수 있으니 참고하시기 바랍니다. 계속해서 좋은 코드 기대할게요!

[chjung99]

@DaleStudy 승인

[dalestudy]

승인되었습니다! 👍