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

[JaeHyeok] 학생 성적 관리 프로그램 #30

Open
wants to merge 1 commit into
base: 2_JaeHyeok
Choose a base branch
from

Conversation

silverjaehyeok
Copy link

학생 성적 관리 프로그램 👨🏻‍🎓

안녕하세요 @tastycodeTeam JaeHyeok입니다. 첫 Pull Request입니다!

프로그램을 작성하면서 고민했던 점

프로그램의 전체적인 설계

  • 처음 과제를 당면했을 때 들었던 생각은 이 프로그램은 Dictionary의 특성을 이용해서 만들자 였습니다. 이유는 학생의 성적은 한 과목에서 여러 개가 발생할 수 없기 때문에 Dictionary의 한 key에 하나의 value 만 존재하는 특성을 이용하여 만들었습니다.

유지보수

  • 메뉴의 기능을 따로 class로 분리하여 새로운 기능을 추가하거나 기능을 삭제할 때 용이하게 분리를 해놓았습니다
1 2 3

해결되지 않은 점 🔒

존재하지 않는 성적의 삭제

  • Dictionary의 removevalue 메소드를 사용하면 값이 존재하지 않더라도, 삭제가 되는 것을 인지하지 못한 상태에서 코드를 작성해 등록된 학생의 존재하지 않는 성적을 삭제해도 삭제가 되는 버그가 있었습니다. 접근하는 성적의 값이 nil인지 확인을 해주는 조건을 만들어 해결하였습니다.

들여쓰기 2번 이내 프로그램 작성하기

  • guard let을 사용해서 옵셔널 바인딩을 하거나 따로 열거형을 사용해 Error를 관리한다면 조금 더 들여쓰기를 줄일 수 있을 것 같습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant