refactor: StatusController 메서드 분리

[whxogus215]

문제 정의

현재 StatusController의 sendStudentId()의 경우, 다음과 같은 로직을 수행하고 있습니다.

• 인증된 사용자의 학번으로 UserDomain 객체를 불러온다.
• 인증 현황 데이터를 불러온다.
• 인증 추천 과목을 불러온다.

하지만 sendStudentId라는 메서드 이름으로 위에서 언급한 세 가지 로직을 잘 표현하고 있는가?에 대해서 의문이 들었습니다. 따라서 하나의 메서드가 한 가지 일만 할 수 있도록 추상화하는 작업을 하였습니다.

변경한 내용

• 기존 sendStudentId() 에서 readGonghakStatusResult()으로 변경하였습니다. 단순히 학번을 전달하는 것보다 최종적으로 사용자의 인증 현황과 추천과목을 불러오는 것이 주된 로직이라고 판단하였고, 이들을 GonghakStatusResult로 표현하였습니다.
• 인증 현황 데이터를 불러오는 로직을 readUserResultRatio()로 분리시켰습니다.
• 인증 추천 과목을 불러오는 로직을 readUserRecommendCourses()로 분리시켰습니다.

[jiseop121]

아마 제가 작성한 컨트롤러 같긴 한데 오래돼서 기억은 안나네요 확실히 getmapping의 특징과 로직의 목적에 둘 다 부합하는 readGonghakStatusResult 이름으로 수정한 것이 좋은 것 같습니다. 그리고 추가 메서드 분리하신 것도 더 좋은 것 같습니다.