Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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를 써야 되긴 합니다만.
sjmmics created a review comment on a pull request on woowacourse-precourse/java-calculator-7
값을 반환하는 메소드 네이밍 컨벤션에 따르는 앞이 get으로 시작합니다. 다음 블로그 참고하시면 좋을 듯 합니다. https://tecoble.techcourse.co.kr/post/2020-04-26-Method-Naming/
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
mvc 패턴에 대한 전반적인 이해가 보이네요. 전체적인 구조를 잘 짜신 것 같습니다. 클래스, 메서드, 변수 네이밍 컨벤션만 살펴보시면 더욱 좋은 코드를 작성하실 것 같습니다.
sjmmics created a review on a pull request on woowacourse-precourse/java-calculator-7
mvc 패턴에 대한 전반적인 이해가 보이네요. 전체적인 구조를 잘 짜신 것 같습니다. 클래스, 메서드, 변수 네이밍 컨벤션만 살펴보시면 더욱 좋은 코드를 작성하실 것 같습니다.
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
구분자 추출+검증은 별도 메서드로 추출하는게 더 좋겠네요! 감사합니다!
kaswhy created a review comment on a pull request on woowacourse-precourse/java-calculator-7
저도 코드 리뷰를 받으면서 많이 받은 리뷰가 검증 로직의 책임이 어디에 있어야 하는지 생각해 보면 좋을 것 같다는 말이었는데요, 정말 좋은 규칙인 것 같아요! getter와 setter 부분에서도 어떻게 하면 같은 로직을 getter와 setter를 사용하지 않고 구현했는지도 고민되었는데, 많이 배워가요!
kaswhy created a review comment on a pull request on woowacourse-precourse/java-calculator-7
from이라는 메소드명이 조금 모호하게 느껴지네요...!
kaswhy created a review comment on a pull request on woowacourse-precourse/java-calculator-7
이런 부분 설명 정말 좋네요! 제가 고민했던 부분들이 많이 완화되는 부분인 것 같습니다. 추가적으로, 리뷰 요청하실 때 MVC 패턴을 적용하지 않은 코드라고 하셨는데 저는 개인적으로 지금 준서님이 작성하신 코드가 MVC 패턴과 유사하게 작동하고 있다고 생각합니다! 이 부분은 어떻게 생각하시는지 궁금해요~
kaswhy created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요, 준서님! 밀린 리뷰 요청이 많아 이제야 리뷰 드립니다. MVC 패턴을 사용하지 않으셨다고 하셨는데, 큰 흐름 자체가 저에게는 MVC와 비슷하게 느껴지는 코드였어요. 그리고 객체지향 원칙을 철저히 지키시고, 자바의 기본 문법이 왜 쓰이는지를 잘 알고 사용하시는 것 같아 배울 점이 정말 많은 코드라고 생각했습니다! 다른 분들과 리뷰하신 부분...
kaswhy created a review on a pull request on woowacourse-precourse/java-calculator-7
안녕하세요, 준서님! 밀린 리뷰 요청이 많아 이제야 리뷰 드립니다. MVC 패턴을 사용하지 않으셨다고 하셨는데, 큰 흐름 자체가 저에게는 MVC와 비슷하게 느껴지는 코드였어요. 그리고 객체지향 원칙을 철저히 지키시고, 자바의 기본 문법이 왜 쓰이는지를 잘 알고 사용하시는 것 같아 배울 점이 정말 많은 코드라고 생각했습니다! 다른 분들과 리뷰하신 부분...
hyojaeKwon created a review comment on a pull request on woowacourse-precourse/java-calculator-7
동의합니다! 2주차 미션에서는 입/출력의 controller기능을 담당하는 별도의 클래스를 도입해보겠습니다!!
mixxeo created a review comment on a pull request on woowacourse-precourse/java-calculator-7
MVC 패턴에서 객체의 레이어를 구분하는게 아직 익숙하지가 않네요 ㅎㅎ 좋은 의견 감사합니다 좀 더 고민해볼게요 ☺️