Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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...
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
59~77이 무슨 역할을 하는지 주석을 추가하시는 게 어떨까요?
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
이 부분은 삭제하셔도 좋을 것 같습니다.
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
잘 봤습니다. 여러 역할과 책임을 가진 클래스로 나누는 편이 가독성과 유지보수에 좋을 것 같습니다. 문자열 검증 부분을 직접 짜신 것 같은데 기존에 있는 메소드를 활용하는 편도 나쁘지 않으니 한 번 그렇게 짜려면 필요한 게 무엇인지 생각해보시면 도움이 될 것입니다.
CISXO 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
변수명에 컬렌션을 명시하는 건 좋지 않은 네이밍이라고 하네요.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스의 역할이 너무 많습니다. 위에서 말씀드린 것처럼 view 클래스를 따로 만드시는 편이 좋을 것 같습니다. veiw -> controller -> service -> controller -> view의 흐름을 따르면 클래스의 역할도 작게 유지하고 분명해질 것입니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. Number 클래스가 쓰이던데 다른 버젼에서 구현하신 것이겠죠? controller가 직접 메인 기능을 수행하지 않고 service에 넘긴 것이 인상 깊네요. 네이밍 컨벤션을 잘 지키신 부분도 있고 조금 미흡한 부분도 있지만 금방 보완하신 것이라고 믿습니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. Number 클래스가 쓰이던데 다른 버젼에서 구현하신 것이겠죠? controller가 직접 메인 기능을 수행하지 않고 service에 넘긴 것이 인상 깊네요. 네이밍 컨벤션을 잘 지키신 부분도 있고 조금 미흡한 부분도 있지만 금방 보완하신 것이라고 믿습니다.
cheoroo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
main메서드에 너무 많은 책임이 있는것 처럼 보여서(사용자 입력 처리, 문자열 분할, 구분자 추출, 덧셈 계산이 모두 main에 들어감) 컨트롤러 클래스를 하나 만들어서 분리해보는건 어떨까요? 아니면 간단한 계산기의 기능을 구현하는 과제였으므로 빠르고 간단하게 기능을 구현하는것이 더 중요하게 판단되었을까요?
cheoroo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
정민님이 작성하신 계산기 메서드에서는 double 타입을 지원하는 계산기 로직으로 작성하셨는데, 해당 메서드에서 커스텀 구분자로 온점이 들어올경우 소수와 구분자를 제대로 구별하지 못할거 같아서 이를 어떤식으로 처리하는지 고민해보는것도 좋을것 같아요 저는 이러한 문제때문에 들어오는 수의 타입을 int타입으로 제한하여 구현했었던것 같아요. 만약 소수...
cheoroo created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 보았습니다. 이번주차에는 없었지만 돌아오는 주차의 요구사항에는 객체지향적 코드를 작성하는것이 들어가 있으므로 요구 사항에 맞추어 많은 고민을 하고 코드를 작성하여 구현하면 좋은 결과가 있을것 같습니다. 수고하셨습니다.
cheoroo created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 보았습니다. 이번주차에는 없었지만 돌아오는 주차의 요구사항에는 객체지향적 코드를 작성하는것이 들어가 있으므로 요구 사항에 맞추어 많은 고민을 하고 코드를 작성하여 구현하면 좋은 결과가 있을것 같습니다. 수고하셨습니다.
CISXO created a review comment on a pull request on woowacourse-precourse/java-calculator-7
ConsoleIO라는 네이밍 때문에 오해한 것 같습니다. 새로운 클래스를 만들고, Application의 내용을 수정해야 하는 것이 연쇄적인 수정이라 생각했습니다. 2주차도 화잇팅입니다!