Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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
minSsan created a review comment on a pull request on woowacourse-precourse/java-calculator-7
사실 이번 과제에서는 불필요했던 부분이긴 한데, 리팩토링 과정에서 들었던 생각이 `커스텀 구분자 파싱 방식이 여러 개가 생긴다면 어떨까` 였습니다. 하지만, 기존 코드에서는 **인터페이스 없이 바로 구현체 내에서 파싱 로직을 넣었**기 때문에, 파싱 방식이 새로 추가될 때마다 _메소드 호출 방식이 일관되지 않을 것 같다_ 는 생각이 들었어요. 그래...
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
String.format을 이용하는것도 방법일 것 같습니다!
na0th created a review on a pull request on woowacourse-precourse/java-calculator-7
보고 배울 점이 많은 PR이었습니다.. 다음 미션도 열심히 참여해주십쇼.. 많이 배우겠습니다.
na0th created a review on a pull request on woowacourse-precourse/java-calculator-7
보고 배울 점이 많은 PR이었습니다.. 다음 미션도 열심히 참여해주십쇼.. 많이 배우겠습니다.
minSsan created a review comment on a pull request on woowacourse-precourse/java-calculator-7
죄송해요 제가 답변이 조금 늦은 것 같네요. 일단 `CustomDelimiter`는 요구사항에 알맞는 입력 형식에 맞게, 커스텀 구분자를 입력했는지 `확인하는 역할`이 분명한 것 같습니다. 그에 반해 `DefaultDelimiter`는 요구사항에 명시된 기본 구분자가 `무엇인지`를 나타내는 것이라고 생각합니다. 즉, 요구사항에 따르면 `C...
Gilbert9172 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
우선 다이어그램 툴은 사용하지 않고 그냥 마크다운에, 어떤 역할을 하는 도메인 모델이 필요한지 적고 시작하는 편입니다! 물론 작업하는 과정에서 기능이 추가되거나 메서드 명이 변경되는 부분도 있어서, 100% 완벽한 설계를 한다고 생각하지는 않아요! 여기 [링크](https://gilbert9172.tistory.com/101) 보시면 이해가 ...
daeuun created a review comment on a pull request on woowacourse-precourse/java-calculator-7
중복로직이 있었네요 감사합니다!
kungbi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 MVC패턴까지 사용해야할 프로젝트라고 판단하지 않아서 사용하지 않았습니다. 그런데 만약 MVC패턴을 사용한다면 service로 빼서 컨트롤러에서의 model과 view의 중간자 역할으로 어떤 데이터를 이동시키는지 잘 확인할 수 있을 것 같습니다.
