Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그러네요 의도가 전달되기 어려울 것 같습니다 결국 해당 숫자에 대한 이해를 요구할 것 같아요 해당 부분을 상수로 관리하거나 예혁님 말씀대로 메서드로 관리하는 것이 더 좋아보입니다! 리팩토링 해보겠습니다! 감사합니다!
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
헉 그러네요 ㅠㅠ
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
지금은 미션 요구사항의 볼륨이 작아서 하나의 클래스로 관리했지만 추후 다뤄야할 메서드가 많아지게 된다면 여러개의 클래스로 관리하기보다는 예외의 공통 분모를 카테고리로 추출하여 중첩 클래스로 관리한다면 유지보수에 용이할 것으로 판단했습니다.
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
생각해보니 계산 결과를 상태로 가지고 있을 필요가 없는 것 같네요 sum()을 정수로 반환받아서 사용하는 것이 더 좋을 것 같다고 생각되네요 제가 생각이 짧았던 것 같습니다!
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
다른 분들이 리뷰를 너무 잘 남겨주셔서 이 부분에 대해서 세세하게 다뤄보겠습니다! 필드를 정적으로 선택한 것은 좋은 선택일수도 있다고 생각하지만 저는 많은 문제점을 발생시킬수 있다고 생각해요 1. 글로벌 상태 문제 static으로 관리되는 객체는 애플리케이션 전역에서 접근이 가능하기에 때문에 해당 객체는 클래스나 메서드에 모두 공유되요 ...
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
mvc패턴에서 view의 역할은 사용자로부터 입력과 출력에 집중해야한다고 생각해요! 입력값을 검증하는 것은 view의 역할로 보기보다는 controller 혹은 model의 역할에 더 부합하지 않나 생각해요!
woogym created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 너무 수고하셨습니다! 😁😁 각 패키지별로 잘 분리되어 있고 메서드 네이밍도 잘 이루어져서 코드를 이해하는데 좋았어요! 미션을 너무 잘 해주셔서 많이 보고 배웠습니다! 다른분들이 리뷰를 너무 잘 남겨주셔서 많이 리류블 적지는 못했지만 특정 부분을 자세히 다뤄봤어요 리뷰가 도움이 되었으면 좋겠습니다!ㅎㅎ 2주차도 화이팅입니다!
woogym created a comment on a pull request on woowacourse-precourse/java-calculator-7
제가 부족했던건지 저는 사실 명확히 잘 읽히지는 못했어요 어떤 패키지가 모델의 역할을 하는지 어떤 패키지가 컨트롤러의 역할을 담당하는지에 대해서 명확히 알기는 힘들었던 것 같아요ㅠㅠ
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
중복되는 코드를 줄이고자 하는 의도는 좋을 것 같습니다! 저는 순수 추상 클래스의 역할을 하는 인터페이스보다는 별도의 클래스(부모)를 두어 자식 클래스에서 사용하면 좀 더 의미에 부합하는 코드를 완성할 수 있지 않을까 생각해요
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아하 그렇군요 배워갑니다 ㅎㅎ
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
맥북을 사용하신다면 전체 줄을 선택(command + a)하신후 `option + command + l` 을 통해서 자동 줄 정렬을 사용하실 수 있어요! 윈도우시면 `alt + ctrl + l`가 단축키로 알고있어요!
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 네이밍 고민에 가장 많은 시간을 할애하는 것 같아요 ㅎㅎ 네이밍이 아직도 가장 어려운 것 같아요😂
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
1 답변: 객체로 주입받기 보다는 해당 메서드가 다른 곳에서 오용되기를 방지하는 방법으로는 접근 제어자를 적절히 잘 활용하여 제어하는 것이 좋아보입니다! 2번 답변: 저도 돌이켜 생각해보니 입력한 계산을 입력받고 그를 계산하는 간단한 유틸리티성 메서드는 `static`으로 관리하는것도 좋아보입니다!
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`static`으로 관리되는 만큼 시스템 전체에서 고려하는 메서드입니다 하지만 `StringBuilder`는 멀티 스레드 환경을 고려했을 때 적절한 선택은 아니라고 생각해요 `String`은 불변 객체지만 `String`의 불변성의 불편함을 보완하여 탄생한 것이 가변 객체 `StringBuilder`에요 가변 객체의 가장 큰 단점은 사이드 이...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
체크예외, 언체크예외에 대해서 알아봅시다!!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`str` 말고 `input` 같은 이름은 어떨까요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`ERROR_PREFIX`가 중복해서 등장하고 있는데요. `OutputView`에 예외 메시지를 출력하는 메서드를 만들고 거기서 `[ERROR]`를 붙여도 좋을 것 같네요 👍
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
매번 new line을 넣기보다는 `System.out.println()`을 하시면 어떨까요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
흐름이 한눈에 들어와서 보기 좋네요 😄 메서드 분리를 잘 하셨다는 방증인 것 같습니다!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
여기는 메서드 분리 없이 바로 리턴해줘도 무방할 것 같습니다! ```suggestion // 결과를 유저에게 안내 OutputView.printSum(calculator.calculateSum()); ```