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
헉 그렇겠네요 조언 감사합니다!
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
MVC패턴처럼 레이어를 나누는 이유 중 한가지는 `각 계층에 대한 요소들의 테스트를 진행하기 용의하다`는 장점이 있는 것 같습니다. 각 레이어의 요소에 대한 단위 테스트도 있다면 좋을 것 같습니다👍
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
isBlank를 사용하면 공백도 검증할 수 있습니다!
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Stringformat을 사용하면 문자열 연산 비용을 줄일 수 있을 것 같습니다!
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
개인적으로 forEach는 접근제어 문제 때문에 for(var a: list)형식으로 쓰는게 더 괜찮다고 생각하시는데 어떻게 생각하시나요?
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
BigInteger를 사용하면 매우 큰 수도 계산할 수 있습니다!
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Converter의 이름이 조금은 추상적인 것 같습니다! CalculateNumbers와 같은 클래스명은 어떨까요?
20HyeonsuLee created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수고하셨습니다! MVC패턴을 적절히 이용하신게 인상깊네요! 2주차 미션도 파이팅입니다🔥
20HyeonsuLee created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수고하셨습니다! MVC패턴을 적절히 이용하신게 인상깊네요! 2주차 미션도 파이팅입니다🔥
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
아하!! 좋습니다 설명까지 자세한 조언 감사드립니다 !
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
감사합니다 👍
