[BE] Repository에서 Optional로 반환하는 메서드를 개선한다.

[seokjin8678]

✨ 세부 내용

대부분의 로직에서 Repository로 엔티티를 찾는 메서드에 orElseThrow 메서드를 사용하고 있습니다.
하지만 공용으로 자주 쓰이는 Repository에서(MemberRepository) 해당 로직이 중복되는 경우가 많기 때문에 default 메서드를 사용하여 중복을 개선합니다.

public interface MemberRepository extends JpaRepository<Member, Long> {

    default Member getOrThrow(Long id) {
        return findById(id)
            .orElseThrow(() -> new NotFoundException(ErrorCode.MEMBER_NOT_FOUND));
    }
}

이때 해당 메서드의 네이밍을 무엇을 하는게 좋을지 논의가 필요합니다.

⏰ 예상 소요 시간

1시간

[carsago]

getByID는 어떤가요? 전부 다 한번에 바꿀 수도 있고 하나씩 바꾸어가는 것도 좋을 것 같네요.

[seokjin8678]

getByID는 어떤가요? 전부 다 한번에 바꿀 수도 있고 하나씩 바꾸어가는 것도 좋을 것 같네요.

getById라는 메서드는 이미 JpaRepository에 구현 메소드로 정의가 되어 있더군요.
게다가 @Deprecated 처리까지 된 상태라서리...

그리고 밑의 링크는 제이슨이 올리신 이슈 내용입니다.
woowacourse/service-apply#565 (comment)
결론으로 getOrThrow 라는 네이밍을 사용하는 것 같네요.

저희 프로젝트에서 코틀린을 사용하지 않으니, get 이라는 메서드 이름을 사용해도 괜찮을 것 같긴 한데...
명시적으로 하려면 getOrThrow가 명확한 것 같네요.

[carsago]

getOrThrow가 더 명시적이네요. 어떠한 명칭이든 우리가 이해하기만 하면 괜찮다고 생각합니다.

---

11월 20일 오후 6시 30분 추가

이렇게 하는 것을 컨벤션으로 모두 동의한다면 PR에서 건드리는 클래스들에 겸사겸사 한두개 같이 수정해도 좋겠네요.

[seokjin8678]

close #648