Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
kwonkeonhyeong created a review comment on a pull request on woowacourse-precourse/java-calculator-7
위처럼 run() 메서드 내부에서 Calculator를 생성하게 되면서 결합도가 증가해버렸네요! ㅜㅜ 구현하면서 위 부분을 의존성을 생성자 주입으로 바꿔줬어야 했는데 간과했습니다! 좋은 피드백 감사드립니다!🤗
rladmstn created a comment on a pull request on woowacourse-precourse/java-calculator-7
> > **상수 클래스**(_DelimiterConstants, InputValidationConstants_)를 확인해보니 > > 생성자를 별도로 private으로 막지 않아 **객체 생성이 가능**해보이며 > > **final 제어자가 없기에** **상속도 가능**한 상태로 보여집니다! > > **일반 클래스로 선언한 이유**가 따로 있으...
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
감사합니다 콘솔로 제공되는 프로그램이라고 생각하고 만들었습니다 ㅎㅎ
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
각 테스트는 독립적으로 수행이 되어야하기 때문에 각 테스트별로 생각하는게 좋다고 생각합니다. 물론 지금 테스트는 동일한 new Spliterator("[,:]");을 사용하고 있지만 다른 케이스의 경우도 있으니까 테스트 별로 생성했습니다
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
양수였군요 ㄷㄷ 엣지케이스...
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이것도 마찬가지로 예외적인 값이기 때문에 split이 정상적으로 되지 않았기 때문에 에러가 발생되도록 따로 처리하지 않았습니다
yushinc created a review comment on a pull request on woowacourse-precourse/java-calculator-7
get을 사용하는 방식이 클래스간 결합도에 영향을 준다고는 정말 생각해보지 못했습니다. 해당 리뷰를 받고 결합도에 관한 정보를 찾아보았습니다. get을 사용함으로써 증가하는 결합도는 외부 결합도로 모듈이 외부에 있는 다른 모듈의 데이터를 참조할 때의 결합도인 것 같은데 데이터를 얻어오기 위해서는 get을 사용하는 것이 당연하다고 생각해왔습니다. 만약...
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네 이런 경우에는 예외처리가 안되어 있습니다. 패턴을 사용해 정규식으로 split을 하고 있는데 ?가 정규식 문자라서 다른 에러가 발생하기도 하네요 그리고 \\n1의 경우에도 에러가 발생하고요. 그런데 정상적인 값이 아닌 정말 예외적인 값이기 때문에 예외가 반환되는 것이 맞다고 생각해서 따로 예외처리는 하지 않았습니다. 그런데 요구사항에서 ...
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
- 단순하게 바깥에서 `Console.readLine()`를 하면 되는데 왜 `InputView`로 감싸야 했나요? - `InputView.getString`라는 메서드 이름으로부터 '콘솔로부터 입력을 받아온다'는 의미를 알아낼 수 있을까요? 보다 좋은 네이밍을 고민해봅시다 😁
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`lastIndex`와 `+2`는 무엇을 의미하나요? 변수의 의미를 이해하기 위해 어디까지 읽어보아야 할까요? 메서드 안에서 해당 단서를 찾지 못했습니다 😢
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`private` 메서드가 많아진다는 이야기는 결국 객체가 너무 많은 일을 하고 있다고도 볼 수 있습니다. 이를 다른 클래스에 역할을 부여했다면 어떨까요? 2주 차 부터는 테스트도 요구사항 중 하나로 들어올 텐데요. `private` 메서드는 테스트하기 어렵지 않을까요?
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
- `setInputString`이면 `inputString`을 설정하는 느낌인데, 내부에서는 `replace`를 부르고 있네요. 다른 어울리는 메서드명을 고민해볼까요? - `replace`는 어떤 방식으로 문자열을 치환할까요?
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
- 해당 부분에 `final`을 붙이지 않은 이유는 변경 가능성이 있어서였을까요? - `inputNumberInt`는 어떤 값을 받을까요? 만약 요구사항이 `Long`으로 바뀌면 변수명도 바뀌어야겠네요. 변경 전파를 최소화할 수는 없을까요? 또, 해당 변수명만으로 어떤 역할을 하는지 알 수 있을까요?
donghoony created a review on a pull request on woowacourse-precourse/java-calculator-7
반갑습니다 👋🏻 네이밍에 대해서 고민을 하면 좋을 듯해 관련된 리뷰를 좀 남겨 보았어요. 댓글로 궁금한 점을 달아 주시면 함께 토론해보겠습니다. 1주 차 미션 고생하셨어요! 💯
donghoony created a review on a pull request on woowacourse-precourse/java-calculator-7
반갑습니다 👋🏻 네이밍에 대해서 고민을 하면 좋을 듯해 관련된 리뷰를 좀 남겨 보았어요. 댓글로 궁금한 점을 달아 주시면 함께 토론해보겠습니다. 1주 차 미션 고생하셨어요! 💯
SeongUk52 created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 보고 많이 배워갑니다 전체적으로 완성도가 높고 노력하신 흔적이 엿보입니다. 한시간 동안 열심히 뜯어봤는데 피드백을 남길 부분이 별로 없어서 죄송합니다 🥲
SeongUk52 created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 보고 많이 배워갑니다 전체적으로 완성도가 높고 노력하신 흔적이 엿보입니다. 한시간 동안 열심히 뜯어봤는데 피드백을 남길 부분이 별로 없어서 죄송합니다 🥲
bowook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
readLine()을 이용하시고, close()를 통해 스트림을 닫아주시는 디테일이 인상깊습니다!
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
분리했으면 좀 더 명확하게 들어났을 것 같네요 감사합니다