Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
2의 의미가 모호한 것 같아요. 상수화해보는 건 어떨까요??
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`//`나 `\\n`를 상수화하면 더 직관적인 코드를 만들 수 있을 것 같습니다!
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
List가 아니라 ArrayList를 사용한 이유가 있을까요??
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
IllegalArgumentException을 잡아 IllegalArgumentException을 던지는 것 같은데 어떤 로직을 구상하신 건지 궁금합니다!
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주차 과제 고생하셨습니다!
kgy1008 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
생성자에 접근 제약을 설정하는 것을 놓쳤네요. 다음 미션부터 적용해보겠습니다!
youngsu5582 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자 생성, 숫자 생성을 담당하는 팩토리 가 있어도 나쁘지 않을거 같습니다. ( 원래는 팩토리 패턴을 안좋아한, 특수한 요구사항(커스텀 구분자)이 있기 때문에 )
youngsu5582 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 부분이 다소 어렵게 느껴지네요. 메소드를 분리해서 메소드 명으로 명시적으로 드러내도 괜찮을 거 같습니다.
youngsu5582 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
InputView 와 OutputView 를 통해 명확하게 처리할 수 있을거 같습니다.
youngsu5582 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
테스트를 위한 getter 로 보이는데 구분자와 숫자 역시도 객체로 분리한다면 필요 없어질 거 같습니다. 구문자들을 가지는 일급 객체가 있다면 이를 생성하고 이 값을 검증하면 될 거 같습니다.
youngsu5582 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 부분에서 문자열 -> 숫자로 변환하는 것도 역시 객체가 담당할 수 있을거 같아요. ```java public clsss PositiveNumber{ ... } ``` 여기서, 음수 변환 예외 또는 숫자 변환 오류 등을 담당할 수 있을거 같습니다.
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
안녕하세요! 밀린 코드리뷰 요청이 많아 리뷰가 늦어졌네요 ㅜㅜ 한 주간 고생 많으셨고, 깔끔하게 코드를 작성하신 것 같아 배울 점이 많은 코드였습니다! 잘 보고 갑니다 ㅎㅎ