Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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주차도 화이팅이에요 ☺️
digitpic created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이 부분은 저도 고민이 들었던 부분입니다.. validate 라는 메서드가 형변환한 값을 return 해버리면 검증과 형변환이라는 두가지의 책임을 지게 된다고 생각하여 두 가지 메서드로 나누어 parseInt() 메서드가 두 번 호출되는 로직이 생겼습니다 결국은 하나의 메서드가 하나의 책임을 지도록 하기 위해 이처럼 구현했습니다! 중복...
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
stream을 활용하면 코드를 더 예쁘게 만들 수 있을 것 같아요! 개인적으로 for문을 사용해야만 하는 코드라면 코드에 문제가 없는지 다시 한 번 확인해보는 습관도 유익했던 것 같습니다
songsunkook 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
`String.join`으로 구분자를 유연하게 활용하는 방향이 정말 유연하고 좋은 구조네요!! 👍 👍
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
Console.readLine()이나 System.out.println()같은 경우는 입력과 출력이라는 관심사에 따라 별도 클래스로 분리할 수 있을 것 같아요!
songsunkook 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
IllegalArgumentException을 잡아 IllegalArgumentException을 던지는 것 같은데 어떤 로직을 구상하신 건지 궁금합니다!
songsunkook created a review comment on a pull request on woowacourse-precourse/java-calculator-7
단일 파일에 모든 로직이 들어있는 건 객체지향스럽지 못한 코딩 방향이라고 생각해요..! 코드별로 관심사를 파악하고 그에 따라 클래스를 분리해보는 연습을 해보면 좋을 것 같습니다! 🥰 다른 사람들의 코드를 리뷰해보면 감이 잡히실 것 같아요!
songsunkook created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다. 관심사에 따라 메서드를 잘 분리해주신 것 같아요. 👍 다음에는 클래스까지 분리해보는 것도 좋은 시도가 될 것 같습니다! 1주차 과제 고생하셨습니다!
songsunkook created a review on a pull request on woowacourse-precourse/java-calculator-7
코드 잘 읽었습니다. 관심사에 따라 메서드를 잘 분리해주신 것 같아요. 👍 다음에는 클래스까지 분리해보는 것도 좋은 시도가 될 것 같습니다! 1주차 과제 고생하셨습니다!
digitpic created a review comment on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요! 저는 메시지 상수를 둘 수 있는 곳이 크게 세 가지의 분류로 나눌 수 있다고 생각합니다! ### 1. print 메서드 내 리터럴 작성 1번 같은 경우에는 코드를 직접 읽으며 어떤 메시지가 출력되는지 확인할 수 있다는 장점이 있습니다 하지만 같은 메시지가 여러 곳에서 사용된 경우 메시지 값에 대한 변경이 발생하게 되...
sjmmics 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
"//"와 "\\n"를 커스텀 구분자 지정 문자 접두사, 접미사로 변수화해서 관리하는 건 어떨까요? "\\n" 인덱스를 한 번 찾아서 커스텀 문자열을 구할 때 재사용하는 게 좋긴 한데 string의 startWith() 메소드와 contains() 메소드를 활용하면 가독성을 높을 수 있습니다. 기존 코드에서 주석을 추가하면 어떨까요?
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
메시지를 system.out.println으로 출력하시니 toString를 오버라이딩해서 사용하시는 건 어떨까요? 대신 outputview에서 printf 대신 println를 써야 되긴 합니다만.