Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
hyojaeKwon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요:) bowook님께서 말씀하신 방법도 좋은 것 같아요! 필요없는 else문이 사라져서 간결한 코드가 될 것 같아요~ 다만, 제가 염려한 부분은 코드의 의미 전달 명확성의 하락 입니다. `if (matcher.find()) { return input.substring(matcher.end()); } throw new Il...
chaeyeon0130 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
코드의 흐름을 이해하는 데 있어, 해당 문자열이 무엇을 의미하는지 확실히 와닿지 않는 부분이 있는 것 같아요! (약간의 매직 넘버랄까,,) 기본 구분자와 커스텀 구분자를 분리하여 각각을 좀 더 명확하게 처리하도록 하는 건 어떨까요? 기본 구분자 같은 경우는 상수로 선언하여 사용할 수도 있을 것 같아요!
chaeyeon0130 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`calculate()` 진입 시 앞에서 입력 값이 없는 경우를 처리하고 있는데! 여기서 다시 한번 검사하는 특정 이유가 있을까요? 특정 의도가 있는 거라면 중복 코드를 줄이기 위해 별도의 메서드로 분리하는 것도 좋아보여요!
chaeyeon0130 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 과제 진행하시느라 수고 많으셨습니다! 전반적으로 책임과 역할을 세분화하여 메서드(혹은 클래스)를 분리한다면, 확장성과 테스트 용이성 측면에서 큰 이점을 가져올 수 있을 것이라 생각합니다!:)
chaeyeon0130 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 과제 진행하시느라 수고 많으셨습니다! 전반적으로 책임과 역할을 세분화하여 메서드(혹은 클래스)를 분리한다면, 확장성과 테스트 용이성 측면에서 큰 이점을 가져올 수 있을 것이라 생각합니다!:)
hyojaeKwon created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 금성님! 1주차 미션 완료하시느라 고생하셨습니다:) 단일 클래스의 책임 분할에 대해 고민하신 것이 느껴져 좋았습니다. 다만 package분리를 조금 해주시면 더욱 구조를 한 눈에 보이게 할 수 있을 것 같습니다!!
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> String을 변환한다는 의미는 매개변수 타입으로 유추 가능하다고 생각해요.String을 변환한다는 의미는 매개변수 타입으로 유추 가능하다고 생각해요. 말씀해주신 이 포인트를 놓쳤네요 감사합니다! `toDoubles` 로 메서드명을 변경하는게 좋을 것 같아요
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클린코드에 대해 고민해보겠습다 :)
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Java의 MVC 패턴을 적용하는데에 아직 익숙하지 않아 이렇게 네이밍을 했습니다 ㅎㅎ 계산기 프로그램의 핵심 제어자이니 `Calculator`로 했는데 일반적인 컨벤션을 따라 `CalculateController`와 같은 네이밍이 더 적절할 것 같네요!
jaepyo-Lee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네이밍에 실수가 있었네요ㅠㅠ 꼼꼼히 봐주셔서 감사합니다!
jaepyo-Lee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그렇네요.. 수정하려했는데 잊고있었네요..ㅠㅠ 꼼꼼히 봐주셔서 감사합니다!
jaepyo-Lee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 메서드의 네이밍이 삭제에 중점을 두고 있어서, 구분자를 찾는 메서드로 분리하는것도 좋을거 같아요! 감사합니다:)
lold2424 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기능 요구사항을 작성하는 docs인 README.md에는 다른 사람들이 봤을때 어떤 동작을 구현할 것인지에 대한 설명이 부족해 보입니다. 기능 요구사항에 대한 상세한 설명과 각 기능이 어떤 흐름으로 동작하는지를 좀 더 구체적으로 작성하는 것이 좋습니다.
lold2424 created a review on a pull request on woowacourse-precourse/java-calculator-7
코드를 작은 단위로 잘 쪼개신 것 같습니다. 이렇게 세분화하면 확장성과 유지보수 측면에서 유리할 수 있겠지만, 한편으로는 코드가 다소 복잡해질 수 있다는 생각도 듭니다. 코드를 잘게 쪼갤수록 유지보수는 유리하지만, 가독성이 떨어진다는 단점 또한 생기게 됩니다. 저는 아직 이러한 균형 잡기가 어려운데 혹시 이에 대해서 어떤 생각을 가지고 계신지 궁금합니다.
lold2424 created a review on a pull request on woowacourse-precourse/java-calculator-7
코드를 작은 단위로 잘 쪼개신 것 같습니다. 이렇게 세분화하면 확장성과 유지보수 측면에서 유리할 수 있겠지만, 한편으로는 코드가 다소 복잡해질 수 있다는 생각도 듭니다. 코드를 잘게 쪼갤수록 유지보수는 유리하지만, 가독성이 떨어진다는 단점 또한 생기게 됩니다. 저는 아직 이러한 균형 잡기가 어려운데 혹시 이에 대해서 어떤 생각을 가지고 계신지 궁금합니다.
hyojaeKwon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
상수를 이렇게 Class로 관리하신 점이 인상깊습니다!
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력을 안내하는 가이드 메시지의 경우에는 InputView가 담당해도 괜찮을 것 같아 이렇게 진행했습니다! 그렇지만, 피드백 주신것처럼 OutputView가 콘솔에 출력하는 책임을 담당하도록 `printMessage()` 같은 메서드를 만들어서 사용하는 것도 괜찮을 것 같아요! 감사합니다 👍
youngsu5582 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
생각을 잘 정리해주신거 같아서 밑에 남깁니다. 이와 같이 간단한 도메인 에서는 getter 를 써도 문제가 없습니다. 하지만 getter 를 사용하면, 코드의 중복이 발생하게 됩니다. 예시로, 여기서 더하기 말고, 빼기를 하는 기능 요구사항이 추가되면 어떻게 될까요? 더하기 하는 부분도 구분자를 `getDelimiters` 로 받고, 빼...
youngsu5582 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
상수를 잘 정리해주셨지만 객체지향 적으로 분리 후 각 객체가 역할에 맞는 상수를 가지는걸 추천합니다. 이 역시도 규모가 커지게 되면, 무엇이 어떤 객체에 쓰이는지 알기 어려워 지기 때문입니다.
youngsu5582 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
- 문자열 더하기 - String.format - StringBuilder 를 사용하는 것의 차이점이 뭔가요?