Skip to content

Conversation

@doh6077
Copy link
Contributor

@doh6077 doh6077 commented Nov 13, 2025

답안 제출 문제

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

@sujeong-dev sujeong-dev self-requested a review November 13, 2025 05:56
if diff in prevMap:
return [prevMap[diff], i]
prevMap[n] = i

No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

스크린샷 2025-11-13 오후 3 31 10

마지막 줄 개행이 누락되신 것 같아요!
해당 가이드 참고하셔서 확인해보시면 좋을 것 같아요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

확인 감사드려요! 수정하도록 하겠습니다!

@doh6077 doh6077 moved this from Solving to In Review in 리트코드 스터디 6기 Nov 15, 2025
@TonyKim9401
Copy link
Contributor

@DaleStudy

@dalestudy
Copy link
Contributor

dalestudy bot commented Nov 16, 2025

좋은 작업으로 여러 문제를 깔끔하게 해결하셨습니다! 특히 set 활용, 정렬 기반 연속 수 찾기, 딕셔너리 활용 등 효율적인 알고리즘을 채택하셨네요. 몇 가지 피드백 드리자면:

  1. 시간/공간 복잡도 명시: 각 솔루션에 TC와 SC를 명확히 적어주세요. 예를 들어, containsDuplicate은 TC: O(n), SC: O(n), longestConsecutive는 TC: O(n log n), SC: O(n), twoSum은 TC: O(n), SC: O(n)입니다. 이렇게 명확히 하면 이해와 평가에 도움이 됩니다.

  2. longestConsecutive 개선 가능성: 정렬 후 연속 수를 찾는 방식은 간단하고 효과적이나, 최적화 가능성을 고려한다면 hash set을 이용한 방법도 있습니다. 해시셋을 이용하면 시간 복잡도를 O(n)으로 줄일 수 있어 더 효율적입니다.

  3. 가독성 및 스타일: 변수명을 조금 더 직관적으로 바꾸거나, 주석을 좀 더 일관성 있게 달면 가독성이 좋아집니다. 예를 들어, l, r 대신 left, right로 풀어 쓰는 것도 고려하세요.

  4. 전반적 격려: 간단하면서도 효율적인 코드 작성이 인상적입니다! 앞으로도 이렇게 여러 접근법을 익히고 최적화하는 연습을 계속 이어가세요. 더 나은 방법도 항상 고민하면서 발전하는 모습을 기대합니다.

계속해서 좋은 성과 기대할게요!

Copy link
Contributor

@TonyKim9401 TonyKim9401 left a comment

Choose a reason for hiding this comment

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

1주차 문제 풀이 고생하셨습니다!
온보딩 및 스터디 적응 하시는데 앞으로 조금 더 힘내주시고 앞으로의 15주간 여정 응원합니다.
2주차 문제 풀이도 파이팅입니다!

@doh6077 doh6077 added this pull request to the merge queue Nov 16, 2025
@DaleSeo DaleSeo removed this pull request from the merge queue due to the queue being cleared Nov 16, 2025
@dalestudy dalestudy bot added this pull request to the merge queue Nov 16, 2025
@DaleSeo DaleSeo removed this pull request from the merge queue due to the queue being cleared Nov 16, 2025
@dalestudy dalestudy bot added this pull request to the merge queue Nov 17, 2025
Merged via the queue into DaleStudy:main with commit 121cf70 Nov 17, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from In Review to Completed in 리트코드 스터디 6기 Nov 17, 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.

3 participants