Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
kysub99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
단순히 1,2,3 같이 숫자로 구분하기 보단 어떤 예외인지 쓰는게 명확할 것 같습니다.
kysub99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
다양한 클래스에 valiate가 있는데 이걸 하나의 클래스에서 관리하면 유지보수하기 좋을 것 같아요
kysub99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분은 caculateSum보다는 파싱 쪽에 가까운 기능이 아닐까 생각됩니다
kysub99 created a review on a pull request on woowacourse-precourse/java-calculator-7
테스트코드도 풍부하고 코드가 전반적으로 가독성이 좋은 것 같아요. 정규 표현식으로 구현한게 인상적이었습니다. 수고하셨습니다! 👍
kysub99 created a review on a pull request on woowacourse-precourse/java-calculator-7
테스트코드도 풍부하고 코드가 전반적으로 가독성이 좋은 것 같아요. 정규 표현식으로 구현한게 인상적이었습니다. 수고하셨습니다! 👍
KKdohyeong created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 그 방향성이 더 좋다고 생각해 service 단으로 비즈니스 로직을 이번 주차에 분리하려고 합니다!! 저도 좋은 방법인 것 같아요. 좋은 의견 감사합니다~
KKdohyeong created a review comment on a pull request on woowacourse-precourse/java-calculator-7
get을 통한 객체 내부의 컬렉션을 외부에 넘기는 것이 좋지 않다고 이해하고 있어서 그렇게 말한 것 같습니다. 저도 안좋다 이정도만 알아 공부를 더 하고 이번 주차에 최대한 적용시킬려 노력하고 있습니다. 첫번 째 링크의 규칙 8, 9와 두번 째 링크의 일급컬렉션을 참고하면 좋을 것 같습니다! https://velog.io/@inwoo920/%EA...
bowook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
aDouble처럼 변수명에 타입이 들어가는 경우는 좋지 않은 것 같습니다! aDouble 대신에 number는 어떠신가요!?
bowook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 부분은 if - else를 이용하지 않아도 될 것 같다는 생각을 했습니다. 왜냐하면, if 조건을 통해 return으로 반환을 하고, if 조건이 거짓이어도 else 문으로 들어가 return 반환하기 때문입니다. 아래처럼 수정해 보시면 더 직관적일 것이라고 생각합니다! `if (doubleNum % 1 != 0){ ...
bowook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Result라고 이름을 지으신 이유가 있으신가요!? 사용자의 입력은 Input으로 하셔서, 출력과 같은 경우는 Output을 안 하시고 Result로 하신 이유가 궁금합니다!
bowook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Input에서는 사용자의 입력만을 받아 반환하는 것이 아닌 비교 후 반환하시는데 이유가 있으신가요? 제 생각에는 Input에서는 사용자의 입력만을 받아서 반환하는 역할을 수행하는게 더 좋다고 판단했습니다. 이유는, Input에서 검증까지 하는 역할을 수행하면 많은 책임이 있는 느낌입니다!
ariimo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
SOLID 개념 설명과 함께 주요 클래스 설명을 미리 해주셨군요. 제가 개발 공부를 시작한 지 얼마 되지 않아 코드 읽는 게 느린 편인데, 덕분에 코드에 대한 이해가 더 잘 되었습니다:) (SOLID 원칙도 정리해가네요!)
ariimo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 함수의 로직을 보면 음수와 0이 하나라도 **있다면** 예외를 던지고 있습니다. 굳이 들어온 음수와 0을 List로 가지고 있을 필요가 없을 것 같은데, 어떻게 생각하시나요? 또한, "0과 음수를 저장할 리스트"와 같이 더 명확히 주석 처리를 해주신다면 좋을 것 같습니다:)
ariimo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
"일회성 변수는 실수 유발 원인이 될 수도 있다"는 이야기를 들은 적 있습니다. 가독성 측면에서도 result변수를 두지 않고 출력 함수 안으로 넣는 게 더 읽기 좋을 것 같습니다.
ariimo created a review on a pull request on woowacourse-precourse/java-calculator-7
객체지향에 대한 고민을 많이 하시며 작성하신 것이 느껴졌습니다. "기능"에 초점 맞춰 객체를 분리하신 것 같은데 (Validator, Parser, DelimiterExtractor 같이) 덕분에 배워갑니다. 코드 잘 읽었습니다. 감사합니다:)
ariimo created a review on a pull request on woowacourse-precourse/java-calculator-7
객체지향에 대한 고민을 많이 하시며 작성하신 것이 느껴졌습니다. "기능"에 초점 맞춰 객체를 분리하신 것 같은데 (Validator, Parser, DelimiterExtractor 같이) 덕분에 배워갑니다. 코드 잘 읽었습니다. 감사합니다:)
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`정상작동`보다 더 구체적인 이름이 있으면 어떤 테스트를 하고 있는지 더 파악하기 쉬워지겠네요!! 테스트 코드도 유지보수 대상이니까요 👍 추가로 `@DisplayName`을 쓰는 방법도 있겠네요!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
경계값 테스트에 익숙하신 게 느껴지네요!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
 EOF에 대해 알아보시면 좋을 것 같습니다! 인텔리제이에서 [자동으로 개행 설정](https://velog.io/@d-h-k/intellij-%ED%8C%8C%EC%9D%BC%EB...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
```suggestion assertThat(numbers.getNumbers()).hasSize(3); } ``` 이렇게도 작성 가능합니다~!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
값이 하나라면 `ParameterizedTest`를 쓰지 않아도 되겠네요!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
부정문을 내부로 밀어넣고 메서드명에 not을 붙이셨군요! 가독성이 좋아져서 저도 이런 방식을 선호합니다! 👍
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
일급 컬렉션 사용 👍
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스명이 separator라서 구분자를 상태로 가질 것으로 예상했는데 그렇지는 않네요!! 현재는 입력 parsing만 담당하고 있는 것일까요??
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드 순서 정렬 기준이 따로 있으실까요?