#334 냉장고 추가 및 정보 수정 API 리팩토링 #343

psyeon1120 · 2024-07-30T14:02:49Z

📍 이슈 번호

[refactor] Fridge 클린 코드 적용 및 성능 개선 #334

🛠️ 작업 내용

냉장고 추가 API 리팩토링
냉장고 정보 수정 API 리팩토링
validation 추가

DTO 클래스의 스웨거 작성 요소를 조금 바꿔봤습니다.
예시를 추가하고, 변수 이름은 중복되길래 지웠습니다. 클래스 이름은 한국어로 지정했어요.

IceButler_Server/src/main/java/com/example/icebutler_server/fridge/dto/request/AddFridgeReq.java

Lines 14 to 23 in 14f6800

    
           @Schema(name = "냉장고 추가 요청 정보") 
        
           public class AddFridgeReq { 
        
             @NotBlank 
        
             @Schema(description = "냉장고 이름", example = "우리집 냉장고") 
        
             private String fridgeName; 
        
             @Schema(description = "냉장고 설명", example = "우리집 냉장고입니다~") 
        
             private String fridgeComment; 
        
             @Schema(description = "냉장고 멤버 ID 리스트", example = "[ 1, 2 ]") 
        
             private List<Long> members; 
        
           }

🎸 기타 사항

냉장고 정보 수정에서 멤버 수정하는 로직은 도저히 이것보다 더 나은 방법이 생각이 안나는데, 혹시 좋은 방법 있다면 디엠으로 공유 부탁드려요 🙏
아직 냉장고 API는 많이 남았기 때문에 브랜치는 삭제하지 말아주세용

sojungpp

복잡한 냉장고 처리하느라 고생했습니답
네이밍이랑 함수 세세하게 만들어줘서 바로 잘 이해할 수 있었어용 체고야.

sojungpp · 2024-07-30T14:26:50Z

src/main/java/com/example/icebutler_server/fridge/entity/FridgeUser.java

+    public void changeRoleToOwner(){
        this.role = FridgeRole.OWNER;
    }
-    public void changeFridgeMember(User user){
+
+    public void changeRoleToMember(){
        this.role = FridgeRole.MEMBER;
    }


네이밍 깔쌈하네욥.

src/main/java/com/example/icebutler_server/fridge/service/FridgeServiceImpl.java

sojungpp · 2024-07-30T14:35:10Z

src/main/java/com/example/icebutler_server/fridge/service/FridgeServiceImpl.java

+    private void updateFridgeUsers(EditFridgeReq editFridgeReq, Fridge fridge) {
+        List<User> currentMembers = this.fridgeUserRepository.findByFridgeAndRoleAndIsEnable(fridge, FridgeRole.MEMBER, true).stream()
+                .map(FridgeUser::getUser).collect(Collectors.toList());


psyeon1120 added 3 commits July 29, 2024 23:26

#334 refactor: 냉장고 추가 API 리팩토링

3939fac

#334 refactor: valid exception 시 ReturnCode.INVALID_PARAM로 고정

404b68a

#334 refactor: 냉장고 정보 수정 API 리팩토링

14f6800

psyeon1120 added the ♻️ refactor 리팩토링 label Jul 30, 2024

psyeon1120 self-assigned this Jul 30, 2024

psyeon1120 requested a review from sojungpp as a code owner July 30, 2024 14:02

sojungpp approved these changes Jul 30, 2024

View reviewed changes

#334 refactor: FridgeUser toEntity 메소드 추가

10ed2c0

psyeon1120 merged commit d5f666b into develop Jul 30, 2024
1 check failed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

#334 냉장고 추가 및 정보 수정 API 리팩토링 #343

#334 냉장고 추가 및 정보 수정 API 리팩토링 #343

psyeon1120 commented Jul 30, 2024 •

edited

Loading

sojungpp left a comment

sojungpp Jul 30, 2024

sojungpp Jul 30, 2024

	@Schema(name = "냉장고 추가 요청 정보")
	public class AddFridgeReq {
	@NotBlank
	@Schema(description = "냉장고 이름", example = "우리집 냉장고")
	private String fridgeName;
	@Schema(description = "냉장고 설명", example = "우리집 냉장고입니다~")
	private String fridgeComment;
	@Schema(description = "냉장고 멤버 ID 리스트", example = "[ 1, 2 ]")
	private List<Long> members;
	}

#334 냉장고 추가 및 정보 수정 API 리팩토링 #343

#334 냉장고 추가 및 정보 수정 API 리팩토링 #343

Conversation

psyeon1120 commented Jul 30, 2024 • edited Loading

📍 이슈 번호

🛠️ 작업 내용

🎸 기타 사항

sojungpp left a comment

Choose a reason for hiding this comment

sojungpp Jul 30, 2024

Choose a reason for hiding this comment

sojungpp Jul 30, 2024

Choose a reason for hiding this comment

psyeon1120 commented Jul 30, 2024 •

edited

Loading