Skip to content

Conversation

jongwanra
Copy link
Contributor

@jongwanra jongwanra commented Aug 11, 2025

답안 제출 문제

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

@sooooo-an sooooo-an self-requested a review August 12, 2025 10:45
Copy link
Contributor

@printjin-gmailcom printjin-gmailcom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다 :)

if node.right:
dfs(depth + 1, node.right)

if not root:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 부분은 문제 없지만, root가 없는 경우는 이미 위에서 처리했기 때문에 if not node: 로 처리해도 더 범용적

current_node = dummy_head
while list1 and list2:
if list1.val > list2.val:
current_node.next = ListNode(list2.val)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

기존 노드를 재사용하지 않고 새 노드를 계속 생성 → 메모리 낭비

return dummy_head.next

def print_list(node:Optional[ListNode])->None:
list = []
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

list라는 변수명이 Python 내장 타입과 충돌

@jongwanra jongwanra moved this from Solving to Completed in 리트코드 스터디 5기 Aug 17, 2025
@jongwanra jongwanra merged commit e33df7b into DaleStudy:main Aug 17, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

2 participants