Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
1주차 공통 피드백에 추가되어있지만, 배열 보다는 컬렉션인 List를 사용하는게 더 좋아보입니다!
halfmoon-mind created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 고생하셨습니다! 제가 생각하기에 보완해야할 점, 질문 사항들을 정리해서 리뷰 남겨놓았습니다! 그리고 테스트 케이스도 추가로 작성해서 요구사항에 대하여 적절한 입력이 들어올 수 있는지에 대해서 검사할 수 있는 기회가 있으면 좋을 것 같아요!
halfmoon-mind created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 고생하셨습니다! 제가 생각하기에 보완해야할 점, 질문 사항들을 정리해서 리뷰 남겨놓았습니다! 그리고 테스트 케이스도 추가로 작성해서 요구사항에 대하여 적절한 입력이 들어올 수 있는지에 대해서 검사할 수 있는 기회가 있으면 좋을 것 같아요!
codingmy reopened a pull request on woowacourse-precourse/java-calculator-7
[문자열 덧셈 계산기] 최민영 미션 제출합니다.
ykmxxi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
여러 테스트 케이스를 진행 하셔서 기능 동작의 완성도를 높이신것 같습니다. 다만 테스트 메서드가 정확히 어떤 테스트 케이스를 실행하면 기대 결과가 무엇인지 보이지 않아 코드를 하나하나 읽고 유추해야 해서 메서드 네이밍을 조금 더 신경쓰신다면 더 좋은 테스트 코드가 될 것 같습니다
ykmxxi created a review on a pull request on woowacourse-precourse/java-calculator-7
사용자 편의성을 고려해 사용 방법을 text block로 생성해 보여주시는 부분이 인상깊었습니다 👍
ykmxxi created a review on a pull request on woowacourse-precourse/java-calculator-7
사용자 편의성을 고려해 사용 방법을 text block로 생성해 보여주시는 부분이 인상깊었습니다 👍
kksshh0612 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
매서드 배치에 대한 기준도 있군요 감사합니다
ykmxxi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좋은 의견 감사합니다. 리뷰를 계속 진행하면서 돌이켜보면 컨트롤러 보다는 `Runner`라는 이름이 더 알맞은것 같네요 > input을 controller 내부에서 받는 방식으로 인해 테스트를 할 수 없는 구조라고 생각합니다. run(String input) 이런식으로 구성한다면 테스트를 할 수 있는 코드가 될 수 있을 것 같습니다. 그리고 결과...
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
0과 - 같은 요소들도 상수로 선언하신 것은 단순 반복을 제거하기 위한 의도일까요?? 제 개인적인 생각으로는 이 요소들은 오히려 상수를 사용하는 것이 가독성을 떨어뜨릴 수도 있다고 생각합니다. '단순한 0, -가 아닌 다른 특별한 정보를 담고 있나?'라고 생각될 가능성도 있는 것 같아서요! 또한 상수도 static 영역이므로 프로그램이 실행될 때 ...
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
각 기능을 아주 세부적으로 분리한 게 인상깊네요!👍 이런 섬세함이 유지보수와 테스트를 더욱 용이하게 해주는 것 같아요. 많이 배워갑니다!!
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
구분자로 숫자로 들어올 경우는 전혀 고려하지 못했는데 세심하게 정의하신 걸 보면 요구사항에서 적시되지 않은 부분도 고심하신 것이 보입니다.