android studioでViewの型変換について

Question

WebViewの終了時の処理を書いています。

override fun onDestroy() {
        super.onDestroy()
        if (myWebView != null) {
            myWebView.stopLoading()
            myWebView.webViewClient = null
            myWebView.destroy()
        }
        myWebView = null
    }

とすると、最後のnullを代入するところで「val cannot be reassigned」と出ました。参考にしたコードはjavaで、findViewByIdを使ってmyWebViewにWebViewを設定していました。私はkotlinでIdを直接参照しています。原因は、javaは変数のnull許容型で、kotlinではViewは定数で非null型だからなのかなと思い、

override fun onDestroy() {
        super.onDestroy()
        if (myWebView != null) {
            myWebView.stopLoading()
            myWebView.webViewClient = null
            myWebView.destroy()
        }
        var a:WebView?=myWebView
        a = null
    }

としてみたのですが、これできちんと型変換できていますか？ViewのIDを使って型変換したことがないので、初歩的な質問になってしまうかもしれませんがよろしくお願い致します。

Answer 1

class XXXX {
  // val myWebView: WebView?  
  var myWebView: WebView? // var にする

  override fun onDestroy() {
     super.onDestroy()
     if (myWebView != null) {
        myWebView.stopLoading()
        myWebView.webViewClient = null
        myWebView.destroy()
     }
     myWebView = null
  }
}

ことが多分やりたいことかと思いました

var a:WebView?=myWebView
a = null

では a は null になりますが、 myWebView には WebView が入ったままかと思います

追記

そもそも null クリアは不要だと思っていて

override fun onDestroy() {
     super.onDestroy()
     // if (myWebView != null) {
     //   myWebView.stopLoading()
     //   myWebView.webViewClient = null
     //   myWebView.destroy()
     // }
     // myWebView = null
  }

だと思います。（もしメモリリークしている等あるのであれば、原因は他にあると思います。

Comments

[noko123]

ご回答ありがとうございます。ということは、最初にfindViewByIdを使って変数のnull許容型を準備してやるしか方法はないでしょうか?そうすると至る所でnullチェックが必要となってしまうので、最後に型変換する方法を探していたのですが、何か良い方法をご存じないでしょうか？

よろしくお願いします。

[unhappychoice]

val は二度と再代入しないという印なので、nullを代入するのであれば var を使うしか無いです。
`最後に型変換する` みたいなことをするのを防いで、常に最初に入れたものであることを保証するのが val なので、矛盾しているというか、そのような方法はないと思います。

質問の内容とそれますが、自分であれば null クリアはそもそも不要だと思うので、回答に追記します

[noko123]

そうなんですね。回答ありがとうございました!

[noko123]

追記された回答を拝見しました。その部分については、ドットインストールの『Androidでブラウザアプリを作ろう』という講座の中で、「webView なのですが、わりとメモリ管理がシビアと言われていて、アクティビティを終わらせる時に、色々後処理をしなくてはいけないと言われているので、そのあたりもやっておきましょう。」とあった部分なのですが、それはどうとらえればいいのでしょうか?情報が古いのか、それともjavaかkotlinかの違いによるものなのか。はたまた別の理由なのか。その部分について意見をいただけると幸いです。

[unhappychoice]

> webView なのですが、わりとメモリ管理がシビアと言われていて、アクティビティを終わらせる時に、色々後処理をしなくてはいけないと言われているので、そのあたりもやっておきましょう。

のようなまた聞きのまた聞きのような情報は多分信用する必要はなくて、Java なので基本はしっかり GC される / Activity は持っているViewの終了処理を行う ものと考えて良いと思います。

WebView がメモリを多く消費してなにか問題を起こすとしても、それは基本的には WebView の問題であって、そのような問題は Android フレームワークの責任として日々改善されています。
（ので、情報が古いというのはある意味そうだと思います

そのため、このようなハックなコードは
- 実際に今古いデバイス等で問題が起きていて
- しかもこのコードを追加することでそれが改善する

のを自身で確認した場合のみ書く、位の温度感でいるべきだという意見です。

[noko123]

ありがとうございます。大手のプログラミング学習サイトであるからと、盲目的になっていたことが分かりました。というかそれ以前に、最終更新日が2015年の情報を鵜呑みにした自分の姿勢に一番問題があったということですね。