Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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가 맞습니다! 분류 후 저장하는 역할을 하고 있습니다.!
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) 범위를 벗어나는 것에 대한 대처도 고려하시면 좋을 것 같습니다.
