Android(Kotlin)でintent.getIntExtraの値をonCreateメソッド外で取得したい。

Question

前提・実現したいこと

簡単なボードゲームを作りたいと考えております。そこで別のアクティビティから数値(colRow)を渡してボードの大きさを変えられるようにしたいです。

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

Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'int android.content.Intent.getIntExtra(java.lang.String, int)' on a null object reference

該当のソースコード

    val colRow =return intent.getIntExtra(EXTRA_COLROW,0)

    private val boardCells = Array(colRow) { arrayOfNulls<Button>(colRow) }

    var board = Board(colRow)

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

        loadBoard()

    private fun loadBoard() {

        for (i in boardCells.indices) {
            for (j in boardCells.indices) {
                boardCells[i][j] = Button(this)
                boardCells[i][j]?.layoutParams = GridLayout.LayoutParams().apply {
                    rowSpec = GridLayout.spec(i)
                    columnSpec = GridLayout.spec(j)
                    width = 285 - (21*colRow)
                    height = 285 - (21*colRow)
                    bottomMargin = 5
                    topMargin = 5
                    leftMargin = 5
                    rightMargin = 5
                }
                layout_board.addView(boardCells[i][j])
                boardCells[i][j]?.setOnClickListener(CellClickListener1(i, j))
                boardCells[i][j]?.setBackgroundColor(
                        ContextCompat.getColor(
                                this,
                                R.color.colorPrimary
                        )
                )
            }
        }
    }

    }

試したこと

intentをメソッド内に入れてみましたが、同じエラーが出ました。

    val colRow =getColrow()

    fun getColrow():Int{
        return intent.getIntExtra(EXTRA_COLROW,0)
    }

intent の値を　onCreate　以外で受け取る方法あるいは、private val boardCells配列へonCreateから値を送る方法をご存じでしたら教えてください。よろしくお願いいたします。

Comments

[hoshi-takanori]

Kotlin なので lazy を使う方法もありますが、var ではダメで、val にしないといけないようです。

[Kiokiokio]

ありがとうございます。申し訳ないのですがどこのvarがダメなのかお教えいただけますでしょうか。

[hoshi-takanori]

誤解を招く書き方ですみません。「var がダメ」と言いたいのではなくて、Kotlin の lazy プロパティという機能が val でしか使えない (var では使えない) ので残念、と言いたかったのです。

lazy プロパティというのは、プロパティの値を、オブジェクトを new した時ではなく、そのプロパティに最初にアクセスした時に設定できるようにする機能で、これを使えば見た目上は onCreate に書かなくてすみます。が、onCreate が呼ばれる前のタイミングで colRow にアクセスしてしまうと intent はまだ nil なのでエラーになります。
また、colRow を lazy にしたら、colRow に依存する他のプロパティ (boardCells, board) も lazy にしないと意味がないので、board が var なために lazy にできなかった、という意味です。

詳しくはこのへんをどうぞ。
https://re-engines.com/2018/11/15/kotlin%E3%81%A7%E5%88%9D%E6%9C%9F%E5%8C%96%E3%82%92%E9%81%85%E5%BB%B6%E3%81%99%E3%82%8B/
https://dogwood008.github.io/kotlin-web-site-ja/docs/reference/delegated-properties.html

[Kiokiokio]

詳しい説明をありがとうございます。今後試していきたいと思います。

Answer 1

intent（getIntent()の戻り値）はonCreate()の開始以降でなければ正しく取得できないので、関数（メソッド）外で参照するのは不可能です。無理やり記述してもonCreate()開始以前のintentの値はnullですから、NullPointerExceptionで落ちるのは必然です。

colRowを関数外（プロパティ）に置く目的がonCreate()以外の関数で参照することなのだとしたら、

    var colRow = 0

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

        colRow = intent.getIntExtra(EXTRA_COLROW, 0)

        // 以下略

    }

のような実装ではだめなのでしょうか？

---

（コメントに対して）

boardCellsやboardCellsも同様にonCreate()内で値を設定するようにすることはできないのでしょうか。（もちろんby lazyも一つの方法）

class MainActivity : AppCompatActivity() {

    private var colRow = 0
    private var boardCells: Array<Array<Button?>>? = null
    private var board: Board? = null

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)
        
        colRow = intent.getIntExtra(EXTRA_COLROW, 0)
        boardCells = Array(colRow) { arrayOfNulls(colRow) }
        board = Board(colRow)

        loadBoard()

    }

    fun loadBoard() {
        // 省略
    }
}

Comments

[Kiokiokio]

回答ありがとうございます。
private val boardCells = Array(colRow) { arrayOfNulls<Button>(colRow) }
boardCellsをonCreatw内で呼び出すメソッドで使用するため、onCreate開始以前に値を渡すか、
onCreate内で他の関数から参照できるboardCells配列を宣言できればと考えおります。
質問内容を書き加えております。ご存知でしたら教えていただければ幸いです。

Answer 2

なぜそのような妙な記述をすることになっているのでしょうか.
onCreate 内で intent から各変数を初期化するだけかと思います.

#追記

  private var colRow = 0 //宣言(＆初期化)
  private lateinit var boardCells: Array<Array<Button?>> //宣言
  override fun onCreate(savedInstanceState: Bundle?) {
    super.onCreate(savedInstanceState)
    setContentView(R.layout.activity_main)

    colRow = intent.getIntExtra(EXTRA_COLROW,0) //(再)初期化
    boardCells = Array(colRow){arrayOfNulls<Button>(colRow)} //初期化

    loadBoard()
  }
  private fun loadBoard() {
    //省略
  }

Comments

[Kiokiokio]

ご回答ありがとうございます。
onCreate内でboardCells配列を利用したメソッドを呼び出しているため、onCreate以前にcolRowや配列を宣言しておりました。

[jimbe]

なぜ
setContentView(R.layout.activity_main)
と
loadBoard()
の間に初期化の行を入れようとはされないのでしょうか.
変数の宣言と変数の(処理に必要な値への)初期化は必ずしも同時とは限りません.