Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
문자열을 별도의 클래스에서 상수화 하셨네요. 어떤 장단점이 있으셨나요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
 파일 맨 끝에는 관습적으로 개행을 추가하는 편인데요. EOF에 대해 찾아보시고 인텔리제이 자동 적용까지 해봅시다!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메서드명이 `isBlank()`면 54번 라인에 작성하신 `isBlank()` 처럼 작동할 것을 예상하게 돼요! 이런 이름은 어떠신가요? ```suggestion private static void validateNotBlank(String str) { if (str.isBlank()) { throw...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
```suggestion public static void validateContainsNegativeNumber(List<Integer> numbers) { numbers.forEach(number -> { if (number < 0) { throw new IllegalA...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`//` `\\n` 이런 값들은 상수화해도 좋을 것 같아요.
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
객체지향 생활 체조 원칙, 디미터 법칙에 대해서 찾아보시면 좋을 것 같습니다! > 한 줄에 점을 하나만 찍는다.
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`element`는 어떤 element인가요? 이름이 더 구체적이면 메서드 역할이 잘 드러날 것 같아요 👍
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
정적 팩터리 메서드를 사용하신 이유가 있었을까요??
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 잠이 안 와서 디스코드 보고 리뷰 남기고 갑니다. 😴 프리코스 화이팅하시고 좋은 하루 되세요!
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 잠이 안 와서 디스코드 보고 리뷰 남기고 갑니다. 😴 프리코스 화이팅하시고 좋은 하루 되세요!
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 메서드들을 보면 `run()`을 통해서 입력에 대해서만 테스트가 이루어지고 있는 것 같아요! 2주차에는 각 클래스와 메서드별로 단위테스트를 구현해보면 어떨까요? 테스트 용이성 상승은 곧 좋은 프로덕트 코드의 신호입니다 반대로 테스트 코드를 위해서 프로덕트 코드의 변경이 필요하다? 이는 프로덕트 코드에 대한 리팩토링 신호라고 생각해요 왜...
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`validateContainsWhitespace`, `validateContainsBlank`와 같은 네이밍은 어떠실까욥? 저는 네이밍이 가장 어려운 것 같아요 ㅠㅠ 네이밍 고민에 시간을 가장 많이 할애하게 되더라구요..
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
확장에 열려있는 인터페이스 설계 너무 좋은 것 같아요! 어떤 확장을 고려해서 view를 인터페이스로 설계하셨는지 여쭤봐도 될까요? 저도 인터페이스를 도입하려 했지만 적용할 지점을 찾지 못했거든요ㅠㅠ
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
입력에 대한 검증은 `Calculator`, `BigIntegerCalculator`에서도 검증하고 있어요 두 곳에서 검증하는 로직에 대한 의도는 어떤건지 잘 알 것 같아요 하지만 검증과정을 추가해야한다면 `InputValidator`, `Calculator` 중 어느곳에서 추가해야할지 개발자가 고민을 부담해야해요 또한 또 다른 개발자가 인...
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`stringAdd()` 해당 메서드가 어떤 역할을 하는지 바로 이해하기 어려운것 같아요..! 저는 처음 봤을때 문자열을 더한다? 와 같은 의미로 다가왔던 것 같아요 반환 타입이 String은 것을 제외하고는 메서드의 실질적인 동작은 정수를 연산을 진행하는 것으로 보여요 해당 메서드가 어떤 역할을 하는지 이해하기 쉽게 네이밍 해보는 것은 어떨...
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`BigIntegerCalculator`와 동일한 동작을 하는 메서드가 존재해요 코드의 중복을 없애기 위해서는 어떻게 관리하는게 좋을까요? 고민해보시면 좋을 것 같아요!
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`split()` 어떤걸 분리하는지, 뭐를 위해 분리하는지 정확히 어떤 역할을 하는 메서드인지 잘 읽히지 않는 네이밍인것 같아요! 구분자를 통해서 숫자를 추출하는 과정으로 보여요 `parseAndSplitWithSeparators`와 같이 정확히 어떤 동작을 수행하는지 명시해주면 코드의 가독성을 더 높일수 있을 것 같아요 메서드와 변수의 네이...
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
하나의 메서드가 2개의 역할을 하는 것 같아요! 각각 분리해보면 어떨까요?
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
해당 시스템이 멀티스레드 환경에서도 수많은 호출받을 메서드를 `static`으로 관리하는 것은 너무 좋은 선택인 것 같아요! 메서드 영역에서 관리되어서 불필요한 객체 생성없이 진행 할 수 있도록 한 점이 효율적이고 합리적인 선택인 것 같아요!
woogym created a review comment on a pull request on woowacourse-precourse/java-calculator-7
계산 결과를 String으로 반환하시는데 이유가 있으실까요? 계산 이후에 값이 불변(불필요한 연산 방지)하도록 불변 객체인 String을 사용하신걸까요? 이유가 궁금해요!
woogym created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 고생하셨습니다! 😁😁 적절하게 메서드도 분리되어 있어서 읽기 편했습니다. 질문에 대해 이야기해보면 좋은 시간이 될 것 같습니다. 감사합니다!
woogym created a review on a pull request on woowacourse-precourse/java-calculator-7
1주차 미션 고생하셨습니다! 😁😁 적절하게 메서드도 분리되어 있어서 읽기 편했습니다. 질문에 대해 이야기해보면 좋은 시간이 될 것 같습니다. 감사합니다!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
여기도 `target` 대신 더 좋은 이름이 있을 것 같네요!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`tokens`는 어떤 의미인가요? 숫자를 담고 있는지 구분자를 담고 있는지 더 구체적인 이름이 있으면 좋겠어요! 😄
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
기능 별로 주석을 작성해주시고 계신데요. 이런 경우 메서드 분리나 클래스 분리를 고려해볼 수 있을 것 같아요! 물론 지금도 분리가 잘 되어 있어서 main 메서드가 간결하게 유지되고 있네요 👍
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 메서드가 값을 반환할 것으로 예상했는데 아니네요. 필드로 `List<Character>`를 가지고 있는 만큼 생성자에서 파싱까지 마치면 어떨까요?
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`valueString`보다는 구체적인 이름이 좋아 보입니다!
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
```suggestion String input = Console.readLine(); ``` 해당 변수가 String이라는 것은 타입만 봐도 충분히 알 수가 있어요! [좋은 코드를 위한 자바 변수명 네이밍](https://tecoble.techcourse.co.kr/post/2020-04-24-variable_naming...
takoyakimchi created a review comment on a pull request on woowacourse-precourse/java-calculator-7
`"//"` `\\n` 같은 값은 상수화하면 좋을 것 같아요. 요구사항이 바뀌면 관리해야 할 포인트가 많아집니다! 숫자 2도 상수값에서 `length()`로 가져와보죠!
takoyakimchi created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 코드가 전체적으로 깔끔하네요! 변수, 메서드 명명 규칙에 대해서 공부해보시면 좋을 것 같습니다! 프리코스 화이팅입니다.