feat: 이수 구분 테이블 general_education_area_id 추가

[Soundbar91]

🔥 연관 이슈

• close 졸업학점 계산기 API 구현 #1140

🚀 작업 내용

• 이수 구분 테이블에 general_education_area_id를 추가했습니다.
• timetableLecture 생성 시 이수구분 생성되는 로직을 변경했습니다.

💬 리뷰 중점사항

[duehee]

간단한 거 하나만 확인해주세요~
고생하셨습니당 :D

[duehee]

A

오.. 세심하셔라

[duehee]

C

이거 추가 된 이유가 있나요?

[Soundbar91]

학기 강의 개설 목록 구현할 때, 조회되는 강의의 이수구분을 유저의 기준으로 할려고 했습니다.
그래서 userId를 가져오기 위해서 어노테이션을 사용했다가, 지우는 과정에서 못 지웠네요,,
감사합니답

[seongjae6751]

고생 많으셨습니다!
간단한거 몇가지만 확인하고 머지해주세요!

[seongjae6751]

C

private CourseType getCourseType(Lecture lecture, Integer userId) {
    Student student = studentRepository.getById(userId);
    Integer studentNumberYear = StudentUtil.parseStudentNumberYear(student.getStudentNumber());

    return catalogRepository.findByLectureNameAndYear(lecture.getName(), String.valueOf(studentNumberYear))
        .stream()
        .findFirst()
        .map(Catalog::getCourseType)
        .orElseGet(() -> findCourseTypeByYearAndCode(lecture));
}

private CourseType findCourseTypeByYearAndCode(Lecture lecture) {
    final int currentYear = LocalDateTime.now().getYear();
    for (int initStudentNumberYear = 2019; initStudentNumberYear <= currentYear; initStudentNumberYear++) {
        Optional<CourseType> courseType = catalogRepository.findByYearAndCode(
            String.valueOf(initStudentNumberYear), lecture.getCode())
            .stream()
            .findFirst()
            .map(Catalog::getCourseType);

        if (courseType.isPresent()) {
            return courseType.get();
        }
    }
    return courseTypeRepository.getByName("이수구분선택");
}

이런식으로 stream을 사용해서 조금 더 깔끔하게 쓸 수 있을 것 같아요~

[seongjae6751]

R

for (int initStudentNumberYear = 2019; initStudentNumberYear <= currentYear; initStudentNumberYear++) {
    catalogs = catalogRepository.findByYearAndCode(String.valueOf(initStudentNumberYear), lecture.getCode());

    if (!catalogs.isEmpty()) {
        return catalogs.get(0).getCourseType();
    }
}

findByYearAndCode()가 빈 리스트를 반환한다면 null 체크가 불필요해서 isEmpty를 검사하는게 좀 더 맞을 것 같아요!

[seongjae6751]

C
2019는 상수 처리 해도 좋을 것 같네요!

[seongjae6751]

C
이수구분선택도 상수 처리 할 수 있을 것 같네요..!

private static final String DEFAULT_COURSE_TYPE = "이수구분선택";

...

return courseTypeRepository.getByName(DEFAULT_COURSE_TYPE);