Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
gyuoo created a review on a pull request on woowacourse-precourse/java-calculator-7
각 클래스가 자신의 책임을 명확히 하고 있어 보입니다! 전반적으로 너무 깔끔하고 좋네요 😄 리뷰하러 왔다가 많이 배우고 갑니다. 클래스 구분이 명확한 게 너무 좋아요 리뷰 요청 주신 덕분에 다른 분들이 달아주신 코멘트도 잘 보고 가요!
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 정적 팩토리 메서드의 단점은 없을까요? 🤔 정적 팩토리 메소드만 제공하면 상속이 불가능하다는 단점이 있습니다!
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`isDigitsOnly` 인 경우일 때도, 분기처리 하지 않고 split 로직 타도록 두는 것도 로직에 대해 이해할 때 문제 없을 것 같습니다!
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
CalculateController에서 메서드 분리가 조금 과한 것 같다는 생각이 듭니다 `calculateSum`과 `displayResult` 메서드를 한번 더 분리해서, run 메서드만 봤을 때는 이해하기가 좋지만 협업하는 관점에서, 두 메서드는 다른 메서드를 한번 호출하는 역할만 수행하는데 로직을 깊게 이해하려면 한번 더 메서드를 타고 들...
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 오버플로우 검사 배워가요 :)
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
결과 출력할 때, printf 사용해도 좋을 것 같습니다! ``` private static final String RESULT_MESSAGE = "결과: %d"; System.out.printf(RESULT_MESSAGE, result) ```
mixxeo created a review on a pull request on woowacourse-precourse/java-calculator-7
예외 처리와 메서드 분리에 신경을 많이 쓰신 것 같아요! 잘 봤습니다 :) 1주차 고생하셨습니다!
mixxeo created a review on a pull request on woowacourse-precourse/java-calculator-7
예외 처리와 메서드 분리에 신경을 많이 쓰신 것 같아요! 잘 봤습니다 :) 1주차 고생하셨습니다!
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 정적 팩토리 메소드를 사용하셨군요! 👀 > > 다만 그렇다면 지금 `MainController` 생성자의 접근 제어자가 `public`으로 설정된 건 의도된 걸까요? > > 현재 상황에서는 `MainController`의 생성자가 퍼블릭으로 열려 있기 때문에 다른 클래스에서 해당 생성자를 가지고 얼마든지 `MainController`를 ...
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
substring 대신 startsWith을 사용하면 더 간결해질 수 있습니다! 성능상으로도 더 유리하다는 것 같습니다
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
overflow의 위험이 있어보입니다. 추가로 stream의 사용을 고려해 보시는 것도 좋을 것 같아요!
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`operand` 변수명이 단수형으로 되어 있지만, 실제로는 여러 값을 담고 있습니다. `operands`로 변경하는 것은 어떤가요?
gyuoo created a review on a pull request on woowacourse-precourse/java-calculator-7
각 클래스가 자신의 책임을 명확히 하고 있어 보입니다! 전반적으로 너무 깔끔하고 좋네요 😄 리뷰하러 왔다가 많이 배우고 갑니다. 클래스 구분이 명확한 게 너무 좋아요
gyuoo created a review on a pull request on woowacourse-precourse/java-calculator-7
각 클래스가 자신의 책임을 명확히 하고 있어 보입니다! 전반적으로 너무 깔끔하고 좋네요 😄 리뷰하러 왔다가 많이 배우고 갑니다. 클래스 구분이 명확한 게 너무 좋아요
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
MVC 패턴을 사용했기 때문에 Controller로 이름 짓게 되었습니다! 그리고 run 이름은 많이 사용하는 것 같아 그렇게 지었는데 혹시 어떤 이름이 좋을까요😂 main은 실행하는 책임만 가진다고 생각해서 그렇게 작성했습니다!
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저에겐 네이밍이 정말 어려운 것 같아요..ㅠㅠ input -> inputNumberStrings, numberStrings validate -> validateNumber 이런식으로 바꾸는게 좋을 것 같습니다! 네이밍으로 변수나 메서드가 그 목적을 드러내도록 더 신경써보도록 하겠습니다!
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
불필요한 임포트가 증가하게 되고 이로 인해 임포트한 클래스 커진다면 컴파일 시간이 증가할 가능성이 있겠네요,, 전 코드가 깔끔해보여서 사용했는데 스타일 가이드에도 와일드 카드를 쓰지 말라는 내용이 있었네요,, 😂 알려주셔서 감사합니다!
AbsoluteValues created a review comment on a pull request on woowacourse-precourse/java-calculator-7
생각을 좀 더 깊게 하고 싶었습니다.
kiwoook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
커스텀 구분자에 여러 개를 입력받아도 전부다 다른 구분자가 될 수 있도록 처리하셨네요! 저 같은 경우에는 _문자열 앞부분의 "//"와 "\n" 사이에 위치하는 **문자**_ 라는 지시사항때문에 커스텀 구분자에는 문자 한개만 들어가는 걸로 제약하였는데 어떻게 생각하시나요?
kiwoook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
특수문자 처리까지 꼼꼼하게 하셨네요!
kiwoook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
객체의 이름이 PositiveNumber이므로, 해당 객체를 생성하는 메소드명에 객체 이름을 포함시키지 않아도 충분히 의미가 전달된다고 생각합니다
kiwoook created a review on a pull request on woowacourse-precourse/java-calculator-7
계층 간에 분리가 잘되어 있는 코드라 많이 배워갑니다. 로직에 대해서는 서로 다른 생각을 가지고 있기에 해당 로직에서는 적절하다고 생각하네요! 그래서 저는 가독성 측면에서 많이 보았습니다. 수고하셨습니다!
kiwoook created a review on a pull request on woowacourse-precourse/java-calculator-7
계층 간에 분리가 잘되어 있는 코드라 많이 배워갑니다. 로직에 대해서는 서로 다른 생각을 가지고 있기에 해당 로직에서는 적절하다고 생각하네요! 그래서 저는 가독성 측면에서 많이 보았습니다. 수고하셨습니다!