Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@DisplayName을 사용하면 @DisplayName("여러개 커스텀 구분자 사용") 이렇게 더 편하게 테스트 이름을 지을 수 있습니다
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Stack의 내부에는 synchoronized로 되어있어서 단일 쓰레드에서는 잘 사용하지 않습니다 ArrayDeque와 같은 자료구조를 사용해보시는건 어떠신가요?
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
for문에 i, j, k 이렇게 관례로 쓰긴하지만 로직을 작성할 때는 의미있는 이름을 주는 것도 고려해보면 좋을 것 같습니다!
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
removedString은 이미 String형이니까 removed로 해도 좋을 것 같습니다
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
팩토리 메소드 패턴을 사용해서 CalculatorController를 더 쉽게 생성할 수 있을 것 같습니다!
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메소드 prefix에 is는 주로 boolean형에 많이 쓰여서 메소드 호출할 때 헷갈릴 것 같아요 그리고 tmpString보단 좀 더 의미있는 이름을 지어주면 읽기 쉬운 쉬코드가 될 것 같습니다
kwongio created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 수비님 코드 잘 봤습니다! 1주차 미션 고생많으셨어요 2주차 미션도 화이팅입니다!
kwongio created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요 수비님 코드 잘 봤습니다! 1주차 미션 고생많으셨어요 2주차 미션도 화이팅입니다!
na0th created a review comment on a pull request on woowacourse-precourse/java-calculator-7
상세하게 readme 적어주셔서 코드 파악하는데 더욱 수월한 것 같습니다. 보고 배우겠습니다^^
yeonnhuu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
extractDelimiters와 extractNumbers 메서드 모두 공통적으로 input.startsWith("//") 확인 조건이 있어서 별도의 메서드로 분리했습니다!
yeonnhuu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
과제명 자체가 문자열 계산기여서 StringCalculator라고 작성했는데 다음 번에 클래스 작성할 때 메서드나 클래스명에 예약어나 기본 제공 클래스명을 사용하는 것을 주의하도록 해야겠네요. 의견 감사합니다!!
ashsty created a review comment on a pull request on woowacourse-precourse/java-calculator-7
final을 빼먹는 휴먼 에러를 일일이 찾아내기 어렵다는 점도 제가 생각하는 단점 중 하나긴 합니다 😚 물론 다른 생각을 가진 분들도 많으니 다른 분이 남겨주신 리뷰처럼 본인의 의견을 정립해나가실 수 있다면 좋겠어요!
yeonnhuu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
클래스를 분리하거나 Extractor와 같이 클래스 기능을 모두 나타내는 제목으로 표현하는 게 맞는 것 같습니다. 의견 감사합니다!
ykmxxi created a comment on a pull request on woowacourse-precourse/java-calculator-7
> > 저는 메서드 호출(요청) == 책임 이라고 생각하지 않습니다. 객체지향은 객체들 간 메시지를 보내며 협력하는 구조를 갖는데, 송신자 객체 내부에서 수신자.메서드(요청 데이터)을 갖고 있는것이을 수신자의 책임을 갖고 있다는 관점에서 바라보면 가장 상위 계층에 존재해 프로그램의 진입점들이 되는 곳들은 서비스의 모든 책임을 갖고 있다고 바라봐야 하니...
yeonnhuu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
코드 컨벤션을 지키고 else 예약어를 사용하지 않도록 아래와 같이 작성하는 게 더 좋을 것 같습니다! if (str.startsWith("//")) { return customParser(str); } return str.split(":|,");
yeonnhuu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
IllegalArgumentException을 catch한 후 다시 throw하셨는데 혹시 그렇게 작성하신 이유가 있는지 궁금합니다!
yeonnhuu created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력과 출력을 하나의 클래스에 구현하셨는데, 입출력 기능을 별도로 분리하는 것을 추천 드립니다.
yeonnhuu created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수행하시느라 고생하셨습니다!
yeonnhuu created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수행하시느라 고생하셨습니다!
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
각 요소를 BigInteger로 변경하고 더하기 보다, Integer 값들을 먼저 더한 뒤 마지막 결과만 BigInteger로 리턴해주는 방식도 괜찮을 것 같습니다!
mixxeo created a review on a pull request on woowacourse-precourse/java-calculator-7
객체의 역할이 잘 분리되어있어서 코드 읽기가 수월했습니다 :) 고생하셨습니다!
mixxeo created a review on a pull request on woowacourse-precourse/java-calculator-7
객체의 역할이 잘 분리되어있어서 코드 읽기가 수월했습니다 :) 고생하셨습니다!
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
위에는 null로 처리하셨는데 여기서는 isEmpty로 사용하신 이유가 있을까요?
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Console.close()로 사용한 자원을 닫아주면 더 좋을 것 같습니다!
kwongio created a review comment on a pull request on woowacourse-precourse/java-calculator-7
@DisplayName() 어노테이션을 사용하면 @DisplayName("기본 합치기 테스트") 이렇게 더 편하게 한글로 작성할 수 있을 것 같아요