lateinitを参照するため初期化したい

Question

前提・実現したいこと

ボタンを押すと画像をダウンロードするアプリを作りたい

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

lateinit property urlConnection has not been initialized

該当のソースコード

class GetImage(context: Context, var url: String) : GetImageBase<Bitmap>(context) {
    private lateinit var urlConnection: HttpURLConnection

    override fun loadInBackground(): Bitmap? {
        Log.v("nullpo", "loadInBackGround")
//        urlConnection = (url as HttpURLConnection?)!!

        val url: URL = URL(url)

        try {

            urlConnection = (url.openConnection() as HttpURLConnection).also {
                it.requestMethod = "GET"
                it.connectTimeout = 1000
                it.readTimeout = 10000
                it.connect()
            }
            return when (urlConnection.responseCode) {
                HttpsURLConnection.HTTP_OK -> {
                    BitmapFactory.decodeStream(urlConnection.inputStream)
                }
                else -> {
                    null
                }
            }
        } catch (e: Exception) {
            return null
        } finally {
            urlConnection.disconnect()
        }
    }
}

E/AndroidRuntime: FATAL EXCEPTION: ModernAsyncTask #1
    Process: jp.co.stv_tech.test_download_2, PID: 4116
    java.lang.RuntimeException: An error occurred while executing doInBackground()
        at androidx.loader.content.ModernAsyncTask$3.done(ModernAsyncTask.java:164)
        at java.util.concurrent.FutureTask.finishCompletion(FutureTask.java:354)
        at java.util.concurrent.FutureTask.setException(FutureTask.java:223)
        at java.util.concurrent.FutureTask.run(FutureTask.java:242)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)
        at java.lang.Thread.run(Thread.java:762)
     Caused by: java.lang.ClassCastException: java.lang.String cannot be cast to java.net.HttpURLConnection
        at jp.co.stv_tech.test_download_2.GetImage.loadInBackground(GetImage.kt:16)
        at jp.co.stv_tech.test_download_2.GetImage.loadInBackground(GetImage.kt:11)
        at androidx.loader.content.AsyncTaskLoader.onLoadInBackground(AsyncTaskLoader.java:307)
        at androidx.loader.content.AsyncTaskLoader$LoadTask.doInBackground(AsyncTaskLoader.java:60)
        at androidx.loader.content.AsyncTaskLoader$LoadTask.doInBackground(AsyncTaskLoader.java:48)
        at androidx.loader.content.ModernAsyncTask$2.call(ModernAsyncTask.java:141)
        at java.util.concurrent.FutureTask.run(FutureTask.java:237)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133) 
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607) 
        at java.lang.Thread.run(Thread.java:762) 

試したこと

調べてみると初期化されていないことに対してのエラーでした。
さらに初期化＝代入との記載もありましたので下記のコードを試してみました。
urlConnection = (url as HttpURLConnection?)!!

しかし、その結果は下記のようなエラーがでてきました。
String cannot be cast to java.net.HttpURLConnection

初期化するために私の試したコードは何が間違っているのでしょうか
どなたかご教示お願いいたします。

E/AndroidRuntime: FATAL EXCEPTION: ModernAsyncTask #1
    Process: jp.co.stv_tech.test_download_2, PID: 2831
    java.lang.RuntimeException: An error occurred while executing doInBackground()
        at androidx.loader.content.ModernAsyncTask$3.done(ModernAsyncTask.java:164)
        at java.util.concurrent.FutureTask.finishCompletion(FutureTask.java:383)
        at java.util.concurrent.FutureTask.setException(FutureTask.java:252)
        at java.util.concurrent.FutureTask.run(FutureTask.java:271)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
        at java.lang.Thread.run(Thread.java:764)
     Caused by: kotlin.UninitializedPropertyAccessException: lateinit property urlConnection has not been initialized
        at jp.co.stv_tech.test_download_2.GetImage.loadInBackground(GetImage.kt:38)
        at jp.co.stv_tech.test_download_2.GetImage.loadInBackground(GetImage.kt:11)
        at androidx.loader.content.AsyncTaskLoader.onLoadInBackground(AsyncTaskLoader.java:307)
        at androidx.loader.content.AsyncTaskLoader$LoadTask.doInBackground(AsyncTaskLoader.java:60)
        at androidx.loader.content.AsyncTaskLoader$LoadTask.doInBackground(AsyncTaskLoader.java:48)
        at androidx.loader.content.ModernAsyncTask$2.call(ModernAsyncTask.java:141)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167) 
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641) 
        at java.lang.Thread.run(Thread.java:764) 

Comments

[hoshi-takanori]

そのエラーはどこで出てますか？
also ブロックの中で例外が発生すると urlConnection は初期化されないままなので、finally でエラーになるかも。
そもそも urlConnection はローカル変数で十分な気がしますが…。

[yu9718s]

申し訳ございませんが、どこからでてるエラーなのか理解不足により説明ができないため
エラー内容を追記いたします。

[hoshi-takanori]

追記していただいたのは String cannot be cast to java.net.HttpURLConnection の方ですね。
文字列は HttpURLConnection にキャストできませんので、urlConnection = (url as HttpURLConnection?)!! すれば当然エラーになります。

お聞きしたかったのは lateinit property urlConnection has not been initialized というエラーがどこで出ていたかなのですが…。

[yu9718s]

汲み取れず申し訳ございません。
再度エラー内容を追記いたしました。

Answer 1

動かそうとしているAndroidのバージョンが9.0以上であるとしたら、デフォルトでHTTPの通信が制限されているので、そのための対処をしないとurl.openConnection()の段階で例外が飛ぶと思われます。そうするとurlConnectionへの代入が行われないままfinallyのurlConnection.disconnect()が実行されようとするので、質問にあるエラーが起きているのではないでしょうか。

下記資料を参考にしてres/xml/network_security_config.xmlを作成し、AndroidManifest.xmlへの記述を追加することで回避できるのではないかと思います。

ネットワーク セキュリティ構成 | Android Developers
Android 9(Pie)でHTTP通信を有効にする | Qiita

---

Android 7.0の端末なら上記の問題は影響しないでしょう。それならば、AndroidManifest.xmlに

    <uses-permission android:name="android.permission.INTERNET" />

の記述はありますか？それがない場合も、同様にurlConnectionへの代入が行われずにfinallyブロックが走るでしょう。

後は、ネットワーク処理がUIスレッド上で走っている場合もやはり代入が行われず例外が飛ぶので、同じ結果になると思います。別スレッドで走るように実装されているでしょうか。

Comments

[yu9718s]

ご丁寧にありがとうございます。
参考にさせていただきます。
ただ、動かそうとしているAndroidのバージョンは7.0なのでもしかしたら違うかもしれませんが試してみます。

[yu9718s]

manifestへの記述を追加したところ正常に動作いたしました。
エラーのパターンをご教示いただきありがとうございます。
今後同じようなエラーが出た際に参考にさせていただきます。