Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
일급 컬렉션을 사용해서 List<number>를 나타내고 일급 컬렉션 내에서 덧셈 로직을 넣으면 객체에게 책임을 위임하는 식으로 코드를 작성할 수도 있지 않을까요? 이 방법은 어떤가요? 도메인에 맞게 데이터를 데이터를 표현하기에도 좋을 것 같습니다
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분도 일급 컬렉션을 이용하면 데이터 중심에서 객체에게 책임을 위임하도록 할 수 있지 않을까요?
koosco created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 리뷰 남겼습니다. 배울 점이 많은 것 같습니다.
koosco created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 리뷰 남겼습니다. 배울 점이 많은 것 같습니다.
Jonghwan-Lee7 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
enum으로 선언하는 구조는 이번에 처음 해보았는데, 프로그램 전체에 상수가 많아질수록 유용할 것 같더라구요!
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 여기에 모아두지 않은 상수들은 위치에 있는 클래스 내에 있을 때 더 확인하기 쉽고 응집성이 높다고 생각해서 옮기지 않았습니다! > `DelimiterConstants` 내에 있는 상수들은 여러 클래스에서 사용되기도 하고, 구분자가 변경될 가능성을 생각하면 모아두는 것이 더 좋을 것 같아서 이렇게 하였습니댜 저도 이와 같은 생각으로 여러 클래...
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 코드와는 상관없지만 개인적으로 궁금한 점은, View의 역할은 어디까지라고 생각하시나요? > > 실제 계산기에서는 출력을 할 때 3자리 단위로 끊어서 보여줘요 ex) 100,000,000 > > 그렇다면, View에서 3자리씩 끊어주는 함수를 만들어도 될까요? 아니면 다른 곳에서 분리를 하고, String을 출력해야하는걸까요? 저는...
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
외부에서 의존성 주입을 하는 방법은 생각해보지 못했는데, 객체 생성과 객체 사용의 책임을 분리할 수 있겠네요! 좋은 방법인 것 같습니다!
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
리스트를 사용하지 않고 배열을 사용하신 이유가 궁금합니다!
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 이번에 전역적으로 Validator 처리를 해줬었는데요! 리뷰 중에, “해당 도메인에 맞는 에러를 전역적으로 처리하는 대신, 각 도메인에서 구체적으로 처리하는 것에 대해 어떻게 생각하세요?“라는 질문을 보았습니다. 처음에는 비즈니스 로직을 처리하면서 유효성 검증까지 맡으면 과도한 책임을 갖는 게 아닐까 하는 생각이 들었지만, 도메인별 에러...
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
반복문으로 numbers 리스트의 요소의 개수만 큼 validatePositivity를 호출하고 있는 것 같아요! 이 부분에 대해서 궁금한 부분이 리스트 형태로 인자로 넘기고 validatePositivity 한 번만 호출해, 그 안에서 for문을 돌려 요소를 확인하는 방법에 대해서는 어떻게 생각하시나요?
hoonyworld created a review on a pull request on woowacourse-precourse/java-calculator-7
미션 너무 고생 많으셨습니다 지현님! 다음 2주차에도 같이 코드리뷰를 진행하면 좋을 것 같아요 ㅎㅎ 궁금한 부분에 대해서는 코멘트 남겼습니다!
hoonyworld created a review on a pull request on woowacourse-precourse/java-calculator-7
미션 너무 고생 많으셨습니다 지현님! 다음 2주차에도 같이 코드리뷰를 진행하면 좋을 것 같아요 ㅎㅎ 궁금한 부분에 대해서는 코멘트 남겼습니다!
Jonghwan-Lee7 created a review comment 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
DI 객체들을 다른 테스트 클래스에서 단위 테스트를 하고 있긴한데, 지환님이 말씀하시는 단위 테스트는 어떤 의미 인가요?? 순수 기능 테스트를 위해서는 모킹이 필요하다는건 백퍼 공감해요 ㅎㅎ Numbers가 간단한 getter 기능 밖에 없어서 모킹하지 않았는데, 요게 쫌 아쉬운 포인트이긴 하네요 ~ ㅎㅎ 좋은 피드백 정말 감사해요~ ☺️
Jonghwan-Lee7 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
순전히 제가 자바 문법에 취약해서... listArray를 잠시 잊고있었습니다... 자바 문법과 내부 원리를 좀 더 공부해야겠습니다.
Jonghwan-Lee7 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
처음에 코드 구조를 잘못짜다보니, 주먹구구식 리팩토링이 많았습니다. 그 과정에서 private으로 가려야 하는 부분을 놓친 것 같네요. 다음부터는 처음부터 기능에 따라 private 여부를 고려해보겠습니다!
rjswjddn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기본 구분자가 바뀌는 경우도 생각해서 이렇게 구현했습니다. 이런거를 주석이나 readme에 남겼으면 더 좋았겠네요 ㅎㅎ.. 부족한 코드 피드백 해주시고 정말 감사합니다. 2주 차 화이팅하세요!!
rjswjddn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
검증하는 메서드로 역할을 나누는게 좋았겠네요!
zzioozzioo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
재사용할 일이 없다면 바로 리턴해줘도 좋을 것 같아요!
zzioozzioo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
한 번에 합까지 구할 수 있다는 점이 좋은 것 같습니다!
zzioozzioo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
단위 테스트 작성에서 꼼꼼함이 돋보이네요! :)
zzioozzioo created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수고하셨습니다~!