Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
yjinlee99 created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 코드 잘 보았습니다! 저와 다르게 mvc로 짜셔서 생각해볼게 아주 많았어요!! 연산과 문자열 처리를 다른 클래스로 구현한것도 좋네요!
yjinlee99 created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 코드 잘 보았습니다! 저와 다르게 mvc로 짜셔서 생각해볼게 아주 많았어요!! 연산과 문자열 처리를 다른 클래스로 구현한것도 좋네요!
coli-geonwoo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
아루의 코멘트를 보고 제가 너무 주관적인 답이 담긴 정적인 리뷰를 남겼다는 성찰을 하게 되네요. 제가 드리고 싶었던 마일스톤도 비슷합니다. 질문을 남겨놓으면 - 우리는 왜 OOP라는 패러다임을 사용할까요? 꼭 객체지향적으로 코드를 짜지 않아도 되는데요? - 객체가 맡는 책임이란 무엇을 의미할까요? - 현재 Calculator는 어떤 책임을 ...
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
한 주간 고생 많으셨습니다! 이미 다른 분들이 많이 코드리뷰를 하셨고 해당 답변들을 보며 저도 많이 배울 수 있던 시간이었습니닷!! 다음 주도 같이 힘냈으면 좋겠습니다!
SeongUk52 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
문제에서 명확하게 제시되진 않았지만.. 커스텀 구분자가 여러번 선언 되는 경우를 고려해보면 어떨까요?
SeongUk52 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
프리코스에서 정한 컨벤션에서는 들여쓰기가 4개의 빈 칸으로 정해져 있는데 작성해주신 코드에서는 2칸으로 보입니다
SeongUk52 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
사소하지만 줄바꿈을 사용하시는 편이 가독성이 높을 것 같습니다.
SeongUk52 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 고생하시느라 고생 많으셨습니다. 3개의 클래스로 굉장히 컴팩트하게 잘 구현해주셨네요 이미 다른분들의 코드리뷰가 많아 많은 피드백을 드리지 못해 죄송합니다. 2주차 미션도 열심히 해봅시다
SeongUk52 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 고생하시느라 고생 많으셨습니다. 3개의 클래스로 굉장히 컴팩트하게 잘 구현해주셨네요 이미 다른분들의 코드리뷰가 많아 많은 피드백을 드리지 못해 죄송합니다. 2주차 미션도 열심히 해봅시다
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 지나다가 코멘트 하나 남깁니다! 저같은 경우는 다르게 생각했는데, 클래스 자체가 Validator이기 때문에 예외 상황이 명확하게 보이는 게 이 클래스에서는 더 중요하다고 생각했습니다 또한 부정/긍정 로직이 있으면 부정로직(혹은 비정상 로직) 이 먼저 나오는 게 가독성이 좋다고 �생각했요. 비정상 로직을 모두 걸러내고 마지막에 정상 로직이 위치하면...
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
stream을 사용하신 부분이 인상깊네요! 저는 이번 코드에서 스트림을 사용하지 않았었는데 이 부분에 대해서도 고려해봐야겠네요. 배워갑니다!
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 동의합니다! 사용자가 커스텀 구분자를 잘못입력하여 endIndexOfCustomDelimiter가 -1이 되는 경우에는 예외를 발생시키는 것이 좋아보입니다
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> 안녕하세요! 실제로 처음에 절차지향적으로 코드를 짠 뒤에 일부 수정을 했습니다. 저는 이 과제의 과정의 대부분이 데이터를 정제하고, 데이터를 노출해야 하는 부분이 많아 절차지향에 조금 더 가깝다고 생각해 객체지향보다는 절차지향에 가깝게 구현하게 된 것 같습니다. 혹시 어떤 부분의 책임을 분리하면 좋을지, 어떤 부분에 객체지향을 적용하면 좋을지 추가...
hyobin-yang created a review comment on a pull request on woowacourse-precourse/java-calculator-7
피연산자 부분만 추출하여 input을 재정의하셨는데 제 개인적인 생각으로는 새로운 변수로 할당하는 게 좋을 것 같습니다! 예를 들어 'operandString'이라는 문자열 변수에 해당 값을 할당하여 '피연산자 문자열'이라는 의미를 드러내는 식으로 다른 변수를 선언해보는 건 어떨까요? 그 이유는 생성자에서 정의한 input이라는 변수의 재사용성이 ...
hyobin-yang created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 넘 수고 많으셨습니다! 가독성이 좋아서 코드 읽기가 편했어요 👍
hyobin-yang created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 넘 수고 많으셨습니다! 가독성이 좋아서 코드 읽기가 편했어요 👍
kwonkeonhyeong created a review comment on a pull request on woowacourse-precourse/java-calculator-7
enum으로 구현해본 이유는 단순히 default mode를 true false로 관리하기 보다 현재 정확히 어떤 모드가 작동하고 있는지 표현해보고 싶었습니다!
JamongFriend created a review comment on a pull request on woowacourse-precourse/java-calculator-7
처음에 private로 선언한 이유가 무엇인가요? 의도가 궁금합니다
JamongFriend created a review comment on a pull request on woowacourse-precourse/java-calculator-7
처음에 private로 선언한 이유가 무엇인가요? 의도가 궁금합니다
ParkHyunS00 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
만약 예외처리나 추가적인 작업이 덧붙여지게 된다면 메서드로 분리하는 것이 좋아보여요!!
bamsanchaeg created a review comment on a pull request on woowacourse-precourse/java-calculator-7
> c.f. `Pattern.quote(seperator)`를 찾아보시는 것을 추천드립니다~ 헉 적용해보니까 잘 됩니다. 알려주셔서 감사합니다!! 새롭게 알아가네요
JamongFriend created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 double을 사용하여 정수일경우 long타입으로 변경하는 번거로운 방법을 사용하여 호출자가 double인지, long일지 직접 확인해야 하는 코드를 작성했는데 BigDecimal같은 정밀한 타입을 사용할 수 있는 방법 배워갑니다.