Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
soonhong99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
개인적인 생각입니다! 해당 initialize는 stringCalculator 안에서 하는 것은 어떨까요? 아래 있는 코드 run은 와닿지만, defaultDelimiters를 stringCalculator를 이용해 initialize화 하는 부분에서는 stringCalculator 클래스 안에 있어도 되지 않을까 생각이 듭니다! main에서 정의하...
jihwankim128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 외부에서 유틸 객체들을 DI 받고 있긴한데, 결합도를 더 낮출려면 어떻게 리팩토링하는게 좋을까요 ?? 🤔 각 DI 되는 객체들을 단위 테스트 한 뒤에 안전성을 보장하는 것이 우선이라고 생각합니다. 이후, Fake 객체나 Mock 객체와 같이 예상하는 동작의 객체를 주입함으로써 순수하게 기능에 대해서만 테스트 할 수 있을 것 같습니다!!
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드명은 parseInput인데 에러까지 반환하도록 설계한 이유가 궁금해요.
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스명, 메서드명으로 표현하기 애매하거나, 조건이 읽기 힘든 경우에만 주석을 남겨보는건 어떨까요?
Astro-Yu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
parsedInput의 첫번째 인덱스 문자가 delimiter 인가요? 0, 1 의 의도를 알고싶어요.
Astro-Yu created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 사빈님. 코드 잘 읽었습니다. 변하지 않을 상수들에 대해선 미리 final static 으로 선언하는게 좋아 보여요. 상수가 코드에서 그대로 쓰일 경우 변화에 취약하거나 의도를 파악하기 어렵습니다. 또한 Application 클래스가 너무 많은 기능을 담당하고 있는 것 같아요. 기능에 따라 분리해보면 좋을 것 같습니다. 구글에서 "객체지향 ...
Astro-Yu created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 사빈님. 코드 잘 읽었습니다. 변하지 않을 상수들에 대해선 미리 final static 으로 선언하는게 좋아 보여요. 상수가 코드에서 그대로 쓰일 경우 변화에 취약하거나 의도를 파악하기 어렵습니다. 또한 Application 클래스가 너무 많은 기능을 담당하고 있는 것 같아요. 기능에 따라 분리해보면 좋을 것 같습니다. 구글에서 "객체지향 ...
chul5 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 코드가 밑에 DefaultDelimiter에 있는 부분과 겹치는 부분이 있는 것 같은데 메서드를 따로 빼서 작성해도 좋지 않을까요??!
chul5 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아.. 소수일 때를 아예 배제한 정수 계산기를 구현하신 건가요?! 해당 부분 문자열을 상수로 관리할 수도 있을 것 같아요
chul5 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
인트 범위를 넘어가는 경우를 고려하시면 더 좋은 코드가 될 것 같아요! 저 또한 소수일 때를 고려하지 않았는데.. Double을 통해서 코드를 작성해보시면 어떨까요?
chul5 created a review on a pull request on woowacourse-precourse/java-calculator-7
한 주간 고생많으셨습니다!! 늦었지만 코드리뷰 해봤습니다! 다음주도 함께 힘내봐요!
chul5 created a review on a pull request on woowacourse-precourse/java-calculator-7
한 주간 고생많으셨습니다!! 늦었지만 코드리뷰 해봤습니다! 다음주도 함께 힘내봐요!
jihwankim128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> OutputView 에서 결과를 BigInteger 타입으로 받고 있기 때문에 동일한 타입을 사용한거에요~ 그리고 이름 그대로 정수형을 더하는 메서드 이기 때문에 파라미터 변경이 필요하다면, 다른 메서드로 선언하는게 맞다고 보거든요~ 지환님 생각은 어떠신가요 ~?? 뒤늦게 제 의견을 보완하자면 입력의 한계를 Integer로 제한하신 것 같은데...
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
정확히 제 의도를 파악하셨습니다 ㅎㅎ 맞습니다! Calculator를 interface로 구현한 이유는 확장성을 고려했기 때문입니다. 현재는 덧셈(sum)의 요구사항만 있지만, 추후 뺄셈, 곱셈, 나눗셈 등 다양한 연산 기능을 추가할 수 있는 가능성을 염두에 두었습니다!
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
num 처리 로직을 별도의 메서드로 빼 indent를 줄이는건 어떨까요? 가독성을 늘리고 메서드 기능을 더 잘게 나눌 수 있을 것 같습니다
soonhong99 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
> 저는 단순히 Exception을 검증 로직에서 IllegalArgumentException를 바로 던지게 구현했었는데, 이렇게 Exception을 더 구체화시켜서 던져주니 코드의 가독성이 더 좋아졌다고 생각합니다! 지금은 CalculatorException만 구현되어 있는데 각각의 예외상황에 맞는 Exception을 더 세분화해서 구현해봐도 좋을 ...
soonhong99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 모든걸 하나의 클래스 안에 집어넣을 생각을 하시다니.. 많이 배워갑니다! 매우 직관적인 코드인 것 같네요.
soonhong99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
README 파일을 엄청 꼼꼼하게 작성하셨네요..! 저도 2주차때 민트님이 작성해주신 것처럼 꼼꼼하고 세심하게 작성해봐야겠어요!
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Stream을 사용하는 것은 어떤가요? 저 또한 이번 미션에서 반복문으로 for문을 사용하였는데 가독성을 위해 Stream으로 바꿨으면 하는 생각이 들었습니다. 수창님의 생각은 어떠신가요?
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
inputString에 대해 null 조건을 판단하시는 것은 어떨까요?
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
불필요한 주석은 삭제하는 게 좋을 것 같습니다.
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
문제에서 "커스텀 구분자는 문자열 앞부분의 "//"와 "\n" 사이에 위치하는 문자를 커스텀 구분자로 사용한다." 라는 문장이 있었습니다. '문자'라고 하였는데 공백과 문자열을 구분자 취급하신 이유가 있나요?