Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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을 참고하시면 좋을 것 같아요!
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 메서드로 분리하는 것은 가독성과 유지보수성 측면에서 좋은 제안이라고 생각합니다. 제안 주신 내용 잘 반영해보겠습니다!
