김이님 코드 잘 읽었습니다. 전체적으로 상수 문자열에 대해서 privat static final ~ 로 선언하신 부분은 좋았던것 같아요. 다른 상수들도 그렇게 선언해보면 어떨까요? 몇몇 함수들은 하나 초과의 책임을 지고 있는 것 같은데, 이 부분도 고려해보시면 좋을 것 같습니다. 1주차 수고하셨습니다.
김이님 코드 잘 읽었습니다. 전체적으로 상수 문자열에 대해서 privat static final ~ 로 선언하신 부분은 좋았던것 같아요. 다른 상수들도 그렇게 선언해보면 어떨까요? 몇몇 함수들은 하나 초과의 책임을 지고 있는 것 같은데, 이 부분도 고려해보시면 좋을 것 같습니다. 1주차 수고하셨습니다.
단순히 상속뿐만 아니라, 코드 스멜을 놓칠 수 있다는 단점도 있다고 생각해요.
저는 개인적으로 정적 팩토리 메서드를 도메인에서 사용하는 것을 좋아하지는 않는데요, `new` 생성자로도 충분히 표현할 수 있다고 생각하기 때문이예요. 오히려 `new`가 **생성한다**는 개념과 더 호응한다고 생각하기도 하고요.
만약 클래스에 이미 정적 팩토리 메서...
로직에 대한 설명이 있는것도 좋지만, 비개발자도 애플리케이션을 이해할 수 있게끔 작성해보면 어떨까요 ~??
README만 봐도 애플리케이션이 어떤 기능을 하는지 알 수 있으면 더욱 친절한 문서가 될 것 같아요 ㅎㅎ
그리고 git에 대해 학습하신 내용에 겪으신 문제와 해결과정을 나열해놓으신건 정말 좋다고 생각해요~ 👍
학습에 대한 열정! 배워 ...
> 사용자가 아무것도 입력하지 않았을 때 "결과 : 0"을 출력하고 재귀적으로 run() 메소드를 호출하는 것으로 보입니다.
> 이는 StackOverflow에 안전하지 않은 방식으로 보입니다! 만약 사용자의 입력을 반복적으로 받아야 한다면,
> 반복문을 통해서 사용자의 입력을 받는 것은 어떨까요? 물론 두 방법다 에러에 취약하다고 개인적으로 생각...