Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
kimsehw created a review comment on a pull request on woowacourse-precourse/java-calculator-7
MVC 패턴에 대해서 빠삭하게 공부하고 제대로 적용해봐야겠네요.. 이것도 좋은게 좋은거지 하고 흉내내봤거든요.. 다음번엔 확실히 공부해서 잘 적용해보겠습니다! 감사합니다! 👍
kimsehw 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
클래스 이름이 추상적인것 같습니다! CalculatorService는 어떨까요?
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
interface를 정의해서 얻은 이점이 있었을까요? 당장 이점을 얻지 못하더라도 추후에 얻을 수 있는 이점이 있을까요?
20HyeonsuLee created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수고하셨습니다👍 enum을 적극적으로 활용하신 부분이 인상깊네요! 2주차 미션도 파이팅입니다🔥
20HyeonsuLee created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수고하셨습니다👍 enum을 적극적으로 활용하신 부분이 인상깊네요! 2주차 미션도 파이팅입니다🔥
nns503 created a review on a pull request on woowacourse-precourse/java-calculator-7
수고하셨습니다 :) 회고를 보니 TDD로 진행한 것이 매우 인상적이었습니다. 2주차도 화이팅!
nns503 created a review on a pull request on woowacourse-precourse/java-calculator-7
수고하셨습니다 :) 회고를 보니 TDD로 진행한 것이 매우 인상적이었습니다. 2주차도 화이팅!
kimsehw created a review comment on a pull request on woowacourse-precourse/java-calculator-7
자세한 피드백 감사드립니다! `static 메서드`에 대해서 제대로 알아보지 않고 단순히 좋은거구나 하고 사용했었던 과거의 저가 아쉽습니다.. 이거 말고도 뭐든지 알맞은 곳에 써야 함을 이번에 뼈저리게 느꼈네요. 자세한 피드백 덕분에 `static 메서드`에 대해서 공부할 방향이 명확해진 것 같아요! 감사합니다!
cherryiJuice 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
다양한 케이스를 고려하다 보니 중간에 존재하는 경우도 떠올렸습니다. 그런 말이 있었군요 ㅎㅎ..
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
나눌 생각 조차를 못했습니다...!!! 규모가 커지면 나누는 것이 훨씬 좋을 것 같아요