-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[자동차 경주] 이해찬 4단계 미션 제출합니다. #61
base: lhc0312
Are you sure you want to change the base?
Conversation
test: 관련 테스트 코드 작성
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
음 테스트를 작성하면서 검증해보면 좋을 것 같아요.
해찬님께서 제시해주신 방향성이 여러개가 있지만, 일단 해보시는 걸 추천 드립니다.
그래야 왜 추천 안하고 추천하는지 알 수 있거든요.
저는 앞으로 정답을 바로 알려드리진 않을 거에요. 해찬이 직접해보고 경험해볼 수 있도록 방향만 잡아줄 뿐입니다.
아직 요구사항이 만족 안된부분도 있는 것 같아서 그런부분에 대해서 조금 더 작성해주시면 좋을 것 같아요.
이번 미션 수고하셨고, 다음 미션 혹은 일요일까지라도 1,2단계에서 못했던 부분들을 더 해보면 좋을 것 같아요
src/main/java/Car.java
Outdated
public class Car{ | ||
|
||
public int pos=0; | ||
public int carId; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
carId라는 필드를 추가한 이유가 있을까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Name을 생성해야 한다는 규칙을 못 봤습니다.. 그래서 키값으로 Name 대신 Id를 두었습니다.
src/main/java/Car.java
Outdated
|
||
public int pos=0; | ||
public int carId; | ||
public int rank; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
자동차는 이름을 가질 수 이썽야 하는데 그부분이 보이지가 않는 것 같아요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3단계에서 있는 항목이라 생각했는데 1단계부터 있었군요..
(체크리스트 작성의 필요성을 확실히 느꼈습니다..)
src/main/java/Car.java
Outdated
go(rand); | ||
} | ||
|
||
public void go(int num) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
go가 외부에서 호출되지 않을 것 같은 함수인데 그러면 private으로 바꾸는 건 어떨까요?
src/main/java/Car.java
Outdated
public int pos=0; | ||
public int carId; | ||
public int rank; | ||
private int RAND_MAX = 9; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
공통적으로 사용하는 변수는 어떻게 정의하면 좋을까요?
수업 때 언급드렸다 싶이 테스트 중에서 단위 테스트가 가장 기반이 되는 테스트입니다. 전체 상황에 대한 테스트가 필요하다 해서 단위 테스트가 무의미한 건 아니거든요. 이에 대해서 한번 생각해보시면 좋을 것 같아요.
이 부분에 대해서는 해찬님이 직접 고민해보셨으면 좋겠어요. 검색보단, private 테스트가 필요한가? 필요하다면 어떻게 테스트해야 하나? public에 대해서만 테스트해야하나? 직접 고민해보시고 일요일에 뵜으면 좋겠습니다.
lombok이 편하긴 하지만, lombok 없이 getter,setter를 추가하지 못해서는 안 될 것 같아요. 의존성을 추가하는 방법이든 다른 방법이든 잘 해보면 좋을 것 같네요. 그리고 꼭 필요한 기능이 아니라면 의존성을 추가하는 것을 지양하길 바랍니다. 많은데에다 의존하는건 좋은 건 아니에요 |
(2024.10.6 과제)
테스트 코드 작성하는데서 어려움을 느꼈습니다. 단위 테스트 보단 전체 상황에 대한 테스트를 해야 했기에 어떤 맥락으로 코드를 작성해야 할지 고민했던 것 같습니다.
궁금한 점은 다음과 같습니다.
test에서 private인 메소드 접근
단위 테스트를 실행하게 되면 private 메소드에 접근은 필수적이라 생각하는데 찾아보니 private 메소드의 테스트는 지양한다는 말과, 접근이 가능하나 복잡한 예제가 보였습니다. test는 되도록 public 메소드에 관해 진행해야 하는지 알고 싶습니다.
lombok 의존성 추가 안됨
getter와 setter 사용을 위해 lombok 의존성을 추가하러 했으나 실패했습니다.. 전에는 start.spring.io 를 이용해 lombok을 추가할 땐 문제 없었는데 직접 추가하러니 버벅거렸었습니다.. 이건 질문이라기 보단, 아직 의존성 추가방법을 잘 못 찾은 것 같아 더욱 찾아볼 예정입니다.
(2024.10.13 과제)
README 파일을 거의 일기장처럼 작성해버렸는데, 궁금했던 점이나 막혔던 점들이 적혀있습니다.
대부분 해답을 알게 되었거나, 일단 옳다고 생각하는 방향으로 코드를 작성했습니다.
한가지 아직도 헷갈리는 것은 예외처리는 Service에서 처리를 해야하는지, InputView에서 Vaildator로 넘겨 처리를 해야하는지 궁금합니다..
스터디 진행하면서 체크해보니,