Как решить проблему java.lang.NullPointerException в Kotlin\Android?

Question

[uvins_s]

Есть список Recycle View, он выводит пользователей из бд. Есть функция, которая позволяет изменить описание профиля определённого пользователя. При изменении описания, приложение вылетает с ошибкой
java.lang.NullPointerException
После повторного запуска описание успешно изменено.

Код изменения описания:

binding.button2.setOnClickListener {
            auth.currentUser?.let { it1 -> db.child(it1.uid).child("description").setValue(binding.editTextText2.text.toString()) }
}

Адаптер Recycle View:

class ItemsAdapter(private var items: ArrayList<Item>, private val context: Context) : RecyclerView.Adapter<ItemsAdapter.ViewHolder>() {

    class ViewHolder(val binding: UserItemBinding) : RecyclerView.ViewHolder(binding.root) {

    }

    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ViewHolder {
        return ViewHolder((UserItemBinding.inflate(LayoutInflater.from(parent.context), parent, false)))
    }

    override fun getItemCount(): Int = items.size

    override fun onBindViewHolder(holder: ViewHolder, position: Int) {
        val item = items[position]
        holder.apply {
            binding.apply {
                cardName.text = item.login
                cardDescription.text = item.description
            }
        }
    }
}

Код самого recycle view:

class HomeFragment : Fragment() {
        usersList = arrayListOf()

        fetchData()

        // Инициализация RecyclerView
        binding.rvContacts.apply {
            setHasFixedSize(true)

            layoutManager = LinearLayoutManager(context)
        }
        // ...
        private fun fetchData() {
        firebaseRef.addValueEventListener(object : ValueEventListener {
            override fun onDataChange(snapshot: DataSnapshot) {
                usersList.clear()

                if (snapshot.exists()) {
                    for (userSnap in snapshot.children) {
                        val users = userSnap.getValue(Item::class.java)

                        usersList.add(users!!)
                    }
                }

                val rvAdapter = context?.let { ItemsAdapter(usersList, it) }

                binding.rvContacts.adapter = rvAdapter
            }

            override fun onCancelled(error: DatabaseError) {
                Toast.makeText(context, "Ошибка: ${error.message}", Toast.LENGTH_SHORT).show()
            }
        })
    }
}

Comments

[Jacen11]

ну а логи где? где падает то. Там же пишет какой объект оказался null

[uvins_s]

Jacen11,


63 строчка
binding.rvContacts.adapter = rvAdapter

[Jacen11]

uvins_s, https://stackoverflow.com/questions/58248031/nullp...

[Jacen11]

HomeFragment это не ресайклер, каша какая то, куча кода явно вырезана, это даже не скомпилируется

[uvins_s]

Jacen11, Не работает

[uvins_s]

Jacen11, Вот прям весь код Recycler

package com.example.findio.ui.home

import android.os.Bundle
import android.view.LayoutInflater
import android.view.View
import android.view.ViewGroup
import android.widget.Toast
import androidx.fragment.app.Fragment
import androidx.recyclerview.widget.LinearLayoutManager
import com.example.findio.adapter.ItemsAdapter
import com.example.findio.databinding.FragmentHomeBinding
import com.example.findio.models.Item
import com.google.firebase.database.DataSnapshot
import com.google.firebase.database.DatabaseError
import com.google.firebase.database.DatabaseReference
import com.google.firebase.database.FirebaseDatabase
import com.google.firebase.database.ValueEventListener

class HomeFragment : Fragment() {

    private var _binding: FragmentHomeBinding? = null
    private val binding get() = _binding!!
    private var firebaseRef: DatabaseReference = FirebaseDatabase.getInstance().getReference("Users")
    private lateinit var usersList: ArrayList<Item>

    override fun onCreateView(
        inflater: LayoutInflater,
        container: ViewGroup?,
        savedInstanceState: Bundle?
    ): View {
        _binding = FragmentHomeBinding.inflate(inflater, container, false)
        val root: View = binding.root

        usersList = arrayListOf()

        fetchData()

        // Инициализация RecyclerView
        binding.rvContacts.apply {
            setHasFixedSize(true)

            layoutManager = LinearLayoutManager(context)
        }

        return root
    }

    private fun fetchData() {
        firebaseRef.addValueEventListener(object : ValueEventListener {
            override fun onDataChange(snapshot: DataSnapshot) {
                usersList.clear()

                if (snapshot.exists()) {
                    for (userSnap in snapshot.children) {
                        val users = userSnap.getValue(Item::class.java)

                        usersList.add(users!!)
                    }
                }

                val rvAdapter = context?.let { ItemsAdapter(usersList, it) }

                binding.rvContacts.adapter = rvAdapter
            }

            override fun onCancelled(error: DatabaseError) {
                Toast.makeText(context, "Ошибка: ${error.message}", Toast.LENGTH_SHORT).show()
            }
        })
    }

    override fun onDestroyView() {
        super.onDestroyView()
        _binding = null
    }
}

[Jacen11]

uvins_s,
!!
это не просто так два восклицательных знака, это потому делать так нельзя

onDataChange в addValueEventListener скорее всего отрабатывает в какой то не тот момент, надо дебажить, какая то сложная последовательность действий, сложно отследить порядок выполнения. Чтобы проще было придумали mvvm или mvi и всякие лайвдата и флоу

https://developer.android.com/studio/debug