refactor: init file data

[gmltn9233]

🔧연결된 이슈

• closed

🛠️작업 내용

• 1. enum MajorName 소프트웨어학과, 데이터사이언스학과 추가
• 2. mapToGonghakCoursesDomain 메서드 리팩토링

🤷‍♂️PR이 필요한 이유

1. enum MajorName

2학과가 새로 추가됨에 따라 추가하였습니다.

2. mapToGonghakCoursesDomain 메서드 리팩토링

존재하지 않는 강의

위는 소프트웨어학과의 17년도 교과과정표입니다. 하지만 이중 고급실시간그래픽스 ,디지털사운드 등은 실제 교과가 개설되지 않아 이를 처리할 로직이 필요했습니다.

CoursesDomain courseDomain = coursesDao.findByName(data[6].replaceAll("\\s+", ""));
if (courseDomain == null) {
    // Return an empty Optional if courseDomain is null
    return Optional.empty();
}

Optional<GonghakCoursesDomain> course = mapToGonghakCoursesDomain(data);
if (course.isPresent()) {
    // Save to repository only if the Optional contains a value
    gonghakCorusesDao.save(course.get());
    }

원래는 GongakCoursesDomain을 return하는 mapToGonghakCoursesDomain 메서드를 Optional 객체를 return 하도록 리팩토링하여, 해당 객체가 empty면 저장하지 않고, 이후 불필요한 로직도 진행하지 않게 하였습니다.

데이터 포맷팅

아래는 소프트웨어학과의 데이터형식이고,

아래는 전자정보통신공학과의 데이터 형식입니다.

• 교과구분

학과별로 교과구분 항목이 상이하여 csv 파일들을 살펴보며 가능한 모든 경우의 수를 추스려서 전문교양, BSM, 전공 ,MSC 4가지 항목으로 통일시켰습니다.

switch (courseCategory) {
    case "중핵필수", "교양필수", "교양선택", "교양선택I", "교양":
        courseCategory = "전문교양";
        break;
    case "전공기초교양", "학문기초교양":
        courseCategory = "BSM";
        break;
    case "전공필수", "전공선택", "전공(설계)", "전공주제", "전공기초":
        courseCategory = "전공";
        break;
    default:
        // 기본적으로는 변경하지 않고 원래 값을 유지
        break;
}

• 인증구분

인증구분 데이터도 인선, 인필 형태로 저장되도록 substring을 적용하였습니다.

passCategory(data[5].substring(0, 2))

• 교과목명 불일치

시간표상으로는 "대학생을위한진로설계" 처럼 띄어쓰기가 주로 없으며, 공학인증 홈페이지상으로는 "대학생을 위한 진로설계" 처럼 띄어쓰기가 적용되어있어 학과목을 검색하는데 에러가 발생했습니다.

시간표상으로는 가끔 띄어쓰기가 존재하는 항목도 있고 불규칙적이라, 모든 교과목에 띄어쓰기를 없애도록 통일하였습니다.

CoursesDomain courseDomain = coursesDao.findByName(data[6].replaceAll("\\s+", ""));

✔️PR 체크리스트

• 필요한 테스트를 작성했는가?
• 다른 코드를 깨뜨리지 않았는가?
• 연결된 이슈 외에 다른 이슈를 해결한 코드가 담겨있는가?

[whxogus215]

코드 로직상 문제는 없는 것 같습니다. 수고했습니다 👍

[whxogus215]

해당 주석을 달아둔 의도가 있을까요? 코드 이해를 돕기 위함인거라면 지워도 될 것 같습니다!

[gmltn9233]

해당 주석을 달아둔 의도가 있을까요? 코드 이해를 돕기 위함인거라면 지워도 될 것 같습니다!

불필요한 주석 삭제한 후 커밋하였습니다. 의견 감사합니다~👍