Skip to content

Conversation

@1lsang
Copy link
Contributor

@1lsang 1lsang commented Nov 29, 2025

답안 제출 문제

작성자 체크 리스트

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

검토자 체크 리스트

Important

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

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

@dalestudy
Copy link
Contributor

dalestudy bot commented Nov 29, 2025

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

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

설정 방법

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

📚 자세한 가이드 보기


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

@1lsang
Copy link
Contributor Author

1lsang commented Nov 29, 2025

@DaleStudy
리뷰해주세요.

@dalestudy
Copy link
Contributor

dalestudy bot commented Nov 29, 2025

전체적으로 깔끔하고 명확한 코드입니다. combinationSum의 경우, DP 배열을 활용해 조합을 저장하는 방식이 효율적이며, 배열 초기화와 반복문 구조도 적절합니다. number-of-1-bits는 자리수 계산 방식 대신 비트 연산을 활용하는 방법이 더 간단할 수 있는데, 현재 방식도 이해하기 쉽고 잘 작동합니다. valid-palindrome은 대소문자와 숫자 필터링 후 양쪽 비교를 통해 깔끔하게 구현되어 있습니다. 전반적으로 로직이 명확하고, 변수 이름도 적절하며, 읽기 쉽습니다. 작은 개선점으로는 hammingWeight에서 비트 연산(n & 1)을 활용하는 것도 고려할 수 있습니다. 수고하셨습니다!

@Sol35229 Sol35229 self-requested a review November 29, 2025 12:05
Copy link
Contributor

Choose a reason for hiding this comment

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

풀이 잘 보았습니다!
문자열 변환을 regex로 하는 방법도 있어서 코드 첨부드려요!
s = s.toLowerCase().replace(/[^a-z0-9]/g, '');

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