Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
daeuun created a review comment on a pull request on woowacourse-precourse/java-calculator-7
먼저 테스트 코드를 여러 케이스를 구분해서 작성하다보니 클래스마다 역할을 분배해서 작은 기능부터 만들었는데요. 컴포넌트 별로 추상화된 퍼블릭 메서드를 열어두고 조건에 따라 사용해서 내부 로직은 변동되더라도 유연하게 사용하려는 처음 목적이었습니다! 그런데 조립하는 과정에서 중복되는 로직이 자꾸 생긴거 같아서 더 혼란이 가중된거 같아요. 좋은 제안 감사...
daeuun created a review comment on a pull request on woowacourse-precourse/java-calculator-7
넵! 이부분도 유효성 검사 클래스 분리하면서 적용하는게 좋을거 같은 의견으로 같이 반영해보겠습니다!
daeuun created a review comment on a pull request on woowacourse-precourse/java-calculator-7
넵! 가독성 때문에 분리했었는데 오히려 단순 명확한 조건문에서 분리하는게 더 가독성을 떨어트리는거 같네요..!
daeuun created a review comment on a pull request on woowacourse-precourse/java-calculator-7
넵 반영해보도록 하겠습니다!
daeuun created a review comment on a pull request on woowacourse-precourse/java-calculator-7
초기에 모든 유효성 검사를 분리하려고 클래스를 따로 빼두었는데, 모델 유효성 검사만 적용하였네요. 여러개의 클래스로 다시 분리해야할지 고민하다가 저도 정확한 판단이 서지 않아서 구현하는 과정에서 개별 클래스에서 유효성 검사를 진행했습니다! 다음번엔 원래 목적대로 다른 패턴을 사용해서 분리하는 쪽으로 가볼 생각이에요!
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
맞네요~ 사실 저는 모든 기능을 다 구현했나 확인 차원에서 마지막에 한 거였는데 말씀하신 방법이 보는 입장에서는 더 좋을 거라 생각이 들어요!
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자에 n이 포함되는 경우 문제가 발생할 수 있어보입니다. `\\n`을 상수화하는 등의 과정을 거친 뒤 명확히 사용하는 것이 좋아보여요!
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
오타 확인해 주세용
gyuoo created a review on a pull request on woowacourse-precourse/java-calculator-7
리뷰 요청 주셔서 감사합니다! 각 역할을 세세하게 분리한 것이 좋아보이네요 다만 결합도에 조금 더 신경을 쓰시면 더 좋을 것 같습니다 제 리뷰가 조금이나마 도움이 되셨으면 좋겠습니다 😃
gyuoo created a review on a pull request on woowacourse-precourse/java-calculator-7
리뷰 요청 주셔서 감사합니다! 각 역할을 세세하게 분리한 것이 좋아보이네요 다만 결합도에 조금 더 신경을 쓰시면 더 좋을 것 같습니다 제 리뷰가 조금이나마 도움이 되셨으면 좋겠습니다 😃
honggyuSHIN created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드 마다 테스트를 돌리신 점이 인상 깊었습니다. 제가 잘 아는 건 아니지만 Number에 너무 많은 메서드가 들어있는 것 같아 역할이나 책임에 따라 구분을 하면 더 좋지 않을까 생각했습니다. 많이 배우겠습니다. 감사합니다.
honggyuSHIN created a review on a pull request on woowacourse-precourse/java-calculator-7
수고 많으셨습니다. 감사합니다.
honggyuSHIN created a review on a pull request on woowacourse-precourse/java-calculator-7
수고 많으셨습니다. 감사합니다.
seoyeon-jung created a review comment on a pull request on woowacourse-precourse/java-calculator-7
와 저는 개인적으로 이렇게 오류를 한번에 모아서 보니까 가독성 좋다고 생각합니다!
yeong0jae created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Set으로 관리..! 저는 생각하지 못한 방법이네요. Set으로 관리했을때 장점과 단점을 알아봐야겠습니다ㅎㅎ 감사합니다
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
물론 밑에서 무조건 num에 값을 정의하고 있지만, default value를 선언시에 같이 작성해주는게 조금 더 안전한 코드가 아닐까 싶은데 어떻게 생각하시나요?
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
calculate라는 메소드 안에서 패턴분리하고 계산까지하는 로직이 하나로 합쳐셔 있는 것 같아서 [SRP 원칙](https://inpa.tistory.com/entry/OOP-%F0%9F%92%A0-%EC%95%84%EC%A3%BC-%EC%89%BD%EA%B2%8C-%EC%9D%B4%ED%95%B4%ED%95%98%EB%8A%94-SRP-%EB%8B...
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
formatting을 적극적으로 활용하면 불필요한 공백을 줄일 수 있을 것 같아요!
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
1주차 공통 피드백에도 나와있지만, 불필요한 주석은 제거하는게 조금 더 좋아보여요
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Exception이 발생할 수 있는 부분에 throws를 추가해서 해당 부분에서 exception handling 혹은 발생할 수 있다는 것을 명시해주면 더 좋을 것 같아요!