Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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
이 else에는 어떤 조건들이 들어가는건가요? if로 경우의 수를 나눠서 표현한다면 더 읽기 좋을 것 같아요.
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
여기서 -1이 뜻하는 의미가 무엇인가요?
Astro-Yu created a review on a pull request on woowacourse-precourse/java-calculator-7
지현님 코드 잘 읽었습니다. 제 생각에는 CalculateStirng를 포함한 클래스들이 너무 많은 기능을 가지고 있다고 생각해요. 기능 분리부터 시작해보면 어떨까요? 추가로 여러 상수들(문자열, 숫자)등을 상수로 선언해주시면 좋을 것 같아요. 구글에서 "객체지향 생활체조 원칙"을 검색해보는걸 추천드려요. 1주차 수고하셨습니다!
Astro-Yu created a review on a pull request on woowacourse-precourse/java-calculator-7
지현님 코드 잘 읽었습니다. 제 생각에는 CalculateStirng를 포함한 클래스들이 너무 많은 기능을 가지고 있다고 생각해요. 기능 분리부터 시작해보면 어떨까요? 추가로 여러 상수들(문자열, 숫자)등을 상수로 선언해주시면 좋을 것 같아요. 구글에서 "객체지향 생활체조 원칙"을 검색해보는걸 추천드려요. 1주차 수고하셨습니다!
phk1128 created a review on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 완성도가 높은 코드였어요~ 제가 여쭤본 부분이 도움이 되는 고민 포인트가 됐으면 좋겠네요 ㅎㅎ 그리고 인터페이스와 DTO를 어떻게 적용할지에대해서 질문해주셨는데요~ 이 부분에 대해 고민을 많이 하셨기 때문에 여쭤본거라고 생각해요~ 어떤 고민을 하셨는지 말씀해주실수 있나요~??
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 저도 처음 int 타입을 사용하다가 long타입으로 바꾸며 int타입의 최대값과 long타입의 최대값에 대한 예외 처리를 고민하며 BigIneter도 사용을 해야 하나 고민을 했습니다. > > 그리고 저의 예외 처리 범위에 대한 근거와 기준을 세우기 위해 가장 흔히 사용하는 '갤럭시와 아이폰 계산기'의 최대값을 다루는 타입에 대해 찾아본후 l...
phk1128 created a review on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 완성도가 높은 코드였어요~ 제가 여쭤본 부분이 도움이 되는 고민 포인트가 됐으면 좋겠네요 ㅎㅎ 그럼 2주차도 화이팅입니다 ~ 😌
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 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 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 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
기능 목록 좋아요! 혹시 클래스 목록은 처음부터 생성하셨나요? 아니면 구현하면서 변경이 있었을까요?