Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
soeunnPark created a review on a pull request on woowacourse-precourse/java-calculator-7
많은 고민을 하고 작성하신 코드라는게 느껴졌습니다! 상수 처리와 같은 부분에서 굉장히 꼼꼼하게 코드를 작성하신 모습에서 많이 배웠습니다 👍 1주차 미션 수고 많으셨습니다!! :)
rladmstn created a comment on a pull request on woowacourse-precourse/java-calculator-7
> 아하! 예외를 발생시키는 로직이 있길래 `public`이라고 잘못 인지하고 있었나봐요!! 그렇다면 `private` 생성자를 선언하셨다면 외부에서의 객체 생성은 방지될 것 같네요! > > 다만 `private`으로 막았음에도 내부에 **예외를 발생시키는 로직을 추가한 이유**도 조심스레 여쭤보고 싶습니다!! 해당 클래스에서 스스로의 객체 생성...
Mae-Hyong created a review comment on a pull request on woowacourse-precourse/java-calculator-7
에러의 경우 상수로 분리할 경우 손쉽게 관리할 수 있을 것 같습니다. :)
hyunn522 created a review on a pull request on woowacourse-precourse/java-calculator-7
unit test를 정말 꼼꼼하게 작성하셨네요 잘 봤습니다!
hyunn522 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
하나의 메소드 내에 여러 책임이 포함되어있는 것 같아요! SRP를 위해 메소드를 분리해보시는 건 어떨까요?
hyunn522 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
일급 컬렉션 사용 좋습니다! 다만 일급 컬렉션 객체는 일종의 자료구조와 유사한 형태라고 알고 있어서 `service` 디렉토리보단 `model` 디렉토리가 더 적절할 것 같은데 어떻게 생각하시나요?
hyunn522 created a review on a pull request on woowacourse-precourse/java-calculator-7
unit test가 정말 꼼꼼하게 작성되어 있어서 잘 봤습니다!
hyunn522 created a review on a pull request on woowacourse-precourse/java-calculator-7
unit test가 정말 꼼꼼하게 작성되어 있어서 잘 봤습니다!
yeseong33 created a comment on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 봤습니다. 전체적으로 코드를 잘 짜신것 같습니다. 한 클래스에 여러 역할을 부여하지 않았고, stream, interface, getter, view 등의 기본적인 기능들을 잘 활용하시는 것 같아요! 그리고 개인적으로 항상 함수 이름을 설정할때 고민이 되는데, 작성자님은 길게 쓰더라도 의미를 충분히 부여하려고 한 것 같아요. 이렇...
Jewan1120 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
헛 그러게요.. 따로 메서드로 뺐어야할 것 같아보이네요
Jewan1120 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
간단한 비즈니스 로직이라 하나의 서비스에 담으려고 했던 것 같네요! 나중에는 분리해서 작성해볼게요 감사합니다 ㅎ
sojeong0202 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그러면 예외에 대해 정의하는 부분을 따로 만들고 그 예외에 대한 처리를 Delimiter에서 하는 방식으로 하면 준원님 말씀처럼 분리가 될까요? 좋은 개념 알려주셔서 감사합니다! `Exception Handler` 공부해보겠습니다~
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 해당 상황은 고려하지 못하고 코드를 작성했네요 😅 더 많은 예외상황을 고려해보고 코드에 적용할 수 있는 습관을 길러야겠습니다.