Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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를 제공하는 컬렉션을 사용...
YoungGaLee created a review comment on a pull request on woowacourse-precourse/java-calculator-7
indexOf()를 알았더라면 inputString.substring(4+delimiter.length()); 보다 깔끔한 작성이 가능했을 것 같다. 코드를 더 가독성있고, 간결하게 만들 수 있었다.
digitpic created a review comment on a pull request on woowacourse-precourse/java-calculator-7
static 을 사용하게 되면 프로그램이 실행될 때 프로그램 전역적으로 생성되어 모든 객체가 이를 알 수 있게 된다고 알고 있습니다 이는 객체지향적이지 못하다는 생각이 들었고 그렇기에 상수라고 꼭 static 을 써야 하나? 라는 생각이 들었고 상수라고 해도 클래스끼리 공유하지 않는 경우가 많았기에 꼭 static 을 사용하지 않아...
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
적용하면 가독성이 개선될 것 같아요! 감사합니다 ☺️
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저는 패턴에 대해 매칭되는 부분을 찾는 것도 "구분자 추출" 메서드의 역할이라고 생각했는데요! 아래에서 짚어주신 것처럼, find후 예외처리 하는 영역을 별도로 분리하고 pattern 변수도 static으로 정의하면 좀 더 메서드가 간결해질 것 같네요 ㅎㅎ 감사합니다 :)
kaswhy created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메소드 인자를 2개 정도로 줄이는 것이 어떨까요? 객체지향 원칙에 4개이상의 인자는 허용하지 않는다는 원칙이 있습니다!
kaswhy created a review comment on a pull request on woowacourse-precourse/java-calculator-7
테스트 목적에 맞게 파일 구분하셔서 테스트하신 것 좋은 것 같아요 👍🏻
kaswhy created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 밀린 리뷰 요청이 많아 코드 리뷰가 늦었네요. 이미 다른 분들께서 제가 하고 싶은 말들을 많이 남겨주셔서 간단하게 리뷰해보았습니다! 2주차도 화이팅이에요 ☺️
kaswhy created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 밀린 리뷰 요청이 많아 코드 리뷰가 늦었네요. 이미 다른 분들께서 제가 하고 싶은 말들을 많이 남겨주셔서 간단하게 리뷰해보았습니다! 2주차도 화이팅이에요 ☺️