Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
stream 사용도 고려해보시면 공부가 될 것 같습니다.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
String.join()를 사용하는 것도 고려해보셔요.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
getSum이 적절한 네이밍인 것 같습니다.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
싱글톤 패턴으로 구현하신 건가요?
ChisaeHwang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Util로 옮기는 과정에서 저가 놓쳤나 보군요.. 감사합니다 !!
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이번 미션에서는 정말 테스트 코드 작성에 시간 투자를 못했던 것 같아요.. 꼭 다음 미션부터는 테스트 코드 작성에 대한 공부 및 단위 테스트 적용을 해보도록 하겠습니다!!
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
음 저는 이번 미션 요구사항에서 `camp.nextstep.edu.missionutils.Console`를 이용하여 입력을 받도록 하게 되어있던 점에서 추후에 다른 입력방식으로 바뀔수 있지 않을까 라는 생각에서 우선 InputView를 인터페이스로 설계하였어요! 그리고 위와 같은 생각으로 OutputView에 대한 설계도 그렇게 진행했던 것 같습니다...
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
mvc 패턴을 사용하고 계신데 calculator와 StringParser는 서비스에 더 가깝지 않을까요? 정규 표현식을 잘 사용하고 계시고, 알아보기 쉽게 상수화해서 적확한 이름을 붙이는 센스에 감탄했습니다. 메서드 별로 주석을 다신 것도 좋고요. 자바독 형식을 사용해서 메서드 주석을 달아보시는 것도 괜찮지 않을까요?
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
mvc 패턴을 사용하고 계신데 calculator와 StringParser는 서비스에 더 가깝지 않을까요? 정규 표현식을 잘 사용하고 계시고, 알아보기 쉽게 상수화해서 적확한 이름을 붙이는 센스에 감탄했습니다. 메서드 별로 주석을 다신 것도 좋고요. 자바독 형식을 사용해서 메서드 주석을 달아보시는 것도 괜찮지 않을까요?
holyPigeon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
try-catch에 대한 부분을 말씀하신 게 아닌가 싶네요...? 단순히 에러 발생으로 프로그램이 종료되는 것보다는, 해당 에러를 캐치해서 메시지를 직접 프린트문을 통해 출력했으면 좋겠다는 말로 이해했습니다. 아마 다음 미션에서는 try-catch에 대한 내용이 나오지 않을까 싶은데, 이번 미션 요구사항에는 딱히 그런 내용이 없어서 아직까지는 신...
holyPigeon created a review on a pull request on woowacourse-precourse/java-calculator-7
리뷰가 조금 늦어졌네요...ㅠ 무사히 완료했습니다! 1주차 너무 고생하셨고, 2주차도 같이 화이팅해봐요!
holyPigeon created a review on a pull request on woowacourse-precourse/java-calculator-7
리뷰가 조금 늦어졌네요...ㅠ 무사히 완료했습니다! 1주차 너무 고생하셨고, 2주차도 같이 화이팅해봐요!
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
오 그렇게 하는게 해당 메서드가 무슨역할을 하는지 더 알아보기 쉬울것 같네요!! 저도 네이밍하는게 너무 어려워서 항상 많은 시간을 할애하는 부분중 하나라고 생각해요.....
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
미션을 진행하면서 검증을 어느 부분에서 관리하는게 좋을지 정말 많이 고민하다가 결국엔 분리하지 못했습니다... 다음 미션부터는 검증하는 관리 포인트를 한곳으로 모으는 방식으로 적용해보겠습니다!! 감사합니다!!
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
지금 생각해보면 같은 동작을 하는 메서드를 묶어서 한개의 클래스로 분리하여 관리하는게 좋을 것 같다는 생각이 드네요!! 좋은 의견 감사합니다!!
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
확실히 네이밍에 신경을 더 써야할 것 같다는 생각이 드네요...!! 좋은 정보 감사합니다!!
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
혹시 구분자로 숫자가 들어오는 경우는 어떻게 생각하시나요?
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
정규표현식을 능숙하게 사용하시네요.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분이 정말 좋네요. 많이 배웁니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 보았습니다. 다음 과제에 더 복잡한 기능 요구사항이 있으면 클래스를 조금 더 나눠서 구현하는 경우도 대비하시면 좋을 것 같습니다. 기출문제 중에 대놓고 mvc 패턴을 차용하게끔 되어 있는 경우도 있으니까요.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 보았습니다. 다음 과제에 더 복잡한 기능 요구사항이 있으면 클래스를 조금 더 나눠서 구현하는 경우도 대비하시면 좋을 것 같습니다. 기출문제 중에 대놓고 mvc 패턴을 차용하게끔 되어 있는 경우도 있으니까요.