kotlinのFragmentでJSONデータをspinnerに表示する

Question

前提

fragmentを利用しているSpinnerActivityで、JSONのvalueをspinnerに表示させたいのですが、spinnerにadapterをセットしている箇所でspinnerがnullになってしまい、セットされません。

実現したいこと

fragmentを利用しているSpinnerActivityで、JSONのvalueをspinnerに表示させたい。

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

なし

該当のソースコード

class MainActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)
        val btn = findViewById<Button>(R.id.btn)

        btn.setOnClickListener {
            CoroutineScope(Dispatchers.Default).launch {
                val jText = getAPIData()
                val fragment: Fragment = SpinnerActivity()
                val bundle = Bundle()
                bundle.putString("getTypes", jText)
                fragment.arguments = bundle

                //Fragment呼び出し
                val transaction = supportFragmentManager.beginTransaction()
                transaction.replace(R.id.container, fragment)
                transaction.commit()
            }
        }
        val textView = findViewById<TextView>(R.id.textResult)
        val keyItem = intent.getStringExtra("test_key")
        textView.text = keyItem
    }

    fun getAPIData() :String {
        var jsonText =""
        try {
            // linkにファイルが置かれたURLを指定
            val link = "http://192.168.1.197:8000/getTypes"

            //API叩く
            //URL化→文字に変換→読み込み
            val br = BufferedReader(InputStreamReader(URL(link).openStream()))

            jsonText = br.readText()//テキストファイルを読み込み、文字列として返す

        } catch (e: Exception) {
            Log.e(localClassName, "Cancel", e)
        }
        return jsonText
    }

class SpinnerActivity : Fragment() {
    var rootview: View? = null
    override fun onCreateView(
        inflater: LayoutInflater, container: ViewGroup?,
        savedInstanceState: Bundle?
    ): View? {
        // レイアウトをここでViewとして作成
        rootview = inflater.inflate(R.layout.activity_spinner, container, false)
        return rootview
    }

    val spinner = rootview?.findViewById<Spinner>(R.id.spinner)
    val list = ArrayList<Pair<String, String>>()

    //渡された値を取得⇒spinnerに表示
    override fun onResume() {
        super.onResume()
        val jsontext = requireArguments().getString("getTypes")
        val json = JSONArray(jsontext) 
        for (i in 0 until json.length()) {
            val jsons = json.getJSONObject(i)
            //JSONのデータを文字列としてlistに追加
            list.add(Pair(jsons.getString("typeId"), jsons.getString("typeName")))
        }


        // スピナーにアダプターを設定
        val adapter = KeyValuePair(this.requireContext(), android.R.layout.simple_spinner_item, list)
        adapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item)
        if (spinner != null) {
            spinner.adapter = adapter
        }

        val setBtn = rootview?.findViewById<Button>(R.id.button)

        setBtn?.setOnClickListener {
            val item = adapter.getItem(spinner!!.getSelectedItemPosition())
            val key = item?.first
            val intent = Intent(activity, MainActivity::class.java)
            intent.putExtra("test_key", key)
            startActivity(intent)
        }
    }
}

class KeyValuePair (
    context: Context?,
    resourceId: Int,
    list: ArrayList<Pair<String, String>>

) : ArrayAdapter<Pair<String?, String?>?>(
    context!!,
    resourceId,
    list as List<Pair<String?, String?>?>
) {
    // スピナー表示用view取得
    //ArrayAdapterをオーバーライドしてカスタマイズ
    override fun getView(pos: Int, convertView: View?, parent: ViewGroup): View {
        val textView = super.getView(pos, convertView, parent) as TextView
        textView.text = getItem(pos)!!.second
        return textView
    }

    // スピナードロップダウン表示用view取得
    override fun getDropDownView(pos: Int, convertView: View?, parent: ViewGroup?): View {
        val textView = super.getDropDownView(pos, convertView, parent) as TextView
        textView.text = getItem(pos)!!.second
        return textView
    }
}

<?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/container"
    tools:context=".MainActivity">

    <TextView
        android:id="@+id/textResult"
        android:layout_width="200dp"
        android:layout_height="100dp"
        android:text=""
        android:textSize="20sp"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintHorizontal_bias="0.497"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintVertical_bias="0.713" />

    <Button
        android:id="@+id/btn"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="選択画面へ"
        app:layout_constraintBottom_toTopOf="@+id/textResult"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintVertical_bias="0.129"
        tools:ignore="MissingConstraints" />

</androidx.constraintlayout.widget.ConstraintLayout>

<?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:background="#AEFF9800">

    <Spinner
        android:id="@+id/spinner"
        android:layout_width="250dp"
        android:layout_height="wrap_content"
        android:backgroundTint="#009688"
        android:scrollbarSize="4dp"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintHorizontal_bias="0.496"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintVertical_bias="0.261"
        tools:ignore="MissingConstraints" />

    <Button
        android:id="@+id/button"
        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_toBottomOf="@+id/spinner"
        app:layout_constraintVertical_bias="0.149" />


</androidx.constraintlayout.widget.ConstraintLayout>

補足情報（FW/ツールのバージョンなど）

まだ右も左もわからない初心者です。よろしくお願いいたします。

↓参考にしたサイトです。
https://qiita.com/Helmos/items/6b65d40d355379dc7ffa

Comments

[hoshi-takanori]

エラーの原因は、val spinner = 〜 はプロパティなので、SpinnerActivity のインスタンスが作られる時に初期化されますが、その時点ではまだ onResume が呼ばれてなくて rootview が存在しないためですね。

また、いろいろツッコミどころがあって、
・Fragment のサブクラスなのに SpinnerActivity という名前なのは気持ち悪い
・KeyValuePair の親クラスは ? を付けずに ArrayAdapter<Pair<String, String>> で良いのでは
・Android では CoroutineScope よりも lifecycleScope を使うべきだし、
　getAPIData は withContext(Dispatchers.IO) とかを使ってバックグランドで実行すべき
　(参考: https://qiita.com/b150005/items/dfaf1b5e619131905512 )

[sin_cos_tan]

解決いたしました。ありがとうございます。
ツッコミもありがとうございます、勉強していきたいと思います。

Answer 1

エラーの原因は、val spinner = 〜 はプロパティなので、SpinnerActivity のインスタンスが作られる時に初期化されますが、その時点ではまだ onResume が呼ばれてなくて rootview が存在しないためですね。

override fun onResume() {
        super.onResume()
val spinner = rootview?.findViewById<Spinner>(R.id.spinner)
}

という事で、onResumeの中で生成したら解決しました。
hoshi-takanoriさんありがとうございました。