Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
hyojaeKwon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
동의합니다! config 클래스를 만들어서 DI구조를 만드는 것이 아니라면 클래스 안에 생성자를 두는게 더 좋을 것 같습니다.
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
InputView에서와 비슷하게, 출력값을 포매팅하는 로직은 OutputView의 책임이라고 생각되어서 내부에 구현하게 되었는데요! 확장성을 고려했을 때는 해당 로직을 분리하는 것이 좋아보여요! 감사합니다 ㅎㅎ
gorapaduckoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
추천 감사합니다! 공부해볼게요 👀
ariimo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
split배열에서 쓰이는 건 split[0] 뿐인데, 굳이 배열 전체를 넘기지 않아도 될 것 같습니다:)
ariimo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
IllegalArgumentException발생 뿐만 아니라, 의도했던 예외 메세지와 일치하는 지도 확인하셨군요. 배워가겠습니다:)
ariimo created a review on a pull request on woowacourse-precourse/java-calculator-7
대체적으로 앞 리뷰어 분들과 겹치는 의견이 많아 저는 이만 마무리하겠습니다. 1. MVC 패턴 적용 2. 에러 메세지 상수화 3. 다양한 예외에 대한 고려 위 3가지를 배워 갑니다. 특히 상세한 예외 처리는 깊게 고민하셨다는 걸 느낄 수 있었습니다. 코드 잘 보고 갑니다:)
ariimo created a review on a pull request on woowacourse-precourse/java-calculator-7
대체적으로 앞 리뷰어 분들과 겹치는 의견이 많아 저는 이만 마무리하겠습니다. 1. MVC 패턴 적용 2. 에러 메세지 상수화 3. 다양한 예외에 대한 고려 위 3가지를 배워 갑니다. 특히 상세한 예외 처리는 깊게 고민하셨다는 걸 느낄 수 있었습니다. 코드 잘 보고 갑니다:)
bogyeom0922 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 테스트 코드를 기존 코드만 사용하고 추가로 작성하지 않았었는데, 작성하신 것을 보고 반성하게 되네요..! 다른 분들 말처럼 application, test코드 모두 기능별로 좀 더 자세히 분리시켜서 코드를 작성하면 훨씬 가독성이 좋을 것 같아보여요 !!
bogyeom0922 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
readme에 요구사항을 굉장히 자세히 작성하셨네요 ! 자세히 써두니까 개발할 때 더 수월할 것 같아보여요 👍
bogyeom0922 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 하시느라 고생 많으셨습니다 !! :)
bogyeom0922 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 하시느라 고생 많으셨습니다 !! :)
dye0p 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) > ``` 잘 몰랐는데 하나 배워갑니다! :)
dye0p created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> CalculateController에서 메서드 분리가 조금 과한 것 같다는 생각이 듭니다 `calculateSum`과 `displayResult` 메서드를 한번 더 분리해서, run 메서드만 봤을 때는 이해하기가 좋지만 협업하는 관점에서, 두 메서드는 다른 메서드를 한번 호출하는 역할만 수행하는데 로직을 깊게 이해하려면 한번 더 메서드를 타...
aka-nick created a review comment on a pull request on woowacourse-precourse/java-calculator-7
타입이 String인 이상 어떠한 구문일거니까, `STATEMENT`를 제외하면 좀 더 간결하게 같은 의미를 전달할 수 있지 않을까요? 구문이 아닌 어떤, 구분자라든지.. 헤더라든지.. 하는 것들이 함께 정의되어있다면 경우가 다를 수도 있겠지만요!
aka-nick created a review comment on a pull request on woowacourse-precourse/java-calculator-7
예외 메시지도 좋은 정보이기 때문에 정성들여 작성해주면 스택트레이스를 다 뒤지지 않아도 버그를 찾기 쉬워지고 도움이 되어요 👍
aka-nick created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력 문자에 따라, 입력 문자의 문자셋에 따라, 한 문자를 표현할 수 있는 길이가 달라질 수 있다는 사실 알고 계셨나용? 🤗 입력 문자가 아주 제한되어 있는 것이 아니라면 문자 위치를 추측하는 것(만약 커스텀 구분자가 있으면 몇번째 자리에는 뭐가 있을거야)보다 헤더와 푸터의 위치를 찾고 그 사이의 문자를 찾아내는 것이 정확해요! 물론 길이체크를 따...
aka-nick created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아래의 @dradnats1012 님 의견과 비슷한 코멘트인데요. 메서드 시그니처(`리턴값 메서드명(파라미터)`)가 좀 더 많은 정보를 주면 좋을 것 같아요. 반환받는 값이 `int output`인데 `int`는 원시타입이라 어떤 용도인지 추측이 어렵구(객체타입이면 의도를 줄 수 있겠죠?), `output`은 무엇의 output인지 추측이 어려워...
aka-nick created a review on a pull request on woowacourse-precourse/java-calculator-7
좋은 코드 잘 보고 갑니다! 👍 케이스가 잘 정리된 문서가 인상깊었어요. 간략하면서도 필요한 정보를 알기 쉬웠습니다!
aka-nick created a review on a pull request on woowacourse-precourse/java-calculator-7
좋은 코드 잘 보고 갑니다! 👍 케이스가 잘 정리된 문서가 인상깊었어요. 간략하면서도 필요한 정보를 알기 쉬웠습니다!
digitpic created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분은 저도 고민이 들었던 부분입니다.. validate 라는 메서드가 형변환한 값을 return 해버리면 검증과 형변환이라는 두가지의 책임을 지게 된다고 생각하여 두 가지 메서드로 나누어 parseInt() 메서드가 두 번 호출되는 로직이 생겼습니다 결국은 하나의 메서드가 하나의 책임을 지도록 하기 위해 이처럼 구현했습니다! 중복...
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
작은 서비스인만큼 프로젝트를 간결하게 유지하려고 하다보니 객체에 책임이 많아졌네요 😅 우선 파서가 몰라도되는 유효성 검증 로직을 분리하면 일차적으로 개선이 될 것 같아요! 고민해보겠습니다 :)
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
꼼꼼한 예외처리 좋네요! 👍
