Skip to content

Conversation

@unpo88
Copy link
Contributor

@unpo88 unpo88 commented Dec 19, 2025

답안 제출 문제

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

@dalestudy
Copy link
Contributor

dalestudy bot commented Dec 19, 2025

⚠️ Week 설정이 누락되었습니다

프로젝트에서 Week를 설정해주세요!

설정 방법

  1. PR 우측의 Projects 섹션에서 리트코드 스터디 옆 드롭다운(▼) 클릭
  2. 현재 주차를 선택해주세요 (예: Week 14(current) 또는 Week 14)

📚 자세한 가이드 보기


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

@unpo88 unpo88 moved this from Solving to In Review in 리트코드 스터디 6기 Dec 20, 2025
@unpo88
Copy link
Contributor Author

unpo88 commented Dec 20, 2025

@DaleStudy

@dalestudy
Copy link
Contributor

dalestudy bot commented Dec 20, 2025

전체적으로 깔끔하고 이해하기 쉬운 코드입니다! 두 솔루션 모두 효율적이고, 설명이 상세하여 학습에 도움이 됩니다. 아래는 몇 가지 피드백입니다:

  1. 시간/공간 복잡도 명시 권장:
    두 솔루션에 모두 시간(TC)과 공간(SC) 복잡도를 주석 또는 설명으로 명시하면 좋겠습니다. 예를 들어, # TC: O(n), SC: O(n)으로 표기하세요. 이는 다른 사람과의 협업 또는 자신이 나중에 복습할 때 유용합니다.

  2. 가독성 향상:

    • return not stack 구문을 마지막에 넣은 것은 간결하지만, 초기 버전과 일관성을 위해 return len(stack) == 0으로 명시하는 것도 좋습니다.
    • 특히, 두 번째 솔루션의 return not stack은 파이썬 초심자에게 익숙하지 않을 수 있으니, return len(stack) == 0으로 표기하는 것도 고려하세요.
  3. 베스트 프랙티스:

    • 딕셔너리 pairs를 함수 밖에 상수로 정의하거나, 클래스 변수로 선언하면 코드 재사용성이 높아지고, 가독성도 향상됩니다.
  4. 추가 제안:

    • pairs 딕셔너리와 같은 상수는 class 내부 또는 모듈 최상단에 정의하는 것도 좋습니다.
    • 이와 함께, stack 변수명은 명확하며 적절합니다.
  5. 격려 메시지:

    • 코드는 깔끔하며, 각각의 경우에 대해 상세한 설명과 예시를 제공한 점이 인상적입니다. 이러한 주석은 리뷰를 받는 사람에게 큰 도움이 되고, 학습 자료로도 손색없습니다.
    • 앞으로도 다양한 케이스를 고려하는 자세를 유지하세요!

요약하자면, 복잡도 표기와 일부 스타일 개선만 고려하시면 더욱 완성도 높은 코드가 될 것 같습니다. 수고하셨고, 계속해서 좋은 코드 기대할게요!

@Blossssom Blossssom self-requested a review December 20, 2025 04:10
Copy link
Contributor

@Blossssom Blossssom left a comment

Choose a reason for hiding this comment

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

각 문제의 접근방식과 시도를 적어주셔서 이해하며 보기 좋았습니다! 한주간 고생하셨습니다~

@unpo88 unpo88 merged commit 38e674e into DaleStudy:main Dec 20, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from In Review to Completed in 리트코드 스터디 6기 Dec 20, 2025
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