Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
changbill created a review comment on a pull request on woowacourse-precourse/java-calculator-7
상수화 하는 것에 2를 빠뜨렸군요.. 수정해보겠습니다!
changbill created a review comment on a pull request on woowacourse-precourse/java-calculator-7
맞다고 생각합니다..! 코드 리뷰를 하다보니 어떤 잘못을 한 것인지 명확하게 보이더라고요
cheoroo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기존 작성한 예외 테스트 코드와 같은 기능을 검증하는 테스트 코드를 testNegativeNumberThrowsException로 작성했다고 생각했는데 ([63efa03](https://github.com/woowacourse-precourse/java-calculator-7/pull/1607/commits/63efa03e0e3aeb64cbed70cf...
changbill created a review comment on a pull request on woowacourse-precourse/java-calculator-7
감사합니다 다음 과제때는 단위 테스트로 구현해보도록 하겠습니다!
marin-Rim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
유효성 검증을 계속 사용하셔서 유틸로 유효성 기능을 빼보면 어떨까? 라는 생각도 들었습니다.
marin-Rim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
convert가 문자를 숫자로 바꿔준다는 의미를 표현하면 더 좋을 것 같아요
marin-Rim created a review on a pull request on woowacourse-precourse/java-calculator-7
유효성 검증을 철저히 하시고 폴더 구분을 명확히 하셔서 꼼꼼함과 깔끔함을 배우고 갑니다!
marin-Rim created a review on a pull request on woowacourse-precourse/java-calculator-7
유효성 검증을 철저히 하시고 폴더 구분을 명확히 하셔서 꼼꼼함과 깔끔함을 배우고 갑니다!
saera-yook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네 아무 입력이 주어지지 않았을 때 0을 반환하기 위해서입니다! 미션 예시에 `"" => 0` 이 있고, 숫자는 양수만 입력할 수 있기 때문에 구분자와 구분자 사이에 문자가 없을 때 오류가 아닌 0으로 처리하기 위해 이렇게 구현했습니다.
yjy323 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
MVC 패턴은 그냥 형식일 뿐입니다! 디자인 패턴을 적용했을 때와 아닐 때의 차이, 장단점 등을 경험하시는 것이 더 우선시 되야 한다는게 제 생각입니다. 프리코스라는 짧은 기간 안에 그것을 다 경험하고 배울 수는 없지만, 딱 떨어지는 정답을 찾으려 하시기 보다는 코드를 작성하기 전에 본인만의 의도가 무엇인지, 코드가 그 의도를 만족했는지 고민해보신다면...
yjy323 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 isSeparator의 로직이 복잡해질 가능성이 적다면 그냥 separators.contains(c)를 쓰는 것도 좋아보입니다. 코드를 읽을 때 계속 참조..참조...하면서 확인하는게 번거롭더라고요. 이건 스타일 차이라 한번 스스로 고민해보세요!
yjy323 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 이중 if문으로 인해 가독성이 떨어진다기보단 로직 자체의 가독성, 또는 과도한 주석이 원인인 것 같습니다. 자바 API에 익숙하지 않으시기 때문이니까 시간이 해결해줄 문제라고 생각합니다! 예를 들어 inputString.charAt(i) == '/' && inputString.charAt(i + 1) == '/' 는 inputString.st...
YehyeokBang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
특정 상수가 해당 클래스에서만 사용된다면, 굳이 외부로 분리하지 않고 그 클래스 내부에서 관리하는 것이 더 직관적이라고 생각합니다. 이렇게 하면 불필요한 복잡성을 줄이고, 가독성을 유지할 수 있다고 생각하고 있어요. 만약 여러 곳에서 사용되는 경우 상수를 위한 클래스를 따로 만드는 방법이 있을 수도 있고, 특히 상수들이 같은 타입을 가지고, 카테...
yesjuhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
감사합니다 꼭 읽고 적용해보겠습니다! 리뷰를 받으면서 네이밍에 대한 고민이 많이 부족했다는 것을 느끼네요 👍 🫶
seongwon030 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자를 콜론으로 대체하고 한꺼번에 콜론 기준으로 split하네요. 깔끔하게 잘 짜신 것 같습니다 ㅎㅎ
seongwon030 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
단순히 궁금해서 여쭤보는건데, java에서는 따로 입출력 라이브러리가 제공되는 게 있나용? 프론트엔드는 <code>@woowacourse/mission-utils</code>라이브러리가 따로 제공됩니다.
seongwon030 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 고생 많으셨습니다!
seongwon030 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 고생 많으셨습니다!
saera-yook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
위에서 언급했듯 전처리 과정과 계산 과정으로 나누어 생각해서 책임을 나눠놓았습니다. 추후에 다른 연산 기능이 추가되거나 계산이 복잡해진다면 Calculator의 책임이 커질 것 같아서요. 대신 Preprocessor와 Calculator를 연결하는 컨트롤러가 있으면 좋을 것 같네요!(아직 컨트롤러의 역할에 대해 100% 이해하지는 못해서 잘못된 생각...
yesjuhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좋은 의견 감사합니다 👍 `length()`를 사용하는 것이 오류 방지 뿐만 아니라 의미 전달 차원에서도 좋겠네요!