Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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
각 기능이 서로 독립적으로 유지되고, 확장성에 용이한 구조로 보입니다 리뷰하러 왔다가 많이 배우고 갑니다! 먼저 시간 내서 리뷰 요청 & 리뷰해주신 부분 감사드리고 이번 주도 힘내세요 👍
nns503 created a review on a pull request on woowacourse-precourse/java-calculator-7
수고하셨습니다 :) 배울 수 있는 점이 많았던 코드 같습니다!!
nns503 created a review on a pull request on woowacourse-precourse/java-calculator-7
수고하셨습니다 :) 배울 수 있는 점이 많았던 코드 같습니다!!
GRAY-DDOT created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 테스트 케이스는 두게의 테스트 메서드로 분리하는 게 좋을 것 같아요. 수정을 적용했을때 어떤 케이스에 에러가 발생했는 지 감지하기 힘들 것 같아요
DongchannN created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드명이 눈에 바로 들어오니까 확실히 가독성이 좋아지네요. 좋은 의견 감사합니다! 👍
DongchannN created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`Delimiters` 클래스에 존재하는 `private final Set<Character> delimiters;` 필드를 사용하지 않는 메서드여서 `Delimiters`객체 생성 없이 바로 호출할 수 있는 static으로 처리했습니다!
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네이밍을 좀 더 명확히 해야겠군요 감사합니다!
DongchannN created a review comment on a pull request on woowacourse-precourse/java-calculator-7
감사합니다!
swlee8783 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
parseToLong 메서드 내에서 입력된 문자열이 비었는지 확인 후 0을 반환하는 로직으로 이해했는데, 한편으로 떠올려봤던 것은 빈 문자열은 상위 메서드인 splitInput 메서드에서 확인 및 처리가 가능할 것 같아 이곳에서 처리하고 return하여 효율성을 가져가는 것도 좋겠다는 생각을 해보았습니다..! '''private static St...
