Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
sangxxjin created a review on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 많이 고민하시고 작성하신 흔적이 느껴집니다. 많이 배워갑니다~
eaststar113 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
enum으로 다른분들도 많이 활용하신거같더라구요..저도 참고해서 적용해보겠습니다
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력을 BigInteger로 받는것도 좋은 방법이 되겠네요~ 입력이 정수로 제한 되어 있는데도 불구하고, 반환값이 BigInteger인 이유는 OuputView의 printResult()의 파라미터 타입을 BigInteger로 받고 있어서 인데요~ 그래서 타입이 바뀌더라도 printResult()를 재사용하기 위해서 sumInteger()도...
eaststar113 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아래에 중복되는 부분이 있더라고요..다음부터는 리팩토링을 좀 더 꼼꼼하게 해야될것같네용 의견 감사합니다:)
eaststar113 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
2주차 과제부터 적용해보도록 하겠습니다. 의견감사합니당:)
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
input을 받는 기능, type casting하는 기능, print하는 부분을 private 메서드로 분리하면 가독성이 좋아질 것 같아요
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
조건문을 사용하는 것보다 추상화와 상속을 통해 책임을 나누는 것은 어떨까요? 분기에 따라 로직을 처리하지 않고 각 객체의 역할이 명확해지며 응집도가 높아질 것 같습니다. 또, 파싱하는 방법이 추가된다하더라도 OCP를 지키며 기능 확장을 할 수 있을 것 같아요
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
util로 따로 빼지 않고 Number 객체를 활용하는 것은 어떨까요?
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
일급 컬렉션을 사용해서 로직을 넣는 방법 좋은 것 같습니다
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 응집도를 생각해 상수를 사용하는 클래스에 정의를 하였는데, 상수를 한 곳에 관리하는 방법도 좋은 것 같습니다.
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
별도 객체로 도메인 항목을 나타내면 생성자에서 검증까지 할 수 있어 좋은 것 같습니다
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
각각의 검증 항목을 메서드로 분리하여 가독성이 좋은 것 같습니다
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
ChangeLog를 관리하니 변경사항을 확인하기 좋은 것 같아요! 배워가요
eaststar113 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
문자열 파싱하는 부분으로 계산기에서 합계를 구하기 전 예외처리를 미리 다 해놓으면 좋다고 생각해서 이렇게 설계하였습니다. 다른 부분에서 예외를 줬어야 될까요..?
eaststar113 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 길이가 3이상이 들어오면 커스텀 구분자가 있다고 판단해서 그렇게 작성했습니다. 이제 보니까 변수명을 둬서 했으면 더 직관적이였을거같네용
sangxxjin created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 Vector를 사용한 이유가 궁금하네요
sangxxjin created a review comment on a pull request on woowacourse-precourse/java-calculator-7
모델에서 입력 및 출력에 관한 메서드는 없어도 되지 않을까 라고 생각합니다~ Applicateion를 컨트롤러 처럼 사용한 것으로 보이는데 여기에도 메서드를 만들 필요가 있을까요?
sangxxjin created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이러한 정규식도 상수화를거치면 어떤 정규식인지 가독성이 더 좋아질 것 같습니다
sangxxjin created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이런 멘트나 변함이 없는 것들은 상수처리해도 좋을 것 같네요
sangxxjin created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 읽었습니다. 메서드 분리하려고 노력하신게 보이시네요. 변수명이나 상수처리 신경 조금 더 써주시면 좋을 것 같아요~
sangxxjin created a review on a pull request on woowacourse-precourse/java-calculator-7
잘 읽었습니다. 메서드 분리하려고 노력하신게 보이시네요. 변수명이나 상수처리 신경 조금 더 써주시면 좋을 것 같아요~
eaststar113 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분은 제가 리팩토링을 진행하면서 시간이 부족하여 간과한 것 같습니다. 2주차부터는 변수명에 더욱 신경써야할거같네요..의견감사합니다:)