Skip to content

Conversation

@ys-han00
Copy link
Contributor

@ys-han00 ys-han00 commented Dec 10, 2025

답안 제출 문제

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

@ys-han00
Copy link
Contributor Author

@DaleStudy 리뷰해줘

@dalestudy
Copy link
Contributor

dalestudy bot commented Dec 12, 2025

전체적으로 여러 문제에 대해 깔끔하고 효율적인 접근 방식을 선택하셨네요! 특히 maxProfitgroupAnagrams, wordBreak 구현에서 최적화와 가독성을 잘 고려하셨습니다.

구현 방식과 알고리즘

  • maxProfit: 최솟값을 갱신하며 단일 패스로 최대 이익을 찾는 방법이 시간복잡도 O(n)으로 가장 효율적입니다. 공간복잡도도 O(1)로 적절하며, 더 좋은 접근법은 없습니다.
  • groupAnagrams: 정렬을 이용한 해시맵 기반 그룹핑으로 O(n * k log k) (k는 문자열 평균 길이)입니다. unordered_map 사용으로 성능도 적절합니다.
  • wordBreak: DP를 이용한 최적화된 풀이로, 시간복잡도는 O(n * m * l) (m은 단어 수, l은 최대 단어 길이)로 적당하며, 깔끔하게 구현되어 있습니다.

가독성 및 스타일

  • 변수명과 구조가 명확하며, 주석처리된 이전 답안도 참고하는 방식이 좋아요.
  • buyans 변수명도 직관적입니다.
  • groupAnagrams에서 unordered_map 사용도 성능에 적합하며, auto 사용으로 간결성을 유지했습니다.

추가 제안

  • wordBreaksubstr 호출을 반복하는 대신, s 문자열의 부분 문자열을 미리 저장하거나, 인덱스 기반 비교를 통해 성능을 더 높일 수 있습니다.
  • maxProfit에서 초기 값이 prices[0]인 것도 좋지만, 만약 prices가 비어있을 가능성도 고려한다면 방어적 코드를 넣어도 좋습니다.

요약

  • 현재 구현 모두 시간복잡도와 공간복잡도를 적절히 고려했고, 효율적입니다.
  • 가독성과 명확성도 뛰어나며, 더 나은 접근법은 필요 없는 상태입니다.
  • 계속해서 다양한 문제를 깔끔하게 해결하는 모습이 인상적입니다. 앞으로도 이 방향 유지하시면서, 복잡한 문제는 DP 또는 해시맵 활용을 연습해보세요!

@ys-han00
Copy link
Contributor Author

@DaleStudy 승인해줘

@dalestudy
Copy link
Contributor

dalestudy bot commented Dec 12, 2025

이 PR은 여러 문제에 대해 깔끔한 풀이가 포함되어 있으며, 특히 maxProfit, groupAnagrams, Trie, wordBreak 함수가 잘 구현되어 있습니다. 마지막 maxProfit 솔루션은 최소값을 계속 갱신하며 최대 차익을 찾는 효율적인 방법으로, 시간 복잡도도 O(n)으로 최적화되어 있습니다. 검토 후 승인하겠습니다. 수고하셨습니다!

Copy link
Contributor

@dalestudy dalestudy bot left a comment

Choose a reason for hiding this comment

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

현재 주차가 종료되어 자동으로 승인되었습니다. PR을 병합해주세요!

@DaleSeo DaleSeo merged commit 3292cc6 into DaleStudy:main Dec 14, 2025
3 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Completed in 리트코드 스터디 6기 Dec 14, 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