[Release] v2.1.5 닉네임 숫자 시작 종결 허용 #332
Conversation
fix: 닉네임 양 끝단 숫자 허용
Summary of ChangesHello @yongjun0511, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 사용자 닉네임 유효성 검사 규칙을 업데이트하여 닉네임이 숫자 문자로 시작하거나 끝날 수 있도록 허용합니다. 이는 사용자에게 닉네임 생성에 더 많은 유연성을 제공하고, 기존의 한글 및 영어 문자 제한을 확장합니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. 정규식 변경 숫자도 이제 허용 닉네임 자유 Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이번 PR은 닉네임이 숫자로 시작하고 끝날 수 있도록 유효성 검사 로직을 수정한 것으로 보입니다. 변경 사항은 PR의 목적과 일치하며 올바르게 구현되었습니다.
다만, 몇 가지 개선점을 제안합니다.
- 코드 개선: 정규식에서
\p{IsHangul}을 사용하여 가독성과 유지보수성을 높일 수 있습니다. - 스타일 가이드: 리포지토리 스타일 가이드에 따라 주석에 작성자 정보와 날짜를 추가하는 것을 고려해 보세요.
가장 중요한 점으로, 변경 사항에 맞춰 테스트 코드를 업데이트해야 합니다. 현재 NicknameValidatorTest.java의 일부 테스트 케이스(한글_또는_영어로_시작하지_않는_닉네임인_경우_예외가_발생한다, 한글_또는_영어로_끝나지_않는_닉네임인_경우_예외가_발생한다)는 이번 변경으로 인해 실패할 것으로 예상됩니다. 또한, 숫자로 시작하거나 끝나는 유효한 닉네임에 대한 테스트 케이스를 추가하여 코드 변경의 안정성을 보장해야 합니다.
자세한 내용은 각 리뷰 댓글을 참고해 주세요.
|
|
||
| // 한글 영어가 아닌 것으로 시작 | ||
| if (nickname.matches("^[^A-Za-z가-힣ㄱ-ㅎㅏ-ㅣ].*")) { | ||
| // 한글, 영어, 숫자가 아닌 것으로 시작 |
There was a problem hiding this comment.
리포지토리 스타일 가이드 8번 규칙에 따르면, 모든 주석에는 작성자 이름과 날짜가 포함되어야 합니다. 이 규칙을 이 주석과 110번 줄의 주석 등 파일 내 다른 유사한 주석들에도 적용하는 것을 고려해 보세요. 예를 들어: // 한글, 영어, 숫자가 아닌 것으로 시작 // author, YYYY-MM-DD
References
- Each comment must include the author's name and the date. (link)
| // 한글 영어가 아닌 것으로 시작 | ||
| if (nickname.matches("^[^A-Za-z가-힣ㄱ-ㅎㅏ-ㅣ].*")) { | ||
| // 한글, 영어, 숫자가 아닌 것으로 시작 | ||
| if (nickname.matches("^[^A-Za-z가-힣ㄱ-ㅎㅏ-ㅣ0-9].*")) { |
There was a problem hiding this comment.
정규식에서 한글 문자 범위를 나타내는 가-힣ㄱ-ㅎㅏ-ㅣ 대신 \p{IsHangul}을 사용하는 것을 고려해 보세요. \p{IsHangul}은 유니코드 속성을 사용하여 모든 한글 문자를 포괄하므로, 코드가 더 간결해지고 가독성이 향상됩니다. 또한, 향후 유니코드 표준 변경에도 더 유연하게 대응할 수 있습니다.
if (nickname.matches("^[^A-Za-z\p{IsHangul}0-9].*")) {| // 한글 영어가 아닌 것으로 마무리 | ||
| if (nickname.matches(".*[^A-Za-z가-힣ㄱ-ㅎㅏ-ㅣ]$")) { | ||
| // 한글, 영어, 숫자가 아닌 것으로 끝남 | ||
| if (nickname.matches(".*[^A-Za-z가-힣ㄱ-ㅎㅏ-ㅣ0-9]$")) { |
There was a problem hiding this comment.
106번 줄의 제안과 마찬가지로, 여기에서도 가-힣ㄱ-ㅎㅏ-ㅣ 대신 \p{IsHangul}을 사용하여 정규식을 개선할 수 있습니다. 일관성을 위해 파일 내 다른 정규식(예: 116번 줄)에도 동일한 변경을 적용하는 것을 추천합니다.
if (nickname.matches(".*[^A-Za-z\p{IsHangul}0-9]$")) {
No description provided.