Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
jihwankim128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
static으로 선언하신 이유가 있으실까요? auto method extract를 통해 생성하신 것이라면, static이 사용되는 것이 적절한지 한 번 생각해봐도 좋을 것 같습니다!
jihwankim128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이펙티브 자바 3판, 아이템 11 equals를 재정의하려거든 hashCode도 재정의하라. 이미 충분히 좋은 코드지만 한 번, 참고하시면 조금 더 대중적으로 좋은 코드가 될 것 같습니다!
jihwankim128 created a review on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 고급진 내용이네요. 설계에 대해 스스로 많은 피드백을 거친 흔적이 보입니다. 다만 늦게 리뷰에 참여하는 만큼 개인적으로 아쉬운 부분이 있습니다. 몇 몇 분께서 인터페이스 도입에 대해 의문을 가지고 계십니다. 저도 해당 부분에 대해서 의문점이 들게 되었는데요.. 개인적으로 만약, 테크 리더나 경험이 많은 개발자가 도입한게 아니...
jihwankim128 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
아하,, 정적 팩토리 메소드에 대한 장점들만 알고 있었어서 쓰는게 좋을 거라 생각했는데 그런 관점이 존재할 수 있겠군요 ㅎㅎ 좋은 의견 나눠주셔서 정말 감사합니다! (가능하시다면 다른 답글도 답장해주시면 정말정말 감사하겠습니다 ㅎㅎ...)
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
customSeparatorHandler로 custom separator를 얻고 그것을 Separator 객체에 추가하는 로직입니다. 메소드명 때문에 혼돈될 수 도 있겠네요 그리고 정환님 덕분에 Separator가 VO인지 엔티티인지 일급컬렉션인지 생각해볼 수 있었어요. 이번에는 구분하지 않고 객체를 만들었지만, 2주차 과제에서는 설계전 한...
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
정수와 소수를 한 변수로 커버하기 위함입니다!
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
정수는 정수형식으로 출력하게 하고 소수는 소수형식으로 출력하게 하기 위함입니다 ex) 6.00 -> 6으로 출력 6.12 -> 6.12으로 출력
soeunnPark created a review comment on a pull request on woowacourse-precourse/java-calculator-7
프로그램 동작을 한 눈에 볼 수 있어서 너무 좋은 것 같아요! 👍
soeunnPark created a review comment on a pull request on woowacourse-precourse/java-calculator-7
문자열 처리에 굉장히 능숙하신 것 같아요 👍
soeunnPark created a review comment on a pull request on woowacourse-precourse/java-calculator-7
정적 팩토리 메서드를 통해 객체를 생성하는 목적을 명확하게 드러낼 수 있다고 생각합니다! 기본 구분자를 만드는 것과, 커스텀 구분자를 포함해 만드는 것이 각각 객체를 생성하는 목적과 의도가 다르기 때문에 이를 정적 팩토리 메서드를 통해 분리하는 것이 좋다고 생각합니다. 다만, 말씀해주신대로 커스텀 구분자가 기본 구분자를 포함하기 때문에 현재 로직...
soeunnPark created a review comment on a pull request on woowacourse-precourse/java-calculator-7
""와 1도 상수처리하면 더 완벽해질 것 같아요.. !!
soeunnPark created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Matcher 사용해봐야겠습니다! 👍
soeunnPark created a review comment on a pull request on woowacourse-precourse/java-calculator-7
header와 content라는 네이밍 좋은 것 같습니다!
soeunnPark created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Command라는 네이밍이 조금 혼란을 주는 것 같습니다. Separator나 Delimiter 등의 이름은 어떨까요?
soeunnPark created a review on a pull request on woowacourse-precourse/java-calculator-7
많은 고민을 통해 작성한 코드라는 것이 느껴졌습니다. 👍 읽는데 불편함이 없어서 코드를 보는 사람도 신경썼다는 느낌을 많이 받았습니다 ㅎㅎ 1주차 미션 수고 많으셨습니다 😄
soeunnPark created a review on a pull request on woowacourse-precourse/java-calculator-7
많은 고민을 통해 작성한 코드라는 것이 느껴졌습니다. 👍 읽는데 불편함이 없어서 코드를 보는 사람도 신경썼다는 느낌을 많이 받았습니다 ㅎㅎ 1주차 미션 수고 많으셨습니다 😄
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
객체 분리는 신경썼지만, 메소드 분리는 놓쳤네요 ㅜ 짚어주셔서 감사합니다
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
youngJun99님 말씀에 동의합니다! 그러나 Hwanvely님 말대로 호출할 때는 가독성을 위해 변수명을 `customSeparatorHandler` 대신 `handler`로 쓸 수 있을 것 같아요! 위 프로젝에서 handler는 하나밖에 없고 만약 해당 클래스에서 어떤 클래스인지 궁금하다면 속성에 선언된 부분을 보면 단번에 알 수 있기 때문입니다!
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
빈 문자열에 대한 예외처리는 놓쳤네요 짚어주셔서 감사합니다!
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 오히려 특별한 제약이 없었기 때문에 List를 사용했습니다. Set은 중복제거 라는 요구사항이 있을 때만 씁니다. 이 경우 정규표현식을 쓰기 때문에 만약 구분자가 중복되어도 상관없어 현행 유지해도 상관없을 것 같습니다 ㅎㅎ
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
예외를 여기서 발생시키도록 설계하신 이유가 궁금해요.
