Conversation
|
안녕하세요 |
|
|
||
| @LastModifiedDate | ||
| private LocalDateTime modifiedAt; | ||
| } |
There was a problem hiding this comment.
작성자와 수정자를 같이 넣어 주시면 요구사항을 만족하고 중복 코드를 줄일 수 있습니다.
| @GeneratedValue(strategy = GenerationType.AUTO) | ||
| @Id | ||
| private Long id; | ||
|
|
There was a problem hiding this comment.
GenerationType을 Auto로 설정한 이유가 있으실까요?
id의 경우는 지금은 Long 으로 설정되어 있고 ddl설정도 auto라서 이대로 실행시에 bigint로 컬럼이 지정될텐데요.
id는 음수값이 들어가면 안되므로 unsigned bigint로 설정 되도록 조정하시는 것이 좋을 것 같습니다.
| public List<Movie> getMovies() { | ||
| return movieRepository.findAll(); | ||
| } | ||
|
|
There was a problem hiding this comment.
1주차에는 상영중인 영화만을 리턴하는 것이 요구사항이었습니다. 이 부분은 구현 중이실까요?
api는 Restful 하게 구현하시려는 의도가 보입니다! 고생하셨습니다.
|
|
||
| @Column | ||
| private List<String> screeningSchedule; | ||
|
|
There was a problem hiding this comment.
상영 스케줄은 따로 테이블로 빼서 관리하시는 것이 좋을 것으로 보입니다.
상영 스케줄에 시작시간과 종료 시간같은 정보가 같이 들어가야 합니다.
위의 상영관을 따로 테이블로 빼고 상영관관 상영 스케줄을 연결 시키는 것은 어떨까요?
|
|
||
| @Column(nullable = false) | ||
| private String releaseDate; | ||
|
|
There was a problem hiding this comment.
date는 타입을 LocalDate로 지정하시면 관리가 수월 할 것 같습니다.
|
|
||
| @Column(nullable = false) | ||
| private String runningTime; | ||
|
|
There was a problem hiding this comment.
러닝 타임은 분으로 저장하므로 int로 설정해 보세요!
좋았던 점
아쉬운 점
추가 리뷰 포인트
|
3 participants
[1주차] 영화 목록 조회 API
작업 내용
발생했던 문제와 해결 과정을 남겨 주세요.
문제1
이번 주차에서 고민되었던 지점이나, 어려웠던 점을 알려 주세요.
리뷰 포인트
기타 질문