Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Semester 클래스의 명칭 변경 #846

Open
zealotzealot opened this issue Sep 30, 2021 · 0 comments
Open

Semester 클래스의 명칭 변경 #846

zealotzealot opened this issue Sep 30, 2021 · 0 comments

Comments

@zealotzealot
Copy link
Member

zealotzealot commented Sep 30, 2021

현재 Semester 클래스(모델)는 year, semester(1: 봄학기, 2: 여름학기, ...), 일정 정보(시작 및 종료 일정, 수강신청 일정 등) 를 담고 있습니다.
하지만 semester라는 명칭이 중복되어 사용되어, semester라는 변수가 주어지면 이것이 Semester의 instance인지 int 타입의 semester(Semester.semester)인지 혼동할 수 있습니다.

다음의 두 가지 안을 제시합니다.

  1. int 형식의 semester를 유지하고 class 이름 Semester를 YearSemester로 변경 (year + semester = YearSemester)
    • 명칭이 약간 더 부자연스러우나 코드 변경 과정이 쉬우며 문제가 발생할 여지가 적음
  2. class 이름 Semester를 유지하고 int 형식의 Semester.semester를 Semester.season으로 변경 (year + season = Semester)
    • 명칭이 조금 더 자연스러우나 이미 year과 semester라는 명칭이 Semester 클래스 도입 이전부터 사용되며 굳어진 상황에서 코드 변경 과정에서 꼬일 가능성이 높아 세심한 작업이 필요함
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant