Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
num 처리 로직을 별도의 메서드로 빼 indent를 줄이는건 어떨까요? 가독성을 늘리고 메서드 기능을 더 잘게 나눌 수 있을 것 같습니다
soonhong99 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
> 저는 단순히 Exception을 검증 로직에서 IllegalArgumentException를 바로 던지게 구현했었는데, 이렇게 Exception을 더 구체화시켜서 던져주니 코드의 가독성이 더 좋아졌다고 생각합니다! 지금은 CalculatorException만 구현되어 있는데 각각의 예외상황에 맞는 Exception을 더 세분화해서 구현해봐도 좋을 ...
soonhong99 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
README 파일을 엄청 꼼꼼하게 작성하셨네요..! 저도 2주차때 민트님이 작성해주신 것처럼 꼼꼼하고 세심하게 작성해봐야겠어요!
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Stream을 사용하는 것은 어떤가요? 저 또한 이번 미션에서 반복문으로 for문을 사용하였는데 가독성을 위해 Stream으로 바꿨으면 하는 생각이 들었습니다. 수창님의 생각은 어떠신가요?
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
inputString에 대해 null 조건을 판단하시는 것은 어떨까요?
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
불필요한 주석은 삭제하는 게 좋을 것 같습니다.
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
문제에서 "커스텀 구분자는 문자열 앞부분의 "//"와 "\n" 사이에 위치하는 문자를 커스텀 구분자로 사용한다." 라는 문장이 있었습니다. '문자'라고 하였는데 공백과 문자열을 구분자 취급하신 이유가 있나요?
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 패키지 안에서 처리 가능하도록 생각해봤습니다. 제가 개념이 부족해서 그런데 응집도라 함은 어떤 개념일까요?