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
역으로 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를 활용했어도 될 부분인데요,,, 뒤늦게 와서 후회가 되는 부분입니다.
ChoiMGyu created a 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
좋은 의견 감사합니다!! 제네릭을 활용해볼까? 라는 생각까지는 했는데 적용하지는 못했습니다 ㅠㅠ 아직 저에게는 열리지 않는 눈인가봅니다 ㅠㅠ 4차 과제동안 시야를 더 넓혀보겠습니다.
201912025 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클라이언트 코드는 변경할 필요없이 팩토리에서 해당 구현체를 생성하는 코드를 수정하여 새로운 클래스를 사용하도록 변경하면 됩니다!
201912025 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
도메인 객체안은 해당 도메인만 관련된 비즈니스 로직들만을 작성하고, 서비스 객체는 도메인들의 로직을 조합하여 처리하게 하여 비즈니스 프로세스의 흐름을 관리 하게 하여 역할을 분리 했습니다! 이렇게 하면 만약 요구사항이 변경돼 Calculator 도메인의 비즈니스 로직에 다른 도메인 객체의 비즈니스 로직을 가져와 사용해 여러 도메인이 섞이게 되면 각 도...
kkonii created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@soeunnPark 제출 전까지도 고민하고 있었던 부분을 딱 집어주셨어요..! 저도 해당 부분을 `Parser`에 위임하는 거 맞아? 아니야? 하다가 Parser의 책임이 너무 무거워지는 것은 아닐까 했어요🤔 여기에 대해서 더 말씀해 주실 게 있을까요?
jihwankim128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
UI에 대한 테스트가 필요했을까요?! 한 번, 테스트 방식들에 대해 고민해보시면 좋은 경험이 될 것 같습니다!!
jihwankim128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
convert 이전에, validator에서 BigInteger로 검증을 하고 있습니다. 큰 입력에 대해서 validator에서는 정상적으로 검증이 되었고, convert 하는 과정에서 예외가 발생하지 않을까? 하는 의심 포인트가 있습니다!
jihwankim128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
MVC 변화에 관심이 많으신 코드라고 느껴지네요!! 실제로 MVC 패턴은 점차 변화되면서 Controller가 많아지는 문제를 MainController에서 처리했죠! 하지만, 현재 필요한 Controller가 그렇게 많지 않아보입니다!! CalculatorController로 모두 처리해 볼 수 있지 않았을까요?! 하지만 학습한 기술을...
jihwankim128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
상수는 어떤 역할을 가지고 있다고 생각하시는지 여쭤봐도 될까요? 저와의 의견 차이가 있는 부분이 있습니다! 저는 메세지 그 자체에서 다 표현할 수 있지 않을까? literal pool의 특성을 잘 사용하고 있는 코드인가? 라는 생각이 듭니다!! 한 번 참고해보시고 지정님의 의견을 들려주시면 감사할 것 같아요!!
jihwankim128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Arrays.asList를 다시 new ArrayList로 재할당하신 이유가 있으실까요? immutable Collection과 mutable Collection을 참고하시면 좋을 것 같아요!