Использование viewmodels с переработчиком просмотров - PullRequest
0 голосов
/ 16 марта 2020

Все это очень сложно, я надеюсь, что смогу объяснить это хорошо. Итак, я использую RecyclelerView, чтобы показать список профилей пользователей в виде карточек. Каждая карточка содержит 2 кнопки для разных действий. Теперь эти кнопки требуют наблюдения объекта LiveData внутри ViewModel (я использую MVVM, Android компоненты архитектуры и Kotlin расширения) при нажатии. В частности, одной кнопке нужно наблюдать за различными данными в реальном времени, основываясь на каком-то условии. Вот почему я создал функцию, которая принимает условие и возвращает подходящее значение OnClickListener. Я передаю это адаптеру переработчика вида.

Теперь проблема в том, что я использую один и тот же RecyclerView в нескольких фрагментах. Все отлично работает, когда я использую RecyclerView в том же фрагменте, где написана функция. В другом месте я получаю сообщение об ошибке:

java.lang.IllegalStateException: Fragment HomeFragment{1ef5330} (b299012d-c9c2-427f-8387-a7888289701b)} not attached to an activity.
        at androidx.fragment.app.Fragment.requireActivity(Fragment.java:833)
        at 
com.halalrishtey.HomeFragment$$special$$inlined$activityViewModels$2.invoke(FragmentViewModelLazy.kt:80)
        at com.halalrishtey.HomeFragment$$special$$inlined$activityViewModels$2.invoke(Unknown Source:0)
        at androidx.lifecycle.ViewModelLazy.getValue(ViewModelProvider.kt:52)
        at androidx.lifecycle.ViewModelLazy.getValue(ViewModelProvider.kt:41)
        at com.halalrishtey.HomeFragment.getUserVM(Unknown Source:7)
        at com.halalrishtey.HomeFragment.access$getUserVM$p(HomeFragment.kt:25)
        at com.halalrishtey.HomeFragment$genInterestBtnListener$1.onClick(HomeFragment.kt:65)
        at android.view.View.performClick(View.java:7125)
        at com.google.android.material.button.MaterialButton.performClick(MaterialButton.java:967)
        at android.view.View.performClickInternal(View.java:7102)
        at android.view.View.access$3500(View.java:801)
        at android.view.View$PerformClick.run(View.java:27336)
        at android.os.Handler.handleCallback(Handler.java:883)
        at android.os.Handler.dispatchMessage(Handler.java:100)
        at android.os.Looper.loop(Looper.java:214)
        at android.app.ActivityThread.main(ActivityThread.java:7356)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:492)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:930)

И функция, которая создает OnClickListener, выглядит примерно так:

fun genInterestBtnListener(
        condition: Boolean,
        v: View
    ): View.OnClickListener {
        return View.OnClickListener {
            if (!condition) {
                userVM.initInterest().observe(viewLifecycleOwner, Observer { msg ->
                    Toast.makeText(context, msg, Toast.LENGTH_SHORT)
                        .show()
                    //More code here
                })
            } else {
                userVM.removeInterest().observe(viewLifecycleOwner, Observer { msg ->
                    Toast.makeText(context, msg, Toast.LENGTH_SHORT)
                        .show()
                })
            }
        }
    }

Я не могу наблюдать за вещами внутри адаптера, так какое оптимальное решение для этого без повторения кода ??

Обновление: Я объявляю userVM следующим образом:

private val userVM: UserViewModel by activityViewModels()

Также Вот мой код адаптера:

class CardDataRVAdapter(private var items: List<ProfileCardData>) :
    RecyclerView.Adapter<CardDataRVAdapter.CardDataViewHolder>() {

    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): CardDataViewHolder {
        val inflatedView =
            LayoutInflater.from(parent.context).inflate(R.layout.profile_card, parent, false)
        return CardDataViewHolder(inflatedView)
    }

    override fun onBindViewHolder(holder: CardDataViewHolder, position: Int) {
        val card = items[position]
        holder.bindCard(card)
    }

    override fun getItemCount() = items.size

    class CardDataViewHolder constructor(
        v: View
    ) : RecyclerView.ViewHolder(v), View.OnClickListener {
        private var view: View = v
        private var cardData: ProfileCardData? = null

        init {
            v.setOnClickListener(this)
        }

        override fun onClick(p0: View?) {
            //TODO: Implement a proper onClickListener
            Toast.makeText(
                p0?.context,
                "${p0?.cardTitleTextView?.text} Card was clicked",
                Toast.LENGTH_SHORT
            ).show()
        }


        companion object {
            private val KEY = "CARD"
        }

        fun bindCard(card: ProfileCardData) {
            this.cardData = card

            if (card.isUserShortlisted) {
                view.showInterestBtn.setIconResource(R.drawable.ic_favorite)
            } else {
                view.showInterestBtn.setIconResource(R.drawable.ic_favorite_border)
            }

            if (card.data.photoUrl.length > 5) {
                Picasso.get().load(card.data.photoUrl)
                    .into(view.cardImageView)

                Picasso.get().load(card.data.photoUrl)
                    .into(view.cardAvatarImageView)
            }

            view.cardTitleTextView.text = card.data.displayName
            view.cardSubtitleTextView.text = "${card.data.age} - ${card.data.height}"

            view.showInterestBtn.setOnClickListener(card.showBtnInterestListener)
            view.sendMessageBtn.setOnClickListener(card.messageBtnListener)
        }
    }
}

Ответы [ 2 ]

0 голосов
/ 21 марта 2020

Я думаю, что ваша проблема в том, что вы используете view. в своей функции bindCard().

Является ли showBtnInterestListener в вашем классе? Вы можете переместить его на ваш Adapter, так как будет более уместно, чтобы Adapter знал о Слушателе, а не data class

Не могли бы вы попробовать изменить Adapter следующим образом:

Вы объявляете объект, с которым будете взаимодействовать, в своем классе, а затем в привязке, которую вы вызываете.

class CardDataViewHolder constructor(
        v: View
    ) : RecyclerView.ViewHolder(v), View.OnClickListener {
        private var view: View = v
        private var cardData: ProfileCardData? = null

        val showInterestBtn = view.findViewById(R.id.show_interest)
        val sendMessageBtn = view.findViewById(R.id.message_btn)
        val title = view.findViewById(R.id.card_title)
        val subtitle = view.findViewById(R.id.card_subtitle)

        companion object {
            private val KEY = "CARD"
        }

        fun bindCard(card: ProfileCardData) {
            this.cardData = card

            if (card.isUserShortlisted) {
                showInterestBtn.setIconResource(R.drawable.ic_favorite)
            } else {
                showInterestBtn.setIconResource(R.drawable.ic_favorite_border)
            }

            if (card.data.photoUrl.length > 5) {
                Picasso.get().load(card.data.photoUrl)
                    .into(view.cardImageView)

                Picasso.get().load(card.data.photoUrl)
                    .into(view.cardAvatarImageView)
            }

            title.text = card.data.displayName
            subtitle.text = "${card.data.age} - ${card.data.height}"

            showInterestBtn.setOnClickListener(card.showBtnInterestListener)
            sendMessageBtn.setOnClickListener(card.messageBtnListener)
        }
    }
0 голосов
/ 19 марта 2020

Насколько я понимаю, каждый LiveData должен наблюдаться внутри фрагмента / действия, в котором создан экземпляр класса ViewModel. Я думаю, что проблема возникает из-за того, что вы пытаетесь наблюдать LiveData вне фрагмента, где он создан. Вы можете наблюдать за LiveData вне фрагмента / действия с помощью observeForever, но вам также нужно будет вызвать removeObserver, чтобы избежать утечек памяти

cmiiw

...