Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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
그러네요.. 문자열 길이가 길어질수록 손해이니 검색할 문자를 정규표현식으로 표현히면 한 번의 메서드 호출로 끝낼 수 있겠네요! 덕분에 다시 생각해 볼 수 있었습니다 :)
Tentennball created a review on a pull request on woowacourse-precourse/java-calculator-7
다른분들이 이미 피드백을 많이 해주셔서 좋네요,,! 전체적으로 enum을 사용한 상수 설정과 도메인을 잘 나누어서 2주차때 구현해보면 좋을것 같아요 수고하셨습니다!
swlee8783 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분도 위에서 언급한 중첩된 if문 내용과 동일하게 분리시키면 좋을 것 같아요!
swlee8783 created a review on a pull request on woowacourse-precourse/java-calculator-7
전반적으로 클래스가 명확하게 구분되어 있고, 예외처리가 꼼꼼하게 되어 있으며, 테스트 코드가 체계적으로 목적이 명시되고 그에 맞게끔 구성되어 있어서 좋았던 것 같아요! 1주차 과제 진행하느라 고생 많으셨습니다..! :)
swlee8783 created a review on a pull request on woowacourse-precourse/java-calculator-7
전반적으로 클래스가 명확하게 구분되어 있고, 예외처리가 꼼꼼하게 되어 있으며, 테스트 코드가 체계적으로 목적이 명시되고 그에 맞게끔 구성되어 있어서 좋았던 것 같아요! 1주차 과제 진행하느라 고생 많으셨습니다..! :)
cheoroo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좋은 리뷰를 남겨주셔서 감사합니다. 돌아보니 해당 부분은 잘못 작성된 코드인것 같습니다. input을 불변처리 해야 사이드 이펙트가 나타나지 않을것 같네요. 코드를 작성할 당시에는 간단한 계산기 기능을 충족시키는 조건에서 매개변수를 직접 변경하는 것이 그렇게 어색하게 느껴지지 않아서 해당 코드로 작성했었던 것 같아요. 다시 코드를 작성한다면 숫자...
saera-yook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
생성자 매개변수로 받는게 객체를 더 유연하게 만들 수 있겠네요! 다음엔 염두에 두고 설계해야겠습니다ㅎㅎ
sonjh919 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
두 부분 다 상수의 역할을 수행하지만 출력의 특성상 상수가 많아진다면 따로 빼는 것이 더 가독성이 좋을 것이라 생각했습니다! 하지만 둘 다 상관없다는 생각은 드네요 ㅎ
sonjh919 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
음.. 솔직하게 말하자면 그부분은 저도 잘 모르겠어요. 저도 이번에 할때 번거로워서 한글로 할까도 생각중인데, 찾아봐도 두 방법의 장단점이 명확하게 보이진 않더라고요. 혹시 두 방법의 명확한 차이점을 아신다면 공유 부탁드려요! ㅎㅎ
sonjh919 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그 부분은 조금 더 공부하고 2주차에 한번 써보겠습니다..!
Tentennball created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 하나의 함수는 하나의 역할만 한다. 하나의 함수에서 여러개의 역할을 가진다는 것은 해당 함수에 대한 의존성이 높아진다는 말 같습니다
Tentennball created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 부분도 상수를 활용하면 좋을것 같습니다!
Tentennball created a review comment on a pull request on woowacourse-precourse/java-calculator-7
에러메시지도 상수화하면 좋을 것 같아요! enum을 사용해도 좋을 것 같아요!
Tentennball created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 보고 갑니다!