Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그것도 좋은 방법이겠네요~ 피드백 감사해요~☺️
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
코드를 되게 꼼꼼히 읽어보셨나보네요~ 지금 보니까 COMMA, COLON이 독립적으로 쓰이는 곳이 없네요~ 민서님이 말씀하신대로 수정하면 가독성이 더 올라가겠네요 ㅎㅎ 좋은 피드백 감사합니다~
daeuun created a review comment on a pull request on woowacourse-precourse/java-calculator-7
목적을 정확하게 전달하는 네이밍 작성에 유의해서 작성해야겠네요! 감사합니다
daeuun created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분은 다른 곳에서 커스텀 메세지를 가지는지 이미 확인하는데, 또 내부에서 검증하고 있어서 로직이 중복되는 것과도 연관되는 문제인거 같아요. 제안 감사합니다!
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
StringSumCalculator에서 double 타입으로 연산이 되고 있어서, `.`을 구분자로 사용못하게 하신것 같은데요~ 근데 미션의 입력 요구사항이 "구분자와 양의 정수" 였잖아요 ?? 그래서 일단은 요구사항에 맞게 양의 정수 연산으로 구현하는게 맞다고 생각해요~ 만약 확장을 고려하고 싶다면, 제네릭이나 메서드 오버로딩을 사용하시는게 어...
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Pattern 타입은 컴파일시에 패턴 분석이 들어가서, 비용이 큰 객체인데요~ 요고를 개선할려면 어떻게 하는게 좋을까요 ??
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
StringParser에는 파싱, 유효성 검증, 데이터 변환이 혼재되어 있어서, 해당 객체의 책임이 많다고 느껴져요 SRP를 지키기 위해 어떻게 개선하면 좋을까요 ??
phk1128 created a review on a pull request on woowacourse-precourse/java-calculator-7
이미 다른분들이 개선 포인트를 잘 짚어주셔서, 리뷰드릴 부분이 많이 없었어요~ 제가 말씀드린게 정답은 아니지만 민서님에게 도움이 됐으면 좋겠네요 ㅎㅎ 그럼 2주차도 화이팅하세요~
phk1128 created a review on a pull request on woowacourse-precourse/java-calculator-7
이미 다른분들이 개선 포인트를 잘 짚어주셔서, 리뷰드릴 부분이 많이 없었어요~ 제가 말씀드린게 정답은 아니지만 민서님에게 도움이 됐으면 좋겠네요 ㅎㅎ 그럼 2주차도 화이팅하세요~
nns503 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드명에서 어떤 연산이 적용되지는 명확히 나타내면 가독성이 더 좋을 것 같습니다.
nns503 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
음수 연산자만 따로 구분한 이유가 궁금합니다!
nns503 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드명을 덧셈 연산자를 의미하는 것으로 바꾸는 것이 이해하기 더 좋을 것 같습니다!
nns503 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
문자열에서 수를 추출하는 방법이 간결하고 깔끔하게 작성되어서 좋은 것 같습니다. 배울 점이 많은 부분 같아요!
nns503 created a review on a pull request on woowacourse-precourse/java-calculator-7
수고하셨습니다!! :) 코드가 전반적으로 간결해서 이해하기 편했고 배울 점이 많았던 것 같습니다. 2주차도 화이팅!!!
nns503 created a review on a pull request on woowacourse-precourse/java-calculator-7
수고하셨습니다!! :) 코드가 전반적으로 간결해서 이해하기 편했고 배울 점이 많았던 것 같습니다. 2주차도 화이팅!!!
gladhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
1. 저는 커스텀구분자로 문자열이 들어오게 되는 경우도 받기로 결정을 했습니다. - `문자`라고 표현이 되어있어서 만약 문자열로 주어지면 한 문자씩을 구분자로 선정했습니다. 2. 맞습니다. 그래서 주어진 예시에선 제 의도대로 `커스텀 구분자`만이 들어온 상황입니다. 그래서 구분자를 제외한 부분은 `""`이기 때문에 숫자 `0`이 출력이 됩니다 ...
gladhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아무래도 리뷰해주신 분들의 얘기를 통합해서 제 코드를 다시 봐보니 일관되지 않는 문제가 있었다는 것을 깨달았습니다. 1. `"CalculatorRequestDto calculatorRequestDto = InputView.beginInput();"` - View 단에서 Dto생성에 대해서 문제가 있었다는 것을 알았습니다. 그래서 컨트롤러에서 Dt...
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메소드 이름에 get를 붙이는 게 나아보입니다. https://tecoble.techcourse.co.kr/post/2020-04-26-Method-Naming/ 참고하셔요.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
https://github.com/woowacourse-precourse/java-christmas-6 6기 프리코스 미션 프로그래밍 요구사항을 보면 들여쓰기 깊이는 최대 2단까지 하라고 되어있습니다. 이번에는 똑같은 요구사항이 없었지만 인덴트 3단 이상인 코드는 가독성이 떨어집니다. 별도의 메소드로 분리하시는 방법은 어떨까요?
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
값을 반환하는 메소드 네이밍은 get으로 시작하는 게 관례입니다.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
9번째 줄이랑 합쳐서 한 번에 선언하고 초기화하는 게 낫지 않을까요?
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
프로그램 요구사항에 자바 코드 인벤션을 지켜야 한다고 되어 있습니다. https://github.com/JunHoPark93/google-java-styleguide 번역해놓은 위 페이지를 참고하시면 다음과 같이 써있습니다. 4.1.1 괄호는 선택사항에서도 쓰인다. 괄호는 if, else, for, do, while 구문에 쓰이는데...
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
고생하셨습니다. mvc패턴에 대해서 검색해보시고 다음에 한 번 적용해보시는 걸 추천드립니다. 먼저 자바 백엔드에서 가장 주도적인 프레임워크는 스프링이고 스프링을 사용해서 mvc 패턴으로 만들어진 사이트가 많습니다. 앞으로도 익숙해지셔야 하는 패턴입니다. 다음으로 예전 우테코 프리코스 과제 요구사항을 보시면 inputview와 outputview를 구현...
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
고생하셨습니다. mvc패턴에 대해서 검색해보시고 다음에 한 번 적용해보시는 걸 추천드립니다. 먼저 자바 백엔드에서 가장 주도적인 프레임워크는 스프링이고 스프링을 사용해서 mvc 패턴으로 만들어진 사이트가 많습니다. 앞으로도 익숙해지셔야 하는 패턴입니다. 다음으로 예전 우테코 프리코스 과제 요구사항을 보시면 inputview와 outputview를 구현...
minSsan created a review comment on a pull request on woowacourse-precourse/java-calculator-7
로직 내에서 사용하는 값들을 모두 불변 값을 유지하기 위해 습관적으로 `final`을 사용했는데, 무의식적으로 컬렉션에도 적용한 것 같습니다. 지적해주셔서 감사해요! :D
