[okyungjin] WEEK 06 Solutions

[okyungjin]

답안 제출 문제

• Valid Parentheses #222
• Container With Most Water #242
• Design Add And Search Words Data Structure #257
• Longest Increasing Subsequence #272

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

[dalestudy]

📊 okyungjin 님의 학습 현황

이번 주 제출 문제

문제	난이도	유형 분석
container-with-most-water	Medium	✅ 의도한 유형
design-add-and-search-words-data-structure	Medium	✅ 의도한 유형
valid-parentheses	Easy	⚠️ 유형 불일치

누적 학습 요약

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

문제 풀이 현황

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

---

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

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

요청	입력 토큰	출력 토큰	합계	비용
1	995	91	1,086	$0.000086
2	1,630	126	1,756	$0.000132
3	1,888	132	2,020	$0.000147
4	1,887	203	2,090	$0.000176
합계	6,400	552	6,952	$0.000541

[okyungjin]

@DaleStudy 풀이 유형 체크

[dalestudy]

다음은 PR에 포함된 풀이 유형 체크 요약입니다.

• container-with-most-water

  • 유형: 그리디(two-pointer) / 활
  • 특징: 좌우 포인터를 한칸씩 좁히며 최대 면적 갱신

• design-add-and-search-words-data-structure

  • 유형:
    • 풀이 1: 브루트포스 + 와일드카드 확장 (데카르트 곱) -> 브루트포스 탐색
    • 풀이 2: Trie(접두사 트리) + DFS를 통한 와일드카드 매칭
  • 특징: 두 가지 접근법 제시, ES에서 성능 차이 비교 가능

• longest-increasing-subsequence

  • 유형: 동적계획법(DP) 2차원적 접근
  • 특징: O(n^2) DP로 LIS 길이 계산

• valid-parentheses

  • 유형: 스택(stack) 기반 매칭
  • 특징: 여는 괄호와 닫는 괄호 매칭 검사, 시간/공간 복잡도 모두 O(n)

추가로, 각 파일은 주석에 시간 복잡도/공간 복잡도 및 풀이 아이디어를 기술하고 있습니다. 필요 시 특정 문제의 최적화 버전(O(n log n) LIS 등) 제안도 가능해요.

[okyungjin]

@DaleStudy 코드 리뷰 부탁해

[dalestudy]

좋은 PR 구성입니다. 주요 포인트와 개선 제안 몇 가지를 정리합니다.

• 전반

  • 각 문제별로 okyungjin.py 파일이 새로 추가되어 있습니다. 기능별로 잘 분리되어 있어 리뷰하기 좋습니다.
  • 주석에 접근법/복잡도 명시가 있어 의도 파악이 쉬워졌습니다.

• 컨테이너 물통 문제 (container-with-most-water)

  • 투 포인터 로직이 올바르게 작동합니다.
  • 단점/보완:
    • area 계산 후 즉시 업데이트하는 흐름이 깔끔합니다. 다만 height[left] == height[right]인 경우 어느 쪽으로 이동할지 결정하는 부분이 현재 else로 처리되는데, 같은 높이일 때 어느 쪽으로 이동하는지 명시적으로 주석으로 남기면 이해가 더 쉬울 것 같습니다.
    • 타입 힌트에 List가 사용되는데, 상단에 from typing import List가 필요합니다. 현재 파일에 임포트가 보이지 않습니다. 코드 실행에 필요한 import를 추가해주세요.
  • 시간 복잡도: O(n), 공간 O(1)로 최적화 잘 구현.

• Add-and-Search-Words 데이터 구조

  • 두 버전 모두 포함되어 있어 비교하기 좋습니다.
  • Trade-off:
    • Version 1: 브루트포스(wildcard 확장) 방식으로 작동, 대체로 느립니다. 간단한 구현으로는 이해가 쉽지만, 대규모 입력에서 시간이 많이 소요될 수 있습니다. 테스트케이스 상 충분히 동작하는지 확인 필요.
    • Version 2: Trie 구현으로 성능 향상 기대됩니다. Trie의 구현이 깔끔하고 와일드카드 검색도 DFS로 다루기 좋아 보입니다.
  • 주의/개선:
    • WordDictionary.addWord에서 매번 Trie를 재구성하는 구조가 아닌지 확인 필요. 현재 구현은 잘 보입니다.
    • 검색 시 '.'를 처리할 때 트리의 자식이 비었거나 해당 경로가 없으면 False를 반환하는 로직이 필요합니다. 현재는 재귀에서 그런 경우를 처리하고 있어 정상 작동 전망이나, char not in node.children인 경우 False를 즉시 반환하도록 구현 보완이 좋습니다. (현재 코드에서 해당 경로는 False로 흐름이 계속되나요? 확인 필요)
    • Type hints를 더 명확히(예: dict[str, Trie]) 하는 게 좋습니다. Python 버전에 따라 호환성 확인 필요.

• 최장 증가 부분 수열 (LIS)

  • O(n^2) 구현으로 설명과 같이 동작합니다.
  • TODO로 O(n log n) 개선 의도가 좋습니다. 가능하면 이후 구현 계획에 대해 간단한 코멘트 남겨두면 리뷰어가 방향 파악에 도움됩니다.

• 유효한 괄호

  • 의도대로 작동하는 것 같지만, 한 가지 의문:
    • "괄호 개수가 짝수인 경우는 항상 실패" 라는 코멘트가 있는데, 반드시 참은 아닙니다. 예를 들어 "()"은 짝수지만 유효합니다. 반례가 있습니다. 이 최적화는 잘못된 판단으로 보입니다.
    • 따라서 이 빠른 검사 대신 실제 스택 검사를 기반으로 작동하도록 유지하거나, 케이스를 잘못 판단하지 않도록 제거하는 것이 좋습니다.
  • 나머지 로직은 스택 기반으로 올바르게 작동합니다.

• 일반 제안

  • 각 파일의 시작에 필요한 import를 명시해 실행 가능하게 만드세요. 예: container-with-most-water에서 List import, typing 등.
  • 동일 문제의 서로 다른 풀이를 같이 두지 말고 PR 설명에 어떤 버전이 더 우수한지 명확히 적어두면 리뷰어가 판단하기 쉽습니다.
  • 테스트 케이스 추가: 각 풀이의 핵심 케이스를 간단한 단위 테스트로 첨부하면 품질 상승에 도움됩니다.

질문이나 특정 부분에 대한 자세한 리뷰가 필요하면 문제 번호를 알려 주세요.

[dahyeong-yun]

한 주간 고생 많으셨습니다. 상세히 적어주신 덕분에 공부가 많이 됐습니다. 개인적으로는 표현상 일관된 것을 선호하는 편이라서 max 값을 만드는 과정에서 if 분기와 max 함수를 번갈아 쓰시는 것이 눈에 띄긴 한 것 같아요. 특별한 이유가 있으셨는지가 살짝 궁금하네요. :)

[dahyeong-yun]

이 케이스가 성공하는 경우가 없을 것 같아요.

[okyungjin]

이 케이스가 성공하는 경우가 없을 것 같아요.

오 그렇네요 감사합니다. 다음 문제 풀때 수정해두겠습니다🙏

[dahyeong-yun]

미리 dictionary 를 만드니, 아래에서 깔끔하게 접근할 수 있네요 👍🏼

[dahyeong-yun]

사소하지만 right - left 가 width의 개념이고 아래서 반복되는 거 같아, 따로 빼도 괜찮을 것 같아요.

[okyungjin]

의견 감사합니다!

[dahyeong-yun]

저는 트라이 문제는 다 skip 했는데, 직관적으로 접근하신 부분에서 트라이의 필요성을 역체감 할 수 있어서 좋았습니다!

[okyungjin]

@dahyeong-yun 정성스러운 리뷰 감사드립니다!

값 2개 비교할 때는 if문 사용하고, 그 이상인 곳은 max를 사용했습니다. 리트코드에서 실행해보면 min , max 함수 사용했을 때 if문보다 오버헤드가 큰 것 같더라구요.

리뷰 주신 부분은 다음 문제 풀 때 적용하겠습니다 🙂 감사합니다!

한 주간 고생 많으셨습니다. 상세히 적어주신 덕분에 공부가 많이 됐습니다. 개인적으로는 표현상 일관된 것을 선호하는 편이라서 max 값을 만드는 과정에서 if 분기와 max 함수를 번갈아 쓰시는 것이 눈에 띄긴 한 것 같아요. 특별한 이유가 있으셨는지가 살짝 궁금하네요. :)