Почему фрагмент накладывается на другой фрагмент при replace?

Question

[Александр Кошелев]

Друзья, всем привет!
Помогите разобраться в следующей ситуации пожалуйста!
Я нахожусь в homeFragment, из него я хочу запустить contentFragment. Все запускается, но фрагменты накладываются друг на друга! Как видно на скриншоте идет наслоение. На одном фрагменте TextView показывает Home Fragment а на втором Я крутой контент.

Вот код фрагмента из которого вызывается другой

class HomeFragment: Fragment() {

    private lateinit var Quotes: TextView
    private var _binding: FragmentHomeBinding? = null
    private val binding: FragmentHomeBinding
    get() = _binding ?: throw RuntimeException("FragmentHomeBinding is null")

    override fun onAttach(context: Context) {
        super.onAttach(context)
    }

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

    override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
        super.onViewCreated(view, savedInstanceState)
        initViews(view)
        addListeners(view)
    }

    private fun initViews(view: View){
        Quotes = view.findViewById(R.id.textViewQuotes)
    }

    private fun addListeners(view: View){
        view.setOnClickListener {
            if (binding.textViewQuotes.id.toString() == R.id.textViewQuotes.toString()){

                requireActivity().supportFragmentManager.beginTransaction()
                    .replace(R.id.homeFragment, ContentFragment.newInstance())
                    .commit()
            }
        }
    }
}

А вот код фрагмента, который вызывается

class ContentFragment: Fragment() {

    override fun onCreateView(
        inflater: LayoutInflater,
        container: ViewGroup?,
        savedInstanceState: Bundle?
    ): View? {
        return inflater.inflate(R.layout.content_fragment, container, false)
    }

    override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
        super.onViewCreated(view, savedInstanceState)
    }

    companion object {
        fun newInstance(): Fragment {
            return ContentFragment()
        }
    }

Comments

[Денис Загаевский]

покажи разметку и код в активити

[Александр Кошелев]

Денис Загаевский, Привет, благодарю, что откликнулся!
Вот активность главная

А вот xml фрагмента, который вызывается и встает в экран, заменяя предыдущий

[Александр Кошелев]

Я нашел решение, Денис!
Ты подтолкнул к ответу. Бэкграунд у меня не был проставлен во фрагменте, который вызывался!
Благодарю!

[Денис Загаевский]

Александр Кошелев, ммм, ну ты этим не решил проблему, а только замёл её под ковёр(под бекграунд). При реплейсе такого не должно быть, ты всё ещё что-то делаешь не так. Эти скрины неинформативны, вставляй код текстом. Код активити и разметку активити.

[Александр Кошелев]

Денис Загаевский, Благодарю!
Вот код активности

package com.example.quotes

import android.os.Bundle
import android.os.Handler
import androidx.appcompat.app.AppCompatActivity
import androidx.fragment.app.Fragment
import com.example.quotes.databinding.ActivityMainBinding

@Suppress("DEPRECATION")
class MainActivity : AppCompatActivity() {

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        var binding = ActivityMainBinding.inflate(layoutInflater)
        setTheme(R.style.splashScreenThemes)

        setContentView(binding.root)
        replaceFragment(HomeFragment())

        binding.bottomNavigationView.setOnItemSelectedListener { item ->
            when(item.itemId) {
                R.id.home -> {
                    replaceFragment(HomeFragment())
                    true
                }
                R.id.profile -> {
                    replaceFragment(ProfileFragment())
                    true
                }
                R.id.settings -> {
                    replaceFragment(SettingsFragment())
                    true
                }
                else -> false
            }
        }
    }

    private fun replaceFragment(fragment: Fragment){
        val transaction = supportFragmentManager.beginTransaction()
        transaction.replace(R.id.frame_layout, fragment)
        transaction.commit()
    }


}

Код фрагмента, из которого вызывается другой фрагмент

package com.example.quotes

import android.content.Context
import android.os.Bundle
import android.util.Log
import android.view.LayoutInflater
import android.view.View
import android.view.ViewGroup
import android.widget.TextView
import androidx.core.view.get
import androidx.fragment.app.Fragment
import com.example.quotes.databinding.FragmentHomeBinding
import java.lang.RuntimeException

class HomeFragment: Fragment() {

    private lateinit var Quotes: TextView
    private var _binding: FragmentHomeBinding? = null
    private val binding: FragmentHomeBinding
    get() = _binding ?: throw RuntimeException("FragmentHomeBinding is null")

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

    override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
        super.onViewCreated(view, savedInstanceState)
        initViews(view)
        addListeners(view)
    }

    private fun initViews(view: View){
        Quotes = view.findViewById(R.id.textViewQuotes)
    }

    private fun addListeners(view: View){
        view.setOnClickListener {
            if (binding.textViewQuotes.id.toString() == R.id.textViewQuotes.toString()){

                requireActivity().supportFragmentManager.beginTransaction()
                    .replace(R.id.homeFragment, ContentFragment.newInstance())
                    .commit()
            }
        }
    }
}

XML этого же фрагмента

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/homeFragment"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:background="@color/purple_200"
    tools:context=".HomeFragment">


    <TextView
        android:id="@+id/textViewQuotes"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="Home Fragment"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        android:textSize="35dp"/>
</androidx.constraintlayout.widget.ConstraintLayout>

XML фрагмента, который вызывается

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/contentFragment"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:background="@color/purple_200"
    tools:context=".ContentFragment">

    <TextView
        android:id="@+id/textView"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="Я  контент"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        android:textSize="30sp"/>
</androidx.constraintlayout.widget.ConstraintLayout>

XML активити

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:id="@+id/main_container"
    android:name="com.example.quotes.HomeFragment"
    tools:context=".MainActivity">

    <FrameLayout
        android:id="@+id/frame_layout"
        android:layout_width="0dp"
        android:layout_height="0dp"
        app:layout_constraintBottom_toTopOf="@+id/bottomNavigationView"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent">
    </FrameLayout>

    <com.google.android.material.bottomnavigation.BottomNavigationView
        android:id="@+id/bottomNavigationView"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:menu="@menu/bottom_nav_menu"/>

</androidx.constraintlayout.widget.ConstraintLayout>

Подскажи пожалуйста как правильно сделать!!!

Answer 1

[Денис Загаевский]

Ну, собственно, всё понятно.

transaction.replace(R.id.frame_layout, fragment)

и

.replace(R.id.homeFragment, ContentFragment.newInstance())

Ты реплейсишь фрагменты в разные контейнеры. Поэтому они находятся на экране одновременно. Должен быть один и тот же контейнер.

Дальше правила хорошего тона:
Фрагмент(в твоем случае — HomeFragment) не имеет морального права брать активити и пихать что-то в её фрагмент менеджер. Это не его ответственность. Тем более он не имеет права закладываться, что в разметке активити есть вьюхи с определенным айди. Максимум он может скастовать активити к какому-то интерфейсу и попросить её что-то сделать:

(requireActivity() as? CallbackInterface)?.showContentPlease(your awesome content)

Comments

[Александр Кошелев]

Мммм, средне понял на самом деле. Про requireActivity тоже были сомнения по поводу правильности данной записи, но лишь таким способом получилось добиться "смены" фрагмента. Денис, можно пожалуйста очень прошу в лс напрмиер в телеграме тебе написать пару уточняющих вопросов? Я много времени не займу, а здесь отмечу решением вопроса твой ответ!