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

[ 지하철 노선도 미션 ] 박민영 미션 제출합니다. #116

Open
wants to merge 49 commits into
base: main
Choose a base branch
from

Conversation

Gomding
Copy link

@Gomding Gomding commented Dec 15, 2020

3주간 정말 감사했습니다.
남은 시간이 없어서 코드를 더 깔끔하게 만들지 못해 밤잠을 이루지 못할것 같습니다.
부족한 부분은 프리코스가 끝나더라도 과제를 좀 더 완성시켜보고 싶습니다.
이번 3주차 미션을 하면서 구현하다보니 점점 MVC 구조와 닮아가고 있어서
중간에 MVC 구조로 바꿔서 구현하느라 시간이 좀 더 걸렸습니다.
구현하면서 구조에 대해 생각하며 코드들이 있어야할 자리에대해 많은 고민을 했습니다.

좋은 과제를 내주셔서 정말 감사합니다!

Gomding and others added 30 commits December 14, 2020 02:21
지하철 노선에 구간을 추가할 때 입력한 역 이름이 StationRepository에 존재하는지 검사하는 기능 추가
클래스명 변경 : SubwayMap -> MainMenu
패키지명 변경 : map -> menu
StationRepository
- stationList 를 ArrayList에서 LinkedList로 변경

Line
- stationList와 관련 기능을 없애고 StationRepository 타입의 변수를 추가
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