Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
KoSeonJe created a review comment on a pull request on woowacourse-precourse/java-calculator-7
System.out.println 이라는 코드가 반복되는 것이 보기 좋지 않다고 생각해서, 그냥 간결하게 사용할 수 있도록 private 메소드로 만들어놓은 것입니다!
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
error는 핸들링 하는게 아니라, Exception을 그대로 노출해주면 좋을 것 같아요
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 부분을 Factory로 만들어줬는데 해당 부분이 전체 application에서 정확하게 어떤 역할을 하는지, 왜 그렇게 하는지 알 수 잇을까요?
halfmoon-mind created a review on a pull request on woowacourse-precourse/java-calculator-7
고생하셨습니다! 코멘트 몇가지 남겼는데 반영하면 좋을 것 같아요~!
halfmoon-mind 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
정규식은 알아보기 힘든 경우가 많아서, 상수화해서 어떤 규칙인지 힌트를 주면 더 편하더라구요! 👍
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
EOF에 대해 알아보고 [인텔리제이 설정에서 자동 개행을 적용](https://hstory0208.tistory.com/entry/IntelliJ-%ED%8C%8C%EC%9D%BC-%EB%81%9D%EC%97%90-%EC%9E%90%EB%8F%99%EC%9C%BC%EB%A1%9C-%EA%B0%9C%ED%96%89-%EC%B6%94%EA%B0%80%ED...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
깔끔하고 견고한 테스트 작성 좋아요!!! 😄 추가로 `1;;2;;3;;` `1;;;;2;;3` 등, 지정한 구분자를 사용은 하고 있지만 올바르지 않게 사용하는 경우에 대해서도 테스트해보면 어떨까요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
인자값이 nullable하지 않다면 primitive type을 써주어도 될 것 같아요!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`default` 접근제어자는 어디까지 접근 가능할까요? 캡슐화란 무엇인가요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
자료형을 그대로 작성하는 것보다는 변수명에 의미가 담겨있으면 더 알아보기 편하겠네요! 👍 ```suggestion /* Controller가 실행 흐름 담당 */ List<Integer> numbers = stringService.parse(inputString); Integer result = ...
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
`IntStream`으로 변환하고 더해보면 어떨까 싶습니다! 😄 추가로 반환값은 nullable 한가요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드 순서에 대한 기준이 따로 있으실까요? 일반적으로 사용되는 기준을 알아보고 따라가보셔도 좋을 것 같습니다 👍
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주차도 달려봅시다!!! 👍
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자들을 아예 하나로 다 통일해서 처리하도록 구현하셨군요! 이 아이디어도 처음 보는 것 같아서 흥미롭네요!
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
와우 저는 "\n"이 구분자가 되는 것은 제한했는데 현욱님은 저랑 다르게 구현하셨군요! 만약 "\n"가 구분자가 된다면 어떻게 구현할까도 잠시 고민했었는데 이런 방법도 있구나 하고 흥미롭게 봤습니다ㅎㅎ
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Calculator 클래스가 너무 여러 기능을 하는 것 같습니다! 클래스명만 보았을 때는 어떤 것을 계산해주는 로직이 주요하게 구현되어있어야 할 것 같은데 여기서는 검증 로직과 입출력 로직의 사용 부분도 함께 있어 기능의 의미가 모호해질 수 있을 듯합니다. 각 기능을 분리하여 구현해보시면 좋을 것 같아요!
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주차 미션 수행하시느라 넘 수고 많으셨습니다!
rladmstn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`NumberFormatException` 캐치해서 input 예외 처리하는거 너무 좋은 것 같아요! 생각 못해본 방법이었는데 배우고 갑니당 :)
rladmstn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
요 메서드는 `NumberInputUtil` 클래스에 넣어두신 이유가 궁금합니다! 코드는 주관이 너무 다양하지만.. 저는 개인적으로 유효하지 않은 symbol이 있는지 확인하는 메소드라면, 결국 input을 검증하는 역할이라고 생각해서 `InputValidator`에서 수행하면 더 어울리지 않을까 싶었거든요-!
rladmstn created a review on a pull request on woowacourse-precourse/java-calculator-7
맛집이라는 소개 받고 왔습니다 ㅎㅎ 전체적으로 코드 구조가 굉장히 깔끔해서 보기도 편했습니다! 그리고 저는 `Util` 클래스들에 대해서 인스턴스화를 막는건 생각 못해봤었는데, 동규님 코드 보니 제가 디테일한 부분을 꽤 놓치고 있었단걸 깨닫고 갑니다..! 코멘트들에 대해서는 서슴없이 의견 달아주세요!
rladmstn created a review on a pull request on woowacourse-precourse/java-calculator-7
맛집이라는 소개 받고 왔습니다 ㅎㅎ 전체적으로 코드 구조가 굉장히 깔끔해서 보기도 편했습니다! 그리고 저는 `Util` 클래스들에 대해서 인스턴스화를 막는건 생각 못해봤었는데, 동규님 코드 보니 제가 디테일한 부분을 꽤 놓치고 있었단걸 깨닫고 갑니다..! 코멘트들에 대해서는 서슴없이 의견 달아주세요!
bowook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Calculator에서 사용하는 상수와 동일한 상수인 것 같아요! Constants 클래스에 공통적으로 사용되는 상수들은 모아놓는 것은 어떻게 생각하시나요!?
bowook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
에러 메시지가 너무 명확해서 주석은 필요 없을 것 같습니다! 주석 없이 코드를 충분히 이해할 수 있을 것 같아요!