Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
절차지향형 프로그래밍을 시도하신 것 같은데 이유가 있으신가요?
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
https://github.com/woowacourse-precourse/java-calculator-7/pull/1466#discussion_r1814803882 장순님의 위에서도 동일한 의견을 주셔서, 답변드린 링크를 첨부합니다!
kkonii created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@soeunnPark 인스턴스 변수를 가지고 그 상태를 변화시키는 책임이 없는 객체여서, 객체로 따로 생성하기보다는 static으로 만들어 쓰는 게 메모리를 아낄 수 있다고 생각했습니다
Astro-Yu created a comment on a pull request on woowacourse-precourse/java-calculator-7
> 전체적으로 완성도가 높은 코드였어요~ 제가 여쭤본 부분이 도움이 되는 고민 포인트가 됐으면 좋겠네요 ㅎㅎ > > 그리고 인터페이스와 DTO를 어떻게 적용할지에대해서 질문해주셨는데요~ 이 부분에 대해 고민을 많이 하셨기 때문에 여쭤본거라고 생각해요~ > > 어떤 고민을 하셨는지 말씀해주실수 있나요~?? Interface의 경우 여러 ...
zzioozzioo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
컨트롤러에서 서비스를 호출할 때 Number 객체까지 알아야 하다보니 필드 주입을 사용했습니다! 코드가 복잡해진다고 생각했거든요. 혹시 이에 대해 조언해 주실 부분이 있나요?
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 저도 DelimiterManager의 역할에 대해 고민해보면 좋겠다는 제이슨님의 의견에 동의하는 바입니다! > > 다만, 저는 패턴을 생성하고 구분자를 관리하는 기능 또한 비즈니스 로직에 해당 하지 않을까? 라는 생각을 하게 되어서 이 부분을 model 영역에 두지만 Manager라는 네이밍을 변경해보는 것은 어떠할까! 라는 생각이 들었습니다...
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
View 자체가 UI를 담당하는 기능이라고 생각하는데, 이때 사용자 입력을 유도하는 메시지 출력은 InputView가 담당해야 한다고 생각했습니다. 혹시 다른 생각이 있으시다면 알려주세요!
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
의도는 실수(real number) 중에서 .0으로 끝나는 것들(ex 16.0, 17.0 ...)을 .0을 제거하기 위한 용도로 사용했는데 좀 조잡해 보입니다. 다른 방법이 있을까요?
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
추후에 출력 사양이 추가될때 이 클래스에서 모두 관리하면 편하다는게 장점일 수 있겠고, 반대로 부담인 점은 view 클래스에서 확인하기 어렵다는 점이 있겠네요. 지금은 하나밖에 없어서 약간 오버같기도 합니다 ㅎㅎ
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 유지보수 용이성에 중점을 뒀어요. 추후 수정이 필요할 때, 상수에서 발생한 에러라면 반드시 constants 패키지 안에서 처리 가능하도록 생각해봤습니다. 제가 개념이 부족해서 그런데 응집도라 함은 어떤 개념일까요?
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
역으로 getter가 강제되는 것에 어떤 문제가 있을까요? 일단 enum을 사용한 이유는 ERROR_PREFIX를 모든 에러 메시지 헤드에 고정시키고 싶어서 작성했습니다.
jihwankim128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
맞습니다. 제가 작성한 custom separate 코드인데, 코드를 보는 분들이 조금 더 이해하기 쉽게 상수로 표현했으면 더 좋았을 것 같네요.. 코드를 보는 상대방을 배려하지 못한 코드라고 생각됩니다.. 좋은 리뷰 감사합니다!!
zzioozzioo created a review comment 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
커스텀 구분자는 왜 숫자가 될 수 없다고 판단하셨는지, 그 이유가 궁금합니다! 문제에서 문자는 커스텀 구분자로 될 수 있다 하여, "0", "1", ... 이러한 한문자로 이루어진 숫자는 구분자가 될 수 있다고 생각했거든요!
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> DelimiterManager 클래스를 보면서 몇 가지 궁금한 점이 생겼습니다. > > 클래스 이름과 역할을 보면 관리자의 성격을 띠는 것 같은데, 혹시 controller에 위치해야 하는 것은 아닌가 하는 생각이 들었습니다. 보통 'Manager'라는 이름이 들어간 클래스는 데이터를 관리하거나 흐름을 제어하는 역할을 많이 맡는데, 이 클래스...
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
단순히 테스트 만을 위해 추상화를 한 것은 아니였지만,, 좋게 봐주셔서 감사합니다. 지금 Fake 객체의 경우 그냥 extends를 활용했어도 될 부분인데요,,, 뒤늦게 와서 후회가 되는 부분입니다.