Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
값을 반환하는 메소드 네이밍 컨벤션에 따르는 앞이 get으로 시작합니다. 다음 블로그 참고하시면 좋을 듯 합니다. https://tecoble.techcourse.co.kr/post/2020-04-26-Method-Naming/
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메시지를 enum으로 관리하는 게 좋네요. 배워갑니다.
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 패턴에 대한 전반적인 이해가 보이네요. 전체적인 구조를 잘 짜신 것 같습니다. 클래스, 메서드, 변수 네이밍 컨벤션만 살펴보시면 더욱 좋은 코드를 작성하실 것 같습니다.
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자 추출+검증은 별도 메서드로 추출하는게 더 좋겠네요! 감사합니다!
kaswhy created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 코드 리뷰를 받으면서 많이 받은 리뷰가 검증 로직의 책임이 어디에 있어야 하는지 생각해 보면 좋을 것 같다는 말이었는데요, 정말 좋은 규칙인 것 같아요! getter와 setter 부분에서도 어떻게 하면 같은 로직을 getter와 setter를 사용하지 않고 구현했는지도 고민되었는데, 많이 배워가요!
kaswhy created a review comment on a pull request on woowacourse-precourse/java-calculator-7
from이라는 메소드명이 조금 모호하게 느껴지네요...!
kaswhy created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이런 부분 설명 정말 좋네요! 제가 고민했던 부분들이 많이 완화되는 부분인 것 같습니다. 추가적으로, 리뷰 요청하실 때 MVC 패턴을 적용하지 않은 코드라고 하셨는데 저는 개인적으로 지금 준서님이 작성하신 코드가 MVC 패턴과 유사하게 작동하고 있다고 생각합니다! 이 부분은 어떻게 생각하시는지 궁금해요~
kaswhy created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요, 준서님! 밀린 리뷰 요청이 많아 이제야 리뷰 드립니다. MVC 패턴을 사용하지 않으셨다고 하셨는데, 큰 흐름 자체가 저에게는 MVC와 비슷하게 느껴지는 코드였어요. 그리고 객체지향 원칙을 철저히 지키시고, 자바의 기본 문법이 왜 쓰이는지를 잘 알고 사용하시는 것 같아 배울 점이 정말 많은 코드라고 생각했습니다! 다른 분들과 리뷰하신 부분...
kaswhy created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요, 준서님! 밀린 리뷰 요청이 많아 이제야 리뷰 드립니다. MVC 패턴을 사용하지 않으셨다고 하셨는데, 큰 흐름 자체가 저에게는 MVC와 비슷하게 느껴지는 코드였어요. 그리고 객체지향 원칙을 철저히 지키시고, 자바의 기본 문법이 왜 쓰이는지를 잘 알고 사용하시는 것 같아 배울 점이 정말 많은 코드라고 생각했습니다! 다른 분들과 리뷰하신 부분...
hyojaeKwon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
동의합니다! 2주차 미션에서는 입/출력의 controller기능을 담당하는 별도의 클래스를 도입해보겠습니다!!
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
MVC 패턴에서 객체의 레이어를 구분하는게 아직 익숙하지가 않네요 ㅎㅎ 좋은 의견 감사합니다 좀 더 고민해볼게요 ☺️
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) > ``` 잘 몰랐는데 하나 배워갑니다! :)
