Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
객체를 생성하는 방법과 정적 메서드로 처리하는 방법 중 고민하다가 정적 메서드로 처리하는게 더 효율적이라고 생각해서 진행했어요!! 하지만 그에 맞는 근거를 제시하기엔 제 지식이 불충분 하다고 생각했는데 집어주셔서 감사합니다!!
yesjuhee created a comment on a pull request on woowacourse-precourse/java-calculator-7
리뷰 내용 반영한 코드는 [개인 저장소](https://github.com/yesjuhee/java-calculator-7)에 커밋하여 푸시했습니다 :)
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
DelimiterConstants의 두 상수가 현재 DelimiterValidator와 StringParser 클래스에서 사용되고 있어, 이를 상수 패키지로 따로 관리하게 되었습니다. 저희 기준은 현재 클래스뿐만 아니라 다른 클래스에서도 사용될 여지가 있는 상수는 상수 패키지에서 관리하자는 것이었는데, 이 부분에 대해 길준님의 의견이 궁금합니다!
eraser502 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
값이 없으면 0을 반환하는 편이 낫지 않을까요? https://velog.io/@oyeon/%EC%95%84%EC%9D%B4%ED%85%9C55.-%EC%98%B5%EC%85%94%EB%84%90-%EB%B0%98%ED%99%98%EC%9D%80-%EC%8B%A0%EC%A4%91%ED%9E%88-%ED%95%98%EB%9D%BC
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
잘 봤습니다. 많이 배우고 갑니다. 건승을 빕니다.
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
메서드명 앞에 get를 붙이는 편이 네이밍 컨벤션에도 걸맞고 역할을 명확히 드러낸다고 봅니다.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 클래스가 model일까요? 전략 패턴에 더 가깝지 않나요? 이런 접근은 처음 봐서 신선하네요. 저라면 stringParingService, NumbersAccumulateService 로 기능 나누어서 분리할 것 같긴 합니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. mvc 패턴을 잘 구현하셨네요. 배우고 갑니다. 특히 테스트를 나누어서 작성하신 것도 배울 점이네요.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. mvc 패턴을 잘 구현하셨네요. 배우고 갑니다. 특히 테스트를 나누어서 작성하신 것도 배울 점이네요.
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
```java public record AdditionResult(String result) { public static AdditionResult from(double result) { return new AdditionResult(formatResult(result)); } private static...
choign3123 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
감사합니다😄
choign3123 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
네 맞습니다!
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네 맞습니다!
yjy323 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
동의합니다 :)
bowook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
효재님께서 말씀하신 명확성 부분도 중요한 고려사항이라고 생각합니다. 다만, 클린 코드 원칙에서는 else 문을 지양하는 것이 가독성을 높이는 이유로 불필요한 중첩을 줄이고 조건에 따른 동작 흐름을 더 직관적으로 만들기 위함이라고 생각합니다. 특히 `if-else` 구조에서는, `else` 문을 읽을 때 "앞의 `if` 조건이 아니라면?"을 전제...
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
인덴트 깊이가 너무 큽니다. 프로그래밍 요구사항에 준수하라고 한 자바 컨벤션에 따르면 최대 2단계입니다. 헬퍼함수로 분리하시거나 하셔서 나누셔야 합니다. https://github.com/woowacourse/woowacourse-docs/blob/main/styleguide/java/README.md https://github.com/Ju...