Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
ashsty created a review comment on a pull request on woowacourse-precourse/java-calculator-7
final을 빼먹는 휴먼 에러를 일일이 찾아내기 어렵다는 점도 제가 생각하는 단점 중 하나긴 합니다 😚 물론 다른 생각을 가진 분들도 많으니 다른 분이 남겨주신 리뷰처럼 본인의 의견을 정립해나가실 수 있다면 좋겠어요!
yeonnhuu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스를 분리하거나 Extractor와 같이 클래스 기능을 모두 나타내는 제목으로 표현하는 게 맞는 것 같습니다. 의견 감사합니다!
ykmxxi created a comment on a pull request on woowacourse-precourse/java-calculator-7
> > 저는 메서드 호출(요청) == 책임 이라고 생각하지 않습니다. 객체지향은 객체들 간 메시지를 보내며 협력하는 구조를 갖는데, 송신자 객체 내부에서 수신자.메서드(요청 데이터)을 갖고 있는것이을 수신자의 책임을 갖고 있다는 관점에서 바라보면 가장 상위 계층에 존재해 프로그램의 진입점들이 되는 곳들은 서비스의 모든 책임을 갖고 있다고 바라봐야 하니...
yeonnhuu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
코드 컨벤션을 지키고 else 예약어를 사용하지 않도록 아래와 같이 작성하는 게 더 좋을 것 같습니다! if (str.startsWith("//")) { return customParser(str); } return str.split(":|,");
yeonnhuu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
IllegalArgumentException을 catch한 후 다시 throw하셨는데 혹시 그렇게 작성하신 이유가 있는지 궁금합니다!
yeonnhuu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력과 출력을 하나의 클래스에 구현하셨는데, 입출력 기능을 별도로 분리하는 것을 추천 드립니다.
yeonnhuu created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수행하시느라 고생하셨습니다!
yeonnhuu created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수행하시느라 고생하셨습니다!
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
각 요소를 BigInteger로 변경하고 더하기 보다, Integer 값들을 먼저 더한 뒤 마지막 결과만 BigInteger로 리턴해주는 방식도 괜찮을 것 같습니다!
mixxeo created a review on a pull request on woowacourse-precourse/java-calculator-7
객체의 역할이 잘 분리되어있어서 코드 읽기가 수월했습니다 :) 고생하셨습니다!
mixxeo created a review on a pull request on woowacourse-precourse/java-calculator-7
객체의 역할이 잘 분리되어있어서 코드 읽기가 수월했습니다 :) 고생하셨습니다!
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
위에는 null로 처리하셨는데 여기서는 isEmpty로 사용하신 이유가 있을까요?
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Console.close()로 사용한 자원을 닫아주면 더 좋을 것 같습니다!
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@DisplayName() 어노테이션을 사용하면 @DisplayName("기본 합치기 테스트") 이렇게 더 편하게 한글로 작성할 수 있을 것 같아요
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
"\\d+" 상수로 의미있는 이름을 지어주면 좋을 것 같습니다
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
null보단 ""이나 Optional을 사용해서 반환하는건 어떨까요?
kwongio created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요. 영진님 1주차 고생많으셨습니다. 2주차도 열심히 해봐요 :)
kwongio created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요. 영진님 1주차 고생많으셨습니다. 2주차도 열심히 해봐요 :)
imjanghyeok created a review comment on a pull request on woowacourse-precourse/java-calculator-7
다음번에는 확실하게 책임을 나눌 수 있게 분리하는 게 낫겠네요. 파싱을 목표를 했는데, 이제 보니 처리 및 검증까지 하고 있네요. 리뷰 감사합니다!
imjanghyeok created a review comment on a pull request on woowacourse-precourse/java-calculator-7
뭐가 run하는지 확실히 할려고 넣었는데, 확실히 그렇게 하는 게 나을거라는 생각이 드네요.
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
positiveNumber면 예외를 던지면 안 될 것 같습니다! 추가로, 예외를 던질 때 사용자에게 메시지를 전달하면 더 좋을 것 같아요 😄
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`isNegativeNumber`로 보입니다!
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분도 분리 가능해 보이는데, 재표님의 생각은 어떠신지 궁금합니다! 그리고 2를 `~_LENGTH`로 상수화 하는 것은 어땠을까요?
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
findDelimiterStart/End로 분리하는 것은 과했을까요?
gyuoo created a review on a pull request on woowacourse-precourse/java-calculator-7
각 기능이 서로 독립적으로 유지되고, 확장성에 용이한 구조로 보입니다 리뷰하러 왔다가 많이 배우고 갑니다! 먼저 시간 내서 리뷰 요청 & 리뷰해주신 부분 감사드리고 이번 주도 힘내세요 👍
gyuoo created a review on a pull request on woowacourse-precourse/java-calculator-7
각 기능이 서로 독립적으로 유지되고, 확장성에 용이한 구조로 보입니다 리뷰하러 왔다가 많이 배우고 갑니다! 먼저 시간 내서 리뷰 요청 & 리뷰해주신 부분 감사드리고 이번 주도 힘내세요 👍