refactor: GonghakDao

[gmltn9233]

🔧연결된 이슈

• refactor: 결과, 추천 로직 변경 #57

🛠️작업 내용

• GonghakDao의 findStandard 메서드를 최신년도에 해당하는 학과 기준을 불러오도록 변경하고 연관된 사항들을 변경사항에 맞게 변경하였습니다.

🤷‍♂️PR이 필요한 이유

• 기존의 '사용자의 입학년도' 를 기준으로 인증 기준을 가져오는 잘못된 방식에서 '최신년도' 기준으로 인증 기준을 가져오는 방식으로 변경하였습니다.
• '기준 학점' 외 '세부 사항' 들은 사용자의 입학년도를 기준삼는것이 맞아서 기존의 추천로직은 따로 수정하지 않았습니다.

✔️PR 체크리스트

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

[whxogus215]

리팩토링한 코드 확인했습니다 👍 공학인증 기준을 가져올 때, 최신년도를 기준으로 가져올 경우, 문제가 발생하는 학과는 없는건가요? 현재 데이터 사이언스학과의 24년도 자료가 없는걸로 알고 있는데 이 경우에도 로직상에 문제가 없는건지 궁금합니다.

[whxogus215]

Standard를 가져오는 기준이 최신년도 기준이군요! YEAR라는 것이 최신년도라는 것을 강조하기 위해 LATEST_YEAR 같이 부가적인 내용을 추가하면 의미가 더 잘 전달될 것 같습니다.

[gmltn9233]

Standard를 가져오는 기준이 최신년도 기준이군요! YEAR라는 것이 최신년도라는 것을 강조하기 위해 LATEST_YEAR 같이 부가적인 내용을 추가하면 의미가 더 잘 전달될 것 같습니다.

저도 해당 부분을 고려했는데, LATEST_YEAR 가 좀 더 직관적인것 같네요. 수정해서 반영했습니다!

[whxogus215]

findLatestStandard라는 뜻은 최신년도의 공학인증 기준 데이터를 가져온다는 의미로 작성한건가요?? 아니면 추후에 findLatestStandard라는 메서드가 추가될 수 있는건가요?

[gmltn9233]

findLatestStandard라는 뜻은 최신년도의 공학인증 기준 데이터를 가져온다는 의미로 작성한건가요?? 아니면 추후에 findLatestStandard라는 메서드가 추가될 수 있는건가요?

제일 처음 findStandard 메서드를 남겨둔 채로 findLatestStandard 메서드를 추가했는데, 리팩토링하다보니 findStandard 메서드를 사용하는 경우가 없어, findLatestStandard 메서드를 findStandard 이름으로 바꿨습니다.
이 과정에서 주석이 최신화가 안된것 같네요 ㅠㅠ 예리한 지적 감사합니다👍

[gmltn9233]

리팩토링한 코드 확인했습니다 👍 공학인증 기준을 가져올 때, 최신년도를 기준으로 가져올 경우, 문제가 발생하는 학과는 없는건가요? 현재 데이터 사이언스학과의 24년도 자료가 없는걸로 알고 있는데 이 경우에도 로직상에 문제가 없는건지 궁금합니다.

findStandard 메서드는 abeek 테이블에 저장된 해당 학과의 최신년도 기준을 불러오는 메서드입니다. 그래서 운영 DB에서는 abeek 테이블에 모든 학과 정보가 존재하므로 로직상 문제가 발생하진 않습니다.

하지만 로컬에서는 현재 initData에서 수동으로 abeek 테이블에 데이터를 넣기 때문에, 로컬에서는 현재까지는 컴공, 전전통 학과만 테스트 가능합니다.

해당 이슈 추가하였으니 참고 부탁드립니다!
#61

[whxogus215]

실수로 잘못 눌렀는데 닫혀버려서 다시 열었습니다...ㅠ 코멘트 확인했습니다!