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
도메인의 경우, 의존성 주입을 해야하는가? 자바의 멀티 스레드 환경에서 어떤 문제점이 발생할 수 있을까? 이 포인트를 한 번 고려해봐도 좋을 것 같습니다.
jihwankim128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 해당 코드의 방식이 나쁘지 않다고 생각합니다. 저수준 컴포넌트와 고수준 컴포넌트의 상호 관계를 생각했을 때, 지정님께서 어느 관점에서 객체들을 분리 했는지 알 수 있는 코드였습니다!
jihwankim128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
현재 3개의 validate에서 BigInteger 객체를 생성하면서 검증하고 있습니다. 그럼 실제 검증이 끝나고 BigInteger 객체를 또 생성할 것으로 보이는데요. 한 번의 숫자 변환을 위해서 4번의 객체 생성이 필요한가? 1번의 객체 생성으로 개선할 수 있을 것으로 보입니다!
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
단위테스트도 진행하시고 인수 테스트도 진행하신 부분이 굉장히 인상 깊습니다!! 단순히 기능 구현 뿐만 아니고 전체 로직에 대한 꼼꼼한 테스트는 좋은 습관이라고 생각합니다. 특히 좋은 기술로 소개되어지는 부분들 또한 접목하려고 하신 것 같더라구요! 다만, 아쉬운 점은 코드에서 무결성을 지키지 않는 부분이 있었다는 것입니다. 그 부분에 대해서는...
soonhong99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
정수 범위를 생각 못했네요. 저도 이런걸 넣어야했는데 미처 생각을 못했네요. 배워갑니다!
kkonii created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@soeunnPark 예외가 복잡하고 많아질수록 빛을 발하는 클래스인 것 같아요✨
soonhong99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
오.. extends를 이용하면 IllegalArgumentException을 쓰지않고, 내가 지정하고 싶은 에러 함수를 써서 무슨 에러인지 알 수 있겠군요! 아주 유용하네요 저도 2주차떄 써봐야겠습니다!
kkonii created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@soeunnPark 바로 에러를 던지게 된다면, 해당 메서드는 조건을 `확인`하는 책임과 그 조건에 따라 `예외를 발생`하는 책임을 모두 가지게 될 것 같아요. `isNotPositive( )`가 내부 검증에서만 사용되긴 하지만 저는 가독성을 위한 분리는 좋은 코드에 가깝지 않을까 싶은데, 이에 대해서는 어떻게 생각하시는지 궁금합니다!🙂
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 장순님! 피드백 주셔서 감사합니다 😊 말씀해주신 대로, validateDelimiterFormat 메서드에서 복잡한 조건식을 private 메서드로 분리하는 것은 가독성과 유지보수성 측면에서 좋은 제안이라고 생각합니다. 제안 주신 내용 잘 반영해보겠습니다!
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
이렇게 이중으로 캐스팅한 이유가 뭔가요 ??