Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
soeunnPark created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Matcher 사용해봐야겠습니다! 👍
soeunnPark created a review comment on a pull request on woowacourse-precourse/java-calculator-7
header와 content라는 네이밍 좋은 것 같습니다!
soeunnPark created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Command라는 네이밍이 조금 혼란을 주는 것 같습니다. Separator나 Delimiter 등의 이름은 어떨까요?
soeunnPark created a review on a pull request on woowacourse-precourse/java-calculator-7
많은 고민을 통해 작성한 코드라는 것이 느껴졌습니다. 👍 읽는데 불편함이 없어서 코드를 보는 사람도 신경썼다는 느낌을 많이 받았습니다 ㅎㅎ 1주차 미션 수고 많으셨습니다 😄
soeunnPark created a review on a pull request on woowacourse-precourse/java-calculator-7
많은 고민을 통해 작성한 코드라는 것이 느껴졌습니다. 👍 읽는데 불편함이 없어서 코드를 보는 사람도 신경썼다는 느낌을 많이 받았습니다 ㅎㅎ 1주차 미션 수고 많으셨습니다 😄
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
객체 분리는 신경썼지만, 메소드 분리는 놓쳤네요 ㅜ 짚어주셔서 감사합니다
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
youngJun99님 말씀에 동의합니다! 그러나 Hwanvely님 말대로 호출할 때는 가독성을 위해 변수명을 `customSeparatorHandler` 대신 `handler`로 쓸 수 있을 것 같아요! 위 프로젝에서 handler는 하나밖에 없고 만약 해당 클래스에서 어떤 클래스인지 궁금하다면 속성에 선언된 부분을 보면 단번에 알 수 있기 때문입니다!
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
빈 문자열에 대한 예외처리는 놓쳤네요 짚어주셔서 감사합니다!
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 오히려 특별한 제약이 없었기 때문에 List를 사용했습니다. Set은 중복제거 라는 요구사항이 있을 때만 씁니다. 이 경우 정규표현식을 쓰기 때문에 만약 구분자가 중복되어도 상관없어 현행 유지해도 상관없을 것 같습니다 ㅎㅎ
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
변수명은 totalSum 인데 String[] 자료형으로 만든 이유가 있을까요? 변수명만 보면 int 값 혹은 하나의 실수 값을 가져야 할것 같아요
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
여기서 숫자 3이 뜻하는 바가 무엇인가요?
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스명을 "CalFinish" 로 지으신 이유가 궁금해요
Astro-Yu created a review on a pull request on woowacourse-precourse/java-calculator-7
김이님 코드 잘 읽었습니다. 전체적으로 상수 문자열에 대해서 privat static final ~ 로 선언하신 부분은 좋았던것 같아요. 다른 상수들도 그렇게 선언해보면 어떨까요? 몇몇 함수들은 하나 초과의 책임을 지고 있는 것 같은데, 이 부분도 고려해보시면 좋을 것 같습니다. 1주차 수고하셨습니다.
Astro-Yu created a review on a pull request on woowacourse-precourse/java-calculator-7
김이님 코드 잘 읽었습니다. 전체적으로 상수 문자열에 대해서 privat static final ~ 로 선언하신 부분은 좋았던것 같아요. 다른 상수들도 그렇게 선언해보면 어떨까요? 몇몇 함수들은 하나 초과의 책임을 지고 있는 것 같은데, 이 부분도 고려해보시면 좋을 것 같습니다. 1주차 수고하셨습니다.
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좋은 의견 감사합니다. 덕분에 String 함수의 무지를 깨달을 수 있었어요. String에 대해서 공부 한 번 해봐야겠네요!
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
**살아있는 문서**를 작성하기 위해 노력해 볼까요? 😁
rladmstn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
요번 과제가 요구사항이 너무 모호했던지라 정말 다양하게 해석이 나오는 것 같아요 ! (사실 저는 너무 과도하게 케이스를 나눠버려서 구현할 때 좀 후회했습니다 ㅎㅎ) 동규님이 고려하신 부분 이해했습니다!! :)
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
단순히 상속뿐만 아니라, 코드 스멜을 놓칠 수 있다는 단점도 있다고 생각해요. 저는 개인적으로 정적 팩토리 메서드를 도메인에서 사용하는 것을 좋아하지는 않는데요, `new` 생성자로도 충분히 표현할 수 있다고 생각하기 때문이예요. 오히려 `new`가 **생성한다**는 개념과 더 호응한다고 생각하기도 하고요. 만약 클래스에 이미 정적 팩토리 메서...
phk1128 created a comment on a pull request on woowacourse-precourse/java-calculator-7
그리고 커밋 메세지 작성하실때 [AngularJS Git Commit Message Conventions](https://gist.github.com/stephenparish/9941e89d80e2bc58a153) 요고 참고해보시면 좋을것 같아요~ 🤗
rladmstn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아티클 잠깐 읽고왔는데 Console이 테스트할 때 저런 이슈가 있군요..!? 저도 처음 알았어요!! 다시 읽어보고 참고해봐야겠어요 좋은 공유 감사합니다!! :)