Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
감사합니다. 혹시 에러 메시지 같은 경우에는 위처럼 클래스에 static final 형태로 하시는 걸 선호하시는지 아니면 Enum 클래스로 정의하는 것을 선호하시는지 궁금합니다!
junghunim07 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네! Separator는 "문자열을 분리해서 수로 반환해라" 라는 메시지만 담당하고자 설계했습니다 !
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
반복문 안이 로직을 별도의 메서드로 분리하면 코드 가독성을 개선시킬 수 있을 것 같아요!
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
if-else를 사용하는 것보다 return을 사용해 if문만을 사용해 작성하면 가독성을 높일 수 있을 것 같습니다
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
delimiter를 파싱하는 것과 숫자 배열을 나누는 기능을 나누어도 좋을 것 같습니다
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Console.readLine을 사용해야할 것 같습니다
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
뒤에 \\n이 오지 않으면 잘못 파싱할 수도 있을 것 같습니다
junghunim07 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력값을 검증을 해야하는데 view에서 하기에는 view는 입력과 출력만 담당을 하고자 해서 Controller로 빼서 검증을 햇습니다 !
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 과정을 사용자로부터 입력을 받고 validation 검사를 먼저 수행하면 되겠다고 생각했습니다. 하지만, 구현 과정에서 '문자열을 분리하고 유효성 검사를 하는 것도 서비스이지 않을까?'라고 생각해서 위와 같이 구현했습니다. 이 과정에서 최대한 중복되는 부분을 제거하고 간단하게 만들려고 했는데, 아직 부족한 부분이 있었네요. 답변해주셔서 ...
junghunim07 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
의존성을 외부에서 주입해주고자 했습니다 !
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
동의합니다. 우테코의 클린 코드 규칙을 지키고자 하였는데 전혀 클린하지 않은 코드가 구현되었습니다. 좀 더 간단한 로직을 고민해보고 리뷰어들이 읽을 수 있는 가독성 높은 코드를 구현해야겠다는 생각이 들었습니다. 복잡한 코드 읽어주셔서 정말 감사합니다!