Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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
`Manager` 클래스는 왜 필요했는지 궁금해요! `DelimiterChecker` `DelimiterExtractor` `DelimiterStore`에 직접 의존하면 안 되는 상황이 있었을까요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
파싱은 예외 발생 여지가 굉장히 많다고 생각해요. - 구분자의 잘못된 사용: `,1,2,3` `1,2,3,` `1,,2` - 등록하지 않은 구분자 사용 - 잘못된 위치에 커스텀 구분자 등록 등등 ... 예외 테스트가 더 있으면 좋겠네요!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
print문을 일일이 확인하는 건 많이 힘들어서 print문을 제거하거나 assert문에서 검증이 이루어지면 더 좋을 것 같네요!
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
안녕하세요. 프리코스 첫번째 미션 고생 많으셨습니다! 자바 사용에 익숙하신 게 코드에서 느껴지네요 👍 리뷰 남겨봤는데 참고만 해주세요!
JunBe created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 보고 갑니다! 저와 접근 방법이 다른 부분이 많은 것 같아서 많이 배우고 갑니다!
JunBe created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 보고 갑니다! 저와 접근 방법이 다른 부분이 많은 것 같아서 많이 배우고 갑니다!
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
현재는 main의 run()메소드를 통해서 테스트가 이루어지는 것으로 확인되요! 인텔리j에서는 `command + shift + t` 단축키를 통해서 해당 클래스에 대한 테스트 클래스를 바로 생성할 수 있어요 각각의 클래스에 대한 단위테스트를 고려해보는 것은 어떨까요?
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
만약 계산기에서 잘못된 입력에 대한 검증 조건이 추가된다면 어떻게 될까요? 하나의 try문에 더 많은 if문이 추가되야 할 것으로 예상되요 검증하는 과정을 어떻게 관리하면 좋을까요? 고민해보시면 좋을거 같아요!
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
빈 입력에 대한 처리가 인상깊어요! 요구사항을 잘 분석하셔서 코드를 작성하신거 같아요! 배우고갑니다!
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자를 상수로 관리하셨네요 좋은 채택 방법인거 같아요!
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
생성자를 통한 의존성 주입 방법을 선택해주셨네요! 의존성 주입 방법에는 3가지 방법이 존재해요 3가지 방법은 어떤것이 있는지, 각각의 방법이 어떤 장점을 가지고 어떠한 경우에 사용되면 유리한지, 단점까지 또 보편적으로는 어떤 방법을 사용하는지 공부해보시면 더욱더 성장하실수 있을 거 같아요!
woogym created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 정말 수고 많으셨어요!☺️☺️ 기능 명세서와 각 클래스의 주석을 통해서 코드를 읽기 편했던거 같아요! mvc 아키텍처 패턴을 고려해서 미션을 진행하신거 같아요! 각각의 mvc패턴에 대한 역할과 책임에 대해서 한 번 찾아보시면 다음 미션하시는데 더 도움이 될 것 같아요! 제가 mvc패턴을 공부할때 도움이 많이 되었던 영상이 있어요...
woogym created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 정말 수고 많으셨어요!☺️☺️ 기능 명세서와 각 클래스의 주석을 통해서 코드를 읽기 편했던거 같아요! mvc 아키텍처 패턴을 고려해서 미션을 진행하신거 같아요! 각각의 mvc패턴에 대한 역할과 책임에 대해서 한 번 찾아보시면 다음 미션하시는데 더 도움이 될 것 같아요! 제가 mvc패턴을 공부할때 도움이 많이 되었던 영상이 있어요...
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
`String.join()` 쓰시면 더 깔끔하게 작성 가능합니다!
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 public void addAllDelimiters(List<String> delimiters) { this.delimiterList.addAll(delimiters); } ``` 여러 개 추가하는만큼 복수형은 어떨까요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
 파일 맨 끝에 개행이 되어 있지 않습니다!! EOF에 대해 알아보시면 좋을 것 같아요.
honggyuSHIN created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 적용하지 못한 부분이지만 문자를 입력 받고 판단하고 반환하는 일련의 과정이 생성자와 getOperandArr에 나눠져 있지 않고 한 메서드에서 동작하도록 구현하면 더 좋지 않을까 생각했습니다. 여러 기능을 만든 뒤에 기능들을 종합해서 문제를 해결하는 과정을 한 메서드에서 동작하도록 하면 읽기 더 편하고 깔끔하지 않을까 싶어서 코멘트 남깁니다. ...
honggyuSHIN created a review on a pull request on woowacourse-precourse/java-calculator-7
수고 많으셨습니다. 감사합니다
honggyuSHIN 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
정규식이 잘 작동하는지를 테스트하는 코드로 보여요. 이름을 바꿔 보면 어떨까요? ```suggestion public class RegexTest { ```
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
 파일 맨 끝에 개행이 되어 있지 않네요. EOF에 대해 알아보시면 좋을 것 같습니다. 인텔리제이 쓰신다면 자동으로 추가하는 옵션도 있어요! [참고](https://hyeon9mak.gi...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
```suggestion public class InputString { private final String input; private String expression; private String operatorRegex; private String[] tokens; ``` `body`, `pattern`...
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요. 리뷰 몇 개 남겨봤습니다. 프리코스 1주차 미션 고생 많으셨습니다!
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요. 리뷰 몇 개 남겨봤습니다. 프리코스 1주차 미션 고생 많으셨습니다!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
테스트 케이스가 하나라서 `@ParameterizedTest`가 크게 필요 없어보이네요!! ```suggestion void 빈_문자열_처리() { //when int result = calculator.calculate(""); //then assertThat(result...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
과하지도 부족하지도 않은 테스트 케이스 좋네요! 😄