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
스트림 사용도 한 번 고려해보셨는지요? 혹시 전혀 고려하지 않으셨다면 스트림도 익숙해지시면 종종 스트림을 활용하면 좋은 경우가 있을 겁니다. 물론 스트림으로 바꾸는 게 능사는 아니지만요.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
null를 반환할 수 있는 메소드에는 이름에 알 수 있도록 표시해주는 게 어떨까요? 예를 들어 getCustomSepartorElseNull()이런 식으로요.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
커스텀 구분자를 지정하는 문자열 "//" 과 "\\n"를 상수로 관리하는 건 어떨까요?
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. IOMessage를 enum으로 관리하신 점, 입력값 형식 검증 예외 기준을 세심하게 짜신 점, 입력값 예외 메시지를 세분화시키신 점에서 세심한 접근이 눈에 띕니다. mvc 패턴에 대해 이해도 있으신 것 같습니다. 만약 확장을 고려한다면 구분자를 클래스로 따로 관리하는 건 어떨까요? 예를 들어 구분자별로 더하기가 아닌 다른 사칙연산이...
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. IOMessage를 enum으로 관리하신 점, 입력값 형식 검증 예외 기준을 세심하게 짜신 점, 입력값 예외 메시지를 세분화시키신 점에서 세심한 접근이 눈에 띕니다. mvc 패턴에 대해 이해도 있으신 것 같습니다. 만약 확장을 고려한다면 구분자를 클래스로 따로 관리하는 건 어떨까요? 예를 들어 구분자별로 더하기가 아닌 다른 사칙연산이...
dbsdndcks created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스 내부에서 사용하는 메서드들은 private 처리하시는 것도 고려해보시면 좋을 것 같습니다!.
dbsdndcks created a review comment on a pull request on woowacourse-precourse/java-calculator-7
다양한 테스트 코드를 작성하셔서 예외를 고려하신게 인상깊습니다!. 배워갑니다😁
dbsdndcks created a review on a pull request on woowacourse-precourse/java-calculator-7
적절한 변수들의 상수화와 다양한 테스트 코드들이 인상깊습니다😁 클래스 내부에서 사용되는 메서드들만 private 처리하시는 것만 고려해보셔도 좋을 것 같습니다. 전체적으로 단일 책임 원칙을 지키려는 모습이 느껴집니다. 수고많으셨습니다.👍
dbsdndcks created a review on a pull request on woowacourse-precourse/java-calculator-7
적절한 변수들의 상수화와 다양한 테스트 코드들이 인상깊습니다😁 클래스 내부에서 사용되는 메서드들만 private 처리하시는 것만 고려해보셔도 좋을 것 같습니다. 전체적으로 단일 책임 원칙을 지키려는 모습이 느껴집니다. 수고많으셨습니다.👍
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
커스텀 구분자에는 기본 구분자를 포함하기 때문에 둘이 나눌 필요성을 못느꼈습니다! 기본 구분자와, 커스텀 구분자를 추가한 방식을 구분하여 구현하면 얻게되는 이점이 있을까요?? 추후 요구사항이 변경된다고 생각해도 기본 구분자, 커스텀 구분자라는 개념은 바뀌지 않을것이라고 생각합니다! 만약 기본 구분자와 커스텀 구분자라는 개념이 바뀐다면 인터페이스를...
dbsdndcks created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 좀 더 기능적으로 분리하는 방식에 대해 고려해볼만 하다고 생각합니다👍 많은 노력이 담긴 코드인것 같습니다
dbsdndcks created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스의 내부 메서드는 private 처리하신게 좋은 것 같습니다 ㅎㅎ!. 👍
dbsdndcks created a review on a pull request on woowacourse-precourse/java-calculator-7
Filter 클래스의 기능을 나누면 좀 더 객체지향적인? 코드가 될 것 같습니다. 그래도 큰 기능부분을 나누신 부분을 보아 많은 노력과 고민을 하신게 느껴집니다 수고하셨습니다.😁
dbsdndcks created a review on a pull request on woowacourse-precourse/java-calculator-7
Filter 클래스의 기능을 나누면 좀 더 객체지향적인? 코드가 될 것 같습니다. 그래도 큰 기능부분을 나누신 부분을 보아 많은 노력과 고민을 하신게 느껴집니다 수고하셨습니다.😁
dbsdndcks created a review comment on a pull request on woowacourse-precourse/java-calculator-7
대부분 패턴 방법을 도입한것 같은데 이렇게 하는 방법도 있다는걸 배웠네요! . 오래 생각하셔서 만든 코드란게 느껴집니다.👍
dbsdndcks created a review comment on a pull request on woowacourse-precourse/java-calculator-7
else if문의 반복을 줄이는 방법을 고려해보시는것도 좋을것 같습니다! 예외 처리를 위해 고민하신게 보입니다👍
dbsdndcks created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력과 출력을 따로 분리하신점도 인상깊습니다!
dbsdndcks created a review on a pull request on woowacourse-precourse/java-calculator-7
우선 README에서부터 코드를 작성하기 위해 많은 고민과 노력을 하신게 느껴집니다!. 다만 한개의 클래스에서 많은 기능이 들어가 있는 부분은 기능을 살짝 나누는 것도 고려해볼만 할 것 같습니다. 정말 고생하셨습니다😁
dbsdndcks created a review on a pull request on woowacourse-precourse/java-calculator-7
우선 README에서부터 코드를 작성하기 위해 많은 고민과 노력을 하신게 느껴집니다!. 다만 한개의 클래스에서 많은 기능이 들어가 있는 부분은 기능을 살짝 나누는 것도 고려해볼만 할 것 같습니다. 정말 고생하셨습니다😁
dbsdndcks created a review comment on a pull request on woowacourse-precourse/java-calculator-7
가끔식 enum클래스를 사용하는데, 이번 과제에서 사용하는 것까진 생각하지 못했네요! 굉장히 인상적이였고 고려할만 할 것 같습니다.👍
dbsdndcks created a review on a pull request on woowacourse-precourse/java-calculator-7
우선 객체화가 굉장히 잘되어있어 자바를 잘하시는분 같습니다!. 다양한 테스트 코드도 작성하신부분과 enum 클래스로 상수들을 관리하는점이 인상깊었습니다😁
dbsdndcks created a review on a pull request on woowacourse-precourse/java-calculator-7
우선 객체화가 굉장히 잘되어있어 자바를 잘하시는분 같습니다!. 다양한 테스트 코드도 작성하신부분과 enum 클래스로 상수들을 관리하는점이 인상깊었습니다😁
Jin409 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 저도 이번 과제 제출 당시에는 현재 과제 상황에 집중해 굳이 역할을 잘 나누지 않았는데, 리뷰를 받다보며 제가 추후의 변경 사항들에 대한 고려가 부족했다는 것을 느끼게 됐어요. 홍규님 말씀대로 다양한 상황들을 가정하여 변경에 용이하도록 다음 과제에서는 조금 더 신경 써보겠습니다! 피드백 감사해요 :)
honggyuSHIN created a review comment on a pull request on woowacourse-precourse/java-calculator-7
제가 자바를 시작한 지 얼마 안 돼서 문제를 처리하는 과정 중 필요한 기능을 클래스로 구현한 뒤 한 클래스(메서드) 안에서 기능들을 문제 해결 흐름에 맞게 줄세우는 방법만 알고 있었는데 이렇게 문제 해결의 흐름을 다른 메서드로 넘겨서 처리하는 방식을 처음 알게 되었습니다. 감사합니다. 제가 아직 아는 것이 많지 않아 다음 주에는 더 좋은 피드백 들고...
honggyuSHIN created a review on a pull request on woowacourse-precourse/java-calculator-7
수고 많으셨습니다. 덕분에 더 많이 공부해야겠다는 생각을 하게 되었습니다. 감사합니다.
honggyuSHIN created a review on a pull request on woowacourse-precourse/java-calculator-7
수고 많으셨습니다. 덕분에 더 많이 공부해야겠다는 생각을 하게 되었습니다. 감사합니다.
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
[[Java] 상수를 활용하자](https://lovethefeel.tistory.com/82) [상수 선언의 장점](https://velog.io/@yohanblessyou/short-%EC%83%81%EC%88%98%EC%9D%98-%EC%9E%A5%EC%A0%90) 가볍게 정리 잘 해둔 글 같습니다! 참고해 보세용
