Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력값이기 때문에 따로 input 변수로 빼봤습니다. 계산기가 calcultate로 input을 받아 계산한다라는 관점에서 본다면 Console.readLine()은 조금 어색한 것 같아서 그렇게 했습니다!
bowook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
비어있는 문자열을 체크하고 NumberFormatException() 대신에 아래와 같이 코드를 작성하시면 더 좋을 것 같다는 생각이 듭니다! `int num; try { num = Integer.parseInt(s);} catch(NumberFormatException e){ throw new IllegalArgumentException();`
kwongio created a review on a pull request on woowacourse-precourse/java-calculator-7
먼저 리뷰 주셔서 감사합니다. 저도 객제지향적으로 코드를 구성하려고 노력하고 있습니다. 아직도 많이 부족하네요 ㅠ 저는 문자열 계산기라는 관점에서 Calculator가 다른 객체들을 가지고 사용하는 쪽으로 리뷰를 달아봤습니다. 1주차 고생많으셨습니다
kwongio created a review on a pull request on woowacourse-precourse/java-calculator-7
먼저 리뷰 주셔서 감사합니다. 저도 객제지향적으로 코드를 구성하려고 노력하고 있습니다. 아직도 많이 부족하네요 ㅠ 저는 문자열 계산기라는 관점에서 Calculator가 다른 객체들을 가지고 사용하는 쪽으로 리뷰를 달아봤습니다. 1주차 고생많으셨습니다
mixxeo created a review on a pull request on woowacourse-precourse/java-calculator-7
PR description에 적어주신 것처럼 코드가 간결해서 가독성이 좋았습니다! 1주차 고생하셨습니다!!
mixxeo created a review on a pull request on woowacourse-precourse/java-calculator-7
PR description에 적어주신 것처럼 코드가 간결해서 가독성이 좋았습니다! 1주차 고생하셨습니다!!
sunkong25 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
오 좋은 방법인 것 같아요! 이번 프리코스 목표 중 하나가 유지보수 쉽게 하도록 코드 구현하는 것인데 클래스나 Enum으로 만들어서 관리하는 방법을 새롭게 알아갑니다:) 좋은 팁 감사합니다😊
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 약간 반대 이야기를 조금 해 보겠습니다.. 패턴의 학습 이후에 적용보다는, 객체의 책임을 먼저 생각한 뒤 그에 따른 코드가 MVC에 가까어지는 코드가 더 바람직하다고 생각해요. 아직 객체지향이 어렵게 느껴지신다면, 객체 책임 분리나 객체지향에 대한 공부를 더 해도 좋겠습니다. 물론 이런 결정은 민혁님 몫이고요! 😁
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Java에서는 camelCase를 사용합니다. 컨벤션을 지키는 것은 왜 중요할까요?
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
왜 Java로 프로그램을 작성할까요? 변수명만 길어지고 책임 분리는 귀찮아요. Python과 같은 언어로 프로그램을 작성한다면 훨씬 적은 코드로 같은 결과를 낼 수도 있고요. 그럼에도 Java를 사용하는 이유는 무엇일까요? 🤔 저는 협업과 유지보수에 큰 가치를 두었다고 생각해요. 혼자 개발하는 것이 아니라 둘 이상이 규칙을 가지고 개발하기. 변경 사항...
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`Calculator`는 계산기이고, 계산기는 수의 연산을 합니다. 계산기에게 `add`라는 연산을 하라고 하면서 아무것도 주지 않는 것이 조금 어색하게 보이는데요, 이는 `input`을 적절하게 나누는 역할이 계산기에게 존재하지 않는다는 이야기로 흘러갈 수 있겠습니다.
donghoony created a review on a pull request on woowacourse-precourse/java-calculator-7
반갑습니다 👋🏻 다른 리뷰어분들이 코멘트를 잘 달아두어서, 저는 컨벤션과 의도에 관한 리뷰를 남겨 보았어요. 도움이 되셨으면 좋겠습니다. 남은 미션도 화이팅이예요! 🚀
donghoony created a review on a pull request on woowacourse-precourse/java-calculator-7
반갑습니다 👋🏻 다른 리뷰어분들이 코멘트를 잘 달아두어서, 저는 컨벤션과 의도에 관한 리뷰를 남겨 보았어요. 도움이 되셨으면 좋겠습니다. 남은 미션도 화이팅이예요! 🚀
sunkong25 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
알려주셔서 감사합니다:) 2주차에서는 참고해서 구현해봐야겠어요!
bowook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
답변해주셔서 감사합니다! 1주차 고생하셨습니다!
miniminjae92 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 예외 처리하면서 예외가 발생하게 된 값도 반환 시키면 예외 처리할 때 도움이 되겠네요! 배워갑니다! 열정적인 리뷰 감사합니다!
miniminjae92 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 우테코 코드 원칙에도 나와있는데, 코드 depth가 깊은 건 좋지 않아서 depth를 줄이면 좋을 것 같아요! 15줄을 넘지 않도록 신경을 써야겠어요!
miniminjae92 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 하나의 메서드에서 많은 동작들을 하고 있어 메서드 분리하면 좋을 것 같네요! SRP 를 신경을 써볼게요!
BaSak0630 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자를 추가할 때 특수 문자나 숫자 등 문제가 발생할수 있는 문자들을 전부 ','로 치환해서 split을 해주면 문제가 발생하지 않을 것 같아서 다시 배열에 저장하였습니다.
fr35wo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
개인적으로 생각했을때 확장 가능성이 있다면 적용시킵니다
