Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
leeje0506 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좋은 말씀 감사합니다!! 그렇게 생각해볼 수 있겠네요. 배워갑니다.
pentorb 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 패턴을 잘 사용하시려고 많이 노력하신 게 보입니다. 특히 controller가 view 에서 요청을 가져와서 서비스에 넘기기만 한 부분이 클래스의 역할을 아주 잘 구분한 점이라고 생각합니다. 다만 의문인 게 서비스 클래스가 왜 모델 폴더에 있을까요? delimiter 클래스가 모델에 있고 서비스는 서비스 폴더에 두는 게 어떨까요? 테스트도 단위...
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
콘솔을 이용해서 출력하고 있으니 클래스 이름을 ConsoleInputView라고 하면 어떨까요?
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
mvc 패턴을 잘 사용하시려고 많이 노력하신 게 보입니다. 특히 controller가 view 에서 요청을 가져와서 서비스에 넘기기만 한 부분이 클래스의 역할을 아주 잘 구분한 점이라고 생각합니다. 다만 의문인 게 서비스 클래스가 왜 모델 폴더에 있을까요? delimiter 클래스가 모델에 있고 서비스는 서비스 폴더에 두는 게 어떨까요? 테스트도 단위...
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
mvc 패턴을 잘 사용하시려고 많이 노력하신 게 보입니다. 특히 controller가 view 에서 요청을 가져와서 서비스에 넘기기만 한 부분이 클래스의 역할을 아주 잘 구분한 점이라고 생각합니다. 다만 의문인 게 서비스 클래스가 왜 모델 폴더에 있을까요? delimiter 클래스가 모델에 있고 서비스는 서비스 폴더에 두는 게 어떨까요? 테스트도 단위...
yummygyudon created a comment on a pull request on woowacourse-precourse/java-calculator-7
> `DelimiterConstants`랑 `InputValidationConstants`를 말씀하시는거라면 생성자는 `private`으로 막아져있는 상태입니다! 아하! 예외를 발생시키는 로직이 있길래 `public`이라고 잘못 인지하고 있었나봐요!! 그렇다면 `private` 생성자를 선언하셨다면 외부에서의 객체 생성은 방지될 것 같네요! ...
leeje0506 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
불필요한 클래스도 같이 불러와지기도 하고, 코드가 불명확해질 것 같네요. 이 부분 더욱 생각해보며 코드 작성해보겠습니다.
chul5 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
주석을 달아놓으셔서 어떤 목적으로 작성하신지 알 수 있어서 좋았습니다! 다만 클래스이름이 Context여서 정확히 어떤 역할인지 느낌이 바로 안 왔는데 이름을 좀더 구체적으로 작성해보면 어떨까요?!
chul5 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 Validator객체의 메서드를 클래스 메서드 (static)으로 구현하신 이유가 궁금합니다.
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
상수 이름에 맞게 에러메세지에 음수가 입력되었다는 것을 나타내면 어떨까요?
