-
Notifications
You must be signed in to change notification settings - Fork 1
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
[#15] 경기 목록 조회 API 구현 #17
Conversation
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.
- 테스트 시 실제 dto가 아닌 any()로 설정해야하는이유
- 현재는 테스트에 사용되는 메소드들이 object(=dto)를 전달받는데, 이 친구는 로직상에서 생성해서 전달하는 객체이고 그러면 로직에서 만든 dto는 내용은 같을지라도 객체상에선 다른 친구이니
any()
를 써야해요.
- 현재는 테스트에 사용되는 메소드들이 object(=dto)를 전달받는데, 이 친구는 로직상에서 생성해서 전달하는 객체이고 그러면 로직에서 만든 dto는 내용은 같을지라도 객체상에선 다른 친구이니
- ResponseEntity로 감싸야하는 이유
- 이건 spring이 가진 표준 포맷이기 때문에 쓰는거라..크게 중요한 이유는 없다고 생각됩니다.
- mybatis는 어떻게 매핑을 하는가
- 앗 요건 어떤 맵핑을 말씀하시는건지..?
- 스프링 빈은 어떻게 thread-safe한가
- 생성을 말씀하시는거면 double checked locking을 쓰구요, 사용하는걸 말씀하시는거면 객체 내에서 공용으로 쓰이는 변수가 아닌 이상은 전부 stack method에 생성되는 친구들이라 thread safe 합니다.
- dto 필드가 하나일때 역직렬화의 문제점
- 필드가 하나일때는 dto를 쓰기보단 그냥 String 같은걸로 받는게 더 일반적이긴 해요 🙂
this.month = month != null ? month : LocalDateTime.now() | ||
.getMonthValue(); |
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.
nit: 사소하지만 보통 ternary operator로 연산하는 케이스면 조건을 긍정 형태(month == null ? ...
)를 쓰는게 이해하는데 더 편합니다.
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.
넵!
관련 이슈
작업 내용
schedule controller
,service
성공 시 유닛테스트 추가참고사항
궁금한 점 or 공부할 내용 🤔
ResponseEntity
로 감싸야하는 이유