Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
koosco 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
맞습니다.. 현재 서비스 로직 자체가 가독성이 매우 안좋은 방식으로 하드 코딩되어 있어 제가 다시 봐도 한 번에 순서를 이해하기에 어려운 코드라고 생각이 듭니다.. 자바 기본 문법에 익숙하지 않아 이미 존재하는 API를 활용하지 못하는 것도 원인 중 하나인 것 같습니다. C언어를 사용하진 않았었는데 정규표현식 개념이 익숙하지 않아 다른 방식으로 ...
jaepyo-Lee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
출력의 책임을 OutputView에서 가지고있는것으로 보이는데, 출력 로직의 경우 OutputView에서 처리해도 괜찮을것 같아요!
jaepyo-Lee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
calculate메서드이어서 계산에 대한 구현이 있을줄 알았는데, Calculator라는 객체에게 계산을 넘기게되는군요! 제 짧은 생각으로는 CalculatorUseCase가 계산에 대한 행위를 가진 인터페이스로 multipleService에서 이를 구현해도 되지않을까 생각되는데, 종인님의 생각이 궁금합니다!
kgy1008 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
밑에 답변을 적어두어서 링크로 대신하겠습니당!! [답변](https://github.com/woowacourse-precourse/java-calculator-7/pull/376#discussion_r1814313785)
kgy1008 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
맡게되는 책임은 똑같지만, 세부적으로 보았을 때 동작하는 로직이 다르기 때문에 두 클래스를 나눈다면, 커스텀 구분자 관련하여 요구사항에 변경이 생겼을 때, `CustomDelimiter`객체만 수정을 하면 되기 때문에 유지보수 측면에서 좋다고 생각했습니다. 관점의 차이인 것 같아요..! 설계는 결국 트레이드오프의 산물이기 때문에 저도 그 부분에 대해...
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
고정되는 문자열을 static 변수로 빼는 것 좋은 것 같아요!
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
분기문을 별도의 private method로 빼면 가독성을 높일 수 있을 것 같아요
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
검증 로직을 책임지는 객체를 별도로 분리하면 클래스의 역할이 더 명확해질 것 같습니다. SumCalculator라는 이름이지만 검증과 파싱까지 모두 하고 있어 응집도가 낮아지고 변경 포인트가 많아지는 것 같습니다!
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
setter를 외부에 모두 오픈하면 캡슐화를 지키기 어려울 것 같습니다. 생성자 쪽에서 받아서 처음에 초기화를 하도록 하는 건 어떨까요?
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Dto를 sumParsedNumbers 메서드에 그대로 넘기면 Calculator는 Dto 내부 데이터에 대해 모르게 할 수 있을 것 같습니다. Dto가 변경되더라도 Calculator 코드는 변경되지 않도록 할 수 있을 것 같아요
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 로직을 DelimiterInputDto에서 수행하도록 책임을 이동시키면 의존성을 줄일 수 있을 것 같습니다.
koosco created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다. Calculator의 책임을 나누면 더 좋을 것 같습니다!
koosco created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다. Calculator의 책임을 나누면 더 좋을 것 같습니다!
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분에 대해서 상수처리를 하면 좋을 것 같아요! 배열의 두번째 요소를 가져오는 이유를 설명하는 변수명이 좋을 것 같네요!
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
경연님은 유효성 검증을 초기화 전에 수행하도록 구현하신 것 같습니다! 생성자에서 delimiter 변수를 초기화 하기전에 검증하기 위해서 인가요?
hoonyworld created a review on a pull request on woowacourse-precourse/java-calculator-7
경연님! 1주차 미션 고생 많으셨습니다 말씀하신 것처럼, 저도 2주차에도 같이 많은 의견 나누고 싶습니다 ㅎㅎ 궁금한 부분은 커멘트 남겼습니다!
hoonyworld created a review on a pull request on woowacourse-precourse/java-calculator-7
경연님! 1주차 미션 고생 많으셨습니다 말씀하신 것처럼, 저도 2주차에도 같이 많은 의견 나누고 싶습니다 ㅎㅎ 궁금한 부분은 커멘트 남겼습니다!
Jonghwan-Lee7 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
ConsoleInput이 View레이어로 보이는데, InputRepository라는 네이밍이 개인적으로 조금 어색한것같습니다..! 저는 View레이어를 외부와 주고받는 레이어로 역할이라 생각되는데, Repository는 Model레이어에서 만들어진 데이터를 저장하기 위한 역할이라 생각되어 역할이 조금 다른 레이어가 아닐까 생각되네요! Rep...
HwiYul-G created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@zzioozzioo 리뷰 인기가 너무 많으셔서 저에게 계속 알람이 와요ㅎㅎ! 사용하신 방식처럼 `Numbers`로 숫자를 관리하는 방법은 `DDD(domain driven design, 도메인 주도 개발`에서 사용되는 방법처럼 느껴집니다. 사실 저도 DDD를 공부해 본 적도 경험한 적도 없어서 잘 모릅니다. 제가 이해하기로 `Numbers...
ckstj8027 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아하 감사합니다 제가 전체 줄을 선택을 안하고 단축키를 사용했었네요
