Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`operand` 변수명이 단수형으로 되어 있지만, 실제로는 여러 값을 담고 있습니다. `operands`로 변경하는 것은 어떤가요?
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
각 클래스가 자신의 책임을 명확히 하고 있어 보입니다! 전반적으로 너무 깔끔하고 좋네요 😄 리뷰하러 왔다가 많이 배우고 갑니다. 클래스 구분이 명확한 게 너무 좋아요
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
MVC 패턴을 사용했기 때문에 Controller로 이름 짓게 되었습니다! 그리고 run 이름은 많이 사용하는 것 같아 그렇게 지었는데 혹시 어떤 이름이 좋을까요😂 main은 실행하는 책임만 가진다고 생각해서 그렇게 작성했습니다!
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저에겐 네이밍이 정말 어려운 것 같아요..ㅠㅠ input -> inputNumberStrings, numberStrings validate -> validateNumber 이런식으로 바꾸는게 좋을 것 같습니다! 네이밍으로 변수나 메서드가 그 목적을 드러내도록 더 신경써보도록 하겠습니다!
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
불필요한 임포트가 증가하게 되고 이로 인해 임포트한 클래스 커진다면 컴파일 시간이 증가할 가능성이 있겠네요,, 전 코드가 깔끔해보여서 사용했는데 스타일 가이드에도 와일드 카드를 쓰지 말라는 내용이 있었네요,, 😂 알려주셔서 감사합니다!
AbsoluteValues created a review comment on a pull request on woowacourse-precourse/java-calculator-7
생각을 좀 더 깊게 하고 싶었습니다.
kiwoook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
커스텀 구분자에 여러 개를 입력받아도 전부다 다른 구분자가 될 수 있도록 처리하셨네요! 저 같은 경우에는 _문자열 앞부분의 "//"와 "\n" 사이에 위치하는 **문자**_ 라는 지시사항때문에 커스텀 구분자에는 문자 한개만 들어가는 걸로 제약하였는데 어떻게 생각하시나요?
kiwoook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
특수문자 처리까지 꼼꼼하게 하셨네요!
kiwoook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
객체의 이름이 PositiveNumber이므로, 해당 객체를 생성하는 메소드명에 객체 이름을 포함시키지 않아도 충분히 의미가 전달된다고 생각합니다
kiwoook created a review on a pull request on woowacourse-precourse/java-calculator-7
계층 간에 분리가 잘되어 있는 코드라 많이 배워갑니다. 로직에 대해서는 서로 다른 생각을 가지고 있기에 해당 로직에서는 적절하다고 생각하네요! 그래서 저는 가독성 측면에서 많이 보았습니다. 수고하셨습니다!
kiwoook created a review 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
이런 부분 짚어주셔서 감사합니다..!! 아직 좋은 코드를 짜는 건 멀고도 먼 길이군요,,
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주차도 화이팅!