Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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
코드 잘 보고 갑니다!
Tentennball 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
말씀하신대로 Extractor가 FormatValidator의 책임을 갖는게 어색한 것 같네요! 저는 문자열 계산기를 숫자를 추출하는 전처리 과정과 합계를 구하는 계산 과정, 이렇게 2개의 큰 기능으로 나누어 생각해서 Preprocessor와 Calculator 두 객체를 구현했습니다. 이 기준에서 Preprocessor가 Extractor, Fo...
yesjuhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
감사합니다 :) 기능 요구 사항에 위배되지 않으면서 확장 가능한 부분을 염두에 두고 작성했습니다
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력값 예외를 상황별로 자세하게 잘 구별해놓으신 것이 돋보이네요. 에러 메세지를 enum으로 관리하시는 분들이 있는데 한 번 고려해보시면 좋을 것 같습니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
상황별로 자세하게 테스트 코드를 작성하셨네요. 입력 값 검증을 위해 많이 노력하신 것이 보입니다. 예외 상황도 많이 분류해놓으셨네요. 다만 한 가지 의문인 점이 service 클래스를 왜 model 폴더에 넣으셨을까요?
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
상황별로 자세하게 테스트 코드를 작성하셨네요. 입력 값 검증을 위해 많이 노력하신 것이 보입니다. 예외 상황도 많이 분류해놓으셨네요. 다만 한 가지 의문인 점이 service 클래스를 왜 model 폴더에 넣으셨을까요?
yesjuhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좋은 의견과 리뷰 감사합니다!! 🙇 🙇♂️ 🙇♀️ 이 부분은 구현 당시 깊게 생각하지 않고 위처럼 작성을 했다가 미션 끝나고 디스코드에서 의견 나누는걸 보면서 "아 맞다 수학적으로 0은 양수에 포함이 안되는구나"하고 생각을 한 케이스인데요ㅎㅎ 그래서 "왜 내가 큰 고민 없이 0을 양수에 포함 시켰을까?"하고 나중에 추측해보고 다음과 같...
bogyeom0922 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드를 두 개로 나누어 작성하신 이유는 뭔가요? 하나로 합치면 간단할 것 같은데 나눠서 쓰신 이유가 궁금해서요!
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
1주차 미션 수행하느라 수고하셨습니다 !!
bogyeom0922 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수행하느라 수고하셨습니다 !!