[rivkode] WEEK 12 solutions

[rivkode]

답안 제출 문제

• Same Tree #228
• Remove Nth Node From End of List #248

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

[TonyKim9401]

@DaleStudy 리뷰해줘

[dalestudy]

전체적으로 깔끔한 구조와 재귀적 접근이 돋보입니다! 특히 isSameTree 메서드의 재귀 구현은 명확하고 효율적입니다. 다만, 몇 가지 개선점과 고려할 점을 말씀드리고 싶어요.

1. 시간/공간 복잡도 분석: 현재 코드에는 포함되어 있지 않으니, "TC: O(n), SC: O(h)" 정도로 명시해 주세요. 여기서 n은 노드 수, h는 트리의 높이입니다. 재귀 호출이 전체 노드를 방문하므로 시간 복잡도는 O(n), 호출 스택 깊이는 트리의 높이 h로 예상됩니다.

2. 접근법 제안: 현재 재귀 방식이 가장 직관적이고 효율적입니다. BFS/큐 방식도 가능하지만, 재귀가 더 간단하고 가독성도 좋습니다. 따라서 지금 방식이 적절하며, 특별히 더 나은 알고리즘은 필요 없다고 봅니다.

3. 코드 가독성 및 스타일:

   • checkLeaf 함수는 현재 사용되지 않으며, 의미상 이름이 조금 모호합니다(리프 체크처럼 보이지만, 내용이 단순히 참/거짓 반환). 이 함수는 삭제하거나, 목적에 맞게 제대로 활용하는 게 좋습니다.
   • 주석처리된 BFS 방식 코드는 삭제하거나 정리하는 것도 추천드립니다. 혼란을 줄 수 있고, 불필요한 코드가 남아있으면 유지보수에 방해가 됩니다.

4. 추가 제언:

   • 재귀 호출이 직관적이고 깔끔하지만, 트리의 깊이에 따라 스택 오버플로우 가능성이 있으니, 매우 깊은 트리인 경우 반복문 방식으로 변경하는 것도 고려할 수 있습니다.
   • 함수에 적절한 주석과 문서화(예: 시간/공간 복잡도) 추가로 가독성을 높일 수 있습니다.

이 프로젝트는 깔끔하게 잘 작성되어 있고, 재귀 방식이 적절하므로, 약간의 정리와 분석 추가만으로 충분히 발전시킬 수 있을 것 같습니다. 수고 많으셨고, 앞으로 더 발전하는 모습 기대합니다!

[TonyKim9401]

12주차 문제 풀이 고생하셨습니다.
Binary Tree 문제 잘 풀어주셨네요, 아래쪽 메소드는 풀이중에 남겨두신 걸까요?
이제 마지막달인 2월이네요. 남은 3주 동안 마무리 잘 하시고 유종의 미를 거두실 수 있길 바라겠습니다!