Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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
```suggestion private void validateNumber(String rawNumber) { ``` 아직 파싱되지 않아서 `number`라고 부르면 조금 위험할 수도 있을 것 같아요. 저는 구분해주기 위해서 `raw`라는 키워드를 붙이는 것을 선호합니다! 참고만 해 주세요 😄
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
```suggestion public String[] parseNumbers(String input) { List<String> delimiters = new ArrayList<>(Arrays.asList(",", ":")); ``` `List`라는 것은 자료형만으로 충분히 파악이 가능해요. 이름을 바꿔 보는 건 어떨까요??
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드명과 실제 역할이 달라 보이네요! ```suggestion if (doesNotEndWithNumber(input)) { throw new IllegalArgumentException("구분자 뒤에 숫자가 없습니다: " + input); } ```
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
상수화 👍👍👍 상수 이름을 그대로 설명만 하는 주석이라서, 없어도 좋아 보입니다! ```suggestion private static final String CUSTOM_DELIMITER_PREFIX = "//"; private static final String CUSTOM_DELIMITER_SUFFIX = "\\n"; ```
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 로직이 전체적으로 깔끔해서 리뷰하기가 편했네요 😄 남은 프리코스 기간도 얻어가시는 것이 많았으면 좋겠습니다!!
takoyakimchi created a review 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
상수화 꼼꼼하게 해주신 것 너무 좋네요!!! `NEWLINE`은 `"\n"`인데, 지금은 escape 문자를 사용하고 있어서 `NEWLINE`은 아닌 것으로 보이네요. prefix와 suffix 이름을 맞추면 더 관리하기 쉬워지지 않을까 싶습니다! 👍 ```suggestion private static final String CUSTO...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 작업은 `Validator`가 해야 할 역할과는 거리가 멀어 보입니다! 다른 개발자가 커스텀 구분자 관련 요구사항을 확인하고 지은님의 코드를 확인했을 때 과연 어떤 파일을 먼저 열어볼까? 를 생각해보시면 좋을 것 같아요!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
작업이 끝났으니 TODO 주석은 지워주셔도 좋을 것 같아요!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
**입력을 받아 검증**하고 **숫자 배열을 반환**하는 메서드 메서드명은 `validate()`인데 검증 + 파싱 두 가지 역할을 하고 있네요! 해당 메서드를 쓰는 부분에서 파싱이 될 것이라고는 예상하지 못할 것 같아요. 메서드 분리 혹은 메서드명이 길어지더라도 `validateAndParse()` 같이 더 명확하게 이름을 지어 보시면 좋을...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
주석 없이도 깔끔하게 코드를 작성하는 연습을 해보시면 어떨까 싶어요! 예를 들면 `if (delimiter.isEmpty())`는 `구분자가 비어 있는지 확인`하는 역할이라는 것이 너무나도 명확해요. 코드를 그대로 설명하는 주석보다는 꼭 필요한 주석만을 사용하면 좋을 것 같습니다! ```java // 커스텀 구분자 처리 및 검증하는 메서드 ...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
junit과 assertJ 어떤 것을 선호하시나요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
와일드카드 import가 있네요! 어떤 단점이 있을까요?
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
Java8의 stream에 대해서 공부해 보시면 좋을 것 같아요! 😄 추가로 `add()`라는 메서드명은 개인적으로 '숫자 하나를 더한다' 라는 행위로 느껴지는데요. `calculateSum()`이나 `summate()` 같은 이름은 어떨지 제안해봅니다!
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요. 1주차 미션 구현 깔끔하게 잘 하셨네요!! 몇 가지 리뷰 남겨보았습니다. 2주차도 화이팅입니다 👊
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요. 1주차 미션 구현 깔끔하게 잘 하셨네요!! 몇 가지 리뷰 남겨보았습니다. 2주차도 화이팅입니다 👊
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 😄 리뷰 몇 가지 남겨보았습니다. 프리코스 과정에서 많이 얻어가시기를 기원하겠습니다! 2주차 프리코스도 화이팅입니다!
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 😄 리뷰 몇 가지 남겨보았습니다. 프리코스 과정에서 많이 얻어가시기를 기원하겠습니다! 2주차 프리코스도 화이팅입니다!
KoSeonJe created a review comment on a pull request on woowacourse-precourse/java-calculator-7
제 의도는 input, output을 담기 위해 한번 더추상화를 시킨 것입니다. 입력과 출력, 두 개의 책임을 가졌다고 생각할 수 있지만, 큰 틀에서 하나의 화면을 담당하고 있는 책임을 가진 클래스를 의도하였습니다. 그 내부 구현은 책임을 분리하기 위해 inputHandler, OutputHandler로 구분한것이고요! 굳이 Wrapping...
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 변수는 default로 선언하신 걸까요? 만약 그렇다면 그 이유가 궁금합니다! 그런 의도가 아니라면 접근제어자를 붙여 접근 범위를 제한하는 게 좋을 것 같습니다!
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분에서 indent depth가 3 이상입니다. 프로그래밍 요구사항을 보시면 들여쓰기는 2까지만 허용한다고 나와있습니다. 이에 따라 코드를 수정하시면 좋을 것 같습니다!
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
코드의 가독성을 위해 "=" 양 옆은 한 칸씩 띄는 것이 좋을 것 같습니다! `String[] onlyNumber = splitInput(input);` 예를 들어 이렇게요!
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 메소드에서는 값이 0인지 아닌지만 체크하여 boolean값을 반환하는 정도의 기능만 하는 것이 좋을 것 같습니다! printResult 메소드를 사용해서 결과값을 출력한다는 것은 곧 프로그램이 종료되어야 함을 의미합니다. 우리가 이 프로그램으로 얻길 원하는 최종 값이 바로 이 결과값이니까요. 그런데 이 최종 결과값의 출력이 단순한 검증 로직에 ...
hyobin-yang created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수행하시느라 고생 많으셨습니다!! 앞으로도 화이팅입니당
hyobin-yang created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수행하시느라 고생 많으셨습니다!! 앞으로도 화이팅입니당
KoSeonJe created a review comment on a pull request on woowacourse-precourse/java-calculator-7
굉장히 중요한 부분인데요! 클래스 전체에서 delimiterStore를 하나로 공유해야 하는 상황인데, 여러 인스턴스가 생성된다면 데이터 정합성이 맞지 않아서 싱글톤으로 구현했습니다! static 키워드로 생성하는 방법도 있지만, 그래도 여러 개 인스턴스를 만드는 것은 좋지 않다고 생각해서 싱글톤을 선택했습니다!