Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
Tentennball created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 보고 갑니다!
saera-yook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
말씀하신대로 Extractor가 FormatValidator의 책임을 갖는게 어색한 것 같네요! 저는 문자열 계산기를 숫자를 추출하는 전처리 과정과 합계를 구하는 계산 과정, 이렇게 2개의 큰 기능으로 나누어 생각해서 Preprocessor와 Calculator 두 객체를 구현했습니다. 이 기준에서 Preprocessor가 Extractor, Fo...
yesjuhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
감사합니다 :) 기능 요구 사항에 위배되지 않으면서 확장 가능한 부분을 염두에 두고 작성했습니다
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력값 예외를 상황별로 자세하게 잘 구별해놓으신 것이 돋보이네요. 에러 메세지를 enum으로 관리하시는 분들이 있는데 한 번 고려해보시면 좋을 것 같습니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
상황별로 자세하게 테스트 코드를 작성하셨네요. 입력 값 검증을 위해 많이 노력하신 것이 보입니다. 예외 상황도 많이 분류해놓으셨네요. 다만 한 가지 의문인 점이 service 클래스를 왜 model 폴더에 넣으셨을까요?
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
상황별로 자세하게 테스트 코드를 작성하셨네요. 입력 값 검증을 위해 많이 노력하신 것이 보입니다. 예외 상황도 많이 분류해놓으셨네요. 다만 한 가지 의문인 점이 service 클래스를 왜 model 폴더에 넣으셨을까요?
yesjuhee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
좋은 의견과 리뷰 감사합니다!! 🙇 🙇♂️ 🙇♀️ 이 부분은 구현 당시 깊게 생각하지 않고 위처럼 작성을 했다가 미션 끝나고 디스코드에서 의견 나누는걸 보면서 "아 맞다 수학적으로 0은 양수에 포함이 안되는구나"하고 생각을 한 케이스인데요ㅎㅎ 그래서 "왜 내가 큰 고민 없이 0을 양수에 포함 시켰을까?"하고 나중에 추측해보고 다음과 같...
bogyeom0922 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드를 두 개로 나누어 작성하신 이유는 뭔가요? 하나로 합치면 간단할 것 같은데 나눠서 쓰신 이유가 궁금해서요!
bogyeom0922 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
숫자들을 추출하는 코드와 예외 상황에 대한 처리 코드를 한 파일에 작성하셨는데, 두 기능을 분리하여 따로 작성해 보는 것은 어떨까요? 책임이 더 명확해지고 유지 보수가 쉬워질 것 같아요 !
bogyeom0922 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수행하느라 수고하셨습니다 !!
bogyeom0922 created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 수행하느라 수고하셨습니다 !!
Tentennball created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이것 같은 경우에는 1. 단순 출력 메시지(String) 2. 0을 출력해야하는 경우(int) 두가지가 있었는데 이에 대해 오버로딩을 사용하여 메소드를 두개 만들기 보다는 내부의 로직은 똑같기 때문에 제네릭을 사용하는 것이 맞다고 판단하여 제네릭을 사용하였습니다!
Tentennball created a review comment on a pull request on woowacourse-precourse/java-calculator-7
사실 꼼수적으로 구현한 부분이긴 한데 특수문자들의 경우에 이스케이프가 필요해서 그냥 모든 구분자들에 대해 \\를 붙였습니다..ㅎㅎ
Tentennball created a review comment on a pull request on woowacourse-precourse/java-calculator-7
private으로 했어야하는데 변경하는것을 깜빡한것 같습니다!
Tentennball created a review comment on a pull request on woowacourse-precourse/java-calculator-7
커스텀 구분자를 받게 되었을때 숫자를 분리시켜줄 구분자 정규식을 생성해주는 로직입니다! joining메서드를 설명하자면 첫 번째 파라미터: 각 요소를 이어 붙일 때 사이에 들어갈 구분자입니다. 두 번째 파라미터: 연결된 문자열의 앞에 추가할 내용입니다. 세 번째 파라미터: 연결된 문자열의 뒤에 추가할 내용입니다.
Tentennball created a review comment on a pull request on woowacourse-precourse/java-calculator-7
어떤 점에서 여러개의 도트를 사용하는것이 별로인지 알 수 있을까요?
thsghdud13 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
지금 현재 코드에서는 nullable 하지 않습니다. 빈 리스트가 들어와도 0으로 반환이 되지만, integerList가 null로 들어왔을 때의 처리는 좀 더 고민해봐야 할 것 같습니다. ```java return integerList.stream() .mapToInt(Integer::intValue) ...
phk1128 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
제가 말씀드린게 정답은 아니에요~ 다만 저는 입력(사용자가 데이터를 입력하는 행위)과 출력(사용자에게 메세지를 보여주는 행위)의 개념을 명확하게 구분하고, 더해서 책임 분리도 되면 좋지 않을까 생각했어요~ 구홍님이 의도하신거를 잘 설명해주시면 그게 정답이 되지 않을까해요~ ☺️
saera-yook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
네 저도 2주차 미션에서는 Stream을 사용해보려구요!ㅎㅎ 코딩테스트 문제에서 Stream을 쓰면 보통 시간초과가 떠서 간단한건 반복문으로 처리하던 습관이 남아있었네요😅
saera-yook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
계산기 객체를 구현할 때 현실 세계의 계산기를 떠올리며 계산한 결과 값을 기억하고 추가로 연속적인 연산이 가능하게 만들어야겠다는 생각에 result 상태값을 갖고 있고 reset() 메서드로 초기화 하는 방식으로 구현했었는데, 생각해보니 너무 현실세계 객체를 그대로 반영하려 했던 것 같더라구요😅 그래서 리팩터링하며 현재 요구사항에서 굳이 필요없는...
kgy1008 created a review comment on a pull request on woowacourse-precourse/java-calculator-7
현재 코드에서는 기본 구분자라는 데이터를 조작할 일이 존재하지 않는 간단한 선언 형태의 로직이기에 굳이 컬렉션을 사용하지 않았습니다! 또한 컬렉션 대신 배열로 선언함으로써 기본 구분자가 2개다! 라는 사실을 고정(?)시키고 싶어 배열을 사용하였습니다. 하지만, 이번 피드백처럼 데이터를 조작할 일이 생긴다면 확실히 여러 API를 제공하는 컬렉션을 사용...
