Conversation
Summary of ChangesHello @yeongsinkeem, 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은 시스템의 학교 관련 데이터 모델을 'School'에서 'University'로 전면 개편하고, 지역 정보를 위한 'Region' 엔티티를 도입합니다. 이를 통해 공공데이터 기반의 대학 정보 초기화 및 관리 기능을 강화하며, 사용자에게 대학 이름 및 초성 검색 기능을 제공하여 조직 생성 및 검색 경험을 개선합니다. 전반적인 데이터 구조의 현대화와 기능 확장을 목표로 합니다. Highlights
Changelog
Activity
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}
|
|
||
| // commandLineRunner의 run 메서드는 String... args로 유지 (String[] args와 유사) -> 스프링 부트 공식 문서 참고 | ||
| // args를 꼭 배열로 넘겨야 하는 건 아니고 가변적으로 받을 수 있다는 의도를 나타냄 | ||
| @Override |
There was a problem hiding this comment.
여기서 RuntimeException을 사용하는 대신, OrganizationService에서 사용되는 MemberNotFoundException과 같이 더 구체적인 예외를 사용하거나, ErrorCode를 포함하는 커스텀 예외를 사용하는 것이 좋습니다. 이는 애플리케이션 전반의 예외 처리 일관성을 높이고 클라이언트가 오류를 더 쉽게 이해할 수 있도록 돕습니다.
.orElseThrow(() -> new MemberNotFoundException(ErrorCode.MEMBER_NOT_FOUND));| initializeRegions(); | ||
|
|
||
| // 2. University 대학 데이터 | ||
| initializeUniversities(); |
There was a problem hiding this comment.
마찬가지로, RuntimeException 대신 UniversityNotFoundException과 같이 더 구체적인 예외를 정의하고 사용하는 것이 좋습니다. 이는 오류의 의미를 명확히 하고 디버깅을 용이하게 합니다.
.orElseThrow(() -> new UniversityNotFoundException(ErrorCode.UNIVERSITY_NOT_FOUND)); // 예시|
|
||
| // 3. 테스트/관리자 데이터 | ||
| initializeAdmin(); | ||
| } |
There was a problem hiding this comment.
이곳에서도 RuntimeException 대신 DuplicateOrganizationException과 같이 비즈니스 로직에 맞는 구체적인 예외를 사용하는 것이 좋습니다. 이는 예외 처리 계층을 명확히 하고, 클라이언트에게 더 의미 있는 오류 메시지를 전달할 수 있게 합니다.
throw new DuplicateOrganizationException(ErrorCode.DUPLICATE_CLUB_NAME_IN_UNIVERSITY); // 예시| private void initializeRegions() { | ||
| if (regionRepository.count() == 0) { | ||
| List<Region> regions = new ArrayList<>(); | ||
|
|
There was a problem hiding this comment.
이 경우에도 RuntimeException 대신 DuplicateOrganizationException과 같은 구체적인 예외를 사용하는 것이 좋습니다. 이는 오류 유형을 명확히 하고, 애플리케이션의 견고성을 높입니다.
throw new DuplicateOrganizationException(ErrorCode.DUPLICATE_COMMUNITY_NAME); // 예시
src/main/java/org/example/tackit/domain/organization/service/OrganizationService.java
Show resolved
Hide resolved
src/main/java/org/example/tackit/domain/organization/service/OrganizationService.java
Show resolved
Hide resolved
| if (organizationRepository.existsByNameAndUniversityAndType(dto.getOrgName(), university, OrgType.CLUB)) { | ||
| throw new RuntimeException("해당 학교에 이미 동일한 이름의 동아리가 존재합니다."); |
There was a problem hiding this comment.
여기서도 RuntimeException 대신 DuplicateOrganizationException과 같이 비즈니스 로직에 특화된 예외를 사용하는 것이 좋습니다. 이는 오류 처리 로직을 더 세분화하고, 클라이언트에게 더 정확한 오류 정보를 제공할 수 있게 합니다.
throw new DuplicateOrganizationException(ErrorCode.DUPLICATE_CLUB_NAME_IN_UNIVERSITY); // 예시| if (organizationRepository.existsByNameAndType(dto.getOrgName(), OrgType.COMMUNITY)) { | ||
| throw new RuntimeException("이미 동일한 이름의 소모임이 존재합니다."); |
There was a problem hiding this comment.
이 경우에도 RuntimeException 대신 DuplicateOrganizationException과 같은 구체적인 예외를 사용하는 것이 좋습니다. 이는 오류의 원인을 명확히 하고, 향후 유지보수 및 확장을 용이하게 합니다.
throw new DuplicateOrganizationException(ErrorCode.DUPLICATE_COMMUNITY_NAME); // 예시
tishakong
left a comment
tishakong
left a comment
There was a problem hiding this comment.
내용 확인했습니다~ 로직에서 이해 안되는 부분은 없습니다.
제미나이 리뷰들도 대부분 응답값 반환 형식 통일 관련 멘트인데, 지금 수정하실 여력이 있으시면 수정하시고, 아니면 머지해도 될 것 같습니다!
| private Long orgId; | ||
| private String orgName; | ||
| private String orgType; | ||
| private String universityName; // CLUB일 때만 값이 있고, COMMUNITY면 "연합/소모임" 등으로 표시 |
There was a problem hiding this comment.
COMMUNITY면 "연합/소모임" 등으로 표시한다는 게, universityName 의 값으로 연합/소모임을 반환한다는 걸까요? 아래 코드를 보니까 null을 반환하는 거 같긴 한데 null 반환이 맞다면 주석이 조금 모호해서 수정하면 좋을 것 같아요!
There was a problem hiding this comment.
수정되어야 할 것 같습니다 !
연합 동아리라면
regionId = 0,
universityName = "연합" 이렇게 되는 게 깔끔할 것 같습니다 !
확인 감사합니다 !
2 participants
이슈 번호
작업 내용