[코드 피드백]
0. Lv5까지 완벽한 코드를 잘 작성해주셨네요, 정말 잘하셨습니다.
1. 클래스와 클래스상속에 대해서 완벽히 이해하셨습니다.
2. coffeeMenu(), drinkMenu()에서 Food자식클래스로 새로운 인스턴스를 만드는 만든 이유가 궁금합니다. 전역변수로 선언한 foods에 이미 동일한 값이 있고, 해당 값을 사용해 중복코드를 줄일 수 있을 것 같아요.
3. Food의 메서드를 자식클래스들에서 모두 동일하게 사용하고있는데요. 자식클래스마다 특별히 다른 구현이 없다면, 부모클래스에서 구현해 사용하는게 좋아보입니다.
[Git 피드백] - 코드 컷밋을 잘 남기려 노력하셨네요: 훌륭합니다. 커밋메시지만 보고도 다른 동료들이 업무 진행과정을 쉽게 알 수 있도록 습관화해보세요
[종합 피드백]
0. 잘하셨습니다. 완벽히 돌아가는 Lv5코드를 작성했다는 것에 칭찬합니다.
1. 맨토들에게 좋은 질문을 하고, 잘 습득하고 있습니다. 빠른 성장을 하고계신데, 기세를 이어서 스프링 과정도 잘 해내리라 믿습니다. 화이팅입니다!
2. 각 메뉴들을 불러올 때 새로운 객체를 생성하지 않고 저장해놓은 리스트를 통해 불러옴
3. 부모 클래스인 Food 와 같은 메소드를 구현하는 자식 클래스들 삭제
목요일에 객체들을 리스트에 넣은 이후 고쳐야 할 부분들을 귀찮다는 핑계로 수정하지 않았었다.
하고 나니 훨씬 깔끔하다!
https://github.com/Re-eun/SpartaKiosk
https://github.com/jeawonlee0327/kiosk/tree/Jaeeun
'왕초보일지' 카테고리의 다른 글
231220 TIL | Spring Service Layer 작성 (1) | 2023.12.20 |
---|---|
231219 TIL API설계/DTO작성/챌린지반주제 (0) | 2023.12.19 |
231218 TIL Spring 입문 1주차 강의 (1) | 2023.12.18 |
231215 TIL (1) | 2023.12.15 |
231214 TIL (0) | 2023.12.14 |