Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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
> '클래스명.메서드명'으로도 충분히 기능이 유추가 되어서 주석을 작성하지 않으셔도 될 것 같습니다! 네 그쵸 제가봐도 과한감이 있네요 ㅎㅎ
jintakkim 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
> 단순 setter는 save보다는 set이 일반적인 표기입니다. 단순히 값을 저장 하는 setter보다 좀더 여러기능이 있어서 save라고 정했습니다
SeoMoonk created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`final`은 불변한다는 의미를 가지고 있는데 저 친구는 구분자를 추출할 때마다 변경되는 친구기에 안붙이려고 했던 것 같습니다! 또한, `ArrayDeque`를 사용하는 이유는 `addLast()` 와 `removeFirst()` 메서드를 통해 큐 구조를 사용함으로써 `특수 구분자가 있다면, 일반 구분자보다 먼저 처리를 진행한다` 는 저만의 생각...
jintakkim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 단순 setter는 save보다는 set이 일반적인 표기입니다. 단순히 값을 저장 하는 setter보다 좀더 여러기능이 있어서 save라고 정했습니다
jintakkim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
단순히 값을 저장 하는 setter보다 좀더 여러기능이 있어서 save라고 정했습니다
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
> 그럼 예를 들어 하나의 입력에 대해 덧셈, 뺄셈, 곱셈 등을 한번에 제공하는 것을 염두에 두고 만드신건가요?? 이번 미션이 덧셈뿐이라 그럴일은 없겠지만 쉽게 확장할 수 있게 만들었습니다.
jintakkim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> getNumbers에서 새로운 인스턴스를 생성하신 이유가 있을까요?? 기존 NumberRepository의 저장소를 밖에서 임의로 변경하는 것을 막기위해 값을 한번 복사해서 리턴했습니다.