Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이렇게 출력 메세지를 따로 클래스로 관리하면 어떤 장점과 단점이 있을까요 ??
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
접근제어자로 생성자도 막아두시고~ 깔끔하게 상수를 클래스로 잘 분리해주셨는데요~ 석현님은 응집도와 유지보수 용이성 중 어떤 기준으로 상수들을 분리하는지 궁금하네요~
phk1128 created a review on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 완성도가 높은 코드였어요~ 제가 여쭤본 부분이 도움이 되는 고민 포인트였으면 좋겠네요 ㅎㅎ 그럼 2주차도 화이팅입니다 ~ 😌
phk1128 created a review on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 완성도가 높은 코드였어요~ 제가 여쭤본 부분이 도움이 되는 고민 포인트였으면 좋겠네요 ㅎㅎ 그럼 2주차도 화이팅입니다 ~ 😌
YehyeokBang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 입력을 Command라는 객체로 다루면서, 생성자에서 스스로 검증하고 인스턴스가 생성될 수 있도록 했어요. 이 방식이 응집도 측면에선 좋은 것 같지만, 검증 로직이 여러 곳으로 흩어질 수 있다는 단점이 있어서 매번 고민이 생기는 것 같아요.
YehyeokBang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 사용자의 입장으로 생각했던 것 같아요. 특히, 실제 프로그램을 사용하는 사용자도 있겠지만, `객체의 메서드를 사용하는 입장`으로 생각했던 것 같아요. 예를 들어, 이름을 입력하면 덕담을 출력해주는 프로그램이 있다고 가정했을 때, 우선 "입력과 출력"을 `View` 객체로 분리하는 것은 같은 입장인 것 같아요. 덕담을 출력하기 위해 Vie...
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
오류 구문을 상수로 분리해보면 어떨까요? 또 appendDigit 메서드가 에러를 던지게 설계한 이유가 궁금해요.
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
여기서 0이 뜻하는 바가 무엇인가요?
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
long 자료형을 통해 큰 수도 처리하는 부분 좋아보여요!
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 클래스에서 private 메서드들을 static으로 선언하신 이유가 궁금해요.
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 클래스는 구분자와 input을 모두 다루는 것으로 보이는데 두 개를 분리해보면 어떨까요? 또 해당 클래스에 DTO를 명명하신 이유가 궁금해요.
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기능 목록 좋아요! 혹시 클래스 목록은 처음부터 생성하셨나요? 아니면 구현하면서 변경이 있었을까요?
Astro-Yu created a review on a pull request on woowacourse-precourse/java-calculator-7
시영님 코드 잘 읽었습니다. Long을 통해 큰 수도 처리하려고 고민하신 부분이 보여서 좋았습니다. 1주차 수고하셨습니다!
Astro-Yu created a review on a pull request on woowacourse-precourse/java-calculator-7
시영님 코드 잘 읽었습니다. Long을 통해 큰 수도 처리하려고 고민하신 부분이 보여서 좋았습니다. 1주차 수고하셨습니다!
youngJun99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
객체 생성을 하는 과정에서 validation을 같이 하는 경우가 많더라구요... 그런데 설계마다 다를 수 있을 것 같아요! 저도 똑같이 중복이 발생하는 코드를 작성한 것 같아서 다음에는 중복이 없도록 설계부터 고민해볼 생각입니다.
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
(중간에 잠시 밥먹으러 나와서,,, 금방 달게요 조금만 기다려주세요 ㅎㅎㅎ 🙏)
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분에서 생성자를 통해 구분자를 받게 설계한 이유가 궁금해요. 클래스 내부에 상수로 만드는 것과 어떤 차이를 생각하셨나요?
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
숫자를 추출하는 메서드에서 에러를 반환하는 이유가 있을까요? 또 에러가 발생했을 경우, 어떤 에러 메시지를 반환하는지 적어주면 좋을 것 같습니다.
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
주석이 없어도 명확한 코드 같아요.
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
여기서 숫자 2의 의미가 궁금해요
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
제가 느끼기겐 PlusCalcuator의 기능이 너무 방대한 것 같아요. 커스텀 구분자 추출, 커스텀 구분자 확인 등등... 분리해보면 어떨까요? 아니면 이렇게 설계하신 이유가 궁금합니다.
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 메서드 이름을 input~ 으로 지으신 이유가 궁금해요.
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분에서 interface를 사용해야겠다고 생각하신 이유가 궁금해요. 제가 아직 interface에 대한 이해가 부족해서 이유를 듣고 싶습니다.
Astro-Yu created a review on a pull request on woowacourse-precourse/java-calculator-7
건우님 코드 잘 읽었습니다. view를 통해 입출력을 나누신 부분은 좋아보여요. 여러 기능을 하나의 클래스에서 처리하는 부분은 PlusCalculator의 책임이 과중해 보입니다. SRP를 검색해보는걸 추천드려요. 또한 명확한 코드에 대해선 주석을 굳이 달지 않아도 괜찮아 보입니다. 1주차 수고하셨습니다!
Astro-Yu created a review on a pull request on woowacourse-precourse/java-calculator-7
건우님 코드 잘 읽었습니다. view를 통해 입출력을 나누신 부분은 좋아보여요. 여러 기능을 하나의 클래스에서 처리하는 부분은 PlusCalculator의 책임이 과중해 보입니다. SRP를 검색해보는걸 추천드려요. 또한 명확한 코드에 대해선 주석을 굳이 달지 않아도 괜찮아 보입니다. 1주차 수고하셨습니다!
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이번에 MVC를 정확히 공부하지 않아서 Controller를 정의하지 못했습니다. 이로 인해 Application이 Controller 역할을 하는 것 같네요 2주차 과제에서는 youngjun99님 의견 반영해야겠어요. 좋은 코드를 위해 리뷰해주셔서 감사합니다
