Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
seperate를 하는 별도의 객체가 있으면 좋을 것 같아요! PlusCalculator는 클래스명대로 plus와 관련된 책임만을 가지도록 하는 것은 어떨까요?
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
SOLID의 ISP는 인터페이스는 최소한에 필요한 노출만을 하는 것을 얘기하는데 노출을 줄이고 구현 부분을 캡슐화를 통해 가리는 것은 어떨까요?
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
public 메서드는 외부에 메서드가 노출되는데 private으로 캡슐화하여 외부 노출 포인트를 줄이는 방법은 어떨까요? 최소한의 인터페이스만을 외부에 노출하는 것이 변경 포인트를 줄일 수 있을 것 같아요
koosco created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다! PlusCalculator를 책임에 맞게 여러 객체로 나누면 더 좋을 것 같아요!
koosco created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다! PlusCalculator를 책임에 맞게 여러 객체로 나누면 더 좋을 것 같아요!
WARDKEY created a review comment on a pull request on woowacourse-precourse/java-calculator-7
숫자를 검증할 때 Character.isDigit()도 사용해보시면 좋습니다
WARDKEY created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분도 메서드로 분리해볼 수 있겠네요
WARDKEY created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스를 따로 분리해주시면 가독성이 좋아질 것 같습니다!
WARDKEY created a review on a pull request on woowacourse-precourse/java-calculator-7
시험기간이라 병행하기 힘드셨을텐데 고생하셨습니다!! 다음 주차도 열심히 해봅시다~!
WARDKEY created a review on a pull request on woowacourse-precourse/java-calculator-7
시험기간이라 병행하기 힘드셨을텐데 고생하셨습니다!! 다음 주차도 열심히 해봅시다~!
ckstj8027 created a comment on a pull request on woowacourse-precourse/java-calculator-7
> 패키지를 model view controller 에 model에 해당하는 큰 범위에다가 추출기에 관련된 내용들을 넣고 controller 에 해당하는곳에 service와 controller를 넣어놓으면 좀 더 가독성이 생길까요 ??
Jonghwan-Lee7 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
MVC 패턴에 대해 처음으로 공부해보고 고민하다가 입력받는 부분도 비즈니스 로직이라고 생각해서 포함했었습니다. 조금 더 알아본 지금의 저라면 view를 Inputview와 OutputView로 나누어서 InputView 파트에 넣을 것 같습니다.
Jonghwan-Lee7 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
작년 프리코스제출 코드 중에 인상깊에 본 코드가 있었습니다. is~는 전부 boolean타입으로 validator내에서만 활용되고 validate~는 전부 void 타입으로 외부에서 검증하는 메서드로 활용되는 것을 보고 따라해봐야겠다고 생각했습니다. 그런데 깊은 고민 없이 형태만 어설프게 따라해서 생긴 결과같습니다. 조금 더 고민을 ...
Jonghwan-Lee7 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
enum 으로 예외를 관리하신모습 인상깊습니다~~😁. 저도 고민했던 부분인데 사용하는것도 좋아보여요!
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
인터페이스와 enum을 통해 가독성이 올라갔습니다!. 다양한 테스트 코드도 작성하셔서 많이 노력하시고 고민하신게 느껴집니다 1주차 굉장히 수고많으셨습니다!😁👍
dbsdndcks created a review on a pull request on woowacourse-precourse/java-calculator-7
인터페이스와 enum을 통해 가독성이 올라갔습니다!. 다양한 테스트 코드도 작성하셔서 많이 노력하시고 고민하신게 느껴집니다 1주차 굉장히 수고많으셨습니다!😁👍
Jonghwan-Lee7 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
"읽기 쉬운 코드"를 지향하다가 캡슐화를 지키지 못한 점입니다.. 다음부터는 저도 controller의 하나의 메서드로 처리할 것 같습니다
rjswjddn created a review on a pull request on woowacourse-precourse/java-calculator-7
저랑은 차원이 다르시네요.. 어떻게든 쥐어 짜서 리뷰 했습니다!
rjswjddn created a review on a pull request on woowacourse-precourse/java-calculator-7
저랑은 차원이 다르시네요.. 어떻게든 쥐어 짜서 리뷰 했습니다!
Jonghwan-Lee7 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
에러 메세지도 상수로 처리하려다가 시간부족으로 인하여 구현하지 못했었는데, Astro-Yu님의 의견을 보니 이번에는 꼭 구현해 보겠습니다. 그리고 테스트에 대해서는 전혀 생각치 못한 부분이네요! 제 코드를 짤 때도, 다른 분들의 코드를 볼 때도 좋은 포인트가 될 것 같습니다.
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
숫자와 관련된 도메인 객체를 만들고 내부에서 검증하도록 하면 도메인을 나타내어 가독성이 좋아질 것 같습니다. 또 객체를 생성할 때 검증하도록 하면 코드의 유연성이 높아질 것 같습니다