Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아하!! 좋습니다 설명까지 자세한 조언 감사드립니다 !
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 한 곳에서 관리하기 때문에 일관성이 있다, 재사용할 수 있다, 유지보수가 편리하다는 점이 장점일 것 같습니다! 그러나 너무 많아지게 된다면 하나의 클래스에서 관리하기 어렵겠다는 생각이 드네요,, 또 말씀대로 예외 생성시에 해당하는 `ErrorType`을 선택해야 하기에 어려움을 겪을 수 있겠네요 제 생각엔 `ErrorType`이 많아진다면 유형별로...
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
한 곳에서 관리하기 때문에 일관성이 있다, 재사용할 수 있다, 유지보수가 편리하다는 점이 장점일 것 같습니다! 그러나 너무 많아지게 된다면 하나의 클래스에서 관리하기 어렵겠다는 생각이 드네요,, 또 말씀대로 예외 생성시에 해당하는 `ErrorType`을 선택해야 하기에 어려움을 겪을 수 있을 수 있겠네요 제 생각엔 `ErrorType`이 많아진다...
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좋습니다! 이번 주에는 시도해 보겠습니다. 감사합니다!
kimsehw created a review comment on a pull request on woowacourse-precourse/java-calculator-7
맞습니다.. `else if`도 `else`임을 왜 몰랐을까요 ㅠ 그래도 이런 감사한 리뷰들 덕에 `early return 패턴`을 알게 되었네요. 다음엔 반영해서 `early return` 패턴으로 해보려고 합니다!
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아하~ 이번 주에는 고려해보며 구현하겠습니다! 좋은 말씀 감사드립니다
gyuoo 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
감사합니다 👍
na0th created a review comment on a pull request on woowacourse-precourse/java-calculator-7
단순 기능은 static 처리하는 것도 좋다는 점 배워갑니다^^
na0th created a review comment on a pull request on woowacourse-precourse/java-calculator-7
재귀적으로 커스텀 구분자들을 처리한 부분이 테스트 케이스에서 커스텀 구분자가 여러 번 나오는 경우에 대해 생각하신 것 같아서 많은 예외들을 생각하신 점이 좋네요
na0th created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이름이 다소 길 수 있지만, 볼 때 메서드나 변수명만 보고 어떤 역할을 하는지 확실히 알 수 있어서 다른 분들과 구별되게 좋은 점이라 생각합니다.
na0th created a review on a pull request on woowacourse-precourse/java-calculator-7
변수명, 테스트 케이스쪽에서 많은 고민 하신 흔적이 보입니다. 수고하셨습니다
na0th created a review on a pull request on woowacourse-precourse/java-calculator-7
변수명, 테스트 케이스쪽에서 많은 고민 하신 흔적이 보입니다. 수고하셨습니다
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그렇겠네요 그런 부분까진 미처 고려를 하지 못한 것 같습니다. 따끔한 지적 감사드립니다! 그럼 input의 길이를 제한하거나.. 해야겠네요!
yeonnhuu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
StringCalculator에서는 입력된 문자열 전체가 비어있는 경우를 처리하고, SumCalculator에서는 분리된 문자열들이 비어있는 경우를 처리했습니다!
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
의견 감사합니다!!
na0th created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드 명이 number(명사)로 시작하는 부분이 동사로 시작하도록 바뀌면 더 좋은 메서드명이 될 것 같습니다! 방금 찾아봤던 참조 드리겠습니다 https://tecoble.techcourse.co.kr/post/2020-04-26-Method-Naming/