Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
보통 this는 파라미터명과 인스턴스 변수명이 같을 때 구분을 위해 사용한다고 알고 있습니다. 아래 getDelimiters 메소드에서는 this를 붙이지 않고 이곳에만 this를 붙인 이유가 있을까요??
hyobin-yang created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수행하시느라 고생 많으셨습니다!! 다른 분들이 좋은 리뷰들 많이 달아주셔서 몇 가지만 적고 갑니다☺️
hyobin-yang created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수행하시느라 고생 많으셨습니다!! 다른 분들이 좋은 리뷰들 많이 달아주셔서 몇 가지만 적고 갑니다☺️
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
2주차에는 람다, 스트림을 최대한 이용해봐야겠네요!
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
팩토리 메서드 패턴을 활용하시다니 생각지도 못했습니다 . 패턴의 이해와 활용을 잘하시는 분 같습니다.
dbsdndcks created a review comment 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
추상클래스와 인터페이스를 정말 잘 활용하신 것 같습니다!. 과제 해결을 위해 상당히 많은 노력을 쏟으신분 같습니다. 다양한 테스트 작성도 인상깊습니다!. 여러가지로 많이 배워갑니다.👍😁
dbsdndcks created a review 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
main과 controller에 의존성이 생기는 부분은 미처 생각하지 못한것 같네요!
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 봤습니다. IOMessage를 enum으로 관리하신 점, 입력값 형식 검증 예외 기준을 세심하게 짜신 점, 입력값 예외 메시지를 세분화시키신 점에서 세심한 접근이 눈에 띕니다. mvc 패턴에 대해 이해도 있으신 것 같습니다. 만약 확장을 고려한다면 구분자를 클래스로 따로 관리하는 건 어떨까요? 예를 들어 구분자별로 더하기가 아닌 다른 사칙연...
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 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
대부분 패턴 방법을 도입한것 같은데 이렇게 하는 방법도 있다는걸 배웠네요! . 오래 생각하셔서 만든 코드란게 느껴집니다.👍