Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
junghunim07 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 하시느라 고생 많으셨습니다 !!! 👍 이번 코드를 보면서 구분자를 Enum으로 관리하는 아이디어는 한 수 배웟습니다 ㅎㅎ 😄 학습하시면 좋을 것 같은 제안 두가지 하고 가겠습니다 !! 1. Enum 활용법 2. 테스트 코드 API 에 대해서 학습하시면 더욱 좋은 개발자로 성장하실 것 같아요 !!
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기능이 확장되어 연산 클래스를 추가하게 된다면, 연산 클래스의 Interface를 정의하고 이것을 구현하는 방식으로 진행해도 좋겠네요!
mixxeo created a review on a pull request on woowacourse-precourse/java-calculator-7
팩토리 패턴이 어떻게 적용되는지 블로그 보고 궁금해서 찾아왔습니다 :) 많은 분들이 정규표현식을 사용해 문자열을 통째로 분석하는 방식으로 구현하신 것 같은데, 제이슨님께서 접근하신 방식은 생각하지 못했어서 코드 너무 잘 봤습니다! 2주차도 화이팅입니다 👍
mixxeo created a review on a pull request on woowacourse-precourse/java-calculator-7
팩토리 패턴이 어떻게 적용되는지 블로그 보고 궁금해서 찾아왔습니다 :) 많은 분들이 정규표현식을 사용해 문자열을 통째로 분석하는 방식으로 구현하신 것 같은데, 제이슨님께서 접근하신 방식은 생각하지 못했어서 코드 너무 잘 봤습니다! 2주차도 화이팅입니다 👍
junghunim07 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
테스트 코드에 대해서 학습을 하시면 좋을 것 같습니다 !! 👍 테스트 코드 API를 사용하시면 더욱 가독성 좋은 테스트 코드를 작성하실 수 있어요 !!
junghunim07 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
if-else 문 방식보단 메서드를 통해 반환하는 형식이 더 좋지 않을까요? ㅎㅎ 👍
junghunim07 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
함수 작명 학습해보시면 좋을 것 같습니다 !! 👍 ex ) preprocessingInput(String input) -> preprocess(String input) 😃
junghunim07 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자가 될 수 없는 Delimiter를 enum으로 생성하는 아이디어는 가독성 면에서 엄청 좋네요 !!! 👍 InputValidator를 따로 분리하신 이유가 궁금합니다. 제 생각엔 Delimiter에서 이 데이터 값들을 꺼내오는 방식이 아닌 구분자 데이터를 Delimiter에게 넘겨줘서 이 안에서 처리하는 로직을 구현했을 것 같습니다 ...
junghunim07 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 하시느라 고생 많으셨습니다 !!! 👍 이번 코드를 보면서 Basic을 Enum으로 관리하고 List에 구분자들을 저장하는 아이디어는 한 수 배웟습니다 ㅎㅎ 😄 학습하시면 좋을 것 같은 제안 두가지 하고 가겠습니다 !! 1. Enum 활용법 2. 테스트 코드 API 에 대해서 학습하시면 더욱 좋은 개발자로 성장하실 것...
junghunim07 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 하시느라 고생 많으셨습니다 !!! 👍 이번 코드를 보면서 Basic을 Enum으로 관리하고 List에 구분자들을 저장하는 아이디어는 한 수 배웟습니다 ㅎㅎ 😄 학습하시면 좋을 것 같은 제안 두가지 하고 가겠습니다 !! 1. Enum 활용법 2. 테스트 코드 API 에 대해서 학습하시면 더욱 좋은 개발자로 성장하실 것...
Yunsu4 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
각 if문을 함수로 따로 만들어도 좋을 것 같아요. 그럼 각 함수가 하는 일이 너무 없다고 생각하실 수도 있지만 의도가 드러나는 함수명을 사용하면 주석이 없어도 validateInput의 검증 과정이 더 명확히 드러날 것 같습니다😊
Yunsu4 created a review on a pull request on woowacourse-precourse/java-calculator-7
다양한 입력을 고려하여 테스트 케이스 상세히 작성하신 점이 인상 깊었습니다!
Yunsu4 created a review on a pull request on woowacourse-precourse/java-calculator-7
다양한 입력을 고려하여 테스트 케이스 상세히 작성하신 점이 인상 깊었습니다!
marin-Rim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
DTO가 프로세스 간의 데이터 전달하는 객체군요. 새롭게 배워갑니다! 구분자들의 처리와 전달 과정을 같이 묶어서 객체로 만든 점이 좋은 것 같아요.
marin-Rim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
delimiters의 접근제어자를 public으로 설정한 이유가 있을까요? StringParser에서 호출하는 용도라면 Getter로 만드는 방법이 더 안전할 것 같다는 생각이 들었습니다! <img width="519" alt="image" src="https://github.com/user-attachments/assets/f1c85b94-b755...
marin-Rim created a review on a pull request on woowacourse-precourse/java-calculator-7
깔끔하게 구분자를 처리하는 방법을 배우고 갑니다 :)
marin-Rim created a review on a pull request on woowacourse-precourse/java-calculator-7
깔끔하게 구분자를 처리하는 방법을 배우고 갑니다 :)
CISXO 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
미션 예시에 입력 ""의 결과는 0이라고 나와 있어서 수정이 필요해요!!
rjswjddn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
커스텀 구분자 추출을 따로 메서드로 빼면 if문이 많이 중첩되는 것을 막을 수 있을 것 같아요!
rjswjddn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 이런식으로 구현했던 것 같은데..... 검증하고 변환 부분을 나누면 더 가독성이 좋아질 것 같습니다!
rjswjddn created a review on a pull request on woowacourse-precourse/java-calculator-7
고생하셨습니다!! 2주 차도 화이팅입니다!!
rjswjddn created a review on a pull request on woowacourse-precourse/java-calculator-7
고생하셨습니다!! 2주 차도 화이팅입니다!!
jintakkim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 전체적으로 어떤 목적성과 기준을 갖고 코드를 분리하려 노력하신 흔적이 보였습니다! 가독성도 좋고, 코드의 논리가 이해하기 좋았습니다. 수고하셨습니다 :) 좋은 리뷰 감사합니다. 저도 yjy님 코드보면서 영감을 많이 받았는데 참고해서 2주차 미션 수행할 수 있도록 노력하겠습니다.
jintakkim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> parse가 계산기 고유의 기능은 아니니 유틸성 기능이라는 해석도 동의하지만, 저는 이번 미션에서는 꽤 비중있는 비즈니스 로직이 아닐까? 라고 생각했습니다! 그쵸 여기가 저도 핵심이라고 생각했습니다.
jintakkim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
맞는 말씀입니다. 기능이 복잡하지 않아서 바로 클레스를 만들까 고민했는데 좋은 지적 감사합니다.