Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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)
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`inputString.indexOf("n") + 1`의 의미가 명확하지 않은 것 같아요. 상수화를 해보는 건 어떨까요??
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
stream을 잘 활용하시네요! 👍
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
필드가 조금 많아보이는데, Service 클래스를 만들어 분리해봐도 좋을 것 같아요!
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
생성자에서 객체를 생성하고 있네요! 의존성 주입은 어떻게 생각하시는지 궁금합니다!
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
접근제어자를 명시하지 않은 이유가 있을까요??
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
엔터도 컨벤션의 일부라는 사실! 알고계셨나요? [Naver 캠퍼스 핵데이 Java 코딩 컨벤션](https://arc.net/l/quote/lzyspvay)
songsunkook created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다. 테스트를 자세히 작성해주신 부분이 인상깊었습니다. 👍 1주차 과제 고생하셨습니다!
songsunkook created a review 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
좀 더 변수명에 의미를 담아보겠습니다! 피드백 감사합니다 👍
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력 요구사항이 "구분자와 양수로 구성된 문자열"이었던 것으로 기억합니다! 그래서 저는 double 타입으로 연산을 하고, . 는 유효하지 않은 구분자로 처리했습니다 :) integer 타입과 double 타입의 연산을 모두 가능하게 하면서 말씀해주신 제너릭이나 메서드 오버로딩도 고려했었는데 적용을 못했습니다 ㅎㅎ 의견 감사합니다! 리팩토링할...
Teddysir created a review comment on a pull request on woowacourse-precourse/java-calculator-7
방어적 복사 용어를 처음 듣고 공부하고보니 진짜 도움이 많이된거같아요! 우선 감사합니다!! new ArrayList<>() / Collections.unmodifiableList() / List.copyOf() 중 하나를 2주차 과제에 적절하게 판단해서 사용해봐야할거같아요! 이런 부분까지 고민해서 지적해주셔서 감사합니다 :)
