Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
gorapaduckoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
DTO의 경우 데이터를 전달하는 것이 목적이기 때문에 접근을 제한할 필요가 있을까 싶었습니다. getter를 사용한다고 해도 필드명이 바뀌면 클라이언트 측 코드가 변경되는건 마찬가지라서 필드를 직접 참조하는게 낫겠다고 생각했어요. `createDelimiterRegex()`를 `DelimiterExtractionResult` 내부로 옮길까도 고...
yushinc created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아직 리팩토링 진행을 하지 못해서 해당 링크의 글을 지금 읽어 보았습니다. 저도 yjy님과 동일하게 일급 컬렉션은 불변성에 중점을 둔 기능이라고 생각하는데 클래스간 결합도와 일급 컬렉션 사이에 어떠한 연관성이 있어서 링크를 첨부해주셨는지 궁금합니다. - 저는 get을 사용하면 다른 클래스의 반환값을 외부 클래스에서 그대로 받아와 사용하는 것이기 ...
rookedsysc created a review comment on a pull request on woowacourse-precourse/java-calculator-7
엇 그러네요..! MultipleService는 사실 모든 과정은 프로세싱하는 Facade 역할만하고 있네요. 이 부분 제가 잘못설계 한 것 같습니다. ㅠㅠ 😭 이걸 이제 발견했네요..
yseo14 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
테스트를 체계적이고 구체적으로 작성하신게 인상적입니다. 저도 테스트코드 작성에 조금 더 신경을 써봐야할 거 같습니다!
yseo14 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
steam을 적재적소에 잘 활용하신 것이 인상적입니다.
yseo14 created a review on a pull request on woowacourse-precourse/java-calculator-7
메서드명들이 다소 길어 가독성이 조금 떨어지는 경향이 있는 거 같습니다! 의미를 전부 담아 전달하려는 의도는 알겠으나 1주차 공통 피드백에서도 언급된 것처럼 한두단어로 유지하도록 노력해보는 것도 좋을 거 같아요!
yseo14 created a review on a pull request on woowacourse-precourse/java-calculator-7
메서드명들이 다소 길어 가독성이 조금 떨어지는 경향이 있는 거 같습니다! 의미를 전부 담아 전달하려는 의도는 알겠으나 1주차 공통 피드백에서도 언급된 것처럼 한두단어로 유지하도록 노력해보는 것도 좋을 거 같아요!
SukJinKim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@kysub99 코드 아래 읽어보시면 파싱 결과인 tokens를 변환하여 합계를 계산합니다. 이를 분리하는 것도 생각해볼만한 주제인 것 같습니다. 감사합니다!
jaehwannnnnn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
"-" 가 구분자가 될 수 있다는 생각을 저는 하지 못했는데 "-"도 충분히 커스텀 구분자로 활용될 수 있다는 생각이 드네요!
HwiYul-G created a review on a pull request on woowacourse-precourse/java-calculator-7
지우님.. 인기가 너무 많으셔서 저는 이제 알람을 끄고 떠납니다!
SukJinKim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@kysub99 2주차 미션부터 참고하겠습니다. 조언 감사합니다!
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
분기문을 별도 메서드로 분리하면 가독성을 높일 수 있을 것 같아요
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
validator를 하나의 인터페이스로 추상화하면 어떨까요?
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
진입점을 controller가 아니라 다른 객체가 수행하는 것은 어떨까요? controller는 데이터를 매개변수로 받아 처리하게 되는데 controller부터 시작하게 되면 입력 로직까지 controller에서 수행하게 될 것 같아요. MVC로 하시는거면 Controller가 처리할 데이터를 받고, View쪽으로 출력을 나누도록 하는 것이 좋을 것 ...
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
SumService라는 클래스는 Sum 관련 로직을 수행할 것으로 예상되는데 delimiter parsing, type casting 등의 역할도 함께 하고 있는 것 같아요. 책임에 따라 객체를 분할하는 것은 어떨까요? 입력으로 들어온 숫자를 계산하도록만 하면 예측하기 쉬운 코드가 될 것 같아요!
koosco created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다. validator 부분을 추상화하고 공통 로직을 상속을 이용해 나타내면 코드 재사용성이 높아질 것 같습니다. 도메인 객체를 도입해 validate하는 부분을 도메인 객체 내부로 옮기면 코드 변경 포인트를 줄일 수 있을 것 같습니다.
jaehwannnnnn created a comment on a pull request on woowacourse-precourse/java-calculator-7
MVC 패턴을 활용해서 코딩을 잘하신 것 같습니다 2주차 과제도 화이팅 하세요!
rookedsysc created a review comment on a pull request on woowacourse-precourse/java-calculator-7
https://martinfowler.com/eaaCatalog/repository.html 여기에서도 유사한 느낌으로 나와있어요!
rookedsysc created a review comment on a pull request on woowacourse-precourse/java-calculator-7
덧셈이 아닌 곱셈 등 다양한 연산에 대응하기 위해서 Interface를 만들고 Usecase Interface를 지켜서 개발할 수 있도록 설계해뒀습니다!
rookedsysc created a review comment on a pull request on woowacourse-precourse/java-calculator-7
데이터 서빙을 AOP 관점에서 보면 Repository는 결국 데이터를 서빙하는 객체를 추상화 해놓은 것을 의미하기도 해요! 그래서 Flutter 같은 Front End Framework에서는 API 호출부를 Repository라고 표현하기도 하는 것 같아요! 그런 의미에서 저도 이 부분을 Repository라고 했습니다! https://code...