Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
Mae-Hyong created a review comment on a pull request on woowacourse-precourse/java-calculator-7
subString을 어떻게 쓰면 좋을지 고민이 많았는데 이렇게 쓰면 됐었군요..! 배워갑니다 ㅠㅠ!
sojeong0202 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`Delimiters`객체를 `Exception Handler`로 설명할 수 있군요! `Exception Handler`를 잘 몰라서 저는 만들면서도 이렇게 생각하지 못했네요 준원님 좋은 개념 알려주셔서 감사합니다!
Jewan1120 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
음수 처리를 위한 필터링을 하고 있는데 제대로 이루어지고 있는 것 같지 않아요 `return str.chars().allMatch(Character::isDigit);`의 isDigit으로는 `-`를 검출할 수 없을 것 같습니다! 어떻게 생각하시나요?
Jewan1120 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
음수나 `0`에 대한 테스트라면 더 좋은 테스트였을 것 같아요!
yseo14 created a comment on a pull request on woowacourse-precourse/java-calculator-7
> mvc 패턴을 사용하고 계신데 calculator와 StringParser는 서비스에 더 가깝지 않을까요? 정규 표현식을 잘 사용하고 계시고, 알아보기 쉽게 상수화해서 적확한 이름을 붙이는 센스에 감탄했습니다. 메서드 별로 주석을 다신 것도 좋고요. 자바독 형식을 사용해서 메서드 주석을 달아보시는 것도 괜찮지 않을까요? 제가 MVC 패턴에서 ...
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
mvc 패턴도 잘 적용하시고 클래스별로 역할과 기능이 적절하게 나누어져 있네요. 구분자와 관련된 상수도 잘 선언하셨고, 에러 메시지를 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 패턴도 잘 적요하시고 클래스별로 역할과 기능이 적절하게 나누어져 있네요. 구분자와 관련된 상수도 잘 선언하셨고, 에러 메시지를 enum으로 관리하신 것도 앞으로 복잡한 요구사항이 생길 때 확장해서 사용하기 유용한 것 같습니다. 컨트롤러와 서비스의 역할을 명확히 한 점도 좋고 네이밍도 적절하게 하셔서 가독성이 높아지네요. 테스트만 조금 보충하시면...
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
mvc 패턴도 잘 적요하시고 클래스별로 역할과 기능이 적절하게 나누어져 있네요. 구분자와 관련된 상수도 잘 선언하셨고, 에러 메시지를 enum으로 관리하신 것도 앞으로 복잡한 요구사항이 생길 때 확장해서 사용하기 유용한 것 같습니다. 컨트롤러와 서비스의 역할을 명확히 한 점도 좋고 네이밍도 적절하게 하셔서 가독성이 높아지네요. 테스트만 조금 보충하시면...
yeseong33 created a comment on a pull request on woowacourse-precourse/java-calculator-7
sjmmics 답변 감사합니다. 덕분에 알지 못했던 장점도 알 수 있었고, 스스로 잘하고 있다고 생각한 부분이 오히려 문제일 수도 있겠다는 생각을 갖게 되었습니다! 저도 리뷰 한번 하러 가겠습니다. 😁
yseo14 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@sjmmics 해당 상황은 고려하지 못하고 코드를 작성했네요 😅 더 많은 예외상황을 고려해보고 코드에 적용할 수 있는 습관을 길러야겠습니다.
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
객체 생성이 되기 전 시점이여서, 크게 상관은 없다고 생각하긴 합니다. 그래도 다음부터는 고려하여 짜보겠습니다!