Skip to content

Conversation

@Blossssom
Copy link
Contributor

@Blossssom Blossssom commented Nov 22, 2025

답안 제출 문제

  • 문제 1
  • 문제 2
  • 문제 3
  • 문제 4

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

@Blossssom
Copy link
Contributor Author

@DaleStudy 리뷰해줘요

@dalestudy
Copy link
Contributor

dalestudy bot commented Nov 22, 2025

이 PR은 4문제 모두 깔끔하게 해결했으며, 효율적 알고리즘을 적용하셨습니다. 특히 threeSum은 투 포인터, 중복 제거 로직이 잘 구현되었고, productExceptSelf는 공간 복잡도 최적화도 적절합니다. climbStairs는 피보나치 패턴을 활용해 반복문으로 간단하게 해결했고, valid-anagram도 해시맵을 이용한 효율적 방법입니다. 전반적으로 코드 가독성도 좋으며, 주석과 설명도 이해에 도움됩니다. 몇 가지 개선점으로는 변수명 명확화, 타입 표기 일관성 유지가 있겠습니다. 수고하셨습니다!

@sujeong-dev sujeong-dev self-requested a review November 22, 2025 10:18
nums.sort((a, b) => a - b);

for (let i = 0; i < nums.length - 2; i++) {
if (i && nums[i] === nums[i - 1]) {
Copy link
Contributor

Choose a reason for hiding this comment

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

i가 true라는건 0보다 클 때를 칭하는것이죠?!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 0은 falsy니까 명시하지 않고 i만 적었습니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

리뷰남겨 주셔서 감사합니다!!!!


const nums = [-1, 0, 1, 2, -1, -4];
threeSum(nums);

Copy link
Contributor

Choose a reason for hiding this comment

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

저는 set으로 해서 연산이 오래걸린던데 투포인터로 변경해봐야겠네요! 👍

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