Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
songsunkook 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
의견 감사합니다. 2주차 과제 정규표현식에 반영 중입니다.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
OutputView에 결과값을 Result 객체로 전달해서 콘솔에 출력하려면 필요합니다. 다시 보니까 return String.valueOf(value); 를 쓰는 게 낫겠네요.
seoyeon-jung created a review comment on a pull request on woowacourse-precourse/java-calculator-7
다른 분들 코드를 보고 상수화 하신 분이 많더라구요. 저는 상수화를 고려본적이 없는데 혹시나 상수화하는 이점이 있을까요?
seoyeon-jung created a review comment on a pull request on woowacourse-precourse/java-calculator-7
넵넵 의견 감사합니다! 확실히 입출력 클래스를 분리하는게 더 적절해 보이네요!
seoyeon-jung created a review comment on a pull request on woowacourse-precourse/java-calculator-7
콘솔 출력을 한 줄만 하면 된다 생각해서 분리하지 않고 단순하게 생각했던 것 같네요 2주차부터는 참고해서 적용하겠습니다!!
seoyeon-jung 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
InputtedLine 정도로 고치는 게 좋을 것 같네요.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네 지적 감사합니다. 제일 멍청한 실수였습니다. 제출하기 콘솔 테스트가 실패하길래 다급해서 이것 저것 마구 건드리다 보니 가장 기초적인 실수를 했습니다.
venus-y created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 오버로딩 이용하신 점이 인상깊습니다. 다만, 상수들을 합한 다른 상수(예시 : DEFAULT_SPLIT_PATTERN 등)를 정의하여 split하는 건 어떨까 생각해봅니다! `(REGEX_PREFIX + DEFAULT_DELIMITERS + REGEX_SUFFIX` 이 부분을 합치는 것은 생각해보지 않았네요. 감사합니다!
pakaheo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 정적 팩터리 메서드를 필요한 곳에 잘 배치해주신 것 같습니다!
pakaheo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
DelimiterParser에 DEFAULT_PATTERN이 이미 정의되어있지 않나요?
venus-y created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 오버로딩 이용하신 점이 인상깊습니다. 다만, 상수들을 합한 다른 상수(예시 : DEFAULT_SPLIT_PATTERN 등)를 정의하여 split하는 건 어떨까 생각해봅니다! 말씀해주신 이 부분을 제가 잘 이해하지 못한 것 같습니다. 혹시 예시를 들어주시면 감사하겠습니다!
venus-y created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 혹시 Prompter와 Receiver를 따로 나눈 이유를 알 수 있을까요? 입력을 요청하는 역할과 사용자로부터 입력받는 역할을 분리해보고 싶어서 두 개의 클래스를 사용하게 됐습니다.
venus-y created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 혹시 리터값을 String으로 하신 의도를 여쭤볼 수 있을까요? > > 저는 validate가 '유효성을 검증하다'라는 의미이기 때문에 리턴값은 void로 하고 입력 값이 유효하지 않으면 Exception을 발생시키는 것이 낫다고 생각해서요. 혹시 제가 이해하지 못 한 의도가 있는지 여쭙고 싶습니다! 석진님 말씀이 맞습니다. 반환타입을 ...
venus-y created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 상수를 모아둔 클래스를 따로 정의한 점이 인상깊습니다. 다만, 제가 잘 작성했다고 생각하는 다른 분의 코드를 보면 구분자 패턴의 변경을 고려하여 enum을 사용하셨더라고요. #1676 enum을 사용하셔서 관련성이 있는 상수끼리 묶으면 좀 더 가독성 높은 코드가 될 것 같습니다! enum까지 사용하는 것은 생각하지 못했네요. 이 부분 참고해...