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
문자열이 "//"로 시작하는데 "\\n"이 없으면 String.substring 메서드가 예외를 호출하지 않나요?
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
예외 입력값에 대해 조금 더 다양한 접근을 하시면 어떨까요?
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
예외 입력값에 대해 조금 더 다양한 접근을 하시면 어떨까요?
eunseobb created a review comment on a pull request on woowacourse-precourse/java-calculator-7
static으로 작성하신 이유가 궁금합니다 !
pentorb created a review comment on a pull request on woowacourse-precourse/java-calculator-7
정적 메서드에 그런 단점이 있군요. 인스턴스를 생성해서 메서드를 사용해보겠습니다.
leeje0506 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
저의 경우, "관심사"에 따라 다른 것 같습니다. 관심사에 대해 "**단일 값만으로 표현**"할 수 있다면 보통 `static final`을 통해 불변 상수 선언을 하고 관심사에 대해 "**여러 형태의 값으로 표현**" 수 있다면 (_ex. 에러 코드, 도메인 관련 코드_) 하나의 상수 객체로 묶어야할 필요가 있다고 판단되어 Enum을 활용하는...
somin-jeong created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 수고하셨습니다!
somin-jeong created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 수고하셨습니다!
leeje0506 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분은 아직 제가 테스트 작성과 검증을 많이 해보지 않아서 어렵네요ㅠㅠ 장단점을 좀 더 파악해서 공부해보겠습니다!
leeje0506 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좋은 말씀 감사합니다!! 그렇게 생각해볼 수 있겠네요. 배워갑니다.
pentorb 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 패턴을 잘 사용하시려고 많이 노력하신 게 보입니다. 특히 controller가 view 에서 요청을 가져와서 서비스에 넘기기만 한 부분이 클래스의 역할을 아주 잘 구분한 점이라고 생각합니다. 다만 의문인 게 서비스 클래스가 왜 모델 폴더에 있을까요? delimiter 클래스가 모델에 있고 서비스는 서비스 폴더에 두는 게 어떨까요? 테스트도 단위...
sjmmics 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
원시 타입이 아니라 래퍼 타입을 쓰신 이유가 있을까요?
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
콘솔을 이용해서 출력하고 있으니 클래스 이름을 ConsoleInputView라고 하면 어떨까요?
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
mvc 패턴을 잘 사용하시려고 많이 노력하신 게 보입니다. 특히 controller가 view 에서 요청을 가져와서 서비스에 넘기기만 한 부분이 클래스의 역할을 아주 잘 구분한 점이라고 생각합니다. 다만 의문인 게 서비스 클래스가 왜 모델 폴더에 있을까요? delimiter 클래스가 모델에 있고 서비스는 서비스 폴더에 두는 게 어떨까요? 테스트도 단위...
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
mvc 패턴을 잘 사용하시려고 많이 노력하신 게 보입니다. 특히 controller가 view 에서 요청을 가져와서 서비스에 넘기기만 한 부분이 클래스의 역할을 아주 잘 구분한 점이라고 생각합니다. 다만 의문인 게 서비스 클래스가 왜 모델 폴더에 있을까요? delimiter 클래스가 모델에 있고 서비스는 서비스 폴더에 두는 게 어떨까요? 테스트도 단위...
yummygyudon created a comment on a pull request on woowacourse-precourse/java-calculator-7
> `DelimiterConstants`랑 `InputValidationConstants`를 말씀하시는거라면 생성자는 `private`으로 막아져있는 상태입니다! 아하! 예외를 발생시키는 로직이 있길래 `public`이라고 잘못 인지하고 있었나봐요!! 그렇다면 `private` 생성자를 선언하셨다면 외부에서의 객체 생성은 방지될 것 같네요! ...
leeje0506 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
주석을 달아놓으셔서 어떤 목적으로 작성하신지 알 수 있어서 좋았습니다! 다만 클래스이름이 Context여서 정확히 어떤 역할인지 느낌이 바로 안 왔는데 이름을 좀더 구체적으로 작성해보면 어떨까요?!
chul5 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 Validator객체의 메서드를 클래스 메서드 (static)으로 구현하신 이유가 궁금합니다.