[ppxyn1] WEEK 13 solutions

[ppxyn1]

답안 제출 문제

• Meeting Rooms #231
• Lowest Common Ancestor of a Binary Search Tree #249
• Kth Smallest Element In a Bst #252
• Insert Interval #277
• Find Median From Data Stream #289

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

[ppxyn1]

@DaleStudy 리뷰해줘

[dalestudy]

좋은 작업입니다! 여러 문제에 대해 깔끔하게 구현하셨고, 시간복잡도도 적절히 분석하셨네요. 특히 이진 탐색 트리 문제에서 BST의 특성을 잘 이용하셨고, 삽입/찾기/공통 조상 문제 모두 효과적인 방식입니다.

몇 가지 피드백 드리자면:

1. 시간/공간 복잡도 명시: 각 솔루션에 TC: O(n) 또는 O(h) 등 시간복잡도와 SC: O(h) 등 공간복잡도 분석을 추가하는 것이 좋겠습니다. 특히, 트리 관련 문제는 최악의 경우 O(n)이 될 수 있으니 참고하세요.

2. 가독성 향상: insert 함수에서 변수명을 좀 더 직관적으로 바꾸거나, 주석을 더 명확히 하면 이해가 쉽습니다. 예를 들어, res 대신 result 또는 merged_intervals 등으로, 어떤 역할인지 명확히 하면 좋겠어요.

3. 코드 스타일: PEP8 가이드라인에 따라, 함수와 변수 이름 사이에 일관성을 유지하고, 주석의 문장 끝에 마침표를 넣는 것이 좋습니다. 예를 들어, # idea: -는 # Idea: 또는 제거하는 것도 고려하세요.

4. 추가 제안: kthSmallest 구현은 인오더 순회로 간단히 해결하셨네요. 더 최적화된 방법(예: 이진 탐색 트리의 크기를 저장하는 방식)을 고려할 수도 있겠지만, 현재 방식도 충분히 적합합니다.

전반적으로 깔끔하고 논리적이며, 학습에 도움이 되는 좋은 코드입니다! 앞으로도 계속 이렇게 문제 해결력과 가독성을 함께 높이시면 좋겠습니다.

[ppxyn1]

@DaleStudy 승인

[dalestudy]

승인되었습니다! 👍