Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
음수가 들어오는 경우도 Validator 안에서 미리 해결하면 어떨까요?
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
만약 문자열에 "//"만 있고 "\\n" 없는 경우 예외 호출이 되나요?
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
에러 메시지를 enum으로 관리하는 것을 검색해보세요. 아마 도움이 될 겁니다.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
스트림을 잘 활용하셨네요. 중간에 한 줄 정도 나눠서 쓰셔도 좋을 것 같습니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. 메서드를 적절하게 작게 나누고 역할과 기능이 잘 드러나는 이름을 지으셨네요. 한 메서드가 역할과 기능이 너무 많으면 이름도 장황해지기 쉬운데 두 마리 토끼를 모두 잡으신 것 같습니다. mvc 패턴도 잘 적용하신 것 같습니다. 남은 프리코스 과제도 화이팅하세요!
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. 메서드를 적절하게 작게 나누고 역할과 기능이 잘 드러나는 이름을 지으셨네요. 한 메서드가 역할과 기능이 너무 많으면 이름도 장황해지기 쉬운데 두 마리 토끼를 모두 잡으신 것 같습니다. mvc 패턴도 잘 적용하신 것 같습니다. 남은 프리코스 과제도 화이팅하세요!
yeseong33 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
평소에 백엔드 위주로 코드를 짜서 제가 알고 있기로는 view 는 시각적으로 보여주는 부분이라고 생각되어서 구현에서는 제외했습니다. 백엔드에서 적용해 볼 수 있는 데이터 입출력 관리를 위한 DTO는 간단한 입출력이라 구현하지는 않았지만, 충분히 고려해볼 만하네요!
eunseobb created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기능마다 메소드를 분리해보시는 건 어떨까요? 그리고 역할에 맞게 클래스를 만들어 메소드를 모으면 가독성이 좋아질 것 같습니다!
yummygyudon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
음 은수님 말씀대로 처음에 단순 잘못된 값을 잡아내고 예외를 뱉어내는 "검증"에만 초점을 둔 바람에 역할 분리가 미숙했던 것 같네요 InputValidator에 포함되는 것이 조금 더 적합할 것 같다는 생각이 들어요!! 의견 너무 감사드립니다 🤤
speculatingwook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네 어제 코드 뜯어보았는데, Scanner 객체를 싱글톤으로 생성하고 관리하는 것 같더라구요! 다음부터는 readLine() 사용 후, 자원 반환을 목적으로 사용해려고 합니다! 감사합니다!
speculatingwook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
객체 생성이 되기 전 시점이여서, 크게 상관은 없다고 생각하긴 합니다. 그래도 다음부터는 고려하여 짜보겠습니다!
speculatingwook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아 넵 감사합니다! 다음부터 적용해보겠습니다.
yummygyudon created a comment on a pull request on woowacourse-precourse/java-calculator-7
> 맛집이라는 소개 받고 왔습니다 ㅎㅎ > > 전체적으로 코드 구조가 굉장히 깔끔해서 보기도 편했습니다! 그리고 저는 `Util` 클래스들에 대해서 인스턴스화를 막는건 생각 못해봤었는데, 동규님 코드 보니 제가 디테일한 부분을 꽤 놓치고 있었단걸 깨닫고 갑니다..! > > 코멘트들에 대해서는 서슴없이 의견 달아주세요! 지적 받을만한 내...
speculatingwook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
객체지향의 관점에서 구현한 만큼, 추후 다른 계산들이 들어올 것을 감안하고 상태를 잡아두었습니다.!
chul5 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 문자열을 상수로 관리해보면 어떨까요?!
chul5 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
모델 객체에서 처리하신 것 너무 좋은 것 같습니다.
chul5 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스이름은 동사보다는 명사로 작성하는 것은 어떨까요?
chul5 created a review on a pull request on woowacourse-precourse/java-calculator-7
한 주간 고생많으셨습니다! 늦었지만 코드리뷰 남김니다! 다음주도 함께 힘내봐요!
chul5 created a review on a pull request on woowacourse-precourse/java-calculator-7
한 주간 고생많으셨습니다! 늦었지만 코드리뷰 남김니다! 다음주도 함께 힘내봐요!
yummygyudon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
요구사항 기준으로 최대한 간단하게 고민했습니다. (미션의 핵심 의의는 쓰레기이더라도 돌아가는 것을 빠르게 만들어 내자였던 것을 의식했던 것 같아요...ㅎ) "구분자"의 요구사항을 살펴봤을 때, "**문자열 앞부분의 "//"와 "\n" 사이에 위치하는 문자를 커스텀 구분자로 사용한다.**"라는 표현에서 "사이에 위치하는 문자"라는 표현뿐 이 ...
speculatingwook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
실제로 분류를 하는 주체는 Classifier가 맞습니다! 분류 후 저장하는 역할을 하고 있습니다.!