Merge branch 'develop' into feature/changeConstructor

README.md

@@ -37,7 +37,6 @@
 - [패키지 구조](https://github.com/f-lab-edu/Ticketing/wiki#-%ED%8C%A8%ED%82%A4%EC%A7%80-%EA%B5%AC%EC%A1%B0)
 - [Use Case](https://github.com/f-lab-edu/Ticketing/wiki/Use-Case)
 - [Prototype](https://github.com/f-lab-edu/Ticketing/wiki/Prototype)
-- [Issue Posting](https://github.com/f-lab-edu/Ticketing/wiki/Issue-Posting)
 
 <br/>
 

server/src/main/java/com/ticketing/server/global/exception/ErrorCode.java

@@ -28,15 +28,12 @@ public enum ErrorCode {
 	MOVIE_NOT_FOUND(NOT_FOUND, "해당 제목의 영화를 찾을 수 없습니다."),
 	REFRESH_TOKEN_NOT_FOUND(NOT_FOUND, "리프레쉬 토큰을 찾을 수 없습니다."),
 	PAYMENT_ID_NOT_FOUND(NOT_FOUND, "결제정보를 찾을 수 없습니다."),
-	THEATER_NOT_FOUND(NOT_FOUND, "상영관 정보를 찾을 수 없습니다."),
 
 	/* 409 CONFLICT : Resource 의 현재 상태와 충돌. 보통 중복된 데이터 존재 */
 	DUPLICATE_EMAIL(CONFLICT, "이메일이 이미 존재합니다."),
 	DUPLICATE_PAYMENT(CONFLICT, "해당 좌석은 현재 판매된 좌석입니다."),
 	DUPLICATE_MOVIE(CONFLICT, "해당 영화 정보가 이미 존재합니다."),
-	DUPLICATE_MOVIE_TIME(CONFLICT, "해당 영화 시간표 정보가 이미 존재합니다."),
-	DELETED_EMAIL(CONFLICT, "이미 삭제된 이메일 입니다."),
-	DELETED_MOVIE(CONFLICT, "이미 삭제된 영화 입니다.");
+	DELETED_EMAIL(CONFLICT, "이미 삭제된 이메일 입니다.");
 
 	private final HttpStatus httpStatus;
 	private final String detail;
@@ -84,10 +81,6 @@ public enum ErrorCode {
 		throw new TicketingException(PAYMENT_ID_NOT_FOUND);
 	}
 
-	public static TicketingException throwTheaterNotFound() {
-		throw new TicketingException(THEATER_NOT_FOUND);
-	}
-
 	/* 409 CONFLICT : Resource 의 현재 상태와 충돌. 보통 중복된 데이터 존재 */
 	public static TicketingException throwDuplicateEmail() {
 		throw new TicketingException(DUPLICATE_EMAIL);
@@ -101,16 +94,8 @@ public enum ErrorCode {
 		throw new TicketingException(DUPLICATE_MOVIE);
 	}
 
-	public static TicketingException throwDuplicateMovieTime() {
-		throw new TicketingException(DUPLICATE_MOVIE_TIME);
-	}
-
 	public static TicketingException throwDeletedEmail() {
 		throw new TicketingException(DELETED_EMAIL);
 	}
 
-	public static TicketingException throwDeletedMovie() {
-		throw new TicketingException(DELETED_MOVIE);
-	}
-
 }

server/src/main/java/com/ticketing/server/movie/application/MovieController.java

@@ -1,15 +1,8 @@
 package com.ticketing.server.movie.application;
 
-import static com.ticketing.server.user.domain.UserGrade.ROLES.STAFF;
-
-import com.ticketing.server.movie.application.request.MovieDeleteRequest;
 import com.ticketing.server.movie.application.request.MovieRegisterRequest;
-import com.ticketing.server.movie.application.response.MovieDeleteResponse;
 import com.ticketing.server.movie.application.response.MovieListResponse;
-import com.ticketing.server.movie.application.response.MovieInfoResponse;
-import com.ticketing.server.movie.service.dto.DeletedMovieDTO;
-import com.ticketing.server.movie.service.dto.MovieListDTO;
-import com.ticketing.server.movie.service.dto.RegisteredMovieDTO;
+import com.ticketing.server.movie.application.response.MovieTitleResponse;
 import com.ticketing.server.movie.service.interfaces.MovieService;
 import io.swagger.annotations.Api;
 import io.swagger.annotations.ApiOperation;
@@ -19,7 +12,6 @@ import lombok.extern.slf4j.Slf4j;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.security.access.annotation.Secured;
-import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.bind.annotation.RequestBody;
@@ -37,37 +29,25 @@ public class MovieController {
 
 	@PostMapping()
 	@ApiOperation(value = "영화 정보 등록")
-	@Secured(STAFF)
-	public ResponseEntity<MovieInfoResponse> registerMovie(@RequestBody @Valid MovieRegisterRequest request) {
-		RegisteredMovieDTO registeredMovieDto =
-			movieService.registerMovie(request.getTitle(), request.getRunningTime());
-
+	@Secured("ROLE_STAFF")
+	public ResponseEntity<MovieTitleResponse> registerMovie(@RequestBody @Valid MovieRegisterRequest request) {
 		return ResponseEntity.status(HttpStatus.OK)
 			.body(
-				registeredMovieDto.toResponse()
+				MovieTitleResponse.from(
+					movieService.registerMovie(request.toMovieRegisterDTO())
+				)
 			);
 	}
 
 	@GetMapping()
 	@ApiOperation(value = "영화 목록 조회")
 	public ResponseEntity<MovieListResponse> getMovies() {
-		MovieListDTO movieListDto = movieService.getMovies();
-
 		return ResponseEntity.status(HttpStatus.OK)
 			.body(
-				movieListDto.toResponse()
+				MovieListResponse.from(
+					movieService.getMovies()
+				)
 			);
 	}
 
-	@DeleteMapping()
-	@ApiOperation(value = "영화 정보 삭제")
-	@Secured(STAFF)
-	public ResponseEntity<MovieDeleteResponse> deleteMovie(@RequestBody @Valid MovieDeleteRequest request) {
-		DeletedMovieDTO deletedMovieDto = movieService.deleteMovie(request.getId());
-
-		return ResponseEntity.status(HttpStatus.OK)
-			.body(
-				deletedMovieDto.toResponse()
-			);
-	}
 }

server/src/main/java/com/ticketing/server/movie/application/MovieTimeController.java

@@ -1,29 +1,18 @@
 package com.ticketing.server.movie.application;
 
-import static com.ticketing.server.user.domain.UserGrade.ROLES.STAFF;
-
-import com.ticketing.server.movie.application.request.MovieTimeRegisterRequest;
-import com.ticketing.server.movie.application.response.MovieTimeInfoResponse;
 import com.ticketing.server.movie.application.response.MovieTimeListResponse;
-import com.ticketing.server.movie.service.dto.MovieTimeListDTO;
-import com.ticketing.server.movie.service.dto.RegisteredMovieTimeDTO;
 import com.ticketing.server.movie.service.interfaces.MovieTimeService;
 import io.swagger.annotations.Api;
 import io.swagger.annotations.ApiOperation;
 import io.swagger.annotations.ApiParam;
 import java.time.LocalDate;
-import javax.validation.Valid;
-import javax.validation.constraints.NotNull;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.format.annotation.DateTimeFormat;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
-import org.springframework.security.access.annotation.Secured;
 import org.springframework.validation.annotation.Validated;
 import org.springframework.web.bind.annotation.GetMapping;
-import org.springframework.web.bind.annotation.PostMapping;
-import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RequestParam;
 import org.springframework.web.bind.annotation.RestController;
@@ -35,35 +24,16 @@ import org.springframework.web.bind.annotation.RestController;
 @Slf4j
 public class MovieTimeController {
 
-	private final MovieTimeService movieTimeService;
+    private final MovieTimeService movieTimeService;
 
-	@PostMapping
-	@ApiOperation(value = "영화 시간표 등록")
-	@Secured(STAFF)
-	public ResponseEntity<MovieTimeInfoResponse> registerMovieTime(
-		@RequestBody @Valid MovieTimeRegisterRequest movieTimeRegisterRequest) {
-		RegisteredMovieTimeDTO registeredMovieTimeDto = movieTimeService.registerMovieTime(
-			movieTimeRegisterRequest.toMovieTimeRegisterDTO()
-		);
+    @GetMapping
+    @ApiOperation(value = "영화 시간표 조회")
+    @Validated
+    public ResponseEntity<MovieTimeListResponse> getMovieTimes(
+        @ApiParam(value = "영화 제목", required = true) @RequestParam String title,
+        @ApiParam(value = "상영 날짜", required = true) @RequestParam @DateTimeFormat(pattern = "yyyy-MM-dd") LocalDate runningDate) {
+        return ResponseEntity.status(HttpStatus.OK).body(MovieTimeListResponse.from(movieTimeService.getMovieTimes(title, runningDate)));
+    }
 
-		return ResponseEntity.status(HttpStatus.OK)
-			.body(
-				registeredMovieTimeDto.toResponse()
-			);
-	}
-
-	@GetMapping
-	@ApiOperation(value = "영화 시간표 조회")
-	@Validated
-	public ResponseEntity<MovieTimeListResponse> getMovieTimes(
-		@ApiParam(value = "영화 ID", required = true) @RequestParam @NotNull Long movieId,
-		@ApiParam(value = "상영 날짜", required = true) @RequestParam @DateTimeFormat(pattern = "yyyy-MM-dd") LocalDate runningDate) {
-		MovieTimeListDTO movieTimeListDto = movieTimeService.getMovieTimes(movieId, runningDate);
-
-		return ResponseEntity.status(HttpStatus.OK)
-			.body(
-				movieTimeListDto.toResponse()
-			);
-	}
 
 }

server/src/main/java/com/ticketing/server/movie/application/request/MovieDeleteRequest.java

@@ -1,16 +0,0 @@
-package com.ticketing.server.movie.application.request;
-
-import javax.validation.constraints.NotNull;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-import lombok.NoArgsConstructor;
-
-@Getter
-@NoArgsConstructor
-@AllArgsConstructor
-public class MovieDeleteRequest {
-
-	@NotNull(message = "{validation.not.null.id}")
-	private Long id;
-
-}

server/src/main/java/com/ticketing/server/movie/application/request/MovieRegisterRequest.java

@@ -5,8 +5,10 @@ import javax.validation.constraints.NotEmpty;
 import javax.validation.constraints.NotNull;
 import lombok.AllArgsConstructor;
 import lombok.Getter;
+import lombok.NoArgsConstructor;
 
 @Getter
+@NoArgsConstructor
 @AllArgsConstructor
 public class MovieRegisterRequest {
 
@@ -17,7 +19,7 @@ public class MovieRegisterRequest {
 	private Long runningTime;
 
 	public MovieRegisterDTO toMovieRegisterDTO() {
-		return new MovieRegisterDTO(title, runningTime);
+		return new MovieRegisterDTO(this.title, this.runningTime);
 	}
 
 }

server/src/main/java/com/ticketing/server/movie/application/request/MovieTimeRegisterRequest.java

@@ -1,29 +0,0 @@
-package com.ticketing.server.movie.application.request;
-
-import com.fasterxml.jackson.annotation.JsonFormat;
-import com.ticketing.server.movie.service.dto.MovieTimeRegisterDTO;
-import java.time.LocalDateTime;
-import javax.validation.constraints.NotNull;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-
-@Getter
-@AllArgsConstructor
-public class MovieTimeRegisterRequest {
-
-	@NotNull(message = "{validation.not.null.movieId}")
-	private Long movieId;
-
-	@NotNull(message = "{validation.not.null.theaterNumber}")
-	private Integer theaterNumber;
-
-	@NotNull(message = "{validation.not.null.round}")
-	private Integer round;
-
-	private LocalDateTime startAt;
-
-	public MovieTimeRegisterDTO toMovieTimeRegisterDTO() {
-		return new MovieTimeRegisterDTO(movieId, theaterNumber, round, startAt);
-	}
-
-}

server/src/main/java/com/ticketing/server/movie/application/response/MovieDeleteResponse.java

@@ -1,17 +0,0 @@
-package com.ticketing.server.movie.application.response;
-
-import java.time.LocalDateTime;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-
-@Getter
-@AllArgsConstructor
-public class MovieDeleteResponse {
-
-	private final Long id;
-
-	private final String title;
-
-	private final LocalDateTime deletedAt;
-
-}

server/src/main/java/com/ticketing/server/movie/application/response/MovieInfoResponse.java

@@ -1,17 +0,0 @@
-package com.ticketing.server.movie.application.response;
-
-import io.swagger.annotations.ApiModelProperty;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-
-@Getter
-@AllArgsConstructor
-public class MovieInfoResponse {
-
-	@ApiModelProperty(value = "영화 ID")
-	private Long movieId;
-
-	@ApiModelProperty(value = "영화 제목")
-	private String title;
-
-}

server/src/main/java/com/ticketing/server/movie/application/response/MovieListResponse.java

@@ -3,14 +3,19 @@ package com.ticketing.server.movie.application.response;
 import com.ticketing.server.movie.service.dto.MovieDTO;
 import io.swagger.annotations.ApiModelProperty;
 import java.util.List;
+import lombok.AccessLevel;
 import lombok.AllArgsConstructor;
 import lombok.Getter;
 
 @Getter
-@AllArgsConstructor
+@AllArgsConstructor(access = AccessLevel.PRIVATE)
 public class MovieListResponse {
 
     @ApiModelProperty(value = "영화 제목")
     private List<MovieDTO> movieDtos;
 
+    public static MovieListResponse from(List<MovieDTO> movieDtos) {
+        return new MovieListResponse(movieDtos);
+    }
+
 }

server/src/main/java/com/ticketing/server/movie/application/response/MovieTimeInfoResponse.java

@@ -1,28 +0,0 @@
-package com.ticketing.server.movie.application.response;
-
-import io.swagger.annotations.ApiModelProperty;
-import java.time.LocalDateTime;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-
-@Getter
-@AllArgsConstructor
-public class MovieTimeInfoResponse {
-
-	@ApiModelProperty(value = "영화 시간표 ID")
-	private Long movieTimeId;
-
-	@ApiModelProperty(value = "상영관 번호")
-	private Integer theaterNumber;
-
-	@ApiModelProperty(value = "회차")
-	private Integer round;
-
-	@ApiModelProperty(value = "시작 시간")
-	private LocalDateTime startAt;
-
-	@ApiModelProperty(value = "종료 시간")
-	private LocalDateTime endAt;
-
-
-}

server/src/main/java/com/ticketing/server/movie/application/response/MovieTimeListResponse.java

@@ -8,10 +8,14 @@ import lombok.AllArgsConstructor;
 import lombok.Getter;
 
 @Getter
-@AllArgsConstructor
+@AllArgsConstructor(access = AccessLevel.PRIVATE)
 public class MovieTimeListResponse {
 
     @ApiModelProperty(value = "영화 시간표 정보")
-    private List<MovieTimeDTO> movieTimeDtos;
+    private List<MovieTimeDTO> movieTimeDTOS;
+
+    public static MovieTimeListResponse from(List<MovieTimeDTO> movieTimeDtos) {
+        return new MovieTimeListResponse(movieTimeDtos);
+    }
 
 }

server/src/main/java/com/ticketing/server/movie/application/response/MovieTitleResponse.java

@@ -0,0 +1,20 @@
+package com.ticketing.server.movie.application.response;
+
+import com.ticketing.server.movie.service.dto.MovieDTO;
+import io.swagger.annotations.ApiModelProperty;
+import lombok.AccessLevel;
+import lombok.AllArgsConstructor;
+import lombok.Getter;
+
+@Getter
+@AllArgsConstructor(access = AccessLevel.PRIVATE)
+public class MovieTitleResponse {
+
+	@ApiModelProperty(value = "영화 제목")
+	private String title;
+
+	public static MovieTitleResponse from(MovieDTO movieDto) {
+		return new MovieTitleResponse(movieDto.getTitle());
+	}
+
+}

server/src/main/java/com/ticketing/server/movie/domain/Movie.java

@@ -1,9 +1,7 @@
 package com.ticketing.server.movie.domain;
 
 import com.ticketing.server.global.dto.repository.AbstractEntity;
-import com.ticketing.server.global.exception.ErrorCode;
-import com.ticketing.server.movie.service.dto.MovieDTO;
-import java.time.LocalDateTime;
+import javax.persistence.Column;
 import javax.persistence.Entity;
 import javax.validation.constraints.NotNull;
 import lombok.AccessLevel;
@@ -18,6 +16,7 @@ import lombok.NoArgsConstructor;
 public class Movie extends AbstractEntity {
 
 	@NotNull
+	@Column(unique = true)
 	private String title;
 
 	@NotNull
@@ -29,18 +28,4 @@ public class Movie extends AbstractEntity {
 		this.runningTime = runningTime;
 	}
 
-	public Movie delete() {
-		if (deletedAt != null) {
-			throw ErrorCode.throwDeletedMovie();
-		}
-
-		deletedAt = LocalDateTime.now();
-
-		return this;
-	}
-
-	public MovieDTO toMovieDTO() {
-		return new MovieDTO(this.id, this.title);
-	}
-
 }

server/src/main/java/com/ticketing/server/movie/domain/MovieTime.java

@@ -1,8 +1,6 @@
 package com.ticketing.server.movie.domain;
 
-
 import com.ticketing.server.global.dto.repository.AbstractEntity;
-import com.ticketing.server.movie.service.dto.MovieTimeDTO;
 import java.time.LocalDateTime;
 import java.util.ArrayList;
 import java.util.List;
@@ -74,9 +72,4 @@ public class MovieTime extends AbstractEntity {
 		return this.theater.getSeats();
 	}
 
-	public MovieTimeDTO toMovieTimeDTO() {
-		return new MovieTimeDTO(
-			this.id, this.theater.getTheaterNumber(), this.round, this.startAt, this.endAt);
-	}
-
 }

server/src/main/java/com/ticketing/server/movie/domain/repository/MovieRepository.java

@@ -13,14 +13,6 @@ public interface MovieRepository extends JpaRepository<Movie, Long> {
 
     Optional<Movie> findByTitle(String title);
 
-	Optional<Movie> findByIdAndDeletedAtNull(Long id);
-
-	@Query(value = "SELECT m "
-		+ "FROM Movie m "
-		+ "WHERE title = :title "
-		+ "AND deleted_at IS NULL")
-	Optional<Movie> findValidMovieWithTitle(String title);
-
     @Query(value = "SELECT * "
         + "FROM movie "
         + "WHERE deleted_at IS NULL", nativeQuery = true)

server/src/main/java/com/ticketing/server/movie/domain/repository/MovieTimeRepository.java

@@ -2,10 +2,8 @@ package com.ticketing.server.movie.domain.repository;
 
 import com.ticketing.server.movie.domain.Movie;
 import com.ticketing.server.movie.domain.MovieTime;
-import com.ticketing.server.movie.domain.Theater;
 import java.time.LocalDateTime;
 import java.util.List;
-import java.util.Optional;
 import org.springframework.data.jpa.repository.JpaRepository;
 import org.springframework.data.jpa.repository.Query;
 import org.springframework.stereotype.Repository;
@@ -13,13 +11,11 @@ import org.springframework.stereotype.Repository;
 @Repository
 public interface MovieTimeRepository extends JpaRepository<MovieTime, Long> {
 
-	Optional<MovieTime> findByMovieAndTheaterAndRoundAndDeletedAtNull(Movie movie, Theater theater, Integer round);
-
-	@Query(value = "SELECT mt "
-		+ "FROM MovieTime mt " 
-		+ "JOIN FETCH mt.movie "
-		+ "WHERE mt.movie = :movie "
-		+ "AND mt.startAt BETWEEN :startOfDay AND :endOfDay ")
-	List<MovieTime> findValidMovieTimes(Movie movie, LocalDateTime startOfDay, LocalDateTime endOfDay);
+    @Query(value = "SELECT mt "
+        + "FROM MovieTime mt "
+        + "JOIN FETCH mt.movie "
+        + "WHERE mt.movie = :movie "
+        + "AND mt.startAt BETWEEN :startOfDay AND :endOfDay ")
+    List<MovieTime> findValidMovieTimes(Movie movie, LocalDateTime startOfDay, LocalDateTime endOfDay);
 
 }

server/src/main/java/com/ticketing/server/movie/service/MovieServiceImpl.java

@@ -3,15 +3,12 @@ package com.ticketing.server.movie.service;
 import com.ticketing.server.global.exception.ErrorCode;
 import com.ticketing.server.movie.domain.Movie;
 import com.ticketing.server.movie.domain.repository.MovieRepository;
-import com.ticketing.server.movie.service.dto.DeletedMovieDTO;
 import com.ticketing.server.movie.service.dto.MovieDTO;
-import com.ticketing.server.movie.service.dto.MovieListDTO;
-import com.ticketing.server.movie.service.dto.RegisteredMovieDTO;
+import com.ticketing.server.movie.service.dto.MovieRegisterDTO;
 import com.ticketing.server.movie.service.interfaces.MovieService;
 import java.util.List;
 import java.util.Optional;
 import java.util.stream.Collectors;
-import javax.transaction.Transactional;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.stereotype.Service;
@@ -24,39 +21,24 @@ public class MovieServiceImpl implements MovieService {
 	private final MovieRepository movieRepository;
 
 	@Override
-	public RegisteredMovieDTO registerMovie(String title, Long runningTime) {
-		Optional<Movie> movie = movieRepository.findValidMovieWithTitle(title);
+	public MovieDTO registerMovie(MovieRegisterDTO movieRegisterDto) {
+		Optional<Movie> movie = movieRepository.findByTitle(movieRegisterDto.getTitle());
 
 		if(movie.isEmpty()) {
-			Movie newMovie = movieRepository.save(
-				new Movie(title, runningTime)
-			);
-
-			return new RegisteredMovieDTO(newMovie);
+			return MovieDTO.from(movieRepository.save(movieRegisterDto.toMovie()));
 		}
 
 		throw ErrorCode.throwDuplicateMovie();
 	}
 
 	@Override
-	public MovieListDTO getMovies() {
+	public List<MovieDTO> getMovies() {
 		List<Movie> movies = movieRepository.findValidMovies();
 
-		List<MovieDTO> movieDtos = movies.stream()
-			.map(movie -> movie.toMovieDTO())
+		return movies.stream()
+			.map(MovieDTO::from)
 			.collect(Collectors.toList());
 
-		return new MovieListDTO(movieDtos);
 	}
 
-	@Override
-	@Transactional
-	public DeletedMovieDTO deleteMovie(Long id) {
-		Movie movie = movieRepository.findByIdAndDeletedAtNull(id)
-			.orElseThrow(ErrorCode::throwMovieNotFound);
-
-		Movie deletedMovie = movie.delete();
-
-		return new DeletedMovieDTO(deletedMovie);
-	}
 }

server/src/main/java/com/ticketing/server/movie/service/MovieTimeServiceImpl.java

@@ -3,22 +3,14 @@ package com.ticketing.server.movie.service;
 import com.ticketing.server.global.exception.ErrorCode;
 import com.ticketing.server.movie.domain.Movie;
 import com.ticketing.server.movie.domain.MovieTime;
-import com.ticketing.server.movie.domain.Theater;
 import com.ticketing.server.movie.domain.repository.MovieRepository;
 import com.ticketing.server.movie.domain.repository.MovieTimeRepository;
-import com.ticketing.server.movie.domain.repository.TheaterRepository;
 import com.ticketing.server.movie.service.dto.MovieTimeDTO;
-import com.ticketing.server.movie.service.dto.MovieTimeListDTO;
-import com.ticketing.server.movie.service.dto.MovieTimeRegisterDTO;
-import com.ticketing.server.movie.service.dto.RegisteredMovieDTO;
-import com.ticketing.server.movie.service.dto.RegisteredMovieTimeDTO;
 import com.ticketing.server.movie.service.interfaces.MovieTimeService;
 import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.util.List;
-import java.util.Optional;
 import java.util.stream.Collectors;
-import javax.validation.Valid;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.stereotype.Service;
@@ -30,60 +22,22 @@ public class MovieTimeServiceImpl implements MovieTimeService {
 
 	private final MovieRepository movieRepository;
 
-	private final TheaterRepository theaterRepository;
-
 	private final MovieTimeRepository movieTimeRepository;
 
 	@Override
-	public RegisteredMovieTimeDTO registerMovieTime(@Valid MovieTimeRegisterDTO movieTimeRegisterDto) {
-		Movie movie = findMovieById(movieTimeRegisterDto.getMovieId());
-		Theater theater = findTheaterByNumber(movieTimeRegisterDto.getTheaterNumber());
-		int round = movieTimeRegisterDto.getRound();
-
-		Optional<MovieTime> movieTime =
-			movieTimeRepository.findByMovieAndTheaterAndRoundAndDeletedAtNull(movie, theater, round);
-
-		if(movieTime.isEmpty()) {
-			MovieTime newMovieTime = movieTimeRepository.save(
-				new MovieTime(movie, theater, round, movieTimeRegisterDto.getStartAt())
-			);
-
-			return new RegisteredMovieTimeDTO(newMovieTime);
-		}
-
-		throw ErrorCode.throwDuplicateMovieTime();
-	}
-
-	@Override
-	public MovieTimeListDTO getMovieTimes(Long movieId, LocalDate runningDate) {
-		Movie movie = findMovieById(movieId);
+	public List<MovieTimeDTO> getMovieTimes(String title, LocalDate runningDate) {
+		Movie movie = movieRepository.findByTitle(title)
+			.orElseThrow(ErrorCode::throwMovieNotFound);
 
 		LocalDateTime startOfDay = runningDate.atStartOfDay().plusHours(6);
 		LocalDateTime endOfDay = startOfDay.plusDays(1);
 
 		List<MovieTime> movieTimes = movieTimeRepository.findValidMovieTimes(movie, startOfDay, endOfDay);
 
-		List<MovieTimeDTO> movieTimeDtos = movieTimes.stream()
-			.map(movieTime -> movieTime.toMovieTimeDTO())
+		return movieTimes.stream()
+			.map(MovieTimeDTO::from)
 			.collect(Collectors.toList());
 
-		return new MovieTimeListDTO(movieTimeDtos);
-	}
-
-	@Override
-	public Movie findMovieById(Long movieId) {
-		Movie movie = movieRepository.findByIdAndDeletedAtNull(movieId)
-			.orElseThrow(ErrorCode::throwMovieNotFound);
-
-		return movie;
-	}
-
-	@Override
-	public Theater findTheaterByNumber(Integer theaterNumber) {
-		Theater theater = theaterRepository.findByTheaterNumber(theaterNumber)
-			.orElseThrow(ErrorCode::throwTheaterNotFound);
-
-		return theater;
 	}
 
 }

server/src/main/java/com/ticketing/server/movie/service/dto/DeletedMovieDTO.java

@@ -1,32 +0,0 @@
-package com.ticketing.server.movie.service.dto;
-
-import com.ticketing.server.movie.application.response.MovieDeleteResponse;
-import com.ticketing.server.movie.domain.Movie;
-import java.time.LocalDateTime;
-import lombok.AccessLevel;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-
-@Getter
-@AllArgsConstructor(access = AccessLevel.PRIVATE)
-public class DeletedMovieDTO {
-
-	private final Long id;
-
-	private final String title;
-
-	private final LocalDateTime deletedAt;
-
-	public DeletedMovieDTO(Movie movie) {
-		this(
-			movie.getId(),
-			movie.getTitle(),
-			movie.getDeletedAt()
-		);
-	}
-
-	public MovieDeleteResponse toResponse() {
-		return new MovieDeleteResponse(id, title, deletedAt);
-	}
-
-}

server/src/main/java/com/ticketing/server/movie/service/dto/MovieDTO.java

@@ -1,14 +1,18 @@
 package com.ticketing.server.movie.service.dto;
 
+import com.ticketing.server.movie.domain.Movie;
+import lombok.AccessLevel;
 import lombok.AllArgsConstructor;
 import lombok.Getter;
 
 @Getter
-@AllArgsConstructor
+@AllArgsConstructor(access = AccessLevel.PRIVATE)
 public class MovieDTO {
 
-	private Long movieId;
-
     private String title;
 
+    public static MovieDTO from(Movie movie) {
+        return new MovieDTO(movie.getTitle());
+    }
+
 }

server/src/main/java/com/ticketing/server/movie/service/dto/MovieListDTO.java

@@ -1,18 +0,0 @@
-package com.ticketing.server.movie.service.dto;
-
-import com.ticketing.server.movie.application.response.MovieListResponse;
-import java.util.List;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-
-@Getter
-@AllArgsConstructor
-public class MovieListDTO {
-
-	private final List<MovieDTO> movieDtos;
-
-	public MovieListResponse toResponse() {
-		return new MovieListResponse(movieDtos);
-	}
-
-}

server/src/main/java/com/ticketing/server/movie/service/dto/MovieRegisterDTO.java

@@ -21,5 +21,4 @@ public class MovieRegisterDTO {
 	public Movie toMovie() {
 		return new Movie(this.title, this.runningTime);
 	}
-
 }

server/src/main/java/com/ticketing/server/movie/service/dto/MovieTimeDTO.java

@@ -1,11 +1,13 @@
 package com.ticketing.server.movie.service.dto;
 
+import com.ticketing.server.movie.domain.MovieTime;
 import java.time.LocalDateTime;
+import lombok.AccessLevel;
 import lombok.AllArgsConstructor;
 import lombok.Getter;
 
 @Getter
-@AllArgsConstructor
+@AllArgsConstructor(access = AccessLevel.PRIVATE)
 public class MovieTimeDTO {
 
     private Long movieTimeId;
@@ -18,4 +20,9 @@ public class MovieTimeDTO {
 
     private LocalDateTime endAt;
 
+    public static MovieTimeDTO from(MovieTime movieTime) {
+        return new MovieTimeDTO(movieTime.getId(), movieTime.getTheater().getTheaterNumber(),
+            movieTime.getRound(), movieTime.getStartAt(), movieTime.getEndAt());
+    }
+
 }

server/src/main/java/com/ticketing/server/movie/service/dto/MovieTimeListDTO.java

@@ -1,18 +0,0 @@
-package com.ticketing.server.movie.service.dto;
-
-import com.ticketing.server.movie.application.response.MovieTimeListResponse;
-import java.util.List;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-
-@Getter
-@AllArgsConstructor
-public class MovieTimeListDTO {
-
-	private final List<MovieTimeDTO> movieTimeDtos;
-
-	public MovieTimeListResponse toResponse() {
-		return new MovieTimeListResponse(movieTimeDtos);
-	}
-
-}

server/src/main/java/com/ticketing/server/movie/service/dto/MovieTimeRegisterDTO.java

@@ -1,25 +0,0 @@
-package com.ticketing.server.movie.service.dto;
-
-import com.fasterxml.jackson.annotation.JsonFormat;
-import java.time.LocalDateTime;
-import javax.validation.constraints.NotNull;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-
-@Getter
-@AllArgsConstructor
-public class MovieTimeRegisterDTO {
-
-	@NotNull(message = "{validation.not.null.movieId}")
-	private Long movieId;
-
-	@NotNull(message = "{validation.not.null.theaterNumber}")
-	private Integer theaterNumber;
-
-	@NotNull(message = "{validation.not.null.round}")
-	private Integer round;
-
-	@JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd HH:mm:ss", timezone = "Asia/Seoul")
-	private LocalDateTime startAt;
-
-}

server/src/main/java/com/ticketing/server/movie/service/dto/RegisteredMovieDTO.java

@@ -1,28 +0,0 @@
-package com.ticketing.server.movie.service.dto;
-
-import com.ticketing.server.movie.application.response.MovieInfoResponse;
-import com.ticketing.server.movie.domain.Movie;
-import lombok.AccessLevel;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-
-@Getter
-@AllArgsConstructor(access = AccessLevel.PRIVATE)
-public class RegisteredMovieDTO {
-
-	private final Long id;
-
-	private final String title;
-
-	public RegisteredMovieDTO(Movie movie) {
-		this(
-			movie.getId(),
-			movie.getTitle()
-		);
-	}
-
-	public MovieInfoResponse toResponse() {
-		return new MovieInfoResponse(id, title);
-	}
-
-}

server/src/main/java/com/ticketing/server/movie/service/dto/RegisteredMovieTimeDTO.java

@@ -1,38 +0,0 @@
-package com.ticketing.server.movie.service.dto;
-
-import com.ticketing.server.movie.application.response.MovieTimeInfoResponse;
-import com.ticketing.server.movie.domain.MovieTime;
-import java.time.LocalDateTime;
-import lombok.AccessLevel;
-import lombok.AllArgsConstructor;
-import lombok.Getter;
-
-@Getter
-@AllArgsConstructor(access = AccessLevel.PRIVATE)
-public class RegisteredMovieTimeDTO {
-
-	private final Long movieTimeId;
-
-	private final Integer theaterNumber;
-
-	private final Integer round;
-
-	private final LocalDateTime startAt;
-
-	private final LocalDateTime endAt;
-
-	public RegisteredMovieTimeDTO(MovieTime movieTime) {
-		this(
-			movieTime.getId(),
-			movieTime.getTheater().getTheaterNumber(),
-			movieTime.getRound(),
-			movieTime.getStartAt(),
-			movieTime.getEndAt()
-		);
-	}
-
-	public MovieTimeInfoResponse toResponse() {
-		return new MovieTimeInfoResponse(movieTimeId, theaterNumber, round, startAt, endAt);
-	}
-
-}

server/src/main/java/com/ticketing/server/movie/service/interfaces/MovieService.java

@@ -1,14 +1,13 @@
 package com.ticketing.server.movie.service.interfaces;
 
-import com.ticketing.server.movie.service.dto.DeletedMovieDTO;
-import com.ticketing.server.movie.service.dto.MovieListDTO;
-import com.ticketing.server.movie.service.dto.RegisteredMovieDTO;
+import com.ticketing.server.movie.service.dto.MovieDTO;
+import com.ticketing.server.movie.service.dto.MovieRegisterDTO;
+import java.util.List;
 
 public interface MovieService {
 
-	RegisteredMovieDTO registerMovie(String title, Long runningTime);
+	MovieDTO registerMovie(MovieRegisterDTO movieRegisterDto);
 
-	MovieListDTO getMovies();
+    List<MovieDTO> getMovies();
 
-	DeletedMovieDTO deleteMovie(Long id);
 }

server/src/main/java/com/ticketing/server/movie/service/interfaces/MovieTimeService.java

@@ -1,21 +1,11 @@
 package com.ticketing.server.movie.service.interfaces;
 
-import com.ticketing.server.movie.domain.Movie;
-import com.ticketing.server.movie.domain.Theater;
-import com.ticketing.server.movie.service.dto.MovieTimeListDTO;
-import com.ticketing.server.movie.service.dto.MovieTimeRegisterDTO;
-import com.ticketing.server.movie.service.dto.RegisteredMovieTimeDTO;
+import com.ticketing.server.movie.service.dto.MovieTimeDTO;
 import java.time.LocalDate;
-import javax.validation.Valid;
+import java.util.List;
 
 public interface MovieTimeService {
 
-	RegisteredMovieTimeDTO registerMovieTime(@Valid MovieTimeRegisterDTO movieTimeRegisterDto);
-
-	MovieTimeListDTO getMovieTimes(Long movieId, LocalDate runningDate);
-
-	Movie findMovieById(Long movieId);
-
-	Theater findTheaterByNumber(Integer theaterNumber);
+    List<MovieTimeDTO> getMovieTimes(String title, LocalDate runningDate);
 
 }

server/src/main/java/com/ticketing/server/user/application/UserController.java

@@ -12,12 +12,12 @@ import com.ticketing.server.user.application.response.SignUpResponse;
 import com.ticketing.server.user.application.response.UserChangePasswordResponse;
 import com.ticketing.server.user.application.response.UserDeleteResponse;
 import com.ticketing.server.user.application.response.UserDetailResponse;
-import com.ticketing.server.user.domain.ChangeGradeDTO;
-import com.ticketing.server.user.service.dto.ChangePasswordDTO;
+import com.ticketing.server.user.domain.UserGrade;
 import com.ticketing.server.user.service.dto.ChangedPasswordUserDTO;
-import com.ticketing.server.user.service.dto.CreatedUserDTO;
 import com.ticketing.server.user.service.dto.DeletedUserDTO;
 import com.ticketing.server.user.service.dto.PaymentsDTO;
+import com.ticketing.server.user.service.dto.ChangePasswordDTO;
+import com.ticketing.server.user.service.dto.CreatedUserDTO;
 import com.ticketing.server.user.service.dto.UserDetailDTO;
 import com.ticketing.server.user.service.interfaces.UserApisService;
 import com.ticketing.server.user.service.interfaces.UserService;
@@ -44,8 +44,8 @@ import org.springframework.web.bind.annotation.RestController;
 @Slf4j
 public class UserController {
 
-	private final UserService userService;
 	private final UserApisService userApisService;
+	private final UserService userService;
 	private final PasswordEncoder passwordEncoder;
 
 	@PostMapping
@@ -90,9 +90,7 @@ public class UserController {
 	@Secured(ADMIN)
 	public ResponseEntity<UserChangeGradeResponse> changeGrade(@RequestBody @Valid UserChangeGradeRequest request) {
 		ChangeGradeDTO changeGradeDto = userService.changeGrade(request.getEmail(), request.getAfterGrade());
-
-		return ResponseEntity.status(HttpStatus.OK)
-			.body(changeGradeDto.toResponse());
+		return ResponseEntity.status(HttpStatus.OK).body(changeGradeDto.toResponse());
 	}
 
 	@GetMapping("/payments")

server/src/main/java/com/ticketing/server/user/service/interfaces/UserService.java

@@ -2,7 +2,6 @@ package com.ticketing.server.user.service.interfaces;
 
 import com.ticketing.server.user.domain.ChangeGradeDTO;
 import com.ticketing.server.user.domain.User;
-import com.ticketing.server.user.domain.UserGrade;
 import com.ticketing.server.user.service.dto.ChangedPasswordUserDTO;
 import com.ticketing.server.user.service.dto.DeletedUserDTO;
 import com.ticketing.server.user.service.dto.ChangePasswordDTO;
@@ -12,6 +11,7 @@ import com.ticketing.server.user.service.dto.SignUpDTO;
 import com.ticketing.server.user.service.dto.UserDetailDTO;
 import javax.validation.Valid;
 import javax.validation.constraints.NotNull;
+import org.springframework.transaction.annotation.Transactional;
 
 public interface UserService {
 
@@ -21,6 +21,7 @@ public interface UserService {
 
 	ChangedPasswordUserDTO changePassword(@Valid ChangePasswordDTO changePasswordDto);
 
+	@Transactional
 	ChangeGradeDTO changeGrade(@NotNull String email, @NotNull UserGrade grade);
 
 	UserDetailDTO findDetailByEmail(@NotNull String email);

server/src/test/java/com/ticketing/server/movie/service/MovieServiceImplTest.java

@@ -6,7 +6,6 @@ import static org.mockito.Mockito.when;
 import com.ticketing.server.movie.domain.Movie;
 import com.ticketing.server.movie.domain.repository.MovieRepository;
 import com.ticketing.server.movie.service.dto.MovieDTO;
-import com.ticketing.server.movie.service.dto.MovieListDTO;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
@@ -39,10 +38,10 @@ public class MovieServiceImplTest {
             .thenReturn(Collections.emptyList());
 
         // when
-        MovieListDTO movieListDto = movieService.getMovies();
+        List<MovieDTO> movieDTOList = movieService.getMovies();
 
         // then
-        assertTrue(movieListDto.getMovieDtos().isEmpty());
+        assertTrue(movieDTOList.isEmpty());
     }
 
     @Test
@@ -50,16 +49,18 @@ public class MovieServiceImplTest {
     void shouldAbleToGetMovies() {
         // given
         movie = new Movie("범죄도시2", 106L);
+        movieDto = movieDto.from(movie);
         movies.add(movie);
+        movieDTOS.add(movieDto);
 
         when(movieRepository.findValidMovies())
             .thenReturn(movies);
 
         // when
-	    MovieListDTO movieListDto = movieService.getMovies();
+        List<MovieDTO> movieDTOList = movieService.getMovies();
 
         // then
-        assertTrue(!movieListDto.getMovieDtos().isEmpty());
+        assertTrue(!movieDTOList.isEmpty());
     }
 
 }

server/src/test/java/com/ticketing/server/movie/service/MovieTimeServiceImplTest.java

@@ -9,7 +9,7 @@ import com.ticketing.server.movie.domain.MovieTime;
 import com.ticketing.server.movie.domain.Theater;
 import com.ticketing.server.movie.domain.repository.MovieRepository;
 import com.ticketing.server.movie.domain.repository.MovieTimeRepository;
-import com.ticketing.server.movie.service.dto.MovieTimeListDTO;
+import com.ticketing.server.movie.service.dto.MovieTimeDTO;
 import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.util.ArrayList;
@@ -39,26 +39,26 @@ public class MovieTimeServiceImplTest {
     MovieTimeServiceImpl movieTimeService;
 
     @Test
-    @DisplayName("MovieTime Service Test - get empty list when there are no valid movie times")
+    @DisplayName("MovieTime Service Test - get empty list when there is no valid movie times")
     void shouldGetEmptyList() {
         // given
         Movie movie = new Movie(title, 106L);
 
-        when(movieRepository.findByIdAndDeletedAtNull(any()))
+        when(movieRepository.findByTitle(title))
             .thenReturn(Optional.of(movie));
 
         when(movieTimeRepository.findValidMovieTimes(any(), any(), any()))
             .thenReturn(Collections.emptyList());
 
         // when
-        MovieTimeListDTO movieTimeListDto = movieTimeService.getMovieTimes(any(), LocalDate.now());
+        List<MovieTimeDTO> movieTimeDTOList = movieTimeService.getMovieTimes(title, LocalDate.now());
 
         // then
-        assertTrue(movieTimeListDto.getMovieTimeDtos().isEmpty());
+        assertTrue(movieTimeDTOList.isEmpty());
     }
 
     @Test
-    @DisplayName("MovieTime Service Test - get list when there are valid movie times")
+    @DisplayName("MovieTime Service Test - get list when there is valid movie times")
     void shouldGetMovieTimeList() {
         // given
         Movie movie = new Movie(title, 106L);
@@ -69,17 +69,17 @@ public class MovieTimeServiceImplTest {
 
         movieTimes.add(movieTime);
 
-        when(movieRepository.findByIdAndDeletedAtNull(any()))
+        when(movieRepository.findByTitle(title))
             .thenReturn(Optional.of(movie));
 
         when(movieTimeRepository.findValidMovieTimes(any(), any(), any()))
             .thenReturn(movieTimes);
 
         // when
-	    MovieTimeListDTO movieTimeListDto = movieTimeService.getMovieTimes(any(), LocalDate.of(2022, 07, 01));
+        List<MovieTimeDTO> movieTimeDTOList = movieTimeService.getMovieTimes(title, LocalDate.of(2022, 07, 01));
 
         // then
-        assertTrue(!movieTimeListDto.getMovieTimeDtos().isEmpty());
+        assertTrue(!movieTimeDTOList.isEmpty());
     }
 
 }