Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
minSsan created a review comment on a pull request on woowacourse-precourse/java-calculator-7
사실 이번 과제에서는 불필요했던 부분이긴 한데, 리팩토링 과정에서 들었던 생각이 `커스텀 구분자 파싱 방식이 여러 개가 생긴다면 어떨까` 였습니다. 하지만, 기존 코드에서는 **인터페이스 없이 바로 구현체 내에서 파싱 로직을 넣었**기 때문에, 파싱 방식이 새로 추가될 때마다 _메소드 호출 방식이 일관되지 않을 것 같다_ 는 생각이 들었어요. 그래...
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
String.format을 이용하는것도 방법일 것 같습니다!
na0th created a review on a pull request on woowacourse-precourse/java-calculator-7
보고 배울 점이 많은 PR이었습니다.. 다음 미션도 열심히 참여해주십쇼.. 많이 배우겠습니다.
na0th created a review on a pull request on woowacourse-precourse/java-calculator-7
보고 배울 점이 많은 PR이었습니다.. 다음 미션도 열심히 참여해주십쇼.. 많이 배우겠습니다.
minSsan created a review comment on a pull request on woowacourse-precourse/java-calculator-7
죄송해요 제가 답변이 조금 늦은 것 같네요. 일단 `CustomDelimiter`는 요구사항에 알맞는 입력 형식에 맞게, 커스텀 구분자를 입력했는지 `확인하는 역할`이 분명한 것 같습니다. 그에 반해 `DefaultDelimiter`는 요구사항에 명시된 기본 구분자가 `무엇인지`를 나타내는 것이라고 생각합니다. 즉, 요구사항에 따르면 `C...
Gilbert9172 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
우선 다이어그램 툴은 사용하지 않고 그냥 마크다운에, 어떤 역할을 하는 도메인 모델이 필요한지 적고 시작하는 편입니다! 물론 작업하는 과정에서 기능이 추가되거나 메서드 명이 변경되는 부분도 있어서, 100% 완벽한 설계를 한다고 생각하지는 않아요! 여기 [링크](https://gilbert9172.tistory.com/101) 보시면 이해가 ...
daeuun created a review comment on a pull request on woowacourse-precourse/java-calculator-7
중복로직이 있었네요 감사합니다!
kungbi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 MVC패턴까지 사용해야할 프로젝트라고 판단하지 않아서 사용하지 않았습니다. 그런데 만약 MVC패턴을 사용한다면 service로 빼서 컨트롤러에서의 model과 view의 중간자 역할으로 어떤 데이터를 이동시키는지 잘 확인할 수 있을 것 같습니다.
kimsehw created a review comment on a pull request on woowacourse-precourse/java-calculator-7
MVC 패턴에 대해서 빠삭하게 공부하고 제대로 적용해봐야겠네요.. 이것도 좋은게 좋은거지 하고 흉내내봤거든요.. 다음번엔 확실히 공부해서 잘 적용해보겠습니다! 감사합니다! 👍
kimsehw created a review comment on a pull request on woowacourse-precourse/java-calculator-7
오 그렇군요! 리뷰를 읽고 나서 바로 찾아보니 왜 그런지 까지 단번에 이해됐습니다. 다음번엔 복수형으로 이름을 지어 봐야겠어요! 감사합니다!
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스 이름이 추상적인것 같습니다! CalculatorService는 어떨까요?
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
interface를 정의해서 얻은 이점이 있었을까요? 당장 이점을 얻지 못하더라도 추후에 얻을 수 있는 이점이 있을까요?
20HyeonsuLee created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수고하셨습니다👍 enum을 적극적으로 활용하신 부분이 인상깊네요! 2주차 미션도 파이팅입니다🔥
20HyeonsuLee created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수고하셨습니다👍 enum을 적극적으로 활용하신 부분이 인상깊네요! 2주차 미션도 파이팅입니다🔥
nns503 created a review on a pull request on woowacourse-precourse/java-calculator-7
수고하셨습니다 :) 회고를 보니 TDD로 진행한 것이 매우 인상적이었습니다. 2주차도 화이팅!
nns503 created a review on a pull request on woowacourse-precourse/java-calculator-7
수고하셨습니다 :) 회고를 보니 TDD로 진행한 것이 매우 인상적이었습니다. 2주차도 화이팅!
kimsehw created a review comment on a pull request on woowacourse-precourse/java-calculator-7
자세한 피드백 감사드립니다! `static 메서드`에 대해서 제대로 알아보지 않고 단순히 좋은거구나 하고 사용했었던 과거의 저가 아쉽습니다.. 이거 말고도 뭐든지 알맞은 곳에 써야 함을 이번에 뼈저리게 느꼈네요. 자세한 피드백 덕분에 `static 메서드`에 대해서 공부할 방향이 명확해진 것 같아요! 감사합니다!
cherryiJuice created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그런 단점이 있다는 점에서 공감합니다! 그러나 말씀드렸듯 재사용, 메모리 효울, 유지보수 측면에서 좋을 것 같아 도입하게 되었습니다
gyuoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
다양한 케이스를 고려하다 보니 중간에 존재하는 경우도 떠올렸습니다. 그런 말이 있었군요 ㅎㅎ..
