Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
bowook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
처음에는 InputView를 컨트롤러에서 인스턴스화를 하기 전이였습니다! 그래서 private를 접근 제한자를 사용해서 InputView 내부에서만 동작하는 기능으로써 정의를 하였고요! 이후에 InputView를 컨트롤러에서 인스턴스화를 하고 난 후 사용자에게 입력을 받아야 해서 public으로 수정하였습니다!
junghunim07 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
정규식 상수화를 못했네요 ㅜ
junghunim07 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
확장성을 고려해서 했습니다 !!
bowook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 저도 맨 처음에는 double을 사용해서 처리를 고려하였지만, double을 사용하게 될 경우 정확한 값이 나오는 것이 아닌 근사값이 나와 오차가 발생했었습니다! 그래서 BigDecimal을 이용하게 되었습니다!
thsghdud13 created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요..! 우연히 들러서 보게되었는데 많이 배우고 갑니다! 코드 보면서 궁금했던 점들 몇가지 적었습니다! 잘 보고 갑니다!! https://github.com/woowacourse-precourse/java-calculator-7/pull/866 시간 되시면 제 코드도 리뷰해주시면 감사드리겠습니다!
halfmoon-mind created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 고생하셨습니다! 테스트 케이스도 추가하면 더 좋을 것 같아요 :)
halfmoon-mind created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 고생하셨습니다! 테스트 케이스도 추가하면 더 좋을 것 같아요 :)
SeongUk52 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
스트림 혹은 메서드 분리를 통해 한 메서드에 한 단계의 들여쓰기로만 구현하는 것이 클린하다고 생각됩니다
SeongUk52 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
디미터의 법칙에 대해 알아보시면 좋을 것 같습니다
SeongUk52 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스의 인스턴스 변수와 생성자의 인자 수가 너무 많은건 아닌지 조심스레 생각해봅니다
SeongUk52 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 진행하시느라 고생 많으셨습니다. 아키텍쳐 구조가 잘 짜여져 있는 것 같아서 좋네요 그리고 클린코드에 대한 고민을 한 번 해보시면 도움이 될 것 같습니다.
SeongUk52 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 진행하시느라 고생 많으셨습니다. 아키텍쳐 구조가 잘 짜여져 있는 것 같아서 좋네요 그리고 클린코드에 대한 고민을 한 번 해보시면 도움이 될 것 같습니다.
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분은 왜 싱글톤 처리했는지 궁금합니다!
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
한 인터페이스가 두가지 책임을 지는 것 같은데 어떻게 생각하시나요? (ISP 원칙) Input Handler와 Output Handler를 ApplicationView로 묶은 이유도 궁금합니다..!
thsghdud13 created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요..! 우연히 들러서 보게되었는데 많이 배우고 갑니다! 코드 보면서 궁금했던 점들 몇가지 적었습니다! 잘 보고 갑니다!!
thsghdud13 created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요..! 우연히 들러서 보게되었는데 많이 배우고 갑니다! 코드 보면서 궁금했던 점들 몇가지 적었습니다! 잘 보고 갑니다!!
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
우테코에서 커스텀으로 지정한 형식이 일반적이지가 않아서 고려하기가 힘들다고 생각했습니다 한 번 고려해봐야겠네요
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구글자바포멧을 사용했는데 기본적으로 저렇게 정렬을 해 하더라고요 줄바꿈 포멧을 변경해봐야겠네요
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기존 구글자바포맷이랑 우테코 포멧에서 다른 부분이 있었네요 우테코껄로 다시 적용해야겠네요 감사합니다
lold2424 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기능 요구 사항을 너무 작성을 잘하신것 같네요 기능 명세서의 경우에는 미리 계획을 하고 이에 맞춰서 코드를 짠 것이지 아니면 작업을 하면서 그때그때 수정을 한것인지 궁금합니다.
lold2424 created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 봤습니다. 1주일동안 고생하신만큼 좋은 코드가 나온것 같습니다. 이번주 또한 화이팅입니다 :)
lold2424 created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 봤습니다. 1주일동안 고생하신만큼 좋은 코드가 나온것 같습니다. 이번주 또한 화이팅입니다 :)
yjinlee99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
커스텀 구분자는 문자열 앞부분의 "//"와 "\n" 사이에 위치하는 문자를 커스텀 구분자로 사용한다. 문자열 앞부분이라는 말이있어서 중간에 있는 커스텀 구분자는 고려할 필요없다고 생각하는데, 해석의 차이일수도 있겠네요.
yjinlee99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
공백에 대한 생각을 못했었는데 공백까지 고려하는게 좋네요