Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
marin-Rim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
DTO가 프로세스 간의 데이터 전달하는 객체군요. 새롭게 배워갑니다! 구분자들의 처리와 전달 과정을 같이 묶어서 객체로 만든 점이 좋은 것 같아요.
marin-Rim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
delimiters의 접근제어자를 public으로 설정한 이유가 있을까요? StringParser에서 호출하는 용도라면 Getter로 만드는 방법이 더 안전할 것 같다는 생각이 들었습니다! <img width="519" alt="image" src="https://github.com/user-attachments/assets/f1c85b94-b755...
marin-Rim created a review on a pull request on woowacourse-precourse/java-calculator-7
깔끔하게 구분자를 처리하는 방법을 배우고 갑니다 :)
marin-Rim 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
음.. 번갈아 봐야 하는 문제에 대한 해결 방법이 상수화 선언시 네이밍을 자세하게 설명한게 아닐까 생각합니다!
rjswjddn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
미션 예시에 입력 ""의 결과는 0이라고 나와 있어서 수정이 필요해요!!
rjswjddn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
커스텀 구분자 추출을 따로 메서드로 빼면 if문이 많이 중첩되는 것을 막을 수 있을 것 같아요!
rjswjddn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 이런식으로 구현했던 것 같은데..... 검증하고 변환 부분을 나누면 더 가독성이 좋아질 것 같습니다!
rjswjddn created a review on a pull request on woowacourse-precourse/java-calculator-7
고생하셨습니다!! 2주 차도 화이팅입니다!!
rjswjddn created a review on a pull request on woowacourse-precourse/java-calculator-7
고생하셨습니다!! 2주 차도 화이팅입니다!!
jintakkim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 전체적으로 어떤 목적성과 기준을 갖고 코드를 분리하려 노력하신 흔적이 보였습니다! 가독성도 좋고, 코드의 논리가 이해하기 좋았습니다. 수고하셨습니다 :) 좋은 리뷰 감사합니다. 저도 yjy님 코드보면서 영감을 많이 받았는데 참고해서 2주차 미션 수행할 수 있도록 노력하겠습니다.
jintakkim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> parse가 계산기 고유의 기능은 아니니 유틸성 기능이라는 해석도 동의하지만, 저는 이번 미션에서는 꽤 비중있는 비즈니스 로직이 아닐까? 라고 생각했습니다! 그쵸 여기가 저도 핵심이라고 생각했습니다.
jintakkim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
맞는 말씀입니다. 기능이 복잡하지 않아서 바로 클레스를 만들까 고민했는데 좋은 지적 감사합니다.
SeoMoonk created a review comment on a pull request on woowacourse-precourse/java-calculator-7
오호? 이건 또 처음 보는 내용이네요 서비스에서 로직을 처리하는게 맞는다는 생각이 들었었는데 validation(?) 과 같은 것을 처리하는 다른 클래스가 있어도 좋을 것 같다는 생각이 들었어요! 경섭님 코드 확인해 보면서 좀 더 배워보겠습니다!
SeoMoonk created a review comment on a pull request on woowacourse-precourse/java-calculator-7
매우매우 공감합니다. 저도 처음에 계산하는 곳, 문자열을 처리하는 곳 두가지로 생각했는데 한 패키지 안에서 두 개의 서비스를 만드는게 괜찮은걸까? 라는 생각에 이렇게 처리 했었는데. 좀 더 알아봐야 할 것 같아요!
jintakkim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
인풋처리 코드쪽에서 구분자/숫자 추출로직이 들어가있는 것을 밖으로 빼내면 훨씬 좋은 설계가 되겠네요.
SeoMoonk created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그렇네요. view 쪽은 신경을 많이 못쓰고 넘어갔던 것 같아요. 지적 감사합니다!
SeoMoonk created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 부분은 저도 동감하고 있습니다 😢 말씀해 주신대로 메서드 분리에 대해 좀 더 많이 고민하고 스트림도 공부해서 적용해 보도록 노력해 보도록 하겠습니다! 또한 클린코드 원칙은 책 이름은 들어봤는데 정확히 어떤 내용인지 몰라 이것도 공부하도록 해보겠습니다...!
SeoMoonk created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 하면서 계속 복잡하다는 생각을 하긴 했었습니다. 메서드 분리를 더 고안해보거나 알려주신 `stream`을 공부해서 노력해 보도록 하겠습니다!
jintakkim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> '클래스명.메서드명'으로도 충분히 기능이 유추가 되어서 주석을 작성하지 않으셔도 될 것 같습니다! 네 그쵸 제가봐도 과한감이 있네요 ㅎㅎ