Mapper 메서드 파라미터 컨벤션 정하기

[KIMSEI1124]

💢 문제 상황

• 일정 생성 API 작성 #35

프로젝트를 진행하던 중 Mapper의 매개변수를 수정한 상황에 대해서 궁금한 점이 생겨서 질문을 하였습니다.

@jiwon83 “scheduleInfo 엔티티 를 생성하는 데 필요한 필드가 creatorId 1개 뿐이어서 수정했었습니다!
scheduleInfo 에 필드가 추가되면 관련 코드를 변경해야하는 문제가 있을 수 있기는 합니다. 해당 부분 논의해 보면 좋을 것 같습니다 : )”

위와 같은 답변을 받아 회의를 진행하기로 하였습니다.

💦 해결 과정

팀원들과 Mapper 함수의 매개변수에 대한 컨벤션을 정리하였습니다.

우선, 연관된 엔티티는 인터페이스에 직접 넣지 않고 따로 매개변수로 전달하기로 결정하였습니다. 이는 엔티티는 수정될 가능성이 적다고 판단했기 때문입니다.

반면, 그 외의 값들은 필드 변경 가능성이 있으므로 인터페이스로 처리하기로 하였습니다.

✅ 결과

public static ScheduleInfo toJpaScheduleInfo(ConvertibleScheduleInfo convertibleScheduleInfo, Calendar calendar) {
   return ScheduleInfo.builder()
           .creatorId(convertibleScheduleInfo.getCreatorId())
           .calendar(calendar)
           .build();
}

와 같은 형식으로 진행하기로 하였습니다!