[socow] WEEK 13 solutions

[sojae]

답안 제출 문제

• Lowest Common Ancestor of a Binary Search Tree #249

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

[TonyKim9401]

@DaleStudy 리뷰해줘

[dalestudy]

좋은 점이 많습니다! 코드는 깔끔하고, BST의 특성을 잘 활용하여 시간 복잡도를 O(log n)으로 기대할 수 있게 구현하셨네요. 또한, 반복문 방식으로 구현하여 공간 복잡도도 O(1)로 유지한 점도 우수합니다.

피드백 드리자면:

• 시간/공간 복잡도 분석 명시: 현재는 코드 내 주석에 TC, SC를 넣어주셨는데, 조금 더 명확하게 "균형 BST에서 평균 O(log n), 최악 O(n)"임을 명시하는 것도 좋겠습니다. 또한, 공간 복잡도는 반복문이므로 O(1)임을 확실히 전달하는게 좋아요.
• 가독성과 스타일: 변수 이름 cur는 직관적이지만, current 또는 node로 좀 더 명확하게 표기하면 좋겠습니다.
• 접근법 제안: 균형이 아닌 일반 BST라면 최악의 경우 O(n)이 될 수 있음을 주석으로 명시하는 것도 학습에 도움이 될 것 같습니다.

전체적으로 매우 깔끔하고 이해하기 쉬운 해결법입니다! 이 방식은 효율적이고 직관적이기에 추천드릴 만합니다. 앞으로도 이렇게 명확하고 간결한 코드 기대하겠습니다. 수고하셨어요!

[TonyKim9401]

13주차 문제 풀이 고생하셨습니다.
재귀 방식으로 이진 트리 순회 문제 잘 풀어내주셨네요!
이제 2주만 더 진행하시면 졸업이시네요. 스터디 후기 언제든지 펼하실때 남겨주시고 14주차도 파이팅입니다!