refactor: MovieDTO 추가

app/src/main/java/woowacourse/movie/activity/moviedetail/MovieDetailActivity.kt

@@ -7,27 +7,27 @@ import android.widget.TextView
 import android.widget.Toast
 import androidx.appcompat.app.AppCompatActivity
 import woowacourse.movie.R
-import woowacourse.movie.model.Movie
+import woowacourse.movie.model.MovieDTO
 import woowacourse.movie.util.Keys
 import woowacourse.movie.util.getVersionDependentSerializableExtra
 
 class MovieDetailActivity : AppCompatActivity() {
  override fun onCreate(savedInstanceState: Bundle?) {
  super.onCreate(savedInstanceState)
  setContentView(R.layout.activity_movie_detail)
- val movie: Movie? = intent.getVersionDependentSerializableExtra(Keys.MOVIE_KEY)
- if (movie != null) {
- initMovieDetailView(movie)
- initReservationInfoView(savedInstanceState, movie)
+ val movieDTO: MovieDTO? = intent.getVersionDependentSerializableExtra(Keys.MOVIE_KEY)
+ if (movieDTO != null) {
+ initMovieDetailView(movieDTO)
+ initReservationInfoView(savedInstanceState, movieDTO)
  } else {
  Toast.makeText(this, DATA_LOADING_ERROR_MESSAGE, Toast.LENGTH_LONG).show()
  finish()
  }
  supportActionBar?.setDisplayHomeAsUpEnabled(true)
  }
 
- private fun initMovieDetailView(movie: Movie) {
- MovieDetailViewInitializer(movie).run {
+ private fun initMovieDetailView(movieDTO: MovieDTO) {
+ MovieDetailViewInitializer(movieDTO).run {
  initImageView(findViewById(R.id.img_movie))
  initTitle(findViewById(R.id.text_title))
  initPlayingDate(findViewById(R.id.text_playing_date))
@@ -36,12 +36,12 @@ class MovieDetailActivity : AppCompatActivity() {
  }
  }
 
- private fun initReservationInfoView(savedInstanceState: Bundle?, movie: Movie) {
+ private fun initReservationInfoView(savedInstanceState: Bundle?, movieDTO: MovieDTO) {
  val savedCount = savedInstanceState?.getInt(Keys.COUNT_KEY) ?: DEFAULT_COUNT
  val savedDate = savedInstanceState?.getInt(Keys.SPINNER_DATE_KEY) ?: DEFAULT_POSITION
  val savedTime = savedInstanceState?.getInt(Keys.SPINNER_TIME_KEY) ?: DEFAULT_POSITION
 
- ReservationInfoViewInitializer(movie).run {
+ ReservationInfoViewInitializer(movieDTO).run {
  initCount(savedCount, findViewById(R.id.text_count))
  initMinusButton(findViewById(R.id.btn_minus), findViewById(R.id.text_count))
  initPlusButton(findViewById(R.id.btn_plus), findViewById(R.id.text_count))

app/src/main/java/woowacourse/movie/activity/moviedetail/MovieDetailViewInitializer.kt

@@ -3,32 +3,32 @@ package woowacourse.movie.activity.moviedetail
 import android.widget.ImageView
 import android.widget.TextView
 import woowacourse.movie.R
-import woowacourse.movie.model.Movie
+import woowacourse.movie.model.MovieDTO
 import java.time.format.DateTimeFormatter
 
-class MovieDetailViewInitializer(private val movie: Movie) {
+class MovieDetailViewInitializer(private val movieDTO: MovieDTO) {
 
  fun initDescription(textView: TextView) {
- textView.text = movie.description
+ textView.text = movieDTO.description
  }
 
  fun initRunningTime(textView: TextView) {
- textView.text = textView.context.getString(R.string.running_time, movie.runningTime)
+ textView.text = textView.context.getString(R.string.running_time, movieDTO.runningTime)
  }
 
  fun initPlayingDate(textView: TextView) {
  textView.text = textView.context.getString(
  R.string.playing_time,
- DateTimeFormatter.ofPattern(textView.context.getString(R.string.date_format)).format(movie.playingTimes.startDate),
- DateTimeFormatter.ofPattern(textView.context.getString(R.string.date_format)).format(movie.playingTimes.endDate)
+ DateTimeFormatter.ofPattern(textView.context.getString(R.string.date_format)).format(movieDTO.playingTimes.startDate),
+ DateTimeFormatter.ofPattern(textView.context.getString(R.string.date_format)).format(movieDTO.playingTimes.endDate)
  )
  }
 
  fun initTitle(textView: TextView) {
- textView.text = movie.title
+ textView.text = movieDTO.title
  }
 
  fun initImageView(imageView: ImageView) {
- imageView.setImageResource(movie.image)
+ imageView.setImageResource(movieDTO.image)
  }
 }

app/src/main/java/woowacourse/movie/activity/moviedetail/ReservationInfoViewInitializer.kt

@@ -6,21 +6,21 @@ import android.widget.Button
 import android.widget.Spinner
 import android.widget.TextView
 import woowacourse.movie.activity.ticketresult.TicketResultActivity
-import woowacourse.movie.model.Movie
+import woowacourse.movie.model.MovieDTO
 import woowacourse.movie.model.Price
 import woowacourse.movie.model.TicketingInfo
 import woowacourse.movie.util.DiscountPolicies
 import woowacourse.movie.util.Keys
 import java.time.LocalDate
 import java.time.LocalTime
 
-class ReservationInfoViewInitializer(private val movie: Movie) {
+class ReservationInfoViewInitializer(private val movieDTO: MovieDTO) {
  fun initReserveButton(reserveButton: Button, countView: TextView, dateSpinner: Spinner, timeSpinner: Spinner) {
  reserveButton.setOnClickListener {
  val intent = Intent(it.context, TicketResultActivity::class.java)
  val ticketingInfo = TicketingInfo.of(
  DiscountPolicies.policies,
- movie.title,
+ movieDTO.title,
  dateSpinner.selectedItem as LocalDate,
  timeSpinner.selectedItem as LocalTime,
  countView.text.toString().toInt(),
@@ -33,7 +33,7 @@ class ReservationInfoViewInitializer(private val movie: Movie) {
  }
 
  fun initTimeSpinner(savedTimePosition: Int, dateSpinner: Spinner, timeSpinner: Spinner) {
- val times = movie.playingTimes.times[dateSpinner.selectedItem] ?: emptyList()
+ val times = movieDTO.playingTimes.times[dateSpinner.selectedItem] ?: emptyList()
  timeSpinner.adapter =
  ArrayAdapter(timeSpinner.context, android.R.layout.simple_spinner_item, times).apply {
  setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item)
@@ -42,13 +42,13 @@ class ReservationInfoViewInitializer(private val movie: Movie) {
  }
 
  fun initDateSpinner(savedDatePosition: Int, dateSpinner: Spinner, timeSpinner: Spinner) {
- val dates = movie.playingTimes.times.keys.sorted()
+ val dates = movieDTO.playingTimes.times.keys.sorted()
  dateSpinner.adapter =
  ArrayAdapter(dateSpinner.context, android.R.layout.simple_spinner_item, dates).apply {
  setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item)
  }
  dateSpinner.setSelection(savedDatePosition, false)
- dateSpinner.onItemSelectedListener = DateSpinnerListener(movie.playingTimes, dates, timeSpinner)
+ dateSpinner.onItemSelectedListener = DateSpinnerListener(movieDTO.playingTimes, dates, timeSpinner)
  }
 
  fun initMinusButton(minusButton: Button, countView: TextView) {

app/src/main/java/woowacourse/movie/activity/movielist/MovieListActivity.kt

@@ -12,7 +12,7 @@ class MovieListActivity : AppCompatActivity() {
  setContentView(R.layout.activity_movie_list)
 
  val listView = findViewById<ListView>(R.id.list_view)
- val adapter = MovieListAdapter(DummyData.movies)
+ val adapter = MovieListAdapter(DummyData.movieDTOS)
  listView.adapter = adapter
  }
 }

app/src/main/java/woowacourse/movie/activity/movielist/MovieListAdapter.kt

@@ -12,17 +12,17 @@ import android.widget.ImageView
 import android.widget.TextView
 import woowacourse.movie.R
 import woowacourse.movie.activity.moviedetail.MovieDetailActivity
-import woowacourse.movie.model.Movie
+import woowacourse.movie.model.MovieDTO
 import woowacourse.movie.util.Keys
 import java.time.format.DateTimeFormatter
 
-class MovieListAdapter(private val movies: List<Movie>) : BaseAdapter() {
+class MovieListAdapter(private val movieDTOS: List<MovieDTO>) : BaseAdapter() {
  override fun getCount(): Int {
- return movies.size
+ return movieDTOS.size
  }
 
  override fun getItem(position: Int): Any {
- return movies[position]
+ return movieDTOS[position]
  }
 
  override fun getItemId(position: Int): Long {
@@ -34,10 +34,10 @@ class MovieListAdapter(private val movies: List<Movie>) : BaseAdapter() {
  if (convertView == null) view.tag = getViewHolder(view)
 
  val holder = view.tag as ViewHolder
- val movie = getItem(position) as Movie
- setViewHolder(holder, movie, parent?.context) {
+ val movieDTO = getItem(position) as MovieDTO
+ setViewHolder(holder, movieDTO, parent?.context) {
  val intent = Intent(view.context, MovieDetailActivity::class.java)
- intent.putExtra(Keys.MOVIE_KEY, movie)
+ intent.putExtra(Keys.MOVIE_KEY, movieDTO)
  view.context.startActivity(intent)
  }
  return view
@@ -51,15 +51,15 @@ class MovieListAdapter(private val movies: List<Movie>) : BaseAdapter() {
  view.findViewById(R.id.btn_reserve)
  )
 
- private fun setViewHolder(holder: ViewHolder, movie: Movie, context: Context?, clickListener: OnClickListener) {
- holder.image.setImageResource(movie.image)
- holder.title.text = movie.title
+ private fun setViewHolder(holder: ViewHolder, movieDTO: MovieDTO, context: Context?, clickListener: OnClickListener) {
+ holder.image.setImageResource(movieDTO.image)
+ holder.title.text = movieDTO.title
  holder.playingDate.text = context?.getString(
  R.string.playing_time,
- DateTimeFormatter.ofPattern(context.getString(R.string.date_format)).format(movie.playingTimes.startDate),
- DateTimeFormatter.ofPattern(context.getString(R.string.date_format)).format(movie.playingTimes.endDate)
+ DateTimeFormatter.ofPattern(context.getString(R.string.date_format)).format(movieDTO.playingTimes.startDate),
+ DateTimeFormatter.ofPattern(context.getString(R.string.date_format)).format(movieDTO.playingTimes.endDate)
  )
- holder.runningTime.text = context?.getString(R.string.running_time, movie.runningTime)
+ holder.runningTime.text = context?.getString(R.string.running_time, movieDTO.runningTime)
  holder.reserveButton.setOnClickListener(clickListener)
  }
 

app/src/main/java/woowacourse/movie/model/Movie.kt

@@ -1,9 +1,8 @@
 package woowacourse.movie.model
 
 data class Movie(
- val image: Int,
  val title: String,
  val playingTimes: PlayingTimes,
  val runningTime: Int,
  val description: String
-) : java.io.Serializable
+)

app/src/main/java/woowacourse/movie/model/MovieDTO.kt

@@ -0,0 +1,11 @@
+package woowacourse.movie.model
+
+import androidx.annotation.DrawableRes
+
+data class MovieDTO(
+ @DrawableRes val image: Int,
+ val title: String,
+ val playingTimes: PlayingTimes,
+ val runningTime: Int,
+ val description: String
+) : java.io.Serializable

app/src/main/java/woowacourse/movie/util/DummyData.kt

@@ -1,69 +1,69 @@
 package woowacourse.movie.util
 
 import woowacourse.movie.R
-import woowacourse.movie.model.Movie
+import woowacourse.movie.model.MovieDTO
 import woowacourse.movie.model.PlayingTimes
 import java.time.LocalDate
 
 object DummyData {
- val movies = listOf(
- Movie(
+ val movieDTOS = listOf(
+ MovieDTO(
  R.drawable.img,
  "해리포터와 마법사의 돌",
  PlayingTimes(LocalDate.of(2023, 3, 1), LocalDate.of(2023, 3, 31)),
  152,
  "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다."
  ),
- Movie(
+ MovieDTO(
  R.drawable.img,
  "해리포터와 마법사의 돌",
  PlayingTimes(LocalDate.of(2023, 3, 1), LocalDate.of(2023, 3, 31)),
  152,
  "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다."
  ),
- Movie(
+ MovieDTO(
  R.drawable.img,
  "해리포터와 마법사의 돌",
  PlayingTimes(LocalDate.of(2023, 3, 1), LocalDate.of(2023, 3, 31)),
  152,
  "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다."
  ),
- Movie(
+ MovieDTO(
  R.drawable.img,
  "해리포터와 마법사의 돌",
  PlayingTimes(LocalDate.of(2023, 3, 1), LocalDate.of(2023, 3, 31)),
  152,
  "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다."
  ),
- Movie(
+ MovieDTO(
  R.drawable.img,
  "해리포터와 마법사의 돌",
  PlayingTimes(LocalDate.of(2023, 3, 1), LocalDate.of(2023, 3, 31)),
  152,
  "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다."
  ),
- Movie(
+ MovieDTO(
  R.drawable.img,
  "해리포터와 마법사의 돌",
  PlayingTimes(LocalDate.of(2023, 3, 1), LocalDate.of(2023, 3, 31)),
  152,
  "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다."
  ),
- Movie(
+ MovieDTO(
  R.drawable.img,
  "해리포터와 마법사의 돌",
  PlayingTimes(LocalDate.of(2023, 3, 1), LocalDate.of(2023, 3, 31)),
  152,
  "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다."
  ),
- Movie(
+ MovieDTO(
  R.drawable.img,
  "해리포터와 마법사의 돌",
  PlayingTimes(LocalDate.of(2023, 3, 1), LocalDate.of(2023, 3, 31)),
  152,
  "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다."
  ),
- Movie(
+ MovieDTO(
  R.drawable.img,
  "해리포터와 마법사의 돌",
  PlayingTimes(LocalDate.of(2023, 3, 1), LocalDate.of(2023, 3, 31)),

app/src/main/java/woowacourse/movie/util/Mapper.kt

@@ -0,0 +1,13 @@
+package woowacourse.movie.util
+
+import androidx.annotation.DrawableRes
+import woowacourse.movie.model.Movie
+import woowacourse.movie.model.MovieDTO
+
+fun MovieDTO.toDomain(): Movie {
+ return Movie(title, playingTimes, runningTime, description)
+}
+
+fun Movie.toPresentation(@DrawableRes res: Int): MovieDTO {
+ return MovieDTO(res, title, playingTimes, runningTime, description)
+}