Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
chul5 created a review on a pull request on woowacourse-precourse/java-calculator-7
한 주간 고생 많으셨습니다! 늦었지만 코드리뷰 드립니다!! 다음주도 함께 힘냈으면 좋겠습니닷!!
chul5 created a review on a pull request on woowacourse-precourse/java-calculator-7
한 주간 고생 많으셨습니다! 늦었지만 코드리뷰 드립니다!! 다음주도 함께 힘냈으면 좋겠습니닷!!
leeje0506 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
오 고려해서 작성해보겠습니다!
leeje0506 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
생각하지 못했던 부분이네요! 메서드명 정할 때 더 고민해보겠습니다.
leeje0506 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
리뷰를 통해 StringBuilder가 멀티 스레드 환경에서 적절하지 않다는 점은 처음 알게 되었네요..! 말씀해주신 내용을 바탕으로 StringBuilder 대신 불변 객체인 String을 사용하여 처리하는 것이 더 나은 방향이라고 생각합니다. 달아주신 리뷰들이 정말 큰 도움이 되는 것 같아요!!
pentorb created a review comment on a pull request on woowacourse-precourse/java-calculator-7
enum으로 관리하면 더 깔끔한 코드를 만들 수 있겠네요! ㅎㅎ
eunseobb 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
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주차도 같이 화이팅해봐요!