Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
holyPigeon created a review on a pull request on woowacourse-precourse/java-calculator-7
리뷰가 조금 늦어졌네요...ㅠ 무사히 완료했습니다! 1주차 너무 고생하셨고, 2주차도 같이 화이팅해봐요!
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
오 그렇게 하는게 해당 메서드가 무슨역할을 하는지 더 알아보기 쉬울것 같네요!! 저도 네이밍하는게 너무 어려워서 항상 많은 시간을 할애하는 부분중 하나라고 생각해요.....
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
미션을 진행하면서 검증을 어느 부분에서 관리하는게 좋을지 정말 많이 고민하다가 결국엔 분리하지 못했습니다... 다음 미션부터는 검증하는 관리 포인트를 한곳으로 모으는 방식으로 적용해보겠습니다!! 감사합니다!!
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
지금 생각해보면 같은 동작을 하는 메서드를 묶어서 한개의 클래스로 분리하여 관리하는게 좋을 것 같다는 생각이 드네요!! 좋은 의견 감사합니다!!
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
확실히 네이밍에 신경을 더 써야할 것 같다는 생각이 드네요...!! 좋은 정보 감사합니다!!
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
혹시 구분자로 숫자가 들어오는 경우는 어떻게 생각하시나요?
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
정규표현식을 능숙하게 사용하시네요.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분이 정말 좋네요. 많이 배웁니다.
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 패턴을 차용하게끔 되어 있는 경우도 있으니까요.
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
객체를 생성하는 방법과 정적 메서드로 처리하는 방법 중 고민하다가 정적 메서드로 처리하는게 더 효율적이라고 생각해서 진행했어요!! 하지만 그에 맞는 근거를 제시하기엔 제 지식이 불충분 하다고 생각했는데 집어주셔서 감사합니다!!
yesjuhee created a comment on a pull request on woowacourse-precourse/java-calculator-7
리뷰 내용 반영한 코드는 [개인 저장소](https://github.com/yesjuhee/java-calculator-7)에 커밋하여 푸시했습니다 :)
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
DelimiterConstants의 두 상수가 현재 DelimiterValidator와 StringParser 클래스에서 사용되고 있어, 이를 상수 패키지로 따로 관리하게 되었습니다. 저희 기준은 현재 클래스뿐만 아니라 다른 클래스에서도 사용될 여지가 있는 상수는 상수 패키지에서 관리하자는 것이었는데, 이 부분에 대해 길준님의 의견이 궁금합니다!
eraser502 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이번 미션에서는 네이밍을 전혀 신경쓰지 못했던 것 같아요.. 다음 미션부터는 꼭 메서드의 역할에 맞는 네이밍을 해보도록 하겠습니다!!
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
값이 없으면 0을 반환하는 편이 낫지 않을까요? https://velog.io/@oyeon/%EC%95%84%EC%9D%B4%ED%85%9C55.-%EC%98%B5%EC%85%94%EB%84%90-%EB%B0%98%ED%99%98%EC%9D%80-%EC%8B%A0%EC%A4%91%ED%9E%88-%ED%95%98%EB%9D%BC
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. 많이 배우고 갑니다. 건승을 빕니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. 많이 배우고 갑니다. 건승을 빕니다.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이해가 되지 않습니다. 주석을 달아주시면 편이 좋을 것 같습니다.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드명 앞에 get를 붙이는 편이 네이밍 컨벤션에도 걸맞고 역할을 명확히 드러낸다고 봅니다.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 클래스가 model일까요? 전략 패턴에 더 가깝지 않나요? 이런 접근은 처음 봐서 신선하네요. 저라면 stringParingService, NumbersAccumulateService 로 기능 나누어서 분리할 것 같긴 합니다.
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 패턴을 잘 구현하셨네요. 배우고 갑니다. 특히 테스트를 나누어서 작성하신 것도 배울 점이네요.