[sangbeenmoon] WEEK 02 Solutions

[sangbeenmoon]

답안 제출 문제

• 3Sum #241
• Validate Binary Search Tree #251

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

[dalestudy]

📊 sangbeenmoon 님의 학습 현황

이번 주 제출 문제

문제	난이도	유형 분석
3sum	Medium	⚠️ 유형 불일치
validate-binary-search-tree	Medium	✅ 의도한 유형

누적 학습 요약

• 풀이한 문제: 3 / 75개
• 이번 주 유형 일치율: 50% (2문제 중 1문제 일치)

문제 풀이 현황

카테고리	진행도	완료
Heap	■■□□□□□	1 / 3 (Medium 1)
Graph	■□□□□□□	1 / 8 (Medium 1)
Dynamic Programming	■□□□□□□	1 / 11 (Medium 1)
Array	□□□□□□□	0 / 10 ← 아직 시작 안 함
Binary	□□□□□□□	0 / 5 ← 아직 시작 안 함
Interval	□□□□□□□	0 / 5 ← 아직 시작 안 함
Linked List	□□□□□□□	0 / 6 ← 아직 시작 안 함
Matrix	□□□□□□□	0 / 4 ← 아직 시작 안 함
String	□□□□□□□	0 / 10 ← 아직 시작 안 함
Tree	□□□□□□□	0 / 14 ← 아직 시작 안 함

---

🤖 이 댓글은 GitHub App을 통해 자동으로 작성되었습니다.

🔢 API 사용량 (gpt-5-nano)

요청	입력 토큰	출력 토큰	합계	비용
1	1,318	74	1,392	$0.000096
2	1,334	74	1,408	$0.000096
합계	2,652	148	2,800	$0.000192

[DaleSeo]

@sangbeenmoon #2704 (comment)

[parkhojeong]

수고하셨습니다! 몇 가지 개선 포인트 커멘트로 남겨보았어요 :)
달레님 커멘트(프로젝트 주차 설정) 확인해주시면 될 거 같습니다~

[parkhojeong]

None은 is 로 비교하는 것으로 권장되어서 참고하시면 좋을 거 같아요!
Comparisons to singletons like None should always be done with is or is not, never the equality operators.
https://peps.python.org/pep-0008/#programming-recommendations

그리고 변수명은 mm, MM이 어떤 값인지 알기가 어려워서 의미를 담아 보아도 좋을 거 같습니다 :)

Suggested change

	def isValid(mm, MM, cur: TreeNode) -> bool:
	if cur == None:
	return True
	if mm != None and cur.val <= mm:
	return False
	if MM != None and cur.val >= MM:
	return False
	return isValid(mm, cur.val, cur.left) and isValid(cur.val, MM, cur.right)
	return isValid(None, None ,root)
	def isValid(max_val, min_val, cur: TreeNode) -> bool:
	if cur is None:
	return True
	if max_val is not None and cur.val <= max_val:
	return False
	if min_val is not None and cur.val >= min_val:
	return False
	return isValid(max_val, cur.val, cur.left) and isValid(cur.val, min_val, cur.right)
	return isValid(None, None, root)

[sangbeenmoon]

오 이런 권장사항이 있는지는 몰랐네요 ㅎㅎ 공식문서까지 찾아서 제안해주셔서 감사합니다 👍

[parkhojeong]

해시맵을 사용해서 candidate를 담아서 중복을 체크하셨군요!
이미 정렬이 된 상태라서 nums[left] == nums[left+1] 인 케이스를 스킵해주면(right도 동일) candidate를 사용하지 않고 not in 로직을 안타기 때문에 더 최적화가 가능할 거 같아요!

[sangbeenmoon]

좀 더 깔끔하게 중복 체크할 수 있는 방법이 있었네요. 짚어주셔서 감사합니다

[parkhojeong]

성공한 경우 right -= 1 도 함께 해주면 좋을 거 같아요!

[parkhojeong]

정렬이 된 상태라 nums[i] 와 nums[i - 1] 이 같은 경우는 스킵해주고 nums[i] > 0 이 된 상태에서는 break 해주면 최적화가 가능할 거 같습니다 :)

ex. [ -2, -1, -1, -1, 0, 1, 2, 3] , -1이 한번인 경우만 실행하고 이후는 스킵, nums[i]가 0보다 커지면 양수 + 양수 + 양수 이기 때문에 종료

[alphaorderly]

깔끔하게 잘 해결하신것 같습니다! 고생하셨어요!

[dalestudy]

🏷️ 알고리즘 패턴 분석

• 패턴: Two Pointers, Hash Map / Hash Set
• 설명: 삼합(3Sum) 문제에서 정렬 후 양 끝 포인터를 이동하는 방식으로 탐색하므로 Two Pointers 패턴이 적용됩니다. 또한 중복 제거를 위해 Hash Map/Hash Set를 활용해 결과 중복을 방지하는 부분이 있습니다.

📊 시간/공간 복잡도 분석

	유저 분석	실제 분석	결과
Time	O(n^2)	O(n^2)	✅
Space	O(n)	O(1)	❌

피드백: 정렬 후 외부 루프와 내부 투 포인터로 복잡도를 달성했다. 다만 중복 제거를 visited로 추가했는데, 전체적으로 불필요한 해시맵 사용 가능성이 있다. 두 번째 구현처럼 중복 제거를 더 간결하게 처리하는 편이 좋다.

개선 제안: 고려해볼 만한 대안: 중복 제거를 리스트 스킵 방식으로 처리하고 해시맵 없이도 중복 없는 결과를 얻도록 구현을 단순화

[dalestudy]

🏷️ 알고리즘 패턴 분석

• 패턴: Depth-First Search, Binary Search, Hash Map / Hash Set
• 설명: 첫 번째 코드는 재귀로 BST의 각 노드를 탐색하며 자식 노드가 특정 범위 안에 있는지 확인한다. 두 가지 구현 모두 트리 구조의 각 노드를 방문하는 DFS 패턴이며, 자식 노드의 값 범위를 제한해 BST 특성을 검사하는 방식이다. 두 번째 구현은 min/max 범위 추적을 통한 Divide and Conquer식 DFS로 BST 여부를 판단한다.

📊 시간/공간 복잡도 분석

	복잡도
Time	O(n)
Space	O(h)

피드백: 두 가지 구현이 섞여 있는데, 첫 번째 구현은 전역 상태를 이용해 재귀적으로 범위를 검사한다. 두 번째는 재귀적으로 범위를 넘겨주는 일반적인 패턴으로 안정적이다.

개선 제안: 현재 구현이 적절해 보입니다.

💡 풀이에 시간/공간 복잡도를 주석으로 남겨보세요!