Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`extractValue`에 대한 힌트가 조금 더 있으면 다른 개발자가(혹은 미래의 자신이...) 코드를 읽기 더 수월해질 것 같아요! 😄 value란 어떤 값인가요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`split()` 했을 때 empty string이 나오면 edge separator나 continuous separator를 모두 검증할 수 있지 않았을까 싶습니다.
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
```suggestion private static final String PREFIX_CUSTOM_SEPARATOR = "//"; private static final String SUFFIX_CUSTOM_SEPARATOR = "\\\\n"; private final SeparatorService separato...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
합을 구하는 행위(=계산)는 `Calculator`에 있을 것으로 예상했는데 `Operator`에 두셨군요!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
리턴 타입으로 wrapper 타입을 쓰신 이유가 있으셨을까요?
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요!! 간단하게 리뷰 남겨보았습니다 😄 프리코스 2~4주차도 화이팅입니다!
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요!! 간단하게 리뷰 남겨보았습니다 😄 프리코스 2~4주차도 화이팅입니다!
mingdodev created a review comment on a pull request on woowacourse-precourse/java-calculator-7
요구사항을 분석할 때, 저는 오직 숫자와 구분자의 위치가 정해진 입력만이 올바른 입력이라고 판단하였습니다! `,1,`, `1:2::3`과 같은 입력도 충분히 숫자를 추출하여 계산값을 출력해줄 수 있습니다. 그러나 이러한 형식의 입력은 구분자의 의미를 퇴색시키며, 우리가 흔히 접하는 계산기를 떠올려보았을 때 직관적인 입력으로 보기 힘들다는 생각이 ...
mingdodev created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드로 분리해 리팩토링하는 방식만 생각했었는데, 애초에 분리하지 않으면 되었네요! 꼼꼼한 리뷰 감사합니다!
honggyuSHIN created a review comment on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 블로그 회고 및 피드백을 원하시는 부분에 대해 작성하신 글 잘 봤습니다 객체 지향이 항상 옳지는 않다는 것과 이를 코드에 반영해 절차 지향의 코드를 책임에 따라 객체를 분리하셨다는 점이 인상 깊었습니다. "Main 이 아닌 Processor 에서 흐름을 관장"에서 의문이 들어 클래스를 구분하는 기준, java 파일을 구분하는 기준에 대...
honggyuSHIN created a review on a pull request on woowacourse-precourse/java-calculator-7
수고 많으셨습니다 감사합니다.
honggyuSHIN created a review on a pull request on woowacourse-precourse/java-calculator-7
수고 많으셨습니다 감사합니다.
yjinlee99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 사실 생각하지 못한 부분인데 int가 아니라 long으로 하신 분들이 있더라구요. 제약 사항엔 없지만 int + int했을 때 long이 되는 경우도 있으니까 한번 생각해보시면 좋을거같아요.
yjinlee99 created a review on a pull request on woowacourse-precourse/java-calculator-7
메소드를 잘 나눠놓으신게 인상깊었습니다!! 코드를 줄이기 위해 stream을 사용하거나 축약하신 부분들이 있는거같은데 https://github.com/woowacourse/woowacourse-docs/blob/main/cleancode/pr_checklist.md 이런 체크리스트가 있더라구요 점을 한번 이상 쓰지 말라고 되어있어서 참고하시면 ...
yjinlee99 created a review on a pull request on woowacourse-precourse/java-calculator-7
메소드를 잘 나눠놓으신게 인상깊었습니다!! 코드를 줄이기 위해 stream을 사용하거나 축약하신 부분들이 있는거같은데 https://github.com/woowacourse/woowacourse-docs/blob/main/cleancode/pr_checklist.md 이런 체크리스트가 있더라구요 점을 한번 이상 쓰지 말라고 되어있어서 참고하시면 ...
junest66 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
넵! 감사합니다!
junest66 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네 이부분 생각해서 숫자범위를 유효성검사에서 검사를 해서 유효한 값만 들어오는데, 한번더 체크해도 안정성엔 좋겠네요!
junest66 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스에 충분히 숫자를 추출한다는 의미를 담고 있어 축약해도 된다고 생각합니다!
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
재사용성에 대한 측면에 대해서 고민해보았을 때, 해당 부분을 분리할 만큼 재사용성이 뛰어나지 않은 것 같은데 어떻게 생각하세요?
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
try catch를 2번 실행해서 감싸주면 좋을 것 같아요 ```java try { int number = Integer.parseInt(token); if (number < 0) { throw new IllegalArgumentException("음수는 허용되지 않습니다...
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
throws를 사용해서 exception이 발생할 수 있다는 것을 알려주면 좋을 것 같아요!
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
1주차 피드백에서 나와있지만, List와 같이 컬렉션을 적극적으로 사용하면 좋을 것 같아요!
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
다른 클래스로 분리하지 않으면, public 메소드로 만들 이유가 있을까요?
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Scanner가 아니라, import camp.nextstep.edu.missionutils.Console; 에서 사용하는 `Console.readLine();`을 사용해야해요!
halfmoon-mind created a review on a pull request on woowacourse-precourse/java-calculator-7
고생하셨어요! 코드리뷰를 하다보니, 테스트 케이스를 더 추가해주면 좋을 것 같아요!
halfmoon-mind created a review on a pull request on woowacourse-precourse/java-calculator-7
고생하셨어요! 코드리뷰를 하다보니, 테스트 케이스를 더 추가해주면 좋을 것 같아요!