Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
yesjuhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
요것까지 고민해서 리팩토링 해보겠습니다 정말 감사합니다!! 🙇♀️ 😍
yesjuhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좋은 의견 감사합니다! 반영해보겠습니다 😃
thsghdud13 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
축약어는 되도록 쓰지 않으시는 편이 좋습니다.
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
정수의 합이 int의 범위를 벗어나면 오버플로우가 일어납니다. int 범위를 벗어나면 예외 처리할 수 있는 기능이 필요합니다. 아니면 값을 BigInteger로 정의해서 처리하는 분들도 있더라고요.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
''//" 과 "\\n"도 상수로 관리하는 편이 어떨까요?
yesjuhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
커스텀 구분자 이후의 문자열을 변수로 네이밍하기가 어려워서 대충 때웠던 것 같네요..ㅋㅋㅋ 다시 고민해보겠습니다 감사합니당!
yesjuhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
오 그런 생각은 못했는데 생성자에 `parseDelimiters`를 포함시켜 파싱하는게 `DelimiterParser` 클래스를 사용하는 측면에서 더 간편해질 수 있겠네요 좋은 의견 감사합니다 🙇 리팩토링에 적극 반영 하겠습니다!
changbill created a review comment on a pull request on woowacourse-precourse/java-calculator-7
덕분에 계층형 구조에 대해서 많이 배우고 있습니다 감사합니다!
changbill created a review comment on a pull request on woowacourse-precourse/java-calculator-7
상수화 하는 것에 2를 빠뜨렸군요.. 수정해보겠습니다!
changbill created a review comment on a pull request on woowacourse-precourse/java-calculator-7
맞다고 생각합니다..! 코드 리뷰를 하다보니 어떤 잘못을 한 것인지 명확하게 보이더라고요
cheoroo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기존 작성한 예외 테스트 코드와 같은 기능을 검증하는 테스트 코드를 testNegativeNumberThrowsException로 작성했다고 생각했는데 ([63efa03](https://github.com/woowacourse-precourse/java-calculator-7/pull/1607/commits/63efa03e0e3aeb64cbed70cf...
changbill created a review comment on a pull request on woowacourse-precourse/java-calculator-7
감사합니다 다음 과제때는 단위 테스트로 구현해보도록 하겠습니다!
marin-Rim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
유효성 검증을 계속 사용하셔서 유틸로 유효성 기능을 빼보면 어떨까? 라는 생각도 들었습니다.
marin-Rim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
convert가 문자를 숫자로 바꿔준다는 의미를 표현하면 더 좋을 것 같아요
marin-Rim created a review on a pull request on woowacourse-precourse/java-calculator-7
유효성 검증을 철저히 하시고 폴더 구분을 명확히 하셔서 꼼꼼함과 깔끔함을 배우고 갑니다!
marin-Rim created a review on a pull request on woowacourse-precourse/java-calculator-7
유효성 검증을 철저히 하시고 폴더 구분을 명확히 하셔서 꼼꼼함과 깔끔함을 배우고 갑니다!
saera-yook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네 아무 입력이 주어지지 않았을 때 0을 반환하기 위해서입니다! 미션 예시에 `"" => 0` 이 있고, 숫자는 양수만 입력할 수 있기 때문에 구분자와 구분자 사이에 문자가 없을 때 오류가 아닌 0으로 처리하기 위해 이렇게 구현했습니다.
