Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
lold2424 created a review on a pull request on woowacourse-precourse/java-calculator-7
구분자에 대해 고민을 많이 하신것 같습니다. 구분자를 명확하게 구분하신 것 같네요. 1주일동안 고생하셨고 남은 1주일동안 미션 화이팅입니다 :)
lold2424 created a review on a pull request on woowacourse-precourse/java-calculator-7
구분자에 대해 고민을 많이 하신것 같습니다. 구분자를 명확하게 구분하신 것 같네요. 1주일동안 고생하셨고 남은 1주일동안 미션 화이팅입니다 :)
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좀 더 큰 묶음으로 ((ex) InputView와 OutputView를 묶어서 View 추가 생성) 주입받아 생성자 수를 낮추는 것도 좋은 방법일 것 같습니다!
Teddysir created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드, 변수명 작명하는방식을 2주차에는 좀 더 시간을 써서 만들어봐야할거같아요!! 정말 좋은 지적 너무 감사합니다!!
Teddysir created a review comment on a pull request on woowacourse-precourse/java-calculator-7
넵 사용자 입력과 입력에서 받은 값을 Integer배열에 저장해서 사용하는 방식을 처음에 생각해서 그려봤는데 더 다른 좋은 방법들이 많이 있더라구요! 좋게 봐주셔서 감사합니다 :)
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그동안 생각없이 여러개의 도트를 이어서 사용하고 있었는데 디미터의 법칙 위반이군요..! 잘 고민해보겠습니다 :)
hyojaeKwon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요:) bowook님께서 말씀하신 방법도 좋은 것 같아요! 필요없는 else문이 사라져서 간결한 코드가 될 것 같아요~ 다만, 제가 염려한 부분은 코드의 의미 전달 명확성의 하락 입니다. `if (matcher.find()) { return input.substring(matcher.end()); } throw new Il...
chaeyeon0130 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
코드의 흐름을 이해하는 데 있어, 해당 문자열이 무엇을 의미하는지 확실히 와닿지 않는 부분이 있는 것 같아요! (약간의 매직 넘버랄까,,) 기본 구분자와 커스텀 구분자를 분리하여 각각을 좀 더 명확하게 처리하도록 하는 건 어떨까요? 기본 구분자 같은 경우는 상수로 선언하여 사용할 수도 있을 것 같아요!
chaeyeon0130 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`calculate()` 진입 시 앞에서 입력 값이 없는 경우를 처리하고 있는데! 여기서 다시 한번 검사하는 특정 이유가 있을까요? 특정 의도가 있는 거라면 중복 코드를 줄이기 위해 별도의 메서드로 분리하는 것도 좋아보여요!
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주차 과제 진행하시느라 수고 많으셨습니다! 전반적으로 책임과 역할을 세분화하여 메서드(혹은 클래스)를 분리한다면, 확장성과 테스트 용이성 측면에서 큰 이점을 가져올 수 있을 것이라 생각합니다!:)
hyojaeKwon created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 금성님! 1주차 미션 완료하시느라 고생하셨습니다:) 단일 클래스의 책임 분할에 대해 고민하신 것이 느껴져 좋았습니다. 다만 package분리를 조금 해주시면 더욱 구조를 한 눈에 보이게 할 수 있을 것 같습니다!!
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> String을 변환한다는 의미는 매개변수 타입으로 유추 가능하다고 생각해요.String을 변환한다는 의미는 매개변수 타입으로 유추 가능하다고 생각해요. 말씀해주신 이 포인트를 놓쳤네요 감사합니다! `toDoubles` 로 메서드명을 변경하는게 좋을 것 같아요
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클린코드에 대해 고민해보겠습다 :)
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Java의 MVC 패턴을 적용하는데에 아직 익숙하지 않아 이렇게 네이밍을 했습니다 ㅎㅎ 계산기 프로그램의 핵심 제어자이니 `Calculator`로 했는데 일반적인 컨벤션을 따라 `CalculateController`와 같은 네이밍이 더 적절할 것 같네요!
jaepyo-Lee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네이밍에 실수가 있었네요ㅠㅠ 꼼꼼히 봐주셔서 감사합니다!
