Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
honggyuSHIN created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드 마다 테스트를 돌리신 점이 인상 깊었습니다. 제가 잘 아는 건 아니지만 Number에 너무 많은 메서드가 들어있는 것 같아 역할이나 책임에 따라 구분을 하면 더 좋지 않을까 생각했습니다. 많이 배우겠습니다. 감사합니다.
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
수고 많으셨습니다. 감사합니다.
seoyeon-jung created a review comment on a pull request on woowacourse-precourse/java-calculator-7
와 저는 개인적으로 이렇게 오류를 한번에 모아서 보니까 가독성 좋다고 생각합니다!
yeong0jae created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Set으로 관리..! 저는 생각하지 못한 방법이네요. Set으로 관리했을때 장점과 단점을 알아봐야겠습니다ㅎㅎ 감사합니다
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
물론 밑에서 무조건 num에 값을 정의하고 있지만, default value를 선언시에 같이 작성해주는게 조금 더 안전한 코드가 아닐까 싶은데 어떻게 생각하시나요?
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
calculate라는 메소드 안에서 패턴분리하고 계산까지하는 로직이 하나로 합쳐셔 있는 것 같아서 [SRP 원칙](https://inpa.tistory.com/entry/OOP-%F0%9F%92%A0-%EC%95%84%EC%A3%BC-%EC%89%BD%EA%B2%8C-%EC%9D%B4%ED%95%B4%ED%95%98%EB%8A%94-SRP-%EB%8B...
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
formatting을 적극적으로 활용하면 불필요한 공백을 줄일 수 있을 것 같아요!
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
1주차 공통 피드백에도 나와있지만, 불필요한 주석은 제거하는게 조금 더 좋아보여요
halfmoon-mind created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Exception이 발생할 수 있는 부분에 throws를 추가해서 해당 부분에서 exception handling 혹은 발생할 수 있다는 것을 명시해주면 더 좋을 것 같아요!
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
각 기능을 아주 세부적으로 분리한 게 인상깊네요!👍 이런 섬세함이 유지보수와 테스트를 더욱 용이하게 해주는 것 같아요. 많이 배워갑니다!!