Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
hoonyworld created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 `1;2;3//;\\n`를 구분자 선언을 했다고 보고 계산이 될 수 있도록 했는데, 잘못된 구분자 위치로 판단하신 이유가 궁금합니다!
hoonyworld created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 정말 고생많으셨습니다!! 다음 미션도 화이팅입니다!
hoonyworld created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 정말 고생많으셨습니다!! 다음 미션도 화이팅입니다!
kgy1008 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
제가 Controller에서 예외를 처리한 이유는 Spring에서 controller에서 @valid 어노테이션을 통해 검증하는 과정에서 영감을 얻었기 때문이었습니다. 비지니스 로직에서 예외까지 잡는다면, 코드가 너무 복잡해질 것이라 생각했고 때문에 비지니스 로직에서는 예외를 던지고, 컨트롤러에서 `try-catch`문을 통해 예외를 잡거나 처리하는...
yechan-kim created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`null`을 예외 처리할 때, 어떻게 해야 할지 몰라 따로 테스트 처리를 못했었는데, 덕분에 배워갑니다!
yechan-kim created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 과제 고생 많으셨습니다! `README`를 읽어보니 과제에 대해서 고민을 많이 하신 게 드러나네요!!! `CHANGELOG`는 생각도 못했는데, 덕분에 배워갑니다! 파일들을 보니 전반적으로 EOF 문제가 발생하고 있는데, 해당 문제는 인텔리 제이에서 자동으로 처리해 주는 기능이 있으니 한번 사용하시는 것을 추천드립니다!
yechan-kim created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 과제 고생 많으셨습니다! `README`를 읽어보니 과제에 대해서 고민을 많이 하신 게 드러나네요!!! `CHANGELOG`는 생각도 못했는데, 덕분에 배워갑니다! 파일들을 보니 전반적으로 EOF 문제가 발생하고 있는데, 해당 문제는 인텔리 제이에서 자동으로 처리해 주는 기능이 있으니 한번 사용하시는 것을 추천드립니다!
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
enum으로 선언하는 것 좋은 것 같아요! 배워가요!!
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
InputParser라는 클래스가 너무 많은 책임을 가지고 있는 것 같습니다. 클래스 이름은 Parsing을 하는 클래스이지만 전처리나 검증, type casting 등 많은 역할을 하고 있는 것 같아요! 좀 더 객체를 잘게 나누면 코드 가독성이 더 좋아질 것 같아요!
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Vector는 동시성을 고려하기 때문에 ArrayList보다 싱글 쓰레드에서 성능이 떨어지는 것으로 알고 있습니다. ListArray를 사용하는 것은 어떤가요?
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
계산에 대한 책임을 calculator에게 위임하도록 하여 calculator의 메서드 노출 정도를 줄일 수 있을 것 같습니다. 하나의 메서드만을 노출시키면 캡슐화를 높일 수 있을 것 같습니다.
koosco created a review comment on a pull request on woowacourse-precourse/java-calculator-7
InputParser에서도 외부에 노출되는 메서드를 private으로 가리면 좋을 것 같습니다. public으로 메서드를 외부에 많이 노출시킬수록 변경 point가 많아지게 됩니다.
cwjjjjjj 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
List는 배열 크기 변경이 불가능한가요!?
cwjjjjjj created a review comment on a pull request on woowacourse-precourse/java-calculator-7
숫자가 몇 개 들어올지 모르는 상황이라 배열 크기 변경이 가능한 ArrayList를 사용했는데, 지금 생각해보니 length에 맞춰서 List를 사용하는 것이 더 효율적이었겠다는 생각이 드네요. ㅎㅎㅎ
cwjjjjjj created a review comment on a pull request on woowacourse-precourse/java-calculator-7
stream 사용이 익숙하지 않아 생각을 못했네요 적용해보도록 하겠습니다!
cwjjjjjj created a review comment on a pull request on woowacourse-precourse/java-calculator-7
잡아서 똑같은 걸 던져도 괜찮을까 하는 의문이 있었는데, IllegalArgumentException을 발생시키면서 끝나야 한다는 요구사항에 맞추려고 그렇게 작성했습니다. 예외 처리 부분에 있어서 아직 모르는 것이 많아 이렇게 작성했을 때의 문제점이 무엇인지, 어떻게 개선할 수 있을지 공부하고 있습니다.
20HyeonsuLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력받은 명령어를 객체로 표현하다 보니 Command라는 네이밍이 나온 것 같습니다. 말씀해주신 네이밍도 적합해보이네요!
soonhong99 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
잘못된 입력 값을 판단하는 케이스가 적다고 생각합니다. 만약 "/$\n1,2$3"이라는 입력 값이 들어온다면 오류가 발생해야 하지 않을까요?
