Be/main/yeoshin java lv1 #21
Conversation
duckgii
commented
Jan 8, 2025
duckgii
commented
- 단계별 커밋에 신경쓰면서 진행했습니다!!
- 부족한 부분을 적극적으로 알려주시면 감사하겠습니다 :)
saewoo1
left a comment
saewoo1
left a comment
There was a problem hiding this comment.
정성스러운 커밋과 기능분리에 저 울엇어요
명령어와 메뉴를 분리시키신 점이 인상깊었고, 빠른 성장이 보이는 코드였습니다 고생 많으셨습니다!!
There was a problem hiding this comment.
addOne이라는 메서드명에서 더하기만 할줄 알았는데 출력까지 하고잇따니~ 다음에는 분리해봅시다
추가적으로, 1000이라는 값이 반복해서 출현하고 있는데 상수로 선언해서 공통적으로 사용하면 어떨까여
There was a problem hiding this comment.
추가적으로, map의 getOrDefault 기능이나, merge도 존재합니다!!
There was a problem hiding this comment.
파서를 다른 클래스로 분리하시면 더 좋을 것 같습니다!!
There was a problem hiding this comment.
공통적으로 사용되는 케이스가 아직 아니라면, 어떤걸 분리하는건지 메서드를 읽고 바로 이해할 수 있도록 명확히 작성해주시면 더 좋을 것 같습니다!
넵 함수와 클래스의 분리와 코드 가독성에 더 신경을 써서 리팩토링 진행하겠습니다. 꼼꼼한 피드백 감사합니다!! |
There was a problem hiding this comment.
enum을 사용하셨네요! 자바의 enum은 c/c++과 다르게 클래스라 더 다양한 방법으로 사용할 수 있답니다
각 타입에 맞는 출력문도 같이 저장해보면 좋을 것 같아요
There was a problem hiding this comment.
또한 문자열의 특정 위치의 문자 하나를 비교하면 코드를 읽고 해당 로직이 어떤 요구사항을 가지고 있는지 알기 어렵습니다.
"몇 개야?", "시켯나?" 등의 문자열로 비교한다면 더 가독성이 좋을 것 같습니다!
