Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
nns503 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
검증 클래스를 만들어서 활용하는 방법이 좋은 것 같아요 :)
KimEunSeop created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기능 별로 함수를 잘 나눈 것 같아요!
KimEunSeop created a review comment on a pull request on woowacourse-precourse/java-calculator-7
상수처리를 잘 적용한 예시인 것 같아요!
KimEunSeop created a review comment on a pull request on woowacourse-precourse/java-calculator-7
각각 기능의 입력값과 출력값을 명시해주어 흐름이 더 잘 보이는 것 같아욥! 배워갑니다.
KimEunSeop created a review comment on a pull request on woowacourse-precourse/java-calculator-7
계산기와 관련된 작동 흐름만 관장하는 컨트롤러인가욥? 개인적인 생각으로 만약 그렇다면, 전체 흐름을 관장하는 컨트롤러도 만들면 메인 함수의 로직을 조금 더 깔끔하게 할 수 있을 것 같다고 생각이 들어요!
KimEunSeop created a review on a pull request on woowacourse-precourse/java-calculator-7
기능별로 깔끔하게 잘 구현된 1주차 과제 같아요. 좀 더 기능이 많아지고 점점 더 복잡해지더라도 이런 방향성을 계속 추구할 수 있도록 저도 노력해보겠습니다!
KimEunSeop created a review on a pull request on woowacourse-precourse/java-calculator-7
기능별로 깔끔하게 잘 구현된 1주차 과제 같아요. 좀 더 기능이 많아지고 점점 더 복잡해지더라도 이런 방향성을 계속 추구할 수 있도록 저도 노력해보겠습니다!
GRAY-DDOT created a review comment on a pull request on woowacourse-precourse/java-calculator-7
변수와 인자를 줄일 수 있겠네요! 감사합니다!
GRAY-DDOT created a review comment on a pull request on woowacourse-precourse/java-calculator-7
확실히 그런게 좋은 것 같아요 최소한 리턴을 숫자 문자열로 바꾸는게 좋아보이네요...ㅠㅠ
GRAY-DDOT created a review comment on a pull request on woowacourse-precourse/java-calculator-7
유틸은 static! 좋은 지적 감사합니다!
cherryiJuice created a review comment 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
그렇게 해볼까도 생각해봤는데요,, 제가 생각하기에 첫 미션으로는 조금 과할 것 같아 인터페이스 사용은 자제했습니다 ㅎㅎ 좋은 의견 감사합니다! 다음 미션부터는 인터페이스 사용도 고려해볼게요 ☺️
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의 사용을 고려해 보시는 것도 좋을 것 같아요!