Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
zzioozzioo created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수고하셨습니다~!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
👍👍👍
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드 시그니처만 봤을 때는 "추출된 숫자들"로부터 "숫자를 추출" 하는 역할을 하는 메서드로 보입니다! 이미 추출된 숫자들을 다시 추출할 필요가 있었을까요??
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기본 연산자들을 enum으로 관리해주셨네요!! 그런데 새로 들어오는 커스텀 구분자는 `String` 으로 관리할 수밖에 없었을 것 같아요. 그래서 enum을 결국에는 다시 `String`으로 꺼내어서 사용하신 것 같은데요. 적절한 enum의 사용이었을까요??
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
```suggestion public static void validateOnlyDigits(String input) { try { Integer.parseInt(input); } catch (NumberFormatException e) { throw new Ill...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
자료형이 String이면 `number`라고 부를 수 없을 것 같아요. 저는 `rawNumber` `rawNumbers`라는 이름을 선호해요. 어떻게 생각하시나요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`target`보다는 더 구체적인 변수명을 지으면 어떨까요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
```suggestion public void extractNumbers(List<Integer> extractedNumbers) { ```
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
ensure과 validate의 차이는 무엇인가요? 용어가 통일되면 더 깔끔하게 작성할 수 있겠네요!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
도메인이 뷰 로직을 직접 의존하고 있어서, 뷰 요구사항이 바뀌게 되면 도메인 코드에까지 영향이 가겠네요! MVC 패턴을 의식하고 작성하신 코드인가요?의존 관계에 대해서 생각해보시면 좋을 것 같습니다 👍
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요. 간단 피드백 남겨봤습니다! 프리코스 화이팅입니다!
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요. 간단 피드백 남겨봤습니다! 프리코스 화이팅입니다!
rjswjddn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
커스텀 구분자가 //의 다음 위치가 2라서 저렇게 했습니다! 상수로 선언해서 처리했다면 알아보기 편했겠네요
WARDKEY created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드 분리에 집중하다보니 생각해보지 못했던 부분이네요 리뷰 남겨주셔서 감사합니다~!
rjswjddn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
맞는 말씀 같습니다!! 기능에 비해서 너무 클래스가 많아지는 것 같아서 안 했는데 다시 보니까 너무 더럽네요 ㄷㄷㄷ..
WARDKEY created a review comment on a pull request on woowacourse-precourse/java-calculator-7
isBlank() 처음 알았는데 다음에 한 번 사용해보겠습니다! 리뷰 해주셔서 감사합니다!!
rjswjddn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
예전에 수업을 들을 때, 인터페이스는 설명서다 라는 말을 들은 적이 있어서 클래스에 메서드가 너무 많아지길래 사용했습니다.
kkmdevel created a review on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 순서가 잘 구성되어있는 코드였어요. 깔끔하고 읽기 편했습니다! 다음에는 Test 코드도 작성해보시면 좋을거같아요! 1주차 고생많으셨어요. 2주차도 같이 힘내봐요!🔥
kkmdevel created a review on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 순서가 잘 구성되어있는 코드였어요. 깔끔하고 읽기 편했습니다! 다음에는 Test 코드도 작성해보시면 좋을거같아요! 1주차 고생많으셨어요. 2주차도 같이 힘내봐요!🔥
rjswjddn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
혹시 다른 계산기 (뺄셈기, 곱셈기... ) 가 생기면 그 input 뒤에 붙혀서 구분하려고 했습니다 ㅎㅎ... 지금 생각해보니까 굳이 이렇게 할 필요는 없었던 것 같네요
sangxxjin created a review comment on a pull request on woowacourse-precourse/java-calculator-7
input을 위한 출력 메시지와 입력을 받는 메서드로 구분하는 것은 어떨까요? 예를들어 displayInputPrompt와 getInput 또는 readInput과 같은 메서드요
sangxxjin created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이러한 입력, 출력에 관한 메서드들은 static 처리를 하는건 어떠실까요? 그러면 Application에서 생성자 생성을 안하고 필요한 곳에서 바로 사용할 수 있을 것 같습니다. controller와 같은 곳에서 말이죠.
sangxxjin created a review comment on a pull request on woowacourse-precourse/java-calculator-7
에러 메시지들을 enum 처리해서 관리하신 부분 인상깊습니다. 많이 배워가요
sangxxjin created a review on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 많이 고민하시고 작성하신 흔적이 느껴집니다. 많이 배워갑니다~