Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
junghunim07 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
분리해야하는 문자열 입니다 ! 함수명 작성에 조금 더 신경써야 될 것 같네요 !! 👍
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
indent 관련해서 크게 생각을 못하고 있었는데 다른 분들의 코드를 보며 확실하게 느껴지더라구요. 다음 미션 때는 지금보다 더 신경을 쓰면서 만들어보도록 하겠습니다!
junghunim07 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
따로 이유는 없었던 것 같습니다... wrapper타입을 자연스레 썻던 것 같아요... ㅠㅠ (생각하고 코드 작성해야겠습니다...!) 성능을 위해선 원시타입을 사용했어야 했는데... 좋은 지적 감사합니다 😃
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
자세한 리뷰 감사합니다! 저 또한 getter() 사용이 꼭 지양되어야 한다고 생각하진 않습니다. 근데 간단한 도메인이라는 부분이 조금 애매하다고 생각합니다. (어느 정도가 간단한건지?) 예를 들어, 도메인이 가지고 있는 로직(더하기, 빼기)이 간단하다면 해당 객체가 역할을 맡고 수행해도 된다고 생각합니다. 하지만, 로직이 조금이라도 복...
junghunim07 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
검증 메서드를 Util클래스로 분리해 볼 생각을 못해봤던 것 같습니다 !!! 😄 다음 주차에 한번 해봐야겠습니다 !! 👍
kiwoook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아 그리고 잘못된 복잡한 추론으로 테스트를 구현한 점도 있었네요... 문제점 지적도 감사합니다
kiwoook 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
요것까지 고민해서 리팩토링 해보겠습니다 정말 감사합니다!! 🙇♀️ 😍
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
덕분에 계층형 구조에 대해서 많이 배우고 있습니다 감사합니다!
