Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
cherryiJuice 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
이런 부분 짚어주셔서 감사합니다..!! 아직 좋은 코드를 짜는 건 멀고도 먼 길이군요,,
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
설정 방법은 몰랐는데 알려주셔서 감사합니다 😊 불변성을 보장해서 값의 예상치 못한 오류(예: 재할당되서 발생하는 오류) 같은 것들을 줄일 수 있고 추가적인 최적화가 가능하다고 들었습니다! 단점은 이번 기회에 생각해보게 되었는데요 😅 만약, 재할당이 필요할 경우 값을 변경하지 못하기 때문에 유연성이 떨어지는 것 같고 이 때문에 새로운 필드를 생...
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좋은 리뷰 감사합니다! 다른 미션부터 적용시켜보도록 하겠습니다!
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Console을 close해주지 않았네요🥲 다음부터는 더 신경써보도록 하겠습니다!
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
group을 이용해서 찾지 않아도 돼서 그렇게 했습니다! 문자열과 관련된 여러가지 메소드를 찾아보고 다른 미션에 적용하도록 해보겠습니다!
changbill created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 과제 하시느라 고생하셨습니다! 2주차도 화이팅!
changbill created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 과제 하시느라 고생하셨습니다! 2주차도 화이팅!
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
리뷰 요청 주셔서 감사합니다! 각 역할을 세세하게 분리한 것이 좋아보이네요 다만 결합도에 조금 더 신경을 쓰시면 더 좋을 것 같습니다 제 리뷰가 조금이나마 도움이 되셨으면 좋겠습니다 😃