Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
Jonghwan-Lee7 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
설계를 잘못한 결과입니다.... Aplication에 메서드 하나만 나오게 하고 입출력은 오직 view에서만 하고 controller에서 컨트롤하는 걸로 끝내야할 것 같습니다.
kkmdevel created a review comment on a pull request on woowacourse-precourse/java-calculator-7
불필요한 0을 포함하지 않는다고 하셨는데 BigDecimal을 사용하면 바로 0이 없어지는게 아닌가요? 궁금해서 여쭤봅니다!
kkmdevel created a review comment on a pull request on woowacourse-precourse/java-calculator-7
자바 네이밍 규칙을 더 공부하시면 좋을거같아요! ConsoleSerive는 콘솔의 어떤 서비스를 하는지 알려주는 ConsoleIOService로 DelService는 한눈에 보면 삭제 서비스처럼 보이기도해요! 길어져도 좋으니 축약은 안하시는게 좋을거같아요! DependencyInjection보다는 실행을 담당하는 클래스라서 AppRunner등 더...
kkmdevel created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 여기서 빈 문자열을 받으면 요소가 없다는 오류가 떴었는데 어떻게 해결하셨나요?
kkmdevel created a review on a pull request on woowacourse-precourse/java-calculator-7
README.md 정리를 잘해놓으셔서 기능들을 이해하는데 수월했습니다. commit들을 보면서 많이 고치신게 눈에 보여서 노력한 모습이 너무 좋았습니다! 다만, commit 방법을 좀 더 알아보시면 좋을거같아요! docs/test등등 문서와 test는 다 따로 commit해주시면 보기에 편할거같아요! 그리고 자바 네이밍 규칙도 보시면 좋을거같습...
kkmdevel created a review on a pull request on woowacourse-precourse/java-calculator-7
README.md 정리를 잘해놓으셔서 기능들을 이해하는데 수월했습니다. commit들을 보면서 많이 고치신게 눈에 보여서 노력한 모습이 너무 좋았습니다! 다만, commit 방법을 좀 더 알아보시면 좋을거같아요! docs/test등등 문서와 test는 다 따로 commit해주시면 보기에 편할거같아요! 그리고 자바 네이밍 규칙도 보시면 좋을거같습...
WARDKEY created a review on a pull request on woowacourse-precourse/java-calculator-7
역할과 책임에 따라 구조를 잘 나누신거 같습니다! 많이 배울 수 있었고, 이번 주도 화이팅하세요!!
WARDKEY created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자 집합을 Enum으로 선언해보는 것도 괜찮을거 같습니다!
WARDKEY created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@BreforeEach로 미리 객체 생성하는 방식 좋은 것 같습니다!
WARDKEY created a review on a pull request on woowacourse-precourse/java-calculator-7
역할과 책임에 따라 구조를 잘 나누신거 같습니다! 많이 배울 수 있었고, 이번 주도 화이팅하세요!!
HwiYul-G created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@zzioozzioo 저는 도메인 주도 개발을 공부하거나 사용해 본 적이 없어서 잘 모르지만, 현재 `Number` 는 그를 이용하시려 시도한 것처럼 느껴집니다. 배움의 측면에서 적용한 것이 좋다고 생각합니다! Numbers 객체는 비즈니스 로직을 담당하는 service에서만 사용하고 싶은데 이것이 controller까지 가는 점에서 불편함을...
HwiYul-G created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@wjdwnsdnjs13 구현하신 내용을 `main`에 merge 하지 않으셨다면 `main`기반으로 새로운 branch를 만드시면 됩니다
HwiYul-G created a review comment on a pull request on woowacourse-precourse/java-calculator-7
현재 코드 구현이 그렇게 느낄 수 있게 구현되어 있긴 하네요! 일반적으로 service가 비즈니스 로직이고 domain은 사용 경우에 따라 쵸큼 달라질 수 있을 것 같긴한데.. 흠 현 상황에서 비즈니스 로직을 위한 object가 더 적합하지 않을까요?!
rookedsysc created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분 재표님 코드 보고 배웠던 내용인데 수정을 못했네요~
jaepyo-Lee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
계산에 대한 인터페이스 좋네요!
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
파싱하는 객체도 나눠도 좋을 것 같아요
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
검증 로직을 수행하는 객체도 별도로 분리해도 좋을 것 같습니다
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
문자열 입력 책임을 갖는 객체를 분리해도 좋을 것 같아요
koosco created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다. 절차 지향 프로그래밍에 익숙하신 것 같은데 책임에 따라 객체를 나누고 각 객체에게 책임을 할당하여 서로 데이터를 주고 받도록 코드를 리팩토링해봐도 재밌을 것 같아요! SOLID에 대해 읽어보시면 도움 되실 것 같습니다 :)
koosco created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다. 절차 지향 프로그래밍에 익숙하신 것 같은데 책임에 따라 객체를 나누고 각 객체에게 책임을 할당하여 서로 데이터를 주고 받도록 코드를 리팩토링해봐도 재밌을 것 같아요! SOLID에 대해 읽어보시면 도움 되실 것 같습니다 :)
jaepyo-Lee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
중복된 구분자는 없어도 되니, Set자료형을 사용하는 것도 좋을것 같아요!
jaepyo-Lee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자 자체를 클래스로 만들어 사용하면 구분자에 따라 행동을 추가할수 있으니 더 좋을 것 같아요!
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
else를 빼면 가독성을 높일 수 있을 것 같아요
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
type casting과 parsing, 계산 로직이 하나의 객체에 있어 너무 많은 책임을 수행하고 있는 것 같아요. 별도의 객체로 분리하는건 어떻게 생각하시나요?
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
customString과 basicString 메서드는 상황에 따라 한 가지 메서드만 수행되는 것 같아요. 추상화와 상속을 통해 공통되는 로직을 묶고 별도의 객체로 나누면 코드의 응집도가 높아질 것 같습니다!
koosco created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다! 객체를 책임에 맞게 더 나누어도 좋을 것 같아요~