Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
변수명은 totalSum 인데 String[] 자료형으로 만든 이유가 있을까요? 변수명만 보면 int 값 혹은 하나의 실수 값을 가져야 할것 같아요
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
여기서 숫자 3이 뜻하는 바가 무엇인가요?
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스명을 "CalFinish" 로 지으신 이유가 궁금해요
Astro-Yu created a review on a pull request on woowacourse-precourse/java-calculator-7
김이님 코드 잘 읽었습니다. 전체적으로 상수 문자열에 대해서 privat static final ~ 로 선언하신 부분은 좋았던것 같아요. 다른 상수들도 그렇게 선언해보면 어떨까요? 몇몇 함수들은 하나 초과의 책임을 지고 있는 것 같은데, 이 부분도 고려해보시면 좋을 것 같습니다. 1주차 수고하셨습니다.
Astro-Yu created a review on a pull request on woowacourse-precourse/java-calculator-7
김이님 코드 잘 읽었습니다. 전체적으로 상수 문자열에 대해서 privat static final ~ 로 선언하신 부분은 좋았던것 같아요. 다른 상수들도 그렇게 선언해보면 어떨까요? 몇몇 함수들은 하나 초과의 책임을 지고 있는 것 같은데, 이 부분도 고려해보시면 좋을 것 같습니다. 1주차 수고하셨습니다.
sunwon12 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좋은 의견 감사합니다. 덕분에 String 함수의 무지를 깨달을 수 있었어요. String에 대해서 공부 한 번 해봐야겠네요!
donghoony 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
요번 과제가 요구사항이 너무 모호했던지라 정말 다양하게 해석이 나오는 것 같아요 ! (사실 저는 너무 과도하게 케이스를 나눠버려서 구현할 때 좀 후회했습니다 ㅎㅎ) 동규님이 고려하신 부분 이해했습니다!! :)
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
단순히 상속뿐만 아니라, 코드 스멜을 놓칠 수 있다는 단점도 있다고 생각해요. 저는 개인적으로 정적 팩토리 메서드를 도메인에서 사용하는 것을 좋아하지는 않는데요, `new` 생성자로도 충분히 표현할 수 있다고 생각하기 때문이예요. 오히려 `new`가 **생성한다**는 개념과 더 호응한다고 생각하기도 하고요. 만약 클래스에 이미 정적 팩토리 메서...
phk1128 created a comment on a pull request on woowacourse-precourse/java-calculator-7
그리고 커밋 메세지 작성하실때 [AngularJS Git Commit Message Conventions](https://gist.github.com/stephenparish/9941e89d80e2bc58a153) 요고 참고해보시면 좋을것 같아요~ 🤗
rladmstn created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아티클 잠깐 읽고왔는데 Console이 테스트할 때 저런 이슈가 있군요..!? 저도 처음 알았어요!! 다시 읽어보고 참고해봐야겠어요 좋은 공유 감사합니다!! :)
phk1128 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
이 상수는 현재 테스트코드에서 사용하고 있지 않는데, 정의해두신 이유가 있나요?? assertSimpleTest()를 뜯어보시면 해당 상수가 언제쓰이는지 아실 수 있을거에요~
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
요부분은 main 메서드에서 수행해도 될 것 같아요~
phk1128 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
요고는 메서드로 따로 분리하는게 어떨까요 ??
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
요구 사항에 입력은 Console.readLine()을 사용하라고 나와있었는데, 스캐너를 사용하신 특별한 이유가 있으신가요??
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
불필요한 주석은 제거하는게 맞다고 생각해요~ 관련해서 [클린코드-주석](https://effortguy.tistory.com/187) 요고 참고해보시면 좋을것 같아요 ㅎㅎ
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
로직에 대한 설명이 있는것도 좋지만, 비개발자도 애플리케이션을 이해할 수 있게끔 작성해보면 어떨까요 ~?? README만 봐도 애플리케이션이 어떤 기능을 하는지 알 수 있으면 더욱 친절한 문서가 될 것 같아요 ㅎㅎ 그리고 git에 대해 학습하신 내용에 겪으신 문제와 해결과정을 나열해놓으신건 정말 좋다고 생각해요~ 👍 학습에 대한 열정! 배워 ...
phk1128 created a review on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 아쉬움이 많이 남는 코드였는데요 ~ 제가 하나하나 짚어드리는것 보다 다른분들의 코드를 보면서 클래스의 구조, 테스트 코드 작성법 등을 먼저 익히시는게 필요하실것 같아요~ 그럼 2주차는 더 성장된 코드를 기대해볼게요! 고생많으셨어요~ ☺️
phk1128 created a review on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 아쉬움이 많이 남는 코드였는데요 ~ 제가 하나하나 짚어드리는것 보다 다른분들의 코드를 보면서 클래스의 구조, 테스트 코드 작성법 등을 먼저 익히시는게 필요하실것 같아요~ 그럼 2주차는 더 성장된 코드를 기대해볼게요! 고생많으셨어요~ ☺️
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 사용자가 아무것도 입력하지 않았을 때 "결과 : 0"을 출력하고 재귀적으로 run() 메소드를 호출하는 것으로 보입니다. > 이는 StackOverflow에 안전하지 않은 방식으로 보입니다! 만약 사용자의 입력을 반복적으로 받아야 한다면, > 반복문을 통해서 사용자의 입력을 받는 것은 어떨까요? 물론 두 방법다 에러에 취약하다고 개인적으로 생각...
soeunnPark created a review comment on a pull request on woowacourse-precourse/java-calculator-7
여기서 boolean 값을 리턴하지 않고 바로 에러를 던지는 것에 대해서는 어떻게 생각하시나요!? 검증 이외에 boolean 값을 사용하는 곳이 없는 것 같아 여쭤봐요!
soeunnPark created a review on a pull request on woowacourse-precourse/java-calculator-7
많은 고민을 하고 작성하신 코드라는게 느껴졌습니다! 상수 처리와 같은 부분에서 굉장히 꼼꼼하게 코드를 작성하신 모습에서 많이 배웠습니다 👍 1주차 미션 수고 많으셨습니다!! :)