Question

我正在使用MVVM体系结构构建一个简单的订购应用程序。我在我的ProductsFragment中使用RecyclerView列出了所有可以订购的产品。我还在ViewModel中使用LiveData，在Fragment中可以观察到它，以检查对产品列表的任何更改。

在我的列表项中，我有3个按钮：1个按钮用于将产品添加到购物篮中，另一个按钮用于增加客户要添加到购物篮中的数量，第三个按钮用于减少客户想要添加到购物篮中的数量

在我的产品数据类中，每当客户单击增减按钮时，数量都会在数据模型中更新。

我还使用数据绑定将产品绑定到recyclerview列表项布局和Click侦听器。我正在使用ListAdapter开箱即用地访问DiffUtil。

我遇到的问题是，当通知了observable时，我想使用ListAdapter的SubmitList方法，以便在RecyclerView中仅更新已更改的项目。但是，我注意到DiffUtil方法areContentsTheSame（）始终返回true。因此，列表项未更新。我不想使用notifyDatasetChanged，因为这会阻塞UI线程。

我遇到的另一个问题是，当我将产品添加到购物篮中时，会保留对该产品的引用，因此，当我将产品添加到购物篮中时，当我只想将MutableLiveData添加为购物篮时，MutableLiveData也会更新更新。将产品添加到购物篮时，如何停止对正在创建的LiveData的引用？

ProductsViewModel

class ProductsViewModel : ViewModel() {

    // LIVE DATA
    private val _basket = MutableLiveData<Basket>()

    val basket: LiveData<Basket>
        get() = _basket

    private val _products = MutableLiveData<List<Product>>()

    val products: LiveData<List<Product>>
        get() = _products


    init {
        _basket.value = Basket()
        _products.value = dummyData
    }

    fun onIncrementProductQuantityButtonPressed(product: Product) {
        //product.quantity += 1
        //val newList = _products.value

        //_products.value = newList   
        val newProduct = product.copy(quantity = product.quantity.plus(1))
        _basket.value!!.updateProductInBasket(newProduct)
        _basket.value = _basket.value
    }

    fun onDecrementProductQuantityButtonPressed(product: Product) {
        if (product.quantity>1) {
            //product.quantity = product.quantity.minus(1)    
            val newProduct = product.copy(quantity = product.quantity.minus(1))
            _basket.value!!.updateProductInBasket(newProduct)
            _basket.value = _basket.value       
        }
    }
}

产品片段

class ProductsFragment : Fragment() {
    private lateinit var viewModel: ProductsViewModel

    override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?,
        savedInstanceState: Bundle?): View? {
        // Inflate the layout for this fragment
        val binding: FragmentProductsBinding = DataBindingUtil.inflate(
            inflater, R.layout.fragment_products, container, false)

        viewModel = ViewModelProviders.of(this).get(ProductsViewModel::class.java)

        val adapter = ProductsAdapter(ProductListener { product, onClickType ->
            when(onClickType) {
                OnClickType.INCREMENT -> {
                    viewModel.onIncrementProductQuantityButtonPressed(product)
                }
                OnClickType.DECREMENT -> {
                    viewModel.onDecrementProductQuantityButtonPressed(product)
                }
                OnClickType.BASKET -> {
                    viewModel.addToBasketButtonPressed(product)
                }
            }
        })

        viewModel.products.observe(this, Observer { list ->
            adapter.submitList(list)
            //adapter.notifyDataSetChanged() // TODO: check why I have to do notifyDataSetChanged()
        })

        viewModel.basket.observe(this, Observer {
            activity?.invalidateOptionsMenu()
        })

        binding.viewModel = viewModel
        binding.lifecycleOwner = this
        binding.productsRecyclerView.adapter = adapter

        setHasOptionsMenu(true)

        return binding.root
    }

ProductsAdapter

class ProductsAdapter(private val clickListener: ProductListener) : ListAdapter<Product, ProductsAdapter.ProductViewHolder>(ProductDiffUtil()) {

    class ProductDiffUtil: DiffUtil.ItemCallback<Product>() {
        override fun areItemsTheSame(oldItem: Product, newItem: Product): Boolean {
            Log.d("Products", "Are items the same")
            return oldItem.name == newItem.name && oldItem.size == newItem.size
        }

        override fun areContentsTheSame(oldItem: Product, newItem: Product): Boolean {
            Log.d("Products", "Are contents the same ${oldItem == newItem}")
            Timber.d("Are contents the same ${oldItem == newItem}")
            Timber.d("OLD ITEM: $oldItem")
            Timber.d("NEW ITEM: $newItem")
            return oldItem == newItem //need to check this
        }
    }

    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ProductViewHolder {
        return ProductViewHolder.from(parent)
    }

    override fun onBindViewHolder(holder: ProductViewHolder, position: Int) {
        val item = getItem(position)

        //use holder to access all the views in the card item
        holder.bind(clickListener, item)
    }

    class ProductViewHolder private constructor(private val binding: LayoutProductCardBinding) : RecyclerView.ViewHolder(binding.root) {

        fun bind(clickListener: ProductListener, item: Product) {
            binding.product = item
            binding.clickListener = clickListener
            binding.executePendingBindings()
        }

        companion object {
            fun from(parent: ViewGroup): ProductViewHolder {
                val layoutInflater = LayoutInflater.from(parent.context)
                val binding = LayoutProductCardBinding.inflate(layoutInflater, parent, false)

                return ProductViewHolder(binding)
            }
        }
    }

}

class ProductListener(val clickListener: (product: Product, clickType: OnClickType) -> Unit) {
    fun onAddToBasket(product: Product) = clickListener(product, OnClickType.BASKET)
    fun onDecrementProductQuantity(product: Product) = clickListener(product, OnClickType.DECREMENT)
    fun onIncrementProductQuantity(product: Product) = clickListener(product, OnClickType.INCREMENT)
}


enum class OnClickType { BASKET, DECREMENT, INCREMENT  }

Answer 1

您的问题似乎有一个简单的解决方案：)。您正在做的是仅更新从视图传递到VM的产品的价值。您应该要做的是：使用.copy（quantity = ...）方法创建产品的副本，覆盖数量。然后替换列表中的上一个项目，并将新列表传递给LiveData。为什么您需要主动调用notifyDataSetChanged（根据您在代码中的注释）也可能是这种情况。

希望这会有所帮助。干杯！

Answer 2

或者，您也可以创建一个称为诸如creationProductInBasked（product.id）之类的新方法。

fun onIncrementProductQuantityButtonPressed(product: Product) {
        _basket.value!!.incrementProductInBasked(product.id)
        _basket.value = _basket.value
    }

我遇到了类似的问题。如果您的 product 类型为LiveData，并且您更新了 product ，则您的用户界面会自动更新。如果当观察者在听篮子的更改时还调用 updateProductInBasket（product），则观察者将收到的新列表已经与UI上的列表相同。

所以我的建议是删除更新 product 的行，并直接调用负责直接执行增量的方法（我建议 incrementProductInBasked（product.id））在您的存储库上。然后将调用购物篮观察者，并且只有更改的项目才会在RecyclerView列表中得到更新。

当向RecyclerView ListAdapter提交新列表时，diff检查始终为areContentsTheSame（）返回true。

2 个答案: