プログレスバーの表示非表示を切り替えたい

Question

前提・実現したいこと

ログインボタン押下時に
fragmentからactivity_layoutに配置されているプログレスバーの表示非表示を切り替えたい。
表示非表示の切り替えはvisibilityで実装したいです。

発生している問題・エラーメッセージ

id.visibility = View.VISIBLEのような記述をしても変化しない。

該当のソースコード

・LoginActivity

class LoginActivity : AppCompatActivity() {

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_login)

        val actionBar: androidx.appcompat.app.ActionBar? = supportActionBar
        getSupportActionBar()?.setDisplayOptions(ActionBar.DISPLAY_SHOW_CUSTOM);
        getSupportActionBar()?.setCustomView(R.layout.login_header);
        actionBar?.setDisplayHomeAsUpEnabled(true)

        val loginFragment = LoginFragment()
        val fragmentTransaction = supportFragmentManager.beginTransaction()
        fragmentTransaction.add(R.id.login_fragment_container, loginFragment)
        fragmentTransaction.commit()
    }
}

・LoginFragment

class LoginFragment : Fragment() {

    private lateinit var login_binding: FragmentLoginBinding
    private var _binding: ActivityLoginBinding? = null
    private val activity_binding get() = _binding!!

    private val viewModel: LoginViewModel by lazy {
        ViewModelProviders.of(this).get(LoginViewModel::class.java)
    }

    override fun onCreateView(
        inflater: LayoutInflater, container: ViewGroup?,
        savedInstanceState: Bundle?
    ): View? {

        _binding = ActivityLoginBinding.inflate(inflater, container, false)

        login_binding = DataBindingUtil.inflate(inflater, R.layout.fragment_login, container, false)
        login_binding.lifecycleOwner = viewLifecycleOwner
        login_binding.viewModel = viewModel

        return login_binding.root
    }

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

        //パスワードリセット画面遷移ボタン
        login_binding.pwdForgetBtn.setOnClickListener{
            val pwdResetBeforeFragment = PwdResetBeforeFragment()
            Log.d("!ResetBefore!", "" + pwdResetBeforeFragment)
            val fragmentTransaction = fragmentManager?.beginTransaction()
            fragmentTransaction?.addToBackStack(null)
            fragmentTransaction?.replace(R.id.login_fragment_container, pwdResetBeforeFragment)
            fragmentTransaction?.commit()
        }

        //新規登録画面遷移ボタン
        login_binding.memberRegistrationBtn.setOnClickListener{
            val memberRegistrationFragment1 = MemberRegistrationFragment1()
            val fragmentTransaction = fragmentManager?.beginTransaction()
            fragmentTransaction?.addToBackStack(null)
            fragmentTransaction?.replace(R.id.login_fragment_container, memberRegistrationFragment1)
            fragmentTransaction?.commit()
        }

        //------ログインボタン押下時にactivity_loginのプログレスバーのvisibilityを切り替えたい------\
        //ログイン処理
        login_binding.loginBtn.setOnClickListener {
            activity_binding.loginProgressbarLayout.visibility = View.VISIBLE
            var user_token = viewModel.sendJsonData()
            Log.d("トークン", ""+user_token)
        }
    }
}

・activity_login

<?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"
    tools:context=".login.LoginActivity">

    <RelativeLayout
        android:id="@+id/login_fragment_container"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent">

        <RelativeLayout
            android:id="@+id/login_progressbar_layout"
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            android:translationZ="100dp"
            android:background="@color/progress_background"
            android:visibility="invisible"
            app:layout_constraintBottom_toBottomOf="parent"
            app:layout_constraintEnd_toEndOf="parent"
            app:layout_constraintStart_toStartOf="parent"
            app:layout_constraintTop_toTopOf="parent">

            <ProgressBar
                android:id="@+id/progressBar"
                style="?android:attr/progressBarStyleLarge"
                android:layout_width="wrap_content"
                android:layout_height="wrap_content"
                android:layout_gravity="center_horizontal"
                android:indeterminate="true"
                android:indeterminateDrawable="@drawable/progress_bg"
                android:max="100"
                app:layout_constraintBottom_toBottomOf="parent"
                app:layout_constraintEnd_toEndOf="parent"
                app:layout_constraintStart_toStartOf="parent"
                app:layout_constraintTop_toTopOf="parent" />

        </RelativeLayout>

    </RelativeLayout>

</androidx.constraintlayout.widget.ConstraintLayout>

※プログレスバーの背景を薄暗くしたいためvisibilityの機能自体は
プログレスバーの一つ上のRelativeLayoutで行っています。

試したこと

fragment_login_xml(LoginFragmentのlayout)でvisibilityの機能を使ったときは上手く表示非表示が出来たのですが、
新規登録やパスワードリセットの時にもプログレスバーを使う事を考えるとactivity_layoutで表示非表示の機能を作るのが良いと考え実装に取り掛かりました。
そこで下記のような記述をしエラー等は無かったのですが処理が走りませんでした。
(恐らくエラーは無いが取得自体が出来ていない？)

private var _binding: ActivityLoginBinding? = null
private val activity_binding get() = _binding!!

activity_binding.loginProgressbarLayout.visibility = View.VISIBLE

確認したいファイル等ありましたら教えてください。
お手数をおかけしますがご教示をお願いしたいです。

Comments

[hoshi-takanori]

LoginFragment で ActivityLoginBinding.inflate() してはいけません。
loginProgressbarLayout は LoginActivity に属しているので、loginBtn の OnClickListener では、activity as LoginActivity に対して loginProgressbarLayout の表示・非表示を切り替えるようにお願いする必要があります。
また、画面遷移も LoginActivity でやった方がいいと思います。

[j.f15]

回答ありがとうございます。
なるほど。プログレスバーはactivityにあるのでログインボタンのクリック処理はactivityに対して行わないといけなかったのですね。
そのことを踏まえてもう少し調べて見たいと思います。

Answer 1

下にさらに良いと思う実装方法に修正しました。(間違っていたら指摘していただけると助かります)

//ログイン処理
        login_binding.loginBtn.setOnClickListener {
            var loginLayout = (activity as LoginActivity?)!!.layoutInflater.inflate(R.layout.activity_login, login_binding.loginProgressbar)
            loginLayout.findViewById<RelativeLayout>(R.id.login_progressbar_layout).visibility = View.VISIBLE
            var user_token = viewModel.sendJsonData()
            Log.d("トークン", ""+user_token)
        }

ヒントを頂いたり試行錯誤したりで何とか望んでいた実装は出来たのですが、
inflaterの理解をせずに使っていたり間違った実装をしていたりと酷いプログラムに
なっていると思うので引き続き調べていきたいと思います。

※追記。上記の方法よりも良いと思う実装方法

・LoginActivity

class LoginActivity : AppCompatActivity() {

    private lateinit var binding : ActivityLoginBinding

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        binding = ActivityLoginBinding.inflate(layoutInflater)
        setContentView(binding.root)

        val actionBar: androidx.appcompat.app.ActionBar? = supportActionBar
        getSupportActionBar()?.setDisplayOptions(ActionBar.DISPLAY_SHOW_CUSTOM);
        getSupportActionBar()?.setCustomView(R.layout.login_header);
        actionBar?.setDisplayHomeAsUpEnabled(true)


        val loginFragment = LoginFragment()
        val fragmentTransaction = supportFragmentManager.beginTransaction()
        fragmentTransaction.add(R.id.login_fragment_container, loginFragment)
        fragmentTransaction.commit()
    }

    //プログレスバーの表示
    fun progressbarVisible() {
        binding.loginProgressbarLayout.visibility = View.VISIBLE
    }

    //プログレスバーの非表示
    fun progressbarInvisible() {
        binding.loginProgressbarLayout.visibility = View.INVISIBLE
    }
}

・LoginFragment

//ログイン処理
        login_binding.loginBtn.setOnClickListener {
            (activity as LoginActivity?)!!.progressbarVisible()
            var user_token = viewModel.sendJsonData()

            Log.d("トークン", ""+user_token)

//            (activity as LoginActivity?)!!.progressbarInvisible()
        }