Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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 를 사용하는 것의 차이점이 뭔가요?
youngsu5582 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이와 같은 값들은 상수로 나타내고 될 거 같습니다. index 1이 뭘 의미하는지, `/` 가 뭘 의미하는지 에 대해서 변수명으로 나타낼 수 있을겁니다. `\\` 를 CUSTOM_DELIMITER_SUFFIX 와 같이요 🫡
youngsu5582 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
delimeter 도 외부에서 주입해서 생성자에서 매개변수 로 가져도 될 거 같은데 이와 같이 내부에서 초기화 하는 이유가 있나요?
youngsu5582 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`StringAddCalculator` 라고 되어있지만 단순히 DTO 와 다를게 없을거 같습니다. 조금 더 역할을 가지도록 변경해도 좋을거 같아요. 👍
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
지금은 되게 어려운 리뷰일 수 있으나 요구 사항을 모르는 사람이 봐도 대략적으로 알 수 있게 좋은 코드를 작성하도록 하는걸 추천합니다. 이는, 객체 지향적으로 할 수 있다면 매우 좋겠지만 어렵다면 함수로 부터 분리하는 걸 추천합니다. 함수로 잘 분리한 후, 이걸 객체가 역할을 가지게 할 수 없을까? 부터가 객체지향의 시작점인 거 같아요. 🙂
youngsu5582 created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 6기 코레아 팀의 조이썬 입니다. 코드 잘봤습니다 🙂 중간 중간 신경 쓴 부분들이 잘 보인거 같습니다. 현재, MVC 패턴(?) 을 도입하려고 한 거 같은데 Controller - Service 가 조금 더 역할을 가져도 좋을거 같습니다! 추가로, 객체지향적인 관점을 가지려고 노력해도 괜찮을 거 같습니다. 현재는, 서비스가 모든...
youngsu5582 created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 6기 코레아 팀의 조이썬 입니다. 코드 잘봤습니다 🙂 중간 중간 신경 쓴 부분들이 잘 보인거 같습니다. 현재, MVC 패턴(?) 을 도입하려고 한 거 같은데 Controller - Service 가 조금 더 역할을 가져도 좋을거 같습니다! 추가로, 객체지향적인 관점을 가지려고 노력해도 괜찮을 거 같습니다. 현재는, 서비스가 모든...
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
추출한다는 의미에서 메소드 이름을 extract라고 지으신 것 같네요. 컨벤션 상 get이 무난한 선택이긴 하지만 의미 전달은 이쪽이 훨씬 선명하긴 하네요.
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
똑같은 로직을 시험하는 경우 다른 입력값을 반복해서 처리해주는 @ParameterizedTest라는 기능이 있습니다. 검색해보면 반복적인 코드 작성을 피할 수 있어서 도움이 될 것 같습니다.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
mvc 패턴에서 콘트롤러가 view의 요청을 받아서 service 나 repository와 메시지를 주고 받는 것을 생각해보면 view 클래스와 의존관계를 설정하는 건 어떨까요? input view -> controller (과정 생략) -> outputview 요청과 반환이 일어나니까요.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
예외테스트, 입력값 검증할 때 예외 유형을 자세하게 분류하신 것이 눈에 띕니다. 컨트롤러에 너무 많은 책임이 있다는 점만 수정하시면 훨씬 좋은 코드를 짜실 수 있을 것 같습니다. 건승을 빕니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
예외테스트, 입력값 검증할 때 예외 유형을 자세하게 분류하신 것이 눈에 띕니다. 컨트롤러에 너무 많은 책임이 있다는 점만 수정하시면 훨씬 좋은 코드를 짜실 수 있을 것 같습니다. 건승을 빕니다.
yushinc created a review comment on a pull request on woowacourse-precourse/java-calculator-7
답변 감사합니다. 첨부해주신 링크 참고해서 개선해보도록 하겠습니다!
eunseobb created a review comment on a pull request on woowacourse-precourse/java-calculator-7
의존성 주입으로 리팩토링하는 과정에서 놓친 부분이 많은 것 같네요,, 2주차부터는 꼼꼼하게 리팩토링 해봐야겠습니다 ! 감사합니다😀
