Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
유효성 검증을 하는 로직이 데이터를 다루는 객체의 책임이라고 생각해서 내부에 예외 처리 로직도 구현하게 되었습니다 프로그램 자체의 규모도 크지 않으니 구조를 간결하게 유지하고싶기도 했구요! :) [찾아보니](https://stackoverflow.com/questions/2223564/should-a-class-validate-itself-o...
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
놓치고 있던 부분인데 알려주셔서 감사합니다! 바로 자동 개행 적용했습니다 😄
gladhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
맞습니다 사실 해당 과제에서는 Dto가 존재하지 않아도 된다고 생각합니다만 이용한 데엔 두 가지 이유가 있습니다. 1. View와 Controller 사이에서 데이터 교환시 불변으로 받아오는 보장성. 2. Controller에서 Request와 Response라고 사용하게 되면 데이터의 계층간의 이동이 명확히 보이기 때문. 입니다. 이 이유들과 ...
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
결과를 equals로 테스트하면 추후 결과 메시지에 다른 정보가 추가될 경우 테스트까지 수정해줘야 하는 점이 번거로울 수 있을 것 같아요. 기본 제공된 테스트처럼 `contains()`를 적용해보는 건 어떨까요??
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
엔터도 컨벤션의 일부라는 사실! 알고계셨나요? [Naver 캠퍼스 핵데이 Java 코딩 컨벤션](https://arc.net/l/quote/lzyspvay)
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
중요한 메서드에 대해 별도 테스트를 만들어주셨네요! 좋은 테스트 방향성이라고 생각합니다. 👍
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
완전 사소한 내용인데, 왜 파일 끝에 이런 마크가 뜨는지 아시나요!? <img width="86" alt="image" src="https://github.com/user-attachments/assets/28592d8f-1033-44b6-8f54-fa26a321006d"> 사실은 파일 끝에 개행을 하나씩 넣어주는 것도 컨벤션의 일부랍니다! ...
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
String 자체가 null로 들어올 경우를 감안한 예외처리군요. 꼼꼼하시네요! 👍
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드명에 `Is`와 같은 단어 자체에 의미가 없는 경우는 지양하는 편이 좋을 것 같습니다. input임을 명시할 필요도 없을 것 같아요! `validateNegative()`나 `validateNegativeNumber()`같은 방향은 어떨까요? 🤔
songsunkook created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다. 어떻게 하면 관심사를 분리하며 가독성을 높일 수 있을지 고민한 흔적이 보이는 것 같아요. 👍 1주차 과제 고생하셨습니다!
songsunkook created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다. 어떻게 하면 관심사를 분리하며 가독성을 높일 수 있을지 고민한 흔적이 보이는 것 같아요. 👍 1주차 과제 고생하셨습니다!
lold2424 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
BigInteger를 사용하면 오버플로우를 지정하지 않아도 되기 때문에 예외 처리를 할 필요가 없겠네요.
lold2424 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 하나의 테스트 코드에 모든 테스트를 담았는데, 다른분들의 코드를 살펴보면서 느낀거지만 테스트 코드를 하나에 다 담기보다는 분리해서 담는게 가독성 및 유지보수에 좋아보입니다. 예를 들자면 입력 테스트, 구분자 추출 테스트, 출력 테스트로 클래스를 나누는것처럼요!
lold2424 created a review on a pull request on woowacourse-precourse/java-calculator-7
구분자에 대해 고민을 많이 하신것 같습니다. 구분자를 명확하게 구분하신 것 같네요. 1주일동안 고생하셨고 남은 1주일동안 미션 화이팅입니다 :)
lold2424 created a review on a pull request on woowacourse-precourse/java-calculator-7
구분자에 대해 고민을 많이 하신것 같습니다. 구분자를 명확하게 구분하신 것 같네요. 1주일동안 고생하셨고 남은 1주일동안 미션 화이팅입니다 :)
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좀 더 큰 묶음으로 ((ex) InputView와 OutputView를 묶어서 View 추가 생성) 주입받아 생성자 수를 낮추는 것도 좋은 방법일 것 같습니다!
Teddysir created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드, 변수명 작명하는방식을 2주차에는 좀 더 시간을 써서 만들어봐야할거같아요!! 정말 좋은 지적 너무 감사합니다!!
Teddysir created a review comment on a pull request on woowacourse-precourse/java-calculator-7
넵 사용자 입력과 입력에서 받은 값을 Integer배열에 저장해서 사용하는 방식을 처음에 생각해서 그려봤는데 더 다른 좋은 방법들이 많이 있더라구요! 좋게 봐주셔서 감사합니다 :)
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
그동안 생각없이 여러개의 도트를 이어서 사용하고 있었는데 디미터의 법칙 위반이군요..! 잘 고민해보겠습니다 :)