Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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()`를 사용하는 것이 오류 방지 뿐만 아니라 의미 전달 차원에서도 좋겠네요!
bogyeom0922 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 부분의 예외 처리도 별도의 예외 처리 클래스를 사용하면 더 역할이 명확해졌을 것 같아요!
bogyeom0922 created a review on a pull request on woowacourse-precourse/java-calculator-7
굉장히 세분화해서 클래스를 작성하신 것 같아요! 저와 다른 점이 많아 인상 깊게 봤습니다!! 미션 진행하느라 수고하셨습니다
bogyeom0922 created a review on a pull request on woowacourse-precourse/java-calculator-7
굉장히 세분화해서 클래스를 작성하신 것 같아요! 저와 다른 점이 많아 인상 깊게 봤습니다!! 미션 진행하느라 수고하셨습니다
venus-y created a comment on a pull request on woowacourse-precourse/java-calculator-7
> 안녕하세요 금성님! 1주차 미션 완료하시느라 고생하셨습니다:) 단일 클래스의 책임 분할에 대해 고민하신 것이 느껴져 좋았습니다. 다만 package분리를 조금 해주시면 더욱 구조를 한 눈에 보이게 할 수 있을 것 같습니다!! 맞습니다. 코드의 가독성을 챙기는 것뿐만 아니라 보다 직관적인 프로젝트 구조를 챙겨보도록 개선해보도록 하겠습니다. 감...
dPwls0125 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
출력할 String을 재사용 할 가능성에 대해 고려했던 것 같습니다. 지금 생각해보니 해당 과제에서는 굳이 분리하지 않아도 될 부분이었던 것 같아요. 코멘트 감사드립니다!
yesjuhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드의 구현 방식을 먼저 정하고 구현한 것은 아니고 구현 후 인스턴스 변수 사용 여부에 따라 결정했습니다! 클래스의 역할을 정하고 구현 했을 때 `DelimiterParser`는 인스턴스변수를 메서드에서 사용해야해서 인스턴스 메서드로 정의했습니다. `Calculator`는 스트링을 받아서 결과값을 정수로 반환하는 기능만 필요했기 때문에 클...
Tentennball created a review comment on a pull request on woowacourse-precourse/java-calculator-7
하지만 너무 MVC패턴이나 레이어드 아키텍처에 의존하다기 보다는 OOP를 목적으로 짜시는 것을 추천드리는 바이긴 합니다!!
saera-yook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그러네요.. 문자열 길이가 길어질수록 손해이니 검색할 문자를 정규표현식으로 표현히면 한 번의 메서드 호출로 끝낼 수 있겠네요! 덕분에 다시 생각해 볼 수 있었습니다 :)