Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
59~77이 무슨 역할을 하는지 주석을 추가하시는 게 어떨까요?
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
프로그램이 종료될 텐데 굳이 참조를 해제해야 될까요?
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분은 삭제하셔도 좋을 것 같습니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. 여러 역할과 책임을 가진 클래스로 나누는 편이 가독성과 유지보수에 좋을 것 같습니다. 문자열 검증 부분을 직접 짜신 것 같은데 기존에 있는 메소드를 활용하는 편도 나쁘지 않으니 한 번 그렇게 짜려면 필요한 게 무엇인지 생각해보시면 도움이 될 것입니다.
sjmmics 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
해당 프로젝트에서는 불필요할 수 있는 것에 동의합니다! 상수화 시키는 것은 일관성이라 생각하게 되었습니다. 코드 전반에서 일관된 규칙으로 사용하여 예측 가능한 패턴을 따르게 되어 수정하는데 편리성에 도모하지 않을 까 생각합니다!
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
변수명에 컬렌션을 명시하는 건 좋지 않은 네이밍이라고 하네요.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스의 역할이 너무 많습니다. 위에서 말씀드린 것처럼 view 클래스를 따로 만드시는 편이 좋을 것 같습니다. veiw -> controller -> service -> controller -> view의 흐름을 따르면 클래스의 역할도 작게 유지하고 분명해질 것입니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. Number 클래스가 쓰이던데 다른 버젼에서 구현하신 것이겠죠? controller가 직접 메인 기능을 수행하지 않고 service에 넘긴 것이 인상 깊네요. 네이밍 컨벤션을 잘 지키신 부분도 있고 조금 미흡한 부분도 있지만 금방 보완하신 것이라고 믿습니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. Number 클래스가 쓰이던데 다른 버젼에서 구현하신 것이겠죠? controller가 직접 메인 기능을 수행하지 않고 service에 넘긴 것이 인상 깊네요. 네이밍 컨벤션을 잘 지키신 부분도 있고 조금 미흡한 부분도 있지만 금방 보완하신 것이라고 믿습니다.
cheoroo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
main메서드에 너무 많은 책임이 있는것 처럼 보여서(사용자 입력 처리, 문자열 분할, 구분자 추출, 덧셈 계산이 모두 main에 들어감) 컨트롤러 클래스를 하나 만들어서 분리해보는건 어떨까요? 아니면 간단한 계산기의 기능을 구현하는 과제였으므로 빠르고 간단하게 기능을 구현하는것이 더 중요하게 판단되었을까요?
cheoroo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
정민님이 작성하신 계산기 메서드에서는 double 타입을 지원하는 계산기 로직으로 작성하셨는데, 해당 메서드에서 커스텀 구분자로 온점이 들어올경우 소수와 구분자를 제대로 구별하지 못할거 같아서 이를 어떤식으로 처리하는지 고민해보는것도 좋을것 같아요 저는 이러한 문제때문에 들어오는 수의 타입을 int타입으로 제한하여 구현했었던것 같아요. 만약 소수...
cheoroo created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 보았습니다. 이번주차에는 없었지만 돌아오는 주차의 요구사항에는 객체지향적 코드를 작성하는것이 들어가 있으므로 요구 사항에 맞추어 많은 고민을 하고 코드를 작성하여 구현하면 좋은 결과가 있을것 같습니다. 수고하셨습니다.
cheoroo 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
ConsoleIO라는 네이밍 때문에 오해한 것 같습니다. 새로운 클래스를 만들고, Application의 내용을 수정해야 하는 것이 연쇄적인 수정이라 생각했습니다. 2주차도 화잇팅입니다!
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
다양한 입력을 고려하여 테스트 케이스 상세히 작성하신 점이 인상 깊었습니다!