refactor(MovieListFragment): MovieListFragment 리팩터링

app/src/main/java/woowacourse/movie/data/MovieMockRepository.kt

@@ -1,47 +1,56 @@
 package woowacourse.movie.data
 
-import woowacourse.movie.R
 import woowacourse.movie.domain.movie.Minute
 import woowacourse.movie.domain.movie.Movie
+import woowacourse.movie.domain.movie.ScreeningDateTimes
 import woowacourse.movie.domain.repository.MovieRepository
 import java.time.LocalDate
+import java.time.LocalTime
 
 object MovieMockRepository : MovieRepository {
 
  private val movies = List(1000) {
  listOf(
  Movie(
  "해리 포터와 마법사의 돌",
- LocalDate.of(2024, 3, 1),
- LocalDate.of(2024, 3, 31),
+ ScreeningDateTimes.of(
+ LocalDate.of(2024, 3, 1),
+ LocalDate.of(2024, 3, 31),
+ listOf(LocalTime.of(12, 0), LocalTime.of(14, 0), LocalTime.of(16, 0)),
+ ),
  Minute(152),
- R.drawable.harry_potter1_poster,
- "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다."
+ "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다.",
  ),
  Movie(
  "해리 포터와 비밀의 방",
- LocalDate.of(2024, 4, 1),
- LocalDate.of(2024, 4, 28),
+ ScreeningDateTimes.of(
+ LocalDate.of(2024, 4, 1),
+ LocalDate.of(2024, 4, 28),
+ listOf(LocalTime.of(14, 0), LocalTime.of(16, 0), LocalTime.of(18, 0)),
+ ),
  Minute(162),
- R.drawable.harry_potter2_poster,
- "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다."
+ "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다.",
  ),
  Movie(
  "해리 포터와 아즈카반의 죄수",
- LocalDate.of(2024, 5, 1),
- LocalDate.of(2024, 5, 31),
+ ScreeningDateTimes.of(
+ LocalDate.of(2024, 5, 1),
+ LocalDate.of(2024, 5, 31),
+ listOf(LocalTime.of(10, 0), LocalTime.of(12, 0), LocalTime.of(14, 0)),
+ ),
  Minute(141),
- R.drawable.harry_potter3_poster,
- "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다."
+ "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다.",
  ),
  Movie(
  "해리 포터와 불의 잔",
- LocalDate.of(2024, 6, 1),
- LocalDate.of(2024, 6, 30),
+ ScreeningDateTimes.of(
+ LocalDate.of(2024, 6, 1),
+ LocalDate.of(2024, 6, 28),
+ listOf(LocalTime.of(14, 0), LocalTime.of(16, 0), LocalTime.of(18, 0)),
+ ),
  Minute(157),
- R.drawable.harry_potter4_poster,
- "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다."
- )
+ "《해리 포터와 마법사의 돌》은 2001년 J. K. 롤링의 동명 소설을 원작으로 하여 만든, 영국과 미국 합작, 판타지 영화이다. 해리포터 시리즈 영화 8부작 중 첫 번째에 해당하는 작품이다. 크리스 콜럼버스가 감독을 맡았다.",
+ ),
  )
  }.flatten()
 

app/src/main/java/woowacourse/movie/data/Theater.kt

@@ -5,7 +5,7 @@ import woowacourse.movie.domain.policy.MorningPolicy
 import woowacourse.movie.domain.policy.MovieDayPolicy
 import woowacourse.movie.domain.policy.NightPolicy
 import woowacourse.movie.domain.theater.Grade
-import woowacourse.movie.domain.theater.TheaterInfo
+import woowacourse.movie.domain.theater.Theater
 
 object Theater {
  private val rowGrade = mapOf(
@@ -24,7 +24,7 @@ object Theater {
  val row = 5
  val col = 4
 
- val info = TheaterInfo(rowGrade, row, col)
+ val info = Theater("선릉", rowGrade, row, col, mapOf())
  val policies = listOf(
  MovieDayPolicy(),
  MorningPolicy(),

app/src/main/java/woowacourse/movie/view/mapper/MovieMapper.kt

@@ -4,7 +4,7 @@ import androidx.annotation.DrawableRes
 import woowacourse.movie.domain.movie.Minute
 import woowacourse.movie.domain.movie.Movie
 import woowacourse.movie.domain.movie.ScreeningDateTimes
-import woowacourse.movie.view.model.MovieListModel.MovieUiModel
+import woowacourse.movie.view.model.MovieUiModel
 
 fun Movie.toUiModel(@DrawableRes posterResourceId: Int): MovieUiModel = MovieUiModel(
  title,

app/src/main/java/woowacourse/movie/view/model/MovieUiModel.kt

@@ -0,0 +1,15 @@
+package woowacourse.movie.view.model
+
+import android.os.Parcelable
+import kotlinx.parcelize.Parcelize
+import java.time.LocalDate
+import java.time.LocalTime
+
+@Parcelize
+data class MovieUiModel(
+ val title: String,
+ val screeningDateTimes: Map<LocalDate, List<LocalTime>>,
+ val runningTime: Int,
+ val posterResourceId: Int,
+ val summary: String,
+) : Parcelable

app/src/main/java/woowacourse/movie/view/moviemain/movielist/ItemViewHolder.kt

@@ -0,0 +1,35 @@
+package woowacourse.movie.view.moviemain.movielist
+
+import android.view.View
+import androidx.annotation.DrawableRes
+import androidx.recyclerview.widget.RecyclerView
+import woowacourse.movie.R
+import woowacourse.movie.databinding.MovieAdItemBinding
+import woowacourse.movie.databinding.MovieItemBinding
+import woowacourse.movie.util.DATE_FORMATTER
+import woowacourse.movie.view.model.MovieUiModel
+
+sealed class ItemViewHolder(view: View) : RecyclerView.ViewHolder(view) {
+ class MovieItemViewHolder(private val binding: MovieItemBinding) : ItemViewHolder(binding.root) {
+ fun set(movie: MovieUiModel, clickListener: View.OnClickListener) {
+ val context = binding.movieTitle.context
+ binding.moviePoster.setImageResource(movie.posterResourceId)
+ binding.movieTitle.text = movie.title
+ binding.movieScreeningDate.text =
+ context.getString(R.string.screening_date_format).format(
+ movie.screeningDateTimes.keys.min().format(DATE_FORMATTER),
+ movie.screeningDateTimes.keys.max().format(DATE_FORMATTER),
+ )
+
+ binding.movieRunningTime.text =
+ context.getString(R.string.running_time_format, movie.runningTime)
+ binding.reserveNowButton.setOnClickListener(clickListener)
+ }
+ }
+
+ class AdItemViewHolder(private val binding: MovieAdItemBinding) : ItemViewHolder(binding.root) {
+ fun set(@DrawableRes resId: Int) {
+ binding.adImageview.setImageResource(resId)
+ }
+ }
+}

app/src/main/java/woowacourse/movie/view/moviemain/movielist/ListViewType.kt

@@ -0,0 +1,15 @@
+package woowacourse.movie.view.moviemain.movielist
+
+import androidx.annotation.LayoutRes
+import woowacourse.movie.R
+
+enum class ListViewType(@LayoutRes val id: Int) {
+ AD_VIEWTYPE(R.layout.movie_ad_item), NORMAL_VIEWTYPE(R.layout.movie_item);
+
+ companion object {
+ fun getViewType(adInterval: Int, position: Int): ListViewType {
+ if ((position + 1) % (adInterval + 1) == 0) return AD_VIEWTYPE
+ return NORMAL_VIEWTYPE
+ }
+ }
+}

app/src/main/java/woowacourse/movie/view/moviemain/movielist/MovieListAdapter.kt

@@ -3,50 +3,51 @@ package woowacourse.movie.view.moviemain.movielist
 import android.view.LayoutInflater
 import android.view.ViewGroup
 import androidx.recyclerview.widget.RecyclerView
+import woowacourse.movie.R
 import woowacourse.movie.databinding.MovieAdItemBinding
 import woowacourse.movie.databinding.MovieItemBinding
-import woowacourse.movie.view.model.MovieListModel
-import woowacourse.movie.view.model.MovieListModel.MovieAdModel
-import woowacourse.movie.view.model.MovieListModel.MovieUiModel
+import woowacourse.movie.view.model.MovieUiModel
 
 class MovieListAdapter(
- private val dataList: List<MovieListModel>,
+ private val adInterval: Int,
+ private val movies: List<MovieUiModel>,
  private val onItemClick: OnItemClick
 ) : RecyclerView.Adapter<RecyclerView.ViewHolder>() {
 
  fun interface OnItemClick {
- fun onClick(item: MovieListModel)
+ fun onClick(item: MovieUiModel)
  }
 
  override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): RecyclerView.ViewHolder {
- return when (MovieListViewType.values()[viewType]) {
- MovieListViewType.MOVIE_ITEM -> {
- val view = LayoutInflater.from(parent.context).inflate(MovieListViewType.MOVIE_ITEM.id, parent, false)
- MovieItemViewHolder(MovieItemBinding.bind(view), onItemClick)
+ return when (ListViewType.values()[viewType]) {
+ ListViewType.NORMAL_VIEWTYPE -> {
+ val view = LayoutInflater.from(parent.context).inflate(ListViewType.NORMAL_VIEWTYPE.id, parent, false)
+ ItemViewHolder.MovieItemViewHolder(MovieItemBinding.bind(view))
  }
- MovieListViewType.AD_ITEM -> {
- val view = LayoutInflater.from(parent.context).inflate(MovieListViewType.AD_ITEM.id, parent, false)
- MovieAdViewHolder(MovieAdItemBinding.bind(view), onItemClick)
+ ListViewType.AD_VIEWTYPE -> {
+ val view = LayoutInflater.from(parent.context).inflate(ListViewType.AD_VIEWTYPE.id, parent, false)
+ ItemViewHolder.AdItemViewHolder(MovieAdItemBinding.bind(view))
  }
  }
  }
 
- override fun getItemCount(): Int = dataList.size
+ override fun getItemCount(): Int = movies.size
 
  override fun onBindViewHolder(holder: RecyclerView.ViewHolder, position: Int) {
- val item = dataList[position]
+ val item = movies[position - ((position + 1) / (adInterval + 1))]
  when (holder) {
- is MovieItemViewHolder -> {
- holder.bind(item as MovieUiModel)
+ is ItemViewHolder.MovieItemViewHolder -> {
+ holder.set(item) {
+ onItemClick.onClick(item)
+ }
  }
- is MovieAdViewHolder -> {
- holder.bind(item as MovieAdModel)
+ is ItemViewHolder.AdItemViewHolder -> {
+ holder.set(R.drawable.woowacourse_banner)
  }
  }
  }
 
- override fun getItemViewType(position: Int): Int = when (dataList[position]) {
- is MovieUiModel -> MovieListViewType.MOVIE_ITEM.ordinal
- is MovieAdModel -> MovieListViewType.AD_ITEM.ordinal
+ override fun getItemViewType(position: Int): Int {
+ return ListViewType.getViewType(adInterval, position).ordinal
  }
 }

app/src/main/java/woowacourse/movie/view/moviemain/movielist/MovieListContract.kt

@@ -1,9 +1,9 @@
 package woowacourse.movie.view.moviemain.movielist
 
-import woowacourse.movie.view.model.MovieListModel
+import woowacourse.movie.view.model.MovieUiModel
 
 interface MovieListContract {
  interface Presenter {
- fun getMovieListData(): List<MovieListModel>
+ fun getMovieListData(): List<MovieUiModel>
  }
 }

app/src/main/java/woowacourse/movie/view/moviemain/movielist/MovieListFragment.kt

@@ -1,37 +1,27 @@
 package woowacourse.movie.view.moviemain.movielist
 
-import android.content.Intent
-import android.net.Uri
 import android.os.Bundle
 import android.view.View
 import androidx.fragment.app.Fragment
 import androidx.recyclerview.widget.RecyclerView
 import woowacourse.movie.R
-import woowacourse.movie.view.model.MovieListModel
+import woowacourse.movie.view.model.MovieUiModel
 import woowacourse.movie.view.reservation.ReservationActivity
 
 class MovieListFragment : Fragment(R.layout.fragment_movie_list) {
  override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
  super.onViewCreated(view, savedInstanceState)
  val presenter = MovieListPresenter()
- val dataList = presenter.getMovieListData()
+ val movies = presenter.getMovieListData()
 
- val movieAdapter = MovieListAdapter(dataList, ::onClick)
+ val movieAdapter = MovieListAdapter(3, movies, ::onClick)
  val movieListView = view.findViewById<RecyclerView>(R.id.movie_recyclerview)
  movieListView.adapter = movieAdapter
  }
 
- private fun onClick(item: MovieListModel) {
- when (item) {
- is MovieListModel.MovieUiModel -> {
- val intent = ReservationActivity.newIntent(requireContext(), item)
- startActivity(intent)
- }
- is MovieListModel.MovieAdModel -> {
- val intent = Intent(Intent.ACTION_VIEW, Uri.parse(item.url))
- startActivity(intent)
- }
- }
+ private fun onClick(item: MovieUiModel) {
+ val intent = ReservationActivity.newIntent(requireContext(), item)
+ startActivity(intent)
  }
 
  companion object {