Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
youngsu5582 created a review on a pull request on woowacourse-precourse/java-calculator-7
코레아 팀의 조이썬입니다 🙂 리뷰 많이 안남겼는데 구조적으로 잘 작성한거 같아서 그렇습니다. 테스트도 잘 작성해주셨는데 객체를 좀 더 잘개 쪼개서 단위 테스트를 작성해보면 더 쉽고 명확하게 테스트가 가능할 거 같습니다 🫡 궁금한게 있다면 남겨주면 답변 하겠습니당 ☺️
youngsu5582 created a review on a pull request on woowacourse-precourse/java-calculator-7
코레아 팀의 조이썬입니다 🙂 리뷰 많이 안남겼는데 구조적으로 잘 작성한거 같아서 그렇습니다. 테스트도 잘 작성해주셨는데 객체를 좀 더 잘개 쪼개서 단위 테스트를 작성해보면 더 쉽고 명확하게 테스트가 가능할 거 같습니다 🫡 궁금한게 있다면 남겨주면 답변 하겠습니당 ☺️
digitpic created a review comment on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 이 부분은 split 으로 구분된 토큰들을 각각 검증하는 부분입니다! 각 토큰은 현재 문자열 상태입니다 각각을 정수로 변환해보며 정수로 형변환을 진행할 수 없는 경우에 예외가 발생하게 되기에 문자열에는 정수만 입력되어야 한다는 에러 메시지를 출력하게 됩니다!
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
Pattern을 static 변수로 선언해두면 런타임에 오버헤드를 줄일 수 있겠네요! 고려하지 못했던 부분인데 감사합니다 ㅎㅎ
kaswhy created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 컨트롤러에서 모델과 뷰의 객체를 생성할 필요가 없다고 생각해서, 외부에서 만들어진 객체를 주입받아 설정하는 방식을 사용했는데, 이 부분에 대해서 어떻게 생각하시는지 궁금해요!
kaswhy created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이건 생각해보지 못한 예외였는데, 좋네요 👍🏻
kaswhy created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 메소드 명을 명사로 지어서 피드백을 받았었는데, 메소드 명을 꼼꼼하게 지으신 것 같다는 생각이 들어요!
kaswhy created a review comment on a pull request on woowacourse-precourse/java-calculator-7
"\\n"처럼 하드코딩된 부분을 상수화하는 것 어떨까요?
kaswhy created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 밀린 코드리뷰 요청이 많아 리뷰가 늦어졌네요 ㅜㅜ 한 주간 고생 많으셨고, 깔끔하게 코드를 작성하신 것 같아 배울 점이 많은 코드였습니다! 잘 보고 갑니다 ㅎㅎ
kaswhy created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 밀린 코드리뷰 요청이 많아 리뷰가 늦어졌네요 ㅜㅜ 한 주간 고생 많으셨고, 깔끔하게 코드를 작성하신 것 같아 배울 점이 많은 코드였습니다! 잘 보고 갑니다 ㅎㅎ
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
제가 잘못기억하고 있는 부분을 짚어주셔서 감사해요~ 😌 민서님 말씀처럼 양수 조건이였다면, 소수점 고려를 위해서는 `.` 구분자 사용을 제한하는게 맞겠네요~ 덕분에 저도 리팩토링 할 부분이 생겼네요~ 🥳
swlee8783 created a comment on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 메서드들이 하나의 책임을 담당하고 있으며, 구현해주신 부분에 있어서 테스트 구성과 유지보수가 쉽게끔 구성해주신 것 같아요! 1주차 과제 시험기간에 진행하느라 고생 많으셨습니다..!! :)
swlee8783 created a comment on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 메서드들이 하나의 책임을 담당하고 있으며, 구현해주신 부분에 있어서 테스트 구성과 유지보수가 쉽게끔 구성해주신 것 같아요! 1주차 과제 시험기간에 진행하느라 고생 많으셨습니다..!! :)
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
default 접근 제어자는 같은 패키지 안에서만 접근 가능한 것을 말합니다! 이 부분은 캡슐화를 위해 private 접근제어자를 사용하는게 훨씬 좋겠네요.
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
nullable하지 않고 래퍼클래스에서 제공하지 않는 기능을 사용하지 않는다면 래퍼 클래스를 굳이 사용할 필요가 없겠네요!
yjy323 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
IO를 따로 관리하는게 어떤 동작일까요? 저는 지금의 구조가 말씀하신 상황을 처리할 수 있다고 생각했습니다! 제가 의도한 것은 예를 들어, 콘솔 입출력 대신 파일 입출력을 지원해야 한다면 FileIO를 구현하고 10번째 라인을 IO io = new FileIO();로 작성하고 이외의 코드는 수정하지 않으면서도 요구사항을 만족하는 것입니다.
yjy323 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네 맞습니다~ 하드코딩된 부분은 상수화하는게 좋습니다. 하지만 지나친 상수화는 선언된 위치와 사용된 위치를 계속 번갈아 봐야한다는 가독성 측면에서 불편함도 있습니다! 이점도 함께 알고 계시면 좋을 것 같습니다. 특히 상수화는 "//"와 같은 리터럴이 여러번 등장할 때 그 효과가 느껴지는데, 이 코드에서는 1번만 사용되기 때문에 제외한 것이 의도입니다!
yjy323 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
조건문을 별도의 메서드로 관리하는 것을 말씀하신걸까요? 예를 들어 parseOperands의 for{ conditionCheck() }와 같은 동작일까요? 제가 이해한 것이 맞다면 말씀하신 것이 기능의 분리라는 측면에서는 고려할만 하다고 생각합니다. 하지만 기능의 확장성 측면에서는 validate가 그 역할을 충분히 할 수 있고, isEmpty(...
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
테스트 내에서 동일한 구문이 인자만 바뀌며 반복되고 있네요! `@ParameterizedTest`를 활용해보는 건 어떨까요??
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분은 InputView보다는 OutputView의 관심사인 것 같아요!
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
개인적으로 동일한 형태의 구문이 반복되는 건 경계하려고 노력하고 있습니다. Validation 클래스에 정의된 상수 패턴을 전부 통과하기만 된다면 해당 패턴들을 ENUM화하고 stream으로 순회하는 방향은 어떻게 생각하시는지 궁금합니다!
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
커스텀 구분자가 이상하게 인식될까봐 작성해주신 로직 같아요. `Pattern.quote()`를 활용해보면 어떨까요? [Understanding the Pattern.quote Method - baeldung](https://www.baeldung.com/java-pattern-quote)