Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
world-dv created a review comment on a pull request on woowacourse-precourse/java-calculator-7
다른 분들 코드 리뷰 하면서 상속 받는 방식을 알게 되었어요 ! 다음 미션부터는 상속받아 사용하면 좋을 것 같네요! 감사합니다!!
kwonkeonhyeong created a review comment on a pull request on woowacourse-precourse/java-calculator-7
enum으로 구현해본 이유는 단순히 default mode를 true false로 관리하기 보다 현재 정확히 어떤 모드가 작동하고 있는지 표현해보고 싶었습니다!
ParkHyunS00 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 `양수 문자열 덧셈기` 라는 문제 해석을 양의 실수도 계산 가능하도록 만들어야 한다고 생각했어요 정수와 실수 리스트를 각각 만들어서 사용하는 것은 비효율적일 것 같아서 정수와 실수 모두 담을 수 있는 Number 클래스를 선택했어요!
kwonkeonhyeong created a review comment on a pull request on woowacourse-precourse/java-calculator-7
생각하지 못했었던 부분인데 다음부터는 적용해봐야겠어요! 감사합니👍
kwonkeonhyeong created a review comment on a pull request on woowacourse-precourse/java-calculator-7
개행을 신경쓰지 못했는데 좋은 피드백 감사드려용👍👍
ParkHyunS00 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
미쳐 생각하지 못했던 것 같아요! 지적 감사합니다👍
kysub99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
'check'와 같이 유효성을 검사하는 부분은 Service가 아닌 별도의 계층이나 클래스로 관리하는게 좋아보입니다
kysub99 created a review on a pull request on woowacourse-precourse/java-calculator-7
고려해야할 사항을 꼼꼼하게 작성하고 구현하신 것 같아요. 수고하셨습니다!
kysub99 created a review on a pull request on woowacourse-precourse/java-calculator-7
고려해야할 사항을 꼼꼼하게 작성하고 구현하신 것 같아요. 수고하셨습니다!
SeriousBug98 created a comment on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 MVC패턴 고려해서 설계한 것이 매우 인상적이었습니다!! 또 여러 상황에 대한 테스트 코드를 많이 고민한게 눈에 띄는 것 같아요!! 기능별로 코드를 나누는 과정을 이후에 추가하면 더 좋은 설계가 될 것 같습니다. 1주차 고생 많으셨어요!! ㅎㅎ
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자만 모델로 따로 빼셨는데, 문자열에 있는 숫자들은 따로 모델로 표현 안하신 이유가 있을까요?
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
View단의 문자열들을 별도 클래스로 빼는 것은 좋은 것 같습니다! 아이디어 얻고 갑니다!
thsghdud13 created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 조금이나마 리뷰 남겨봤습니다! 다른분들이 워낙 잘 리뷰해주셔서 리뷰하신 내용들 잘 보고갑니다!!
thsghdud13 created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 조금이나마 리뷰 남겨봤습니다! 다른분들이 워낙 잘 리뷰해주셔서 리뷰하신 내용들 잘 보고갑니다!!
dd-jiyun created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 고생하셨습니다! 되게 세부적으로 구현을 잘하신 것 같아요! 아직 저도 부족한게 많아서 코드 보면서 많이 배워갑니다 :) 2주차도 화이팅입니다 💪
dd-jiyun created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 고생하셨습니다! 되게 세부적으로 구현을 잘하신 것 같아요! 아직 저도 부족한게 많아서 코드 보면서 많이 배워갑니다 :) 2주차도 화이팅입니다 💪
JamongFriend created a comment on a pull request on woowacourse-precourse/java-calculator-7
1주차 프리코스 고생많으셨습니다. 깔끔하고 가독성 좋은 코드인 것 같습니다.
kwonkeonhyeong created a review comment on a pull request on woowacourse-precourse/java-calculator-7
처음 코드를 작성하게 된 의도는 단순히 숫자영역들을 담아두는 token 객체를 만들어보자였습니다! dto를 염두해두고 작성하지는 않았지만 리펙토링 과정에서 생각해보니 dto 역할을 하고있다는 생각이 들어서 추후 record로 변경해볼 계획입니다!
rladmstn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분도 제가 미리 리팩토링을 못해서 놓쳤던 부분이었습니다 ㅎㅎ 기존에 `Controller`단에서 받아오는 Input을 필드로 가질 걸 생각하고 계획 했었는데요, 나중에 설계가 조금 바뀌고 나서 필드로 가지지 않아도 되겠다는 생각에 주입 받는 부분 로직을 제거하다가 생성자는 제거를 못해버렸습니다 ㅎㅎ
kwonkeonhyeong created a review comment on a pull request on woowacourse-precourse/java-calculator-7
주석 제거를 제대로 신경쓰지 못했네요!! 좋은 피드백 감사드립니다👍
rladmstn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
사실 이번 과제 자체는 생성자 주입의 이점을 못느꼈습니다! 필드 주입의 단점이라고 한다면 테스트가 어렵다는 정도로 생각하는데, 이번 과제에서는 `Controller`에 대한 테스트도 따로 작성할 계획이 없었기에 단순하게 작성하고 싶어 필드 주입을 채택했습니다!