Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
유효성 검사를 main()에서도 수행하셨는데 둘 중 한 곳에서만 하시면 좋을 것 같습니다.
ChoiMGyu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
하나의 class로 작성하신 이유가 있나요? 객체 및 계층을 만들어서 하는 방법은 왜 선택하시지 않았나요?
ChoiMGyu created a comment on a pull request on woowacourse-precourse/java-calculator-7
README 파일에서 이 프로그램의 중요 역할 부분을 highlighting 하여 보여준 모습이 인상깊었습니다. 다른 사람들이 이 부분만 보고도 어떤 역할을 하는 프로그램일지 유추할 수 있을 것 같네요!
donghoony 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
위에서도 이야기가 나왔겠지만, 변수명에 자료형을 포함시키는 건 좋지 않은 선택일 수 있어요! 😁 좋은 메서드 이름을 위해 열심히 고민해봅시다. 🚀
donghoony created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 `import`도 읽어보는 편입니다. 의존성을 파악할 때에는 `import`만큼 편하게 모여있는 게 없거든요. 이 부분을 읽는 사람을 배려하는 하나의 장치이기도 하죠!
soonhong99 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이러한 interface를 만들게 되신 계기가 있으실까요? 프로그래밍 하면서 인터페이스를 만들어야지, 하는데, 한번도 만들 엄두를 못내서 고민입니다.. 언제 해당 인터페이스가 지금 필요한 것 같다! 라고 떠오르셨을까요?
RedSunSmile created a comment on a pull request on woowacourse-precourse/java-calculator-7
전체적으로 물흐르듯이 controller에서 호출해서,separatorManager, StringHandler,Validator 전체적인 흐름다좋아요 이런식으로 발전시키는구나 한수 배웠습니다 그런데 StringHandler에서 이미 separatorManager기능 이미 다해서 굳이 separatorManager필요없어도 될거같습니다 세부적으로 나눈점...
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)의 요구사항만 있지만, 추후 뺄셈, 곱셈, 나눗셈 등 다양한 연산 기능을 추가할 수 있는 가능성을 염두에 두었습니다!
