[yuseok89] WEEK 06 Solutions - #2786
Conversation
📊 yuseok89 님의 학습 현황이번 주 제출 문제
누적 학습 요약
문제 풀이 현황
🤖 이 댓글은 GitHub App을 통해 자동으로 작성되었습니다. 🔢 API 사용량 (gpt-5-nano)
|
There was a problem hiding this comment.
제 생각에 너무 좋은 코드인데요, Python의 타입 힌팅을 적극적으로 이제 활용해 보시면 코드의 가독성에 큰 긍정적 영향을 줄것이라고 생각해요!
특히 메서드를 추가 작성할때 input param들의 타입과 return type을 작성하면 개인적으로는 상당한 가독성의 향상이 있다고 생각합니다.
There was a problem hiding this comment.
아 물론 개인적 의견이니 꼭 하실 필요는 없습니다!!!
There was a problem hiding this comment.
문제 푸는 코드를 쓰면서는 신경 안쓴 부분이었는데,
리트코드에서 기본으로 주는 코드엔 이미 들어가있네요.
리뷰하는 입장에서도 있는 것이 더 좋을 것 같습니다 !
There was a problem hiding this comment.
와 이건 코드를 아트의 경지로 올리셨군요 ㄷㄷㄷ 대단하십니다.
배우고 갑니다!
| if c in pars: | ||
| stack.append(c) | ||
| else: | ||
| if len(stack) == 0 or pars.get(stack.pop(), '') != c: |
There was a problem hiding this comment.
if not stack or pars[stack.pop()] != c:
제 개인적인 생각으로는 이것으로 수정하면
not stack을 통해 스택이 비어있는 경우를 먼저 확인
단축 평가에 의해 stack이 비어 있으면 False를 리턴하기에 or의 뒤에 있는 부분은 실행시 stack에 값이 있음이 확실시 됩니다!
pars[stack.pop()] != c
이부분은 stack에 값이 있음이 확실시 되고, 어차피 닫는 기호는 스택에 들어가지 않기 때문에 스택 내부에는 여는 기호만 있는것이 확실시 되기 때문에 get과 기본값을 명시하는것이 크게 필요하지 않다는 생각이 들었습니다.
There was a problem hiding this comment.
더 좋네요.
막 쓰다보니, 어차피 여는 기호만 있는 것을 기본값으로 처리해버렸네요.
리뷰 감사합니다 🙏
alphaorderly
left a comment
There was a problem hiding this comment.
전반적으로 깔끔한 풀이 잘 봤습니다!
| # TC: O(N*M) | ||
| # SC: O(N*M) | ||
| class Solution: | ||
| def spiralOrder(self, matrix: List[List[int]]) -> List[int]: |
There was a problem hiding this comment.
진짜 깔끔하게 푸셨네요. 리뷰하러 왔다가 배워갑니다..!!!
| else: | ||
| r -= 1 | ||
|
|
||
| return ans; |
There was a problem hiding this comment.
파이썬이니까 세미콜론은 없어도 될 것 같습니다 😄
| @@ -0,0 +1,16 @@ | |||
| # TC: O(N) | |||
| # SC: O(N/2) | |||
There was a problem hiding this comment.
공간복잡도를 O(N/2) 라고 생각하신 이유가 있을까요?
There was a problem hiding this comment.
잘못 생각했었네요
O(N) 이 맞습니다
리뷰 감사합니다 🙏
There was a problem hiding this comment.
🏷️ 알고리즘 패턴 분석
- 패턴: Two Pointers, Greedy
- 설명: 좌우 포인터를 이용해 서로를 좁혀가며 용량을 계산하는 방식으로 최댓값을 탐색합니다. 각 단계에서 더 작은 벽에 맞춰 포인터를 이동해 최적 해를 빨리 찾는 그리디 특성을 가집니다.
📊 시간/공간 복잡도 분석
| 유저 분석 | 실제 분석 | 결과 | |
|---|---|---|---|
| Time | O(N) | O(n) | ✅ |
| Space | O(1) | O(1) | ✅ |
피드백: 왼쪽/오른쪽 포인터를 한 칸씩만 이동시키며 최댓값을 갱신한다. 루프는 배열 길이에 비례해 수행된다.
개선 제안: 현재 구현이 적절해 보입니다.
There was a problem hiding this comment.
🏷️ 알고리즘 패턴 분석
- 패턴: Trie, Backtracking, Dynamic Programming
- 설명: 코드는 트라이(Trie) 자료구조를 활용해 단어를 저장하고 검색합니다. 검색에서 '.' 와일드카드를 재귀적으로 대입해 가능한 경로를 탐색하므로 부분적으로 백트래킹 패턴이 보이고, 트라이 기반의 저장과 탐색으로 문제의 문자열 매칭을 구현합니다.
📊 시간/공간 복잡도 분석
| 복잡도 | |
|---|---|
| Time | O(len(word)) |
| Space | O(total_chars_inserted) |
피드백: 트라이 구조로 단어를 저장하고 와일드카드를 재귀로 확장 탐색한다.
개선 제안: 현재 구현이 적절해 보입니다.
💡 풀이에 시간/공간 복잡도를 주석으로 남겨보세요!
There was a problem hiding this comment.
🏷️ 알고리즘 패턴 분석
- 패턴: Binary Search, Dynamic Programming
- 설명: LIS 길이를 이분 탐색으로 최적화하는 대표적인 방법으로, arr를 유지하며 각 원소의 삽입 위치를 이분 탐색으로 찾는 패턴입니다. 이로써 O(N log N) 시간 복잡도가 가능해집니다.
📊 시간/공간 복잡도 분석
| 유저 분석 | 실제 분석 | 결과 | |
|---|---|---|---|
| Time | O(NlogN) | O(n log n) | ✅ |
| Space | O(N) | O(n) | ✅ |
피드백: 이분검색으로 LIS의 길이를 효율적으로 관리한다.
개선 제안: 현재 구현이 적절해 보입니다.
There was a problem hiding this comment.
🏷️ 알고리즘 패턴 분석
- 패턴: Two Pointers, Monotonic Stack, Hash Map / Hash Set, Greedy, Divide and Conquer, Binary Search, Dynamic Programming, Breadth First Search, Depth First Search, Backtracking, Union Find, Trie, Bit Manipulation, Heap / Priority Queue, Sliding Window, Fast & Slow Pointers, DFS, BFS
- 설명: 해당 코드는 행렬을 순회하여 나선형으로 방문하는 방식으로 인덱스를 좌우로 움직이며 모든 원소를 연속적으로 수집한다. 방향 전환과 경계 축소를 반복하는 구조는 Two Pointers의 응용으로 볼 수 있다.
📊 시간/공간 복잡도 분석
| 유저 분석 | 실제 분석 | 결과 | |
|---|---|---|---|
| Time | O(N*M) | O(n*m) | ✅ |
| Space | O(N*M) | O(1) | ❌ |
피드백: 끝까지 각 원소를 한 번씩 방문하는 방식으로 구현되어 있다.
개선 제안: 현재 구현이 적절해 보입니다.
There was a problem hiding this comment.
🏷️ 알고리즘 패턴 분석
- 패턴: Stack, Hash Map / Hash Set
- 설명: 괄호 유효성 검사에서 스택을 이용해 여는 괄호를 저장하고 닫는 괄호가 올바르게 매칭되는지 확인하므로 Stack 패턴에 해당하며, 매칭 규칙을 해시맵으로 빠르게 조회하므로 Hash Map / Hash Set 패턴도 함께 해당됩니다.
📊 시간/공간 복잡도 분석
| 유저 분석 | 실제 분석 | 결과 | |
|---|---|---|---|
| Time | O(N) | O(n) | ✅ |
| Space | O(N) | O(n) | ✅ |
피드백: 문자열을 한 번 순회하며 스택으로 매칭 여부를 판단한다.
개선 제안: 현재 구현이 적절해 보입니다.
답안 제출 문제
작성자 체크 리스트
In Review로 설정해주세요.검토자 체크 리스트
Important
본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!