Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 run() 메소드는 Application.java 파일에서 실행됩니다. 현재 CalculatorController에서는 공용 데이터나 인스턴스 필드를 포함하지 않고 있기 때문에 static을 사용하였습니다. 또한, 프로그램이 시작되면서 힙 영역보다는 스택 영역에 할당되는 것이 조금 더 빠르지 않을까 생각했기 때문에 위와 같이 프로그래밍하였습...
holyPigeon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네이밍에 대한 의문점이 있는 것 같습니다! 방금 `Adder` 클래스를 보고 왔는데, classifier 인스턴스로부터 숫자 리스트를 가져오더라구요. 본래 문자열로부터 구분자를 활용하여 숫자를 뽑아내고, 분류하는 게 `Classifier` 클래스의 일인데, `Adder`에서는 이미 분류가 끝나 단순히 구분자 + 숫자 저장소의 역할밖에 안 하는 `...
holyPigeon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자에는 중복을 허용하지 않고, 저장 순서 또한 상관없기 때문에 Set을 사용하는 것이 좋은 것 같습니다!
holyPigeon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`Console` 라이브러리에서는 `readLine()`, `close()` 2개의 메서드를 지원하더라구요! 아무래도 `readLine()` 메서드가 Scanner를 활용하기 때문에 입력을 종료시켜주는 `close()` 메서드도 존재하는 것 같습니다. `close()`도 활용해보시면 좋을 것 같아요!
holyPigeon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
사소한 부분인데 `isCustomDelimiterValid()`가 `this.customDelimiter = getCustomDelimiter(input);` 보다 먼저 나와야 하지 않을까요? 검증이 완료된 값을 주입하는 게 올바른 것 같습니다!
holyPigeon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
혹시 꼭 필드 변수를 둬야 하는 이유가 있었는지 궁금합니다. 저 같은 경우에는, 구분자나 숫자들을 필드에 저장하지 않고 메서드에서 바로 반환했거든요! "덧셈 연산"이라는 기능을 제외하고도 해당 값들을 더 사용할 일이 있을 거라고 판단하셨나요?
holyPigeon created a review on a pull request on woowacourse-precourse/java-calculator-7
리뷰가 좀 늦어졌네요..ㅠㅠ 1주차 너무 고생하셨고, 2주차도 화이팅이에요!!
holyPigeon created a review on a pull request on woowacourse-precourse/java-calculator-7
리뷰가 좀 늦어졌네요..ㅠㅠ 1주차 너무 고생하셨고, 2주차도 화이팅이에요!!
yummygyudon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
사실 입력 기능 Test하면서 `Console.readIne`에서 사용되는 Scanner에 의해 발생되는 예외였고 이를 해결하고자 추가하게 된 예외 처리였습니다! Scanner 사용에 미숙한 탓에 발생한 불필요한 예외 처리인 것 같습니다...ㅠㅜㅠ 다음 미션부터는 잘 개선해야봐야겠네요!! (좋은 부분 질문 주셔서 감사합니다!) 비록 1주...
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
답변해주셔서 감사합니다! negative의 의미에 맞게 은섭님께서 말씀해주신대로 고치면 좋을거 같네요!
yummygyudon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
사실 보통 관습적으로 MVC 패턴을 적용하여 패키지를 나누는 경우가 많다고 생각합니다. 하지만 규모가 작은 어플리케이션 혹은 매우 간단한 어플리케이션을 구현할 때, 패키지와 클래스를 과도하게 분리하는 것은 오히려 개발 생산성을 저하시킨다고 생각해요. (클래스는 별로 없는데 패키지만 많으면 여기저기 찾으러 다니는 것은 비효율적일 거에요) ...
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
합이 정수(int) 범위를 벗어나는 것에 대한 대처도 고려하시면 좋을 것 같습니다.
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
이 부분은 아직 제가 테스트 작성과 검증을 많이 해보지 않아서 어렵네요ㅠㅠ 장단점을 좀 더 파악해서 공부해보겠습니다!