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
잘못된 값이 들어왔을때 예외를 던지고 프로그램을 종료하라는 요구사항이 있었는데, 요고를 요구사항에 맞게 리팩토링 해보시는게 어떨까요??
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주차 미션 수고 많으셨습니다!! :)
soeunnPark created a review on a pull request on woowacourse-precourse/java-calculator-7
많은 고민을 하고 작성하신 코드라는게 느껴졌습니다! 상수 처리와 같은 부분에서 굉장히 꼼꼼하게 코드를 작성하신 모습에서 많이 배웠습니다 👍 1주차 미션 수고 많으셨습니다!! :)
rladmstn created a comment on a pull request on woowacourse-precourse/java-calculator-7
> 아하! 예외를 발생시키는 로직이 있길래 `public`이라고 잘못 인지하고 있었나봐요!! 그렇다면 `private` 생성자를 선언하셨다면 외부에서의 객체 생성은 방지될 것 같네요! > > 다만 `private`으로 막았음에도 내부에 **예외를 발생시키는 로직을 추가한 이유**도 조심스레 여쭤보고 싶습니다!! 해당 클래스에서 스스로의 객체 생성...
Mae-Hyong created a review comment on a pull request on woowacourse-precourse/java-calculator-7
에러의 경우 상수로 분리할 경우 손쉽게 관리할 수 있을 것 같습니다. :)
hyunn522 created a review on a pull request on woowacourse-precourse/java-calculator-7
unit test를 정말 꼼꼼하게 작성하셨네요 잘 봤습니다!
hyunn522 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
하나의 메소드 내에 여러 책임이 포함되어있는 것 같아요! SRP를 위해 메소드를 분리해보시는 건 어떨까요?
hyunn522 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
일급 컬렉션 사용 좋습니다! 다만 일급 컬렉션 객체는 일종의 자료구조와 유사한 형태라고 알고 있어서 `service` 디렉토리보단 `model` 디렉토리가 더 적절할 것 같은데 어떻게 생각하시나요?
hyunn522 created a review on a pull request on woowacourse-precourse/java-calculator-7
unit test가 정말 꼼꼼하게 작성되어 있어서 잘 봤습니다!
hyunn522 created a review on a pull request on woowacourse-precourse/java-calculator-7
unit test가 정말 꼼꼼하게 작성되어 있어서 잘 봤습니다!
yeseong33 created a comment on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 봤습니다. 전체적으로 코드를 잘 짜신것 같습니다. 한 클래스에 여러 역할을 부여하지 않았고, stream, interface, getter, view 등의 기본적인 기능들을 잘 활용하시는 것 같아요! 그리고 개인적으로 항상 함수 이름을 설정할때 고민이 되는데, 작성자님은 길게 쓰더라도 의미를 충분히 부여하려고 한 것 같아요. 이렇...
Jewan1120 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
헛 그러게요.. 따로 메서드로 뺐어야할 것 같아보이네요
Jewan1120 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
간단한 비즈니스 로직이라 하나의 서비스에 담으려고 했던 것 같네요! 나중에는 분리해서 작성해볼게요 감사합니다 ㅎ
sojeong0202 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그러면 예외에 대해 정의하는 부분을 따로 만들고 그 예외에 대한 처리를 Delimiter에서 하는 방식으로 하면 준원님 말씀처럼 분리가 될까요? 좋은 개념 알려주셔서 감사합니다! `Exception Handler` 공부해보겠습니다~
