refactor: '대학교'와 '대학 지원 정보' 용어를 구분하도록 #346
Conversation
nayonsoso
requested review from
Gyuhyeok99,
lsy1307,
whqtker and
wibaek
as code owners
Walkthrough
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🧰 Additional context used🧬 Code Graph Analysis (1)src/main/java/com/example/solidconnection/university/repository/UnivApplyInfoRepository.java (1)
⏰ Context from checks skipped due to timeout of 90000ms (1)
🔇 Additional comments (4)
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🔭 Outside diff range comments (1)
src/test/java/com/example/solidconnection/university/fixture/LanguageRequirementFixtureBuilder.java (1)
35-43: NPE 예방을 위한 널 체크 추가 제안univApplyInfo가 null인 상태에서
addLanguageRequirements()가 호출되면 테스트 픽스처 생성 시 NPE가 발생합니다. 미리 null 검증을 해두면 디버깅 시간을 절약할 수 있습니다.public LanguageRequirement create() { + Objects.requireNonNull(univApplyInfo, "univApplyInfo는 필수 값입니다."); LanguageRequirement languageRequirement = new LanguageRequirement(
🧹 Nitpick comments (7)
src/main/java/com/example/solidconnection/application/dto/ApplyRequest.java (1)
15-15: ✅ DTO 파라미터 타입이 올바르게 업데이트되었습니다
- 타입 변경:
UniversityChoiceRequest에서UnivApplyInfoChoiceRequest로 정확히 변경되었습니다- TODO 주석: 프론트엔드 협의 후 파라미터명 변경 계획이 명시되어 있어 좋습니다
- 임시 불일치 인식: 타입과 파라미터명 간의 불일치를 개발자가 인지하고 있습니다
💡 제안사항: 가능하다면 프론트엔드와의 협의를 조속히 진행하여
univApplyInfoChoiceRequest로 파라미터명도 통일하는 것을 권장합니다. 이렇게 하면 코드베이스 전체의 일관성이 한층 더 향상될 것입니다.src/main/java/com/example/solidconnection/university/domain/LikedUnivApplyInfo.java (1)
22-29: 제약 조건 이름에 남은 ‘liked_university’ 용어Constraint 이름
uk_liked_university_site_user_id_university_info_for_apply_id에 여전히liked_university가 포함되어 있습니다. 스키마 가독성을 위해uk_liked_univ_apply_info_...등으로 맞춰 두면 혼란을 줄일 수 있습니다.src/test/java/com/example/solidconnection/university/service/UnivApplyInfoLikeServiceTest.java (1)
154-160: 헬퍼 메서드 이름을 새로운 도메인 명명 규칙에 맞게 수정해주세요.헬퍼 메서드
saveLikedUniversity가 여전히 기존 명명 규칙을 사용하고 있습니다. 일관성을 위해saveLikedUnivApplyInfo로 변경하는 것이 좋겠습니다.- private void saveLikedUniversity(SiteUser siteUser, UnivApplyInfo univApplyInfo) { + private void saveLikedUnivApplyInfo(SiteUser siteUser, UnivApplyInfo univApplyInfo) {src/main/java/com/example/solidconnection/university/service/UnivApplyInfoRecommendService.java (1)
66-66: 캐시 키를 새로운 도메인 명명 규칙에 맞게 업데이트하는 것을 고려해보세요.캐시 키가 여전히
"university:recommend:general"을 사용하고 있습니다. 일관성을 위해"univApplyInfo:recommend:general"로 변경하는 것이 좋겠습니다.- @ThunderingHerdCaching(key = "university:recommend:general", cacheManager = "customCacheManager", ttlSec = 86400) + @ThunderingHerdCaching(key = "univApplyInfo:recommend:general", cacheManager = "customCacheManager", ttlSec = 86400)참고: 기존 캐시 데이터와의 호환성을 고려하여 배포 전 캐시 무효화가 필요할 수 있습니다.
src/main/java/com/example/solidconnection/university/service/UnivApplyInfoQueryService.java (1)
24-24: TODO 주석의 아키텍처 개선사항을 추적하고 있어 좋습니다.구현체를 직접 주입받는 대신 인터페이스를 통해 사용하도록 하는 것이 의존성 역전 원칙에 더 부합합니다. 이 개선사항을 별도 이슈로 추적하는 것을 권장합니다.
이 리팩토링을 위한 별도 이슈를 생성하여 추적하시겠습니까?
src/main/java/com/example/solidconnection/university/controller/UnivApplyInfoController.java (2)
91-102: 3. 검색 엔드포인트 매개변수 개선 필요현재 빈 문자열을 기본값으로 사용하는 것보다는 null을 허용하는 것이 더 명확할 수 있습니다.
- 빈 문자열 vs null의 의미적 차이
- 선택적 필터링 로직의 명확성- @RequestParam(required = false, defaultValue = "") String region, - @RequestParam(required = false, defaultValue = "") List<String> keyword, - @RequestParam(required = false, defaultValue = "") LanguageTestType testType, - @RequestParam(required = false, defaultValue = "") String testScore + @RequestParam(required = false) String region, + @RequestParam(required = false) List<String> keyword, + @RequestParam(required = false) LanguageTestType testType, + @RequestParam(required = false) String testScore
91-91: 4. TODO 주석 처리 계획반환 타입 변경 관련 TODO 주석이 있습니다. 프론트엔드와의 협의 후 변경 예정이라고 하니, 이슈로 트래킹하는 것이 좋겠습니다.
이 TODO 항목을 별도 이슈로 생성해서 추적하시겠어요? 필요하시면 도와드릴 수 있습니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (42)
src/main/java/com/example/solidconnection/application/dto/ApplyRequest.java(1 hunks)src/main/java/com/example/solidconnection/application/dto/UnivApplyInfoChoiceRequest.java(1 hunks)src/main/java/com/example/solidconnection/application/dto/UniversityChoiceRequest.java(0 hunks)src/main/java/com/example/solidconnection/application/service/ApplicationQueryService.java(4 hunks)src/main/java/com/example/solidconnection/application/service/ApplicationSubmissionService.java(2 hunks)src/main/java/com/example/solidconnection/common/exception/ErrorCode.java(3 hunks)src/main/java/com/example/solidconnection/siteuser/repository/LikedUniversityRepository.java(0 hunks)src/main/java/com/example/solidconnection/siteuser/service/MyPageService.java(3 hunks)src/main/java/com/example/solidconnection/university/controller/UnivApplyInfoController.java(1 hunks)src/main/java/com/example/solidconnection/university/controller/UniversityController.java(0 hunks)src/main/java/com/example/solidconnection/university/domain/LikedUnivApplyInfo.java(1 hunks)src/main/java/com/example/solidconnection/university/dto/UnivApplyInfoDetailResponse.java(2 hunks)src/main/java/com/example/solidconnection/university/dto/UnivApplyInfoPreviewResponse.java(2 hunks)src/main/java/com/example/solidconnection/university/dto/UnivApplyInfoPreviewResponses.java(1 hunks)src/main/java/com/example/solidconnection/university/dto/UnivApplyInfoRecommendsResponse.java(1 hunks)src/main/java/com/example/solidconnection/university/dto/UniversityInfoForApplyPreviewResponses.java(0 hunks)src/main/java/com/example/solidconnection/university/dto/UniversityRecommendsResponse.java(0 hunks)src/main/java/com/example/solidconnection/university/dto/validation/ValidUnivApplyInfoChoice.java(1 hunks)src/main/java/com/example/solidconnection/university/dto/validation/ValidUnivApplyInfoChoiceValidator.java(3 hunks)src/main/java/com/example/solidconnection/university/repository/LikedUnivApplyInfoRepository.java(1 hunks)src/main/java/com/example/solidconnection/university/repository/UnivApplyInfoRepository.java(3 hunks)src/main/java/com/example/solidconnection/university/repository/UniversityRepository.java(1 hunks)src/main/java/com/example/solidconnection/university/repository/custom/UnivApplyInfoFilterRepository.java(1 hunks)src/main/java/com/example/solidconnection/university/repository/custom/UnivApplyInfoFilterRepositoryImpl.java(3 hunks)src/main/java/com/example/solidconnection/university/service/GeneralUnivApplyInfoRecommendService.java(2 hunks)src/main/java/com/example/solidconnection/university/service/UnivApplyInfoLikeService.java(1 hunks)src/main/java/com/example/solidconnection/university/service/UnivApplyInfoQueryService.java(4 hunks)src/main/java/com/example/solidconnection/university/service/UnivApplyInfoRecommendService.java(3 hunks)src/main/java/com/example/solidconnection/university/service/UniversityLikeService.java(0 hunks)src/main/resources/db/migration/V17__rename_like_university.sql(1 hunks)src/test/java/com/example/solidconnection/application/service/ApplicationSubmissionServiceTest.java(5 hunks)src/test/java/com/example/solidconnection/siteuser/service/MyPageServiceTest.java(5 hunks)src/test/java/com/example/solidconnection/university/dto/validation/ValidUnivApplyInfoChoiceValidatorTest.java(7 hunks)src/test/java/com/example/solidconnection/university/fixture/LanguageRequirementFixture.java(1 hunks)src/test/java/com/example/solidconnection/university/fixture/LanguageRequirementFixtureBuilder.java(1 hunks)src/test/java/com/example/solidconnection/university/fixture/UnivApplyInfoFixture.java(1 hunks)src/test/java/com/example/solidconnection/university/fixture/UnivApplyInfoFixtureBuilder.java(3 hunks)src/test/java/com/example/solidconnection/university/repository/LikedUnivApplyInfoRepositoryTest.java(4 hunks)src/test/java/com/example/solidconnection/university/service/GeneralUnivApplyInfoRecommendServiceTest.java(2 hunks)src/test/java/com/example/solidconnection/university/service/UnivApplyInfoLikeServiceTest.java(9 hunks)src/test/java/com/example/solidconnection/university/service/UnivApplyInfoQueryServiceTest.java(10 hunks)src/test/java/com/example/solidconnection/university/service/UnivApplyInfoRecommendServiceTest.java(7 hunks)
💤 Files with no reviewable changes (6)
- src/main/java/com/example/solidconnection/university/dto/UniversityRecommendsResponse.java
- src/main/java/com/example/solidconnection/application/dto/UniversityChoiceRequest.java
- src/main/java/com/example/solidconnection/university/dto/UniversityInfoForApplyPreviewResponses.java
- src/main/java/com/example/solidconnection/siteuser/repository/LikedUniversityRepository.java
- src/main/java/com/example/solidconnection/university/service/UniversityLikeService.java
- src/main/java/com/example/solidconnection/university/controller/UniversityController.java
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build
🔇 Additional comments (59)
src/main/resources/db/migration/V17__rename_like_university.sql (1)
1-1: ✅ 데이터베이스 마이그레이션이 올바르게 구성되었습니다
- 테이블 리네이밍:
liked_university에서liked_university_info_for_apply로의 변경이 도메인 모델 리팩토링과 일치합니다- SQL 구문: ALTER TABLE RENAME 문법이 정확하게 작성되었습니다
이 변경사항은
LikedUniversity엔티티가LikedUnivApplyInfo로 변경되는 것과 완벽하게 일치합니다.src/test/java/com/example/solidconnection/university/fixture/UnivApplyInfoFixture.java (1)
19-91: ✅ 테스트 픽스처 메서드 호출이 일관되게 업데이트되었습니다
- 메서드 명 변경: 모든 픽스처 생성 메서드에서
universityInfoForApply()가univApplyInfo()로 일관되게 변경되었습니다- 리팩토링 일관성:
UnivApplyInfoFixtureBuilder의 메서드명 변경과 완벽하게 일치합니다- 테스트 데이터 무결성: 로직 변경 없이 순수한 네이밍 변경으로 기존 테스트 동작이 보장됩니다
모든 대학 정보 픽스처(괌대학, 네바다주립대학, 메모리얼대학 등)가 새로운 도메인 용어로 통일되어 가독성이 향상되었습니다.
src/test/java/com/example/solidconnection/application/service/ApplicationSubmissionServiceTest.java (2)
7-7: ✅ 테스트 클래스의 임포트가 올바르게 업데이트되었습니다
- DTO 타입 변경:
UniversityChoiceRequest에서UnivApplyInfoChoiceRequest로 정확히 변경되었습니다- 의존성 일관성: 삭제된 DTO 클래스 대신 새로운 DTO를 올바르게 참조합니다
이 변경은 메인 서비스 레이어의 리팩토링과 완벽하게 동기화되어 있습니다.
76-81: ✅ 테스트 메서드들의 DTO 인스턴스 생성이 일관되게 수정되었습니다
생성자 호출 변경: 모든 테스트 메서드에서
UnivApplyInfoChoiceRequest생성자 사용으로 통일되었습니다테스트 시나리오 유지: 기존 테스트 로직과 검증 내용이 그대로 보존되어 테스트 커버리지가 유지됩니다
파라미터 일관성: 대학 선택 ID들이 올바르게 전달되고 있습니다
- 정상 케이스: 3개 대학 모두 선택
- 예외 케이스: 첫 번째 선택만 사용 (나머지는 null)
이러한 변경으로 새로운 DTO 구조에 맞춰 테스트가 완벽하게 적응되었습니다.
Also applies to: 109-114, 129-134, 149-154
src/main/java/com/example/solidconnection/application/service/ApplicationSubmissionService.java (2)
7-7: ✅ 서비스 클래스의 임포트가 정확하게 업데이트되었습니다
- DTO 임포트 변경:
UniversityChoiceRequest에서UnivApplyInfoChoiceRequest로 올바르게 변경되었습니다- 의존성 해결: 삭제된 클래스 대신 새로운 DTO 클래스를 올바르게 참조합니다
이 변경은 리팩토링의 핵심 부분으로 서비스 레이어의 일관성을 보장합니다.
45-51: ✅ 변수명과 메서드 호출이 일관되게 리팩토링되었습니다
- 변수명 업데이트:
universityChoiceRequest가univApplyInfoChoiceRequest로 의미적으로 더 명확하게 변경되었습니다- 메서드 호출 일관성: 새로운 변수명을 사용하여 선택 대학 ID들을 올바르게 추출합니다
firstChoiceUniversityIdsecondChoiceUniversityIdthirdChoiceUniversityId- 비즈니스 로직 보존: 핵심 지원서 제출 로직은 그대로 유지되면서 네이밍만 개선되었습니다
이러한 변경으로 코드의 가독성과 도메인 일관성이 크게 향상되었습니다.
src/test/java/com/example/solidconnection/university/fixture/LanguageRequirementFixtureBuilder.java (1)
30-33: 메서드명 리네임 👍도메인 용어를 반영해
univApplyInfo로 변경된 부분, 가독성이 향상되었습니다.src/main/java/com/example/solidconnection/university/dto/UnivApplyInfoRecommendsResponse.java (1)
5-8: 필드명에 남아있는 ‘Universities’ 용어도메인 전환이 완료되었다면
recommendedUniversities→recommendedUnivApplyInfos로 맞추는 것이 일관성 측면에서 좋습니다. TODO 처리 일정 한 번 더 확인 부탁드립니다.src/main/java/com/example/solidconnection/university/dto/UnivApplyInfoPreviewResponses.java (1)
5-8: 응답 래핑 전략 TODO 확인
univApplyInfoPreviews로 감싸는 형태로 변경 예정이라면, 컨트롤러 반환 타입도 함께 수정되어야 합니다. 프론트 협의 후 일정에 반영해 주세요.src/test/java/com/example/solidconnection/university/dto/validation/ValidUnivApplyInfoChoiceValidatorTest.java (1)
3-4: 테스트 리네이밍 일관성 확인새 DTO·에러 코드로 깔끔하게 교체되어 테스트가 의도대로 동작할 것으로 보입니다.
src/test/java/com/example/solidconnection/university/fixture/UnivApplyInfoFixtureBuilder.java (3)
5-5: 리포지토리 참조가 올바르게 업데이트되었습니다
- 임포트 구문이
UniversityInfoForApplyRepository에서UnivApplyInfoRepository로 정확히 변경되었습니다- 필드 타입도 일관되게 업데이트되었습니다
리팩토링의 일관성이 잘 유지되고 있습니다.
Also applies to: 18-18
24-26: 팩토리 메서드명이 새로운 네이밍 컨벤션에 맞게 변경되었습니다메서드명이
universityInfoForApply()에서univApplyInfo()로 간결하고 명확하게 변경되었네요. 새로운 도메인 용어와 일치합니다.
51-51: 새로운 리포지토리를 사용하여 엔티티 저장이 올바르게 수행됩니다
univApplyInfoRepository.save()호출이 업데이트된 리포지토리 타입과 일치합니다.src/main/java/com/example/solidconnection/university/dto/validation/ValidUnivApplyInfoChoice.java (1)
13-14: 검증 어노테이션이 새로운 도메인 용어에 맞게 일관되게 리네이밍되었습니다
@Constraint어노테이션의validatedBy속성이ValidUnivApplyInfoChoiceValidator.class로 올바르게 변경되었습니다- 인터페이스명도
ValidUnivApplyInfoChoice로 일관성 있게 업데이트되었습니다검증 로직의 기능은 그대로 유지하면서 네이밍 컨벤션만 깔끔하게 변경되었네요.
src/main/java/com/example/solidconnection/university/repository/UniversityRepository.java (1)
15-15: 필터링 책임이 적절히 분리되었는지 확인이 필요합니다
UniversityFilterRepository확장이 제거되어UniversityRepository가 단순화되었습니다. 이는 필터링 기능이 새로운UnivApplyInfoFilterRepository로 이동했음을 시사합니다.책임 분리 관점에서는 좋은 변경이지만, 기존 필터링 기능이 새로운 리포지토리에서 올바르게 구현되었는지 확인해보시기 바랍니다.
#!/bin/bash # Description: 필터링 기능이 새로운 UnivApplyInfoFilterRepository에서 올바르게 구현되었는지 확인 # UnivApplyInfoFilterRepository와 관련 구현체 찾기 echo "=== UnivApplyInfoFilterRepository 관련 파일 검색 ===" fd -t f "UnivApplyInfoFilter" --exec echo "파일: {}" \; # 기존 UniversityFilterRepository의 메서드들이 새로운 곳에서 구현되었는지 확인 echo -e "\n=== 필터링 메서드 구현 확인 ===" rg -A 3 "findByRegionCodeAndKeywords|findAllByRegionCodeAndKeywords" --type javasrc/main/java/com/example/solidconnection/application/service/ApplicationQueryService.java (3)
11-12: 리포지토리 의존성이 새로운 인터페이스로 일관되게 업데이트되었습니다
UniversityInfoForApplyRepository가UnivApplyInfoRepository로 변경UniversityFilterRepositoryImpl이UnivApplyInfoFilterRepositoryImpl로 변경필드 선언과 임포트가 모두 일치하여 일관성이 잘 유지되고 있습니다.
Also applies to: 34-35
44-44: 메서드명이 더 명확하게 개선되었습니다
findByRegionCodeAndKeywords에서findAllByRegionCodeAndKeywords로 변경되어 복수 결과를 반환한다는 의미가 더 명확해졌네요. 좋은 네이밍 개선입니다.
74-74: 새로운 리포지토리 메서드 호출이 올바르게 적용되었습니다
univApplyInfoRepository.findAllByUniversityIds()호출이 새로운 리포지토리 인터페이스와 일치합니다.src/main/java/com/example/solidconnection/university/repository/LikedUnivApplyInfoRepository.java (2)
11-11: 새로운 도메인 모델을 위한 리포지토리 인터페이스가 올바르게 정의되었습니다
LikedUnivApplyInfo엔티티를 관리하기 위한 JPA 리포지토리가 적절히 선언되었습니다. 기존의LikedUniversityRepository를 대체하는 새로운 도메인 모델과 일치합니다.
13-17: 필요한 쿼리 메서드들이 적절히 정의되었습니다
findAllBySiteUser_Id()- 사용자별 좋아요 목록 조회countBySiteUser_Id()- 사용자별 좋아요 개수 카운트findBySiteUserAndUnivApplyInfo()- 특정 사용자와 대학 지원 정보 조합으로 좋아요 검색Spring Data JPA 네이밍 컨벤션을 잘 따르고 있으며, 좋아요 기능에 필요한 핵심 쿼리들이 모두 포함되어 있습니다.
src/main/java/com/example/solidconnection/university/dto/UnivApplyInfoDetailResponse.java (2)
8-8: 클래스명 변경이 올바르게 적용되었습니다
UniversityDetailResponse에서UnivApplyInfoDetailResponse로의 리네이밍이 깔끔하게 처리되었습니다.
36-39: 팩토리 메서드가 일관되게 업데이트되었습니다static factory 메서드의 반환 타입과 생성자 호출이 모두 새로운 클래스명에 맞게 적절히 변경되었습니다. 리팩토링의 일관성이 잘 유지되고 있습니다.
src/test/java/com/example/solidconnection/university/service/UnivApplyInfoRecommendServiceTest.java (3)
13-14: import문이 체계적으로 업데이트되었습니다새로운 DTO 클래스들(
UnivApplyInfoPreviewResponse,UnivApplyInfoRecommendsResponse)로의 import 변경이 올바르게 적용되었습니다.
28-31: 서비스 의존성이 일관되게 변경되었습니다다음과 같은 변경사항들이 체계적으로 적용되었습니다:
1. 테스트 클래스명 변경
2. 서비스 필드명 업데이트
3. 타입 참조 일관성 유지리팩토링의 목적에 완벽하게 부합하는 변경입니다.
84-94: 테스트 로직이 새로운 도메인 모델과 일치합니다테스트 메서드에서 새로운 응답 타입과 DTO를 사용하도록 올바르게 업데이트되었습니다. 테스트의 본질적 검증 로직은 유지되면서 타입만 일관되게 변경된 좋은 예시입니다.
src/test/java/com/example/solidconnection/university/service/GeneralUnivApplyInfoRecommendServiceTest.java (3)
14-14: 상수 import가 올바르게 업데이트되었습니다
RECOMMEND_UNIV_APPLY_INFO_NUM상수를 새로운 서비스 클래스에서 가져오도록 적절히 변경되었습니다.
20-23: 서비스 의존성이 체계적으로 변경되었습니다테스트 클래스명과 서비스 필드 모두 새로운 네이밍 규칙에 맞게 일관되게 업데이트되었습니다.
49-56: 테스트 로직이 새로운 도메인과 일치합니다테스트 메서드에서 새로운 서비스와 상수를 사용하도록 올바르게 변경되었습니다. 검증 로직의 본질은 유지되면서 새로운 타입 시스템에 맞게 적절히 적응되었습니다.
src/test/java/com/example/solidconnection/siteuser/service/MyPageServiceTest.java (3)
13-17: 도메인 엔티티 변경이 올바르게 적용되었습니다다음 변경사항들이 체계적으로 처리되었습니다:
1.LikedUnivApplyInfoRepository사용
2.LikedUnivApplyInfo도메인 엔티티 사용
3.UnivApplyInfoPreviewResponseDTO 사용새로운 도메인 모델과의 일관성이 잘 유지되고 있습니다.
195-204: 헬퍼 메서드가 새로운 도메인에 맞게 리팩토링되었습니다
createLikedUniversities에서createLikedUnivApplyInfos로 메서드명이 변경되고, 내부 로직도 새로운 엔티티(LikedUnivApplyInfo)와 레포지토리를 사용하도록 올바르게 업데이트되었습니다.도메인 변경에 따른 테스트 유틸리티의 적절한 적응입니다.
97-103: 테스트 메서드가 새로운 응답 타입을 사용합니다관심 대학교 목록 조회 테스트에서
UnivApplyInfoPreviewResponse타입을 사용하도록 적절히 변경되었습니다. 테스트의 검증 의도는 그대로 유지되면서 새로운 도메인 모델에 맞게 잘 적응되었습니다.src/main/java/com/example/solidconnection/university/dto/UnivApplyInfoPreviewResponse.java (1)
8-37: 리네이밍 변경사항이 깔끔하게 적용되었습니다
📝 클래스명 변경:
UniversityInfoForApplyPreviewResponse→UnivApplyInfoPreviewResponse
- 도메인 용어 통일에 맞게 정확히 변경됨
🏭 팩토리 메서드 업데이트: 반환 타입과 인스턴스 생성 부분이 일관되게 수정됨
- 메서드 시그니처와 구현부가 모두 올바르게 업데이트됨
✅ 로직 보존: 기존 비즈니스 로직과 필드 구조가 그대로 유지됨
LanguageRequirementResponse변환 및 정렬 로직이 정상적으로 보존됨변경사항이 일관성 있고 정확하게 적용되었습니다.
src/test/java/com/example/solidconnection/university/service/UnivApplyInfoQueryServiceTest.java (5)
7-21: 임포트 및 의존성 변경이 완벽하게 적용되었습니다
📦 DTO 클래스 임포트 업데이트:
UnivApplyInfoDetailResponse,UnivApplyInfoPreviewResponse,UnivApplyInfoPreviewResponses등 모든 DTO가 새 명칭으로 변경됨🏪 레포지토리 인터페이스 업데이트:
UnivApplyInfoRepository,UnivApplyInfoFilterRepository로 일관되게 변경됨🚨 에러 코드 상수 업데이트:
UNIV_APPLY_INFO_NOT_FOUND로 도메인 용어에 맞게 변경됨모든 의존성이 새로운 명명 규칙에 맞게 정확히 업데이트되었습니다.
29-38: 테스트 클래스와 의존성 주입이 올바르게 리팩토링되었습니다
🧪 테스트 클래스명 변경:
UnivApplyInfoQueryServiceTest로 명확하게 리네임됨💉 서비스 의존성 주입:
UnivApplyInfoQueryService타입으로 정확히 변경됨🔧 SpyBean 설정: 레포지토리들이 새 인터페이스 타입으로 올바르게 설정됨
테스트 인프라가 새 도메인 구조에 맞게 완벽하게 적응되었습니다.
52-82: 메서드 호출과 예외 처리가 정확하게 업데이트되었습니다
🔍 메서드 호출 변경:
getUnivApplyInfoDetail()메서드로 일관되게 변경됨- 캐시 검증 로직에서도 올바른 메서드명 사용됨
🎯 레포지토리 메서드 호출:
getUnivApplyInfoById()메서드로 정확히 업데이트됨
⚠️ 예외 처리 검증:
UNIV_APPLY_INFO_NOT_FOUND에러 코드로 올바르게 변경됨메서드 시그니처 변경과 예외 처리가 일관성 있게 적용되었습니다.
96-108: 검색 메서드와 응답 처리가 완벽하게 리팩토링되었습니다
🔎 검색 메서드 호출:
searchUnivApplyInfo()메서드로 명확하게 변경됨🎯 응답 타입 처리:
UnivApplyInfoPreviewResponses타입으로 정확히 업데이트됨univApplyInfoPreviews()필드 접근도 올바르게 변경됨📊 응답 변환 로직:
UnivApplyInfoPreviewResponse.from()팩토리 메서드 사용이 일관됨검색 기능의 모든 부분이 새 도메인 모델에 맞게 정확히 적응되었습니다.
122-131: 캐시 검증 테스트가 새 레포지토리 구조에 맞게 업데이트되었습니다
🔄 캐시 동작 검증:
searchUnivApplyInfo()메서드의 캐시 동작이 올바르게 테스트됨🏪 레포지토리 메서드 검증:
findAllByRegionCodeAndKeywordsAndLanguageTestTypeAndTestScoreAndTerm()메서드 호출이 정확히 검증됨캐시 기능의 테스트가 새 아키텍처에 맞게 완벽하게 유지되었습니다.
src/main/java/com/example/solidconnection/siteuser/service/MyPageService.java (3)
9-12: 임포트 및 의존성이 새 도메인 모델에 맞게 정확히 업데이트되었습니다
📦 레포지토리 임포트 변경:
LikedUnivApplyInfoRepository로 올바르게 업데이트됨🏗️ 엔티티 임포트 변경:
LikedUnivApplyInfo도메인 엔티티로 정확히 변경됨📋 DTO 임포트 변경:
UnivApplyInfoPreviewResponse로 일관되게 업데이트됨모든 임포트가 새로운 도메인 구조에 맞게 완벽하게 적응되었습니다.
34-43: 의존성 주입과 카운트 로직이 올바르게 리팩토링되었습니다
💉 레포지토리 의존성 주입:
LikedUnivApplyInfoRepository타입으로 정확히 변경됨📊 카운트 메서드 호출:
likedUnivApplyInfoRepository.countBySiteUser_Id()메서드 사용이 적절함- 변수명
likedUniversityCount는 기존 Response 필드와의 호환성을 위해 유지된 것으로 보임서비스 로직이 새 엔티티 구조에 맞게 완벽하게 적응되었습니다.
97-102: 위시리스트 조회 메서드가 완벽하게 리팩토링되었습니다
🔄 메서드명 변경:
getWishUnivApplyInfo()로 도메인 용어에 맞게 명확하게 변경됨📋 반환 타입 업데이트:
List<UnivApplyInfoPreviewResponse>로 정확히 변경됨🏪 레포지토리 호출:
likedUnivApplyInfoRepository.findAllBySiteUser_Id()메서드 사용이 적절함🔄 매핑 로직:
UnivApplyInfoPreviewResponse.from()팩토리 메서드를 통한 변환이 올바름위시리스트 기능이 새 도메인 모델에 맞게 완벽하게 업데이트되었습니다.
src/main/java/com/example/solidconnection/common/exception/ErrorCode.java (3)
36-37: 대학 지원 정보 관련 에러 코드가 일관되게 업데이트되었습니다
🏷️ 에러 코드 상수명 변경:
UNIV_APPLY_INFO_NOT_FOUND,UNIV_APPLY_INFO_NOT_FOUND_FOR_TERM으로 명확하게 변경됨✅ 메시지 및 상태 코드 보존:
- 기존 HTTP 상태 코드(404)와 한국어 메시지가 그대로 유지되어 호환성 보장됨
에러 처리 로직이 새 도메인 용어에 맞게 정확히 적응되었습니다.
74-74: 중복 선택 에러 코드가 도메인 용어에 맞게 업데이트되었습니다
🎯 에러 코드 상수명 변경:
DUPLICATE_UNIV_APPLY_INFO_CHOICE로 명확하게 변경됨✅ 기능 보존:
- 기존 HTTP 상태 코드(400)와 메시지가 유지되어 검증 로직의 연속성 보장됨
지망 선택 검증 기능이 새 도메인 구조에 맞게 완벽하게 적응되었습니다.
89-90: 좋아요 관련 에러 코드가 일관되게 리팩토링되었습니다
❤️ 좋아요 에러 코드 업데이트:
ALREADY_LIKED_UNIV_APPLY_INFO,NOT_LIKED_UNIV_APPLY_INFO로 명확하게 변경됨🔄 일관성 유지:
- 기존 HTTP 상태 코드(400)와 한국어 메시지가 보존되어 사용자 경험 연속성 보장됨
좋아요 기능의 에러 처리가 새 도메인 모델에 맞게 완벽하게 업데이트되었습니다.
src/main/java/com/example/solidconnection/university/repository/custom/UnivApplyInfoFilterRepository.java (1)
8-14: 레포지토리 인터페이스가 도메인 용어에 맞게 완벽하게 리팩토링되었습니다
🏷️ 인터페이스명 변경:
UnivApplyInfoFilterRepository로 도메인 용어 통일에 맞게 명확하게 변경됨📝 메서드명 개선:
findBy→findAllBy로 변경되어 복수 결과 반환을 명확히 표현함- 메서드 의도가 더욱 명확해졌습니다
✅ 시그니처 일관성:
- 파라미터 타입과 반환 타입(
List<UnivApplyInfo>)이 그대로 유지되어 구현체 호환성 보장됨레포지토리 계층이 새 도메인 구조에 맞게 완벽하게 적응되었습니다.
src/main/java/com/example/solidconnection/university/service/GeneralUnivApplyInfoRecommendService.java (1)
1-35: 리팩토링이 일관성 있게 잘 적용되었습니다!도메인 용어 통일화 작업이 체계적으로 이루어진 것을 확인할 수 있습니다:
1. 클래스명이 `GeneralUniversityRecommendService`에서 `GeneralUnivApplyInfoRecommendService`로 적절히 변경됨 2. 의존성 주입되는 레포지토리가 `UniversityInfoForApplyRepository`에서 `UnivApplyInfoRepository`로 업데이트됨 3. 상수 임포트가 `RECOMMEND_UNIVERSITY_NUM`에서 `RECOMMEND_UNIV_APPLY_INFO_NUM`으로 일치하게 수정됨 4. 필드명이 `recommendUniversities`에서 `generalRecommends`로 더 명확하게 변경됨 5. 레포지토리 메서드 호출이 새로운 네이밍에 맞게 조정됨전체적으로 네이밍 컨벤션이 통일되어 코드의 가독성과 일관성이 향상되었습니다.
src/test/java/com/example/solidconnection/university/repository/LikedUnivApplyInfoRepositoryTest.java (1)
1-88: 테스트 리팩토링이 완벽하게 수행되었습니다!도메인 모델 변경에 맞춰 테스트 코드가 정확히 업데이트되었습니다:
1. 테스트 클래스명이 `UniversityLikeRepositoryTest`에서 `LikedUnivApplyInfoRepositoryTest`로 변경됨 2. 레포지토리 의존성이 `LikedUniversityRepository`에서 `LikedUnivApplyInfoRepository`로 업데이트됨 3. 엔티티 참조가 모두 `LikedUniversity`에서 `LikedUnivApplyInfo`로 일관되게 수정됨 4. 픽스처 메서드 호출이 새로운 도메인 모델에 맞게 조정됨 5. 헬퍼 메서드 `createLikedUnivApplyInfo`의 네이밍과 구현이 적절하게 변경됨기존 테스트의 논리와 검증 시나리오는 그대로 유지되면서 새로운 도메인 모델에 맞게 완벽히 적응되었습니다.
src/main/java/com/example/solidconnection/university/service/UnivApplyInfoLikeService.java (1)
1-79: 새로운 서비스 클래스가 잘 설계되었습니다!
UnivApplyInfo도메인의 좋아요 기능을 담당하는 서비스가 체계적으로 구현되었습니다:1. **트랜잭션 관리**: 읽기 전용과 쓰기 트랜잭션이 적절히 분리되어 적용됨 2. **예외 처리**: 중복 좋아요 및 존재하지 않는 좋아요에 대한 커스텀 예외 처리가 정확함 3. **메서드 구성**: `like`, `cancelLike`, `getIsLiked` 세 가지 핵심 기능이 명확히 분리됨 4. **상수 관리**: 성공/취소 메시지가 상수로 잘 정의됨 5. **의존성 주입**: 필요한 레포지토리들이 적절히 주입됨코드 구조가 깔끔하고 단일 책임 원칙을 잘 따르고 있어 유지보수성이 좋습니다.
src/main/java/com/example/solidconnection/university/repository/UnivApplyInfoRepository.java (1)
1-86: 레포지토리 리팩토링이 체계적으로 완료되었습니다!인터페이스와 쿼리 구조가 새로운 도메인 모델에 맞게 완벽히 업데이트되었습니다:
1. **인터페이스 확장**: `UnivApplyInfoFilterRepository`를 확장하여 필터링 기능 통합 2. **메서드명 통일**: `getUniversityInfoForApplyById` → `getUnivApplyInfoById` 등 일관된 네이밍 3. **JPQL 별칭 정리**: `uifa`에서 `uai`로 간결하고 명확하게 변경 4. **에러 코드 업데이트**: `UNIV_APPLY_INFO_NOT_FOUND` 등 새로운 상수 적용 5. **쿼리 구조 개선**: JOIN FETCH를 활용한 N+1 문제 방지 쿼리 유지네이티브 쿼리의 테이블명이
university_info_for_apply로 되어있는데, 실제 테이블명과 일치하는지 확인해보시길 권합니다.다음 스크립트로 테이블명 일치 여부를 확인해보세요:
#!/bin/bash # 테이블명과 네이티브 쿼리 일치성 검증 rg -A 5 -B 5 "university_info_for_apply" --type sql rg -A 5 -B 5 "@Table.*university" --type javasrc/test/java/com/example/solidconnection/university/fixture/LanguageRequirementFixture.java (1)
1-54: 픽스처 클래스의 네이밍이 일관성 있게 업데이트되었습니다!테스트 데이터 생성을 위한 픽스처 메서드들이 새로운 도메인 모델에 맞게 정확히 수정되었습니다:
1. **매개변수명 통일**: 모든 메서드에서 `universityInfo` → `univApplyInfo`로 일관되게 변경 2. **빌더 메서드 호출**: `.universityInfoForApply()` → `.univApplyInfo()`로 적절히 업데이트 3. **메서드 시그니처 일관성**: 5개 메서드 모두 동일한 패턴으로 변경되어 일관성 확보 4. **기능 보존**: 기존 로직은 그대로 유지하면서 네이밍만 개선테스트 픽스처의 일관성이 향상되어 테스트 코드의 가독성과 유지보수성이 좋아졌습니다.
src/test/java/com/example/solidconnection/university/service/UnivApplyInfoLikeServiceTest.java (1)
1-161: 도메인 리네이밍이 체계적으로 잘 적용되었습니다.
- 서비스 클래스 의존성 업데이트 완료
- 레포지토리 의존성 변경 완료
- 도메인 엔티티 및 DTO 업데이트 완료
- 에러 코드 상수 일관되게 변경 완료
- 테스트 메서드들의 로직이 올바르게 유지됨
테스트 커버리지와 기능적 검증이 새로운 도메인 모델에 맞게 잘 적용되었습니다.
src/main/java/com/example/solidconnection/university/repository/custom/UnivApplyInfoFilterRepositoryImpl.java (2)
31-50: 메서드명 변경이 적절하게 적용되었습니다.
findByRegionCodeAndKeywords에서findAllByRegionCodeAndKeywords로의 메서드명 변경이 Spring Data JPA 규칙에 더 부합하며, 복수 결과를 반환함을 명확히 나타냅니다.
77-107: 복잡한 필터링 로직이 안전하게 보존되었습니다.
- 메서드명이
findAllBy...로 일관되게 변경됨- QueryDSL 쿼리 로직이 그대로 유지되어 기능적 안정성 확보
- 변수명들이 새로운 명명 규칙에 맞게 적절히 업데이트됨
- 언어 시험 점수 비교 로직이 올바르게 작동함
src/main/java/com/example/solidconnection/university/dto/validation/ValidUnivApplyInfoChoiceValidator.java (2)
12-12: 에러 코드 상수가 새로운 도메인 명명 규칙에 맞게 정확히 업데이트되었습니다.
DUPLICATE_UNIVERSITY_CHOICE에서DUPLICATE_UNIV_APPLY_INFO_CHOICE로의 변경이 일관성 있게 적용되었습니다.
16-61: 검증 로직이 새로운 DTO와 완벽하게 호환됩니다.
- 제네릭 타입이
UnivApplyInfoChoiceRequest로 적절히 변경됨- 첫 번째 선택 필수 검증 유지
- 세 번째 선택 시 두 번째 선택 필수 검증 유지
- 중복 선택 방지 로직 정확히 작동
- 모든 검증 메서드가 새로운 DTO 구조와 호환됨
비즈니스 로직의 무결성이 리네이밍 과정에서 완벽하게 보존되었습니다.
src/main/java/com/example/solidconnection/university/service/UnivApplyInfoRecommendService.java (1)
20-73: 추천 서비스 리팩토링이 체계적으로 완료되었습니다.
- 클래스명과 상수명이 일관되게 변경됨
- 레포지토리 의존성 정확히 업데이트됨
- 메서드명이 명확하고 일관되게 변경됨
- DTO 반환 타입이 적절히 업데이트됨
- 비즈니스 로직과 추천 알고리즘이 그대로 보존됨
- 트랜잭션 및 캐싱 설정이 유지됨
도메인 모델 변경이 서비스 계층에 안전하게 적용되었습니다.
src/main/java/com/example/solidconnection/university/service/UnivApplyInfoQueryService.java (2)
34-40: 캐시 키가 새로운 명명 규칙에 맞게 적절히 업데이트되었습니다.
- 캐시 키가
"univApplyInfo:{0}"로 일관되게 변경됨- 메서드명이
getUnivApplyInfoDetail로 명확하게 변경됨- 레포지토리 메서드 호출이 정확히 업데이트됨
51-60: 검색 기능의 캐시 키와 메서드가 체계적으로 업데이트되었습니다.
- 캐시 키가 다중 파라미터를 포함하여 정확히 구성됨
- 메서드명이
searchUnivApplyInfo로 명확하게 변경됨- 레포지토리 메서드 호출이
findAllBy...로 일관되게 업데이트됨- DTO 매핑이 새로운 응답 타입에 맞게 적절히 변경됨
검색 기능의 캐싱 전략과 비즈니스 로직이 안전하게 보존되었습니다.
src/main/java/com/example/solidconnection/university/controller/UnivApplyInfoController.java (2)
27-36: 컨트롤러 구조가 깔끔하게 설계되었습니다!의존성 주입과 Spring Boot 어노테이션이 적절히 사용되었고, 서비스 계층 분리도 잘 되어 있습니다.
37-46: 5. 인증 처리 로직이 깔끔합니다
@AuthorizedUser(required = false)를 사용해서 선택적 인증을 처리하는 방식이 좋습니다. 비로그인 사용자에게는 일반 추천을, 로그인 사용자에게는 개인화 추천을 제공하는 로직이 명확합니다.
src/main/java/com/example/solidconnection/application/dto/UnivApplyInfoChoiceRequest.java
Outdated
Show resolved
Hide resolved
src/main/java/com/example/solidconnection/university/controller/UnivApplyInfoController.java
Show resolved
Hide resolved
src/main/java/com/example/solidconnection/university/controller/UnivApplyInfoController.java
Show resolved
Hide resolved
- University와 UniversityInformationForApply의 혼용으로 발생한 문제가 있었다. - UnivApplyInfo 를 적용하며 발견했고, 바로잡았다.
- LikedUniversity -> LikedUnivApplyInfo
- siteuser 하위 -> university 하위
- 프론트와 협의가 되어야 하는 요청/응답의 json에 해당하는 이름들을 백엔드가 임의로 바꾸는 것은 바람직하지 않다. - issue를 만들고, todo로 표시하고 넘어간다. - 프론트와 협의 이후, 별도의 PR에서 수정한다.
- Application 도메인의 의미상, '대학에 지원한다'는 내용이 포함되어있다. - 코드 전체적으로 위 내용의 합의되어있기도 하다. (application 앞에 University를 붙이지 않음) - 따라서 University나 UnivApplyInfo이 없는 이름으로 변경한다.
nayonsoso
force-pushed
the
refactor/344-clarify-university-terminology
branch
from
7d11843 to
0b2bfa1
Compare
- JsonProperties로 Json key는 유지하되, 필드와 인자명은 코드 전체에서 통일되게한다.
lsy1307
left a comment
lsy1307
left a comment
There was a problem hiding this comment.
이 많은 걸 하나하나 다 고치신게 진짜 대단하네용 고생하셨어요!
There was a problem hiding this comment.
고생하셨습니다 ~ 확실히 University 와 UnivApplyInfo 간 구분이 명확해진 거 같습니다. 좋은 변경이라고 생각합니다.
워낙 UniversityInfoForApply 가 많았어서 그런지, 제가 추가로 변경되어야 하는 부분을 찾았습니다.
ApplicationQueryService
74, 76, 80, 88, 90, 92, 113, 116번 째 라인:universityInfosForApply->univApplyInfosUnivApplyInfoQueryServiceTest
75, 79번 째 라인:invalidUniversityInfoForApplyId->invalidUnivApplyInfoId
이 부분만 수정하면 완벽한 것 같습니다!!
|
wish -> like 로 바꾸는 것 동의합니다! |
|
@whqtker 꼼꼼한 확인 감사드립니다. 😇 |
Gyuhyeok99
left a comment
Gyuhyeok99
left a comment
There was a problem hiding this comment.
확인했습니다!! 다음부턴 빨리 확인하도록 하겠습니다 🥹
src/main/java/com/example/solidconnection/application/dto/ApplyRequest.java
Show resolved
Hide resolved
| @Transactional(readOnly = true) | ||
| public MyPageResponse getMyPageInfo(SiteUser siteUser) { | ||
| int likedUniversityCount = likedUniversityRepository.countBySiteUser_Id(siteUser.getId()); | ||
| int likedUniversityCount = likedUnivApplyInfoRepository.countBySiteUser_Id(siteUser.getId()); |
There was a problem hiding this comment.
이 지역변수도 likedUnivApplyInfoCount가 되어야할까요?
There was a problem hiding this comment.
네 그게 좋겠습니다😁 꼼꼼한 검토 감사드립니다~
src/main/java/com/example/solidconnection/university/controller/UnivApplyInfoController.java
Show resolved
Hide resolved
|
|
||
| @Override | ||
| public List<UnivApplyInfo> findByRegionCodeAndKeywordsAndLanguageTestTypeAndTestScoreAndTerm( | ||
| public List<UnivApplyInfo> findAllByRegionCodeAndKeywordsAndLanguageTestTypeAndTestScoreAndTerm( |
There was a problem hiding this comment.
영서님은 JPQL이나 QueryDSL을 쓰더라도 함수명을 JPA 표준에 맞추는 것을 선호하시나요? 개인적인 궁금함인데 함수명이 너무 길어지는 경우에도 이걸 지켜야하나 생각이 드네요 ㅎㅎ
There was a problem hiding this comment.
우선, 이런 질문 정말 좋네요! 질문해주셔서 감사합니다. 🤩
구성원들이 생각을 묻는 질문을 주고받으면서 "두뇌 싱크"나 "팀워크"가 많이 좋아지는 것 같아요.
영서님은 JPQL이나 QueryDSL을 쓰더라도 함수명을 JPA 표준에 맞추는 것을 선호하시나요?
네 저는 Jpa가 자바 표준 ORM이라는 점에서 따르는게 좋다 생각해요.
그리고 Jpql은 JPA에서 제공하는 기능이고,
QueryDsl도 여러 프레임워크의 버전을 지원하는 것 같지만 우리 프로젝트에서 사용하는 것은 결국 jpa 버전이거든요.
표준을 따른다는 것은, 개발자들이 이미 이해하고 있는 것을 사용한다는 것이므로
동작 예측이 가능하고, 코드를 이해가 쉬워집니다.
그래서 되도록 JPA 표준 함수이름을 따르는게 좋다 생각합니다.
예를 들어, 위의 변경에서는 List<엔티티>를 반환하니 findBy → findAllBy로 바꾼 것이고요!
함수명이 너무 길어지는 경우에도 이걸 지켜야하나 생각이 드네요 ㅎㅎ
그런데 위와 같이 너무 함수명이 길어지면 조금의 융통성을 발휘해도 좋다 생각해요😅
원칙을 지켰을 때의 이점 : (표준을 따르기에 이해가 쉽고 오해의 여지 x)보다
단점 : (한번에 파악하기 어려움)이 더 큰 것 같더라고요.
There was a problem hiding this comment.
표준을 따른다는 것은, 개발자들이 이미 이해하고 있는 것을 사용한다는 것이므로
동작 예측이 가능하고, 코드를 이해가 쉬워집니다.
그래서 되도록 JPA 표준 함수이름을 따르는게 좋다 생각합니다.
저도 이부분에 대해서는 완전 이해되고 좋다고 생각합니다 ㅎㅎ
그런데 위와 같이 너무 함수명이 길어지면 조금의 융통성을 발휘해도 좋다 생각해요😅
이거에 대해서는 고민이 조금 필요하겠네요!
잘 정리해서 말해주셔서 감사합니다!
| import static com.example.solidconnection.common.exception.ErrorCode.UNIVERSITY_INFO_FOR_APPLY_NOT_FOUND; | ||
| import static com.example.solidconnection.common.exception.ErrorCode.UNIV_APPLY_INFO_NOT_FOUND; | ||
|
|
||
| @Repository |
There was a problem hiding this comment.
이건 그냥 이 pr에서 지우는 건 어떤가요?
There was a problem hiding this comment.
프로젝트 전체에 @Repository가 사용된게 많더라고요!
이건 따로 PR을 만드는게 좋을 것 같아요.
There was a problem hiding this comment.
엇 그러네요! 제가 한 번 작업하겠습니다!
src/main/java/com/example/solidconnection/university/repository/UniversityRepository.java
Show resolved
Hide resolved
| void 마이페이지_정보를_조회한다() { | ||
| // given | ||
| int likedUniversityCount = createLikedUniversities(user); | ||
| int likedUniversityCount = createLikedUnivApplyInfos(user); |
There was a problem hiding this comment.
여기변수명도요! 혹시 제가 잘못생각했나요 ㅎㅎ..
There was a problem hiding this comment.
저도 사소한 변수명도 바꾸는게 좋다 생각해요 ㅎㅎ
나중에 오해를 일으킬 수 있으니까요!
MyPageService 와 MyPageServiceTest에 사용되었던 likedUniversity를 모두 likedUnivApplyInfo로 바꿨습니다 🫡
f781751
| Optional<UnivApplyInfo> findByIdAndTerm(Long id, String term); | ||
|
|
||
| Optional<UnivApplyInfo> findFirstByKoreanNameAndTerm(String koreanName, String term); | ||
|
|
||
| @Query(""" | ||
| SELECT uifa | ||
| FROM UnivApplyInfo uifa | ||
| WHERE uifa.university IN :universities | ||
| AND uifa.term = :term | ||
| SELECT uai | ||
| FROM UnivApplyInfo uai | ||
| WHERE uai.university IN :universities | ||
| AND uai.term = :term | ||
| """) | ||
| List<UnivApplyInfo> findByUniversitiesAndTerm(@Param("universities") List<University> universities, @Param("term") String term); | ||
|
|
There was a problem hiding this comment.
제 인텔리제이가 잠시 이상해진건지 모르겠는데 이거 사용하지 않는 것이라면 지우는 거 어떤가요?
4 participants
관련 이슈
작업 내용
회의에서 이야기한대로, universityInformationForApply를 univApplyInfo로 변경합니다.
특이 사항
1/ 처음에는 ✌️어제 회의✌️에서만 이야기된
/univsersity->/univ-apply-infosURI 변경만 적용하려 했는데요.하나의 PR로 일괄 처리하는것도 괜찮겠다 싶어 이렇게 작업했습니다~
2/ 3dae69a 의 커밋 메세지에도 적었다시피, Dto 이름 변경할 때 요청/응답에 직접적으로 사용되는 이름들은 바꾸지 않았습니다.
이 내용은 따로 이슈 파두었습니다!
→ 수정됨 / 디스코드에서 논의한 내용에 따라, JSON Key는
@JsonProperty로 유지하고 java 코드에서의 이름은 바꿨습니다!리뷰 참고사항
파일 이름 변경을 github에서는 "이전 이름의 파일이 지워지고 새로운 이름의 파일이 생성됨"으로 인식하더라고요?
그래서 커밋 따라가셔도 읽기가 많이 어려우실 것 같아요😭
하시면 더 편하실 수 있습니다.
저는 인텔리제이랑 github PR 연동해서 확인하면 코드 읽기가 더 편하더라고요. 참고하세요~