Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
kwongio created a comment on a pull request on woowacourse-precourse/java-calculator-7
>저는 메서드 호출(요청) == 책임 이라고 생각하지 않습니다. 객체지향은 객체들 간 메시지를 보내며 협력하는 구조를 갖는데, 송신자 객체 내부에서 수신자.메서드(요청 데이터)을 갖고 있는것이을 수신자의 책임을 갖고 있다는 관점에서 바라보면 가장 상위 계층에 존재해 프로그램의 진입점들이 되는 곳들은 서비스의 모든 책임을 갖고 있다고 바라봐야 하니까요....
swlee8783 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 럭키님 의견에 동의합니다! 특수문자를 수동으로 다루는 방식보다 Pattern.quote() 메서드를 사용하게 되면 정규식의 특수문자 이스케이프 처리를 자동으로 해줘 유지보수 측면에서 더 좋을 것 같아요!
chaeyeon0130 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
역할 분배에서도 여러 관점이 있을 것 같은데요! OutputHandler가 화면에 출력하는 역할을 전담하므로, 입력을 요청하는 메시지도 출력의 일환으로 간주하여 해당 클래스에 넣을 수도 있을 것 같은데,, 저는 개인적으로 입력과 관련된 모든 작업(`readInput()`, `printInputRequest()`)을 한 클래스에서 처리하면 역할이...
chaeyeon0130 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`calculateInput()` 메서드가 다소 많은 역할을 수행하고 있는 것 같아요! (입력 문자열 처리 / 구분자 기준 분리 / 합계 계산..) 메서드 분리를 통해 역할을 좀 더 세분화하면 어떨까요?
chaeyeon0130 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
charList에 중복된 문자를 추가하지 않기 위해 검사하는 if문은 depth 4에 해당하는데요! 객체지향 생활체조 원칙에는 들여쓰기의 depth를 2 이상으로 두지 말라는 지침이 있습니다. 메서드 구조를 분리함으로써 한 단락은 하나의 일만 하도록 로직을 잘개 쪼개보는 건 어떨까요?
chaeyeon0130 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 진행하시느라 고생 많으셨습니다. 객체 지향의 단일 책임 관점에서 몇 가지 개선점을 리뷰로 남겨보았습니다! 더불어, 테스트 코드 작성의 중요성을 깨닫고 갑니다. 상현님이 작성해주신 테스트 케이스 일부가 제 코드에서는 실패하더라고요. 코드의 안정성을 위해서는 테스트 코드 작성이 필수일 것 같아요,,🥹
chaeyeon0130 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 진행하시느라 고생 많으셨습니다. 객체 지향의 단일 책임 관점에서 몇 가지 개선점을 리뷰로 남겨보았습니다! 더불어, 테스트 코드 작성의 중요성을 깨닫고 갑니다. 상현님이 작성해주신 테스트 케이스 일부가 제 코드에서는 실패하더라고요. 코드의 안정성을 위해서는 테스트 코드 작성이 필수일 것 같아요,,🥹
leeesooha created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아하 그냥 궁금해서 여쭸습니다! 1주차 과제 고생하셨습니다!
Chaeyoung714 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
```java private void getCalculator(Calculator calculator) { outputView.outputResult(calculator.calculate()); } ```
nns503 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
임시 변수를 만들지 않고 바로 인자로 넣어주면 코드 길이가 많이 줄을 것 같아요! ```suggestion private void validateEdgeChar(String input) { if (!Character.isDigit(input.charAt(0))) { throw new IllegalArg...
nns503 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
멤버 변수에 상수로 선언하는 방법도 좋을 것 같습니다! ```suggestion private static final char PLUS_SIGN = "+"; ```
nns503 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
검증 클래스를 만들어서 활용하는 방법이 좋은 것 같아요 :)
KimEunSeop created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기능 별로 함수를 잘 나눈 것 같아요!
KimEunSeop created a review comment on a pull request on woowacourse-precourse/java-calculator-7
상수처리를 잘 적용한 예시인 것 같아요!
KimEunSeop created a review comment on a pull request on woowacourse-precourse/java-calculator-7
각각 기능의 입력값과 출력값을 명시해주어 흐름이 더 잘 보이는 것 같아욥! 배워갑니다.
KimEunSeop created a review comment on a pull request on woowacourse-precourse/java-calculator-7
계산기와 관련된 작동 흐름만 관장하는 컨트롤러인가욥? 개인적인 생각으로 만약 그렇다면, 전체 흐름을 관장하는 컨트롤러도 만들면 메인 함수의 로직을 조금 더 깔끔하게 할 수 있을 것 같다고 생각이 들어요!
KimEunSeop created a review on a pull request on woowacourse-precourse/java-calculator-7
기능별로 깔끔하게 잘 구현된 1주차 과제 같아요. 좀 더 기능이 많아지고 점점 더 복잡해지더라도 이런 방향성을 계속 추구할 수 있도록 저도 노력해보겠습니다!
KimEunSeop created a review on a pull request on woowacourse-precourse/java-calculator-7
기능별로 깔끔하게 잘 구현된 1주차 과제 같아요. 좀 더 기능이 많아지고 점점 더 복잡해지더라도 이런 방향성을 계속 추구할 수 있도록 저도 노력해보겠습니다!
GRAY-DDOT created a review comment on a pull request on woowacourse-precourse/java-calculator-7
변수와 인자를 줄일 수 있겠네요! 감사합니다!
GRAY-DDOT created a review comment on a pull request on woowacourse-precourse/java-calculator-7
확실히 그런게 좋은 것 같아요 최소한 리턴을 숫자 문자열로 바꾸는게 좋아보이네요...ㅠㅠ
GRAY-DDOT created a review comment on a pull request on woowacourse-precourse/java-calculator-7
유틸은 static! 좋은 지적 감사합니다!
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 제가 생각하는 키워드) 메서드 오버로딩 👀 그렇군요! 감사합니다 ㅎㅎ
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그렇게 해볼까도 생각해봤는데요,, 제가 생각하기에 첫 미션으로는 조금 과할 것 같아 인터페이스 사용은 자제했습니다 ㅎㅎ 좋은 의견 감사합니다! 다음 미션부터는 인터페이스 사용도 고려해볼게요 ☺️