Service()起動数秒後にアプリが落ちる

Question

○やりたいこと
Serviceのバックグラウンド挙動を安定させるためActivityとServiceを連携させる

○課題
以下サイトをもとに、ActivityとServiceの連携をおこなっております。

参考サイト
https://qiita.com/satken2/items/49dd76d848ceb208e937

しかし、以下エラーが発生しService起動数秒後にアプリが落ちます。

2021-11-12 23:03:18.165 1720-24063/? E/ActivityManager: ANR in com.example.servicetest
    PID: 23934
    Reason: Context.startForegroundService() did not then call Service.startForeground(): ServiceRecord{1ee86ba u0 com.example.servicetest/.MyService}

○教えていただきたいこと
エラーが発生する原因を理解し、改善方法を教えていただきたいです。

const val MSG_ACTIVITY_TO_SERVICE = 1
const val MSG_SERVICE_TO_ACTIVITY = 2

class MainActivity : AppCompatActivity() {
    private var bound = false
    private var mActivityMessenger: Messenger? = null
    private var mServiceMessenger: Messenger? = null
    private var mActivityHandler: ActivityHandler? = null

    private val mConnection = object: ServiceConnection {
        override fun onServiceConnected(name: ComponentName?, binder: IBinder?) {
            mServiceMessenger = Messenger(binder)
            try {
                val msg = Message.obtain(null, MSG_ACTIVITY_TO_SERVICE, 0, 0)
                msg.replyTo = mActivityMessenger
                mServiceMessenger!!.send(msg)
            } catch (e: RemoteException) {
                e.printStackTrace()
            }
            bound = true
        }

        override fun onServiceDisconnected(name: ComponentName?) {
            mServiceMessenger = null
            bound = false
        }
    }

    internal class ActivityHandler(
        activity: MainActivity
    ) : Handler() {
        //define weak reference to activity
        private val mActivity = WeakReference<MainActivity>(activity)

        override fun handleMessage(msg: Message) {
            when (msg.what) {
                MSG_SERVICE_TO_ACTIVITY ->{
                    //something
                }
                else -> super.handleMessage(msg)
            }
        }
    }

    override fun onCreate(savedInstanceState: Bundle?) {
        Log.d("debug", "onCreate() mainactivity")
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)

        mActivityHandler = ActivityHandler(this)
        mActivityMessenger = Messenger(mActivityHandler)

        findViewById<Button>(R.id.start).setOnClickListener {
            Log.d("debug","btnTapped")
            if(bound) {
                Log.d("debug", "bound = ture")
                try {
                    val msg = Message.obtain(null, MSG_ACTIVITY_TO_SERVICE, 0, 0)
                    //val msg = Message.obtain(null, MSG_STOP_UTTERANCE, 0, 0)
                    msg.replyTo = mActivityMessenger
                    mServiceMessenger!!.send(msg)

                    intentGo()

                } catch (e: RemoteException) {
                    e.printStackTrace()
                }
            } else {
                Log.d("debug", "bound = false")
            }
        }
    }

    fun intentGo() {
        Log.d("debug", "intentGo")
        //start service
        Intent(this, MyService::class.java).also {
            Log.d("debug", "Intent")
            it.action = ACTION_SET_TIMER
            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
                Log.d("debug", "startForegroundService")
                startForegroundService(it)
                return
            }
            startService(it)
        }
    }

    override fun onStop() {
        Log.d("debug", "onStop")
        super.onStop()
        //unbind from the service
        if(bound){
            unbindService(mConnection)
            bound = false
        }
    }

    override fun onStart(){
        Log.d("debug", "onStart")
        super.onStart()
        //bind to the service
        Intent(this, MyService::class.java).also{ intent ->
            bindService(intent, mConnection, Context.BIND_AUTO_CREATE)
        }
    }
}

class MyService: Service() {
    private var mServiceMessenger: Messenger? = null

    internal class ServiceHandler(
        service: MyService
    ) : Handler() {
        private val mService = WeakReference<MyService>(service)

        override fun handleMessage(msg: Message) {
            when (msg.what) {
                MSG_ACTIVITY_TO_SERVICE ->{
                    //return some value
                    try {
                        val msg2 = Message.obtain(null, MSG_SERVICE_TO_ACTIVITY, 0, 0)
                        msg.replyTo.send(msg2)
                    } catch (e: RemoteException) {
                        e.printStackTrace()
                    }
                }
                else -> super.handleMessage(msg)
            }
        }
    }

    override fun onBind(intent: Intent?): IBinder {
        Log.d("debug", "onBind")
        mServiceMessenger = Messenger(ServiceHandler(this))
        return mServiceMessenger!!.binder
    }

    override fun onCreate() {
        Log.d("debug", "onCreate service")
        super.onCreate()
    }

    override fun onStartCommand(intent: Intent?, flags: Int, startId: Int): Int {
        Log.d("debug", "onStartCommand")

        Thread.sleep(1000)

        val notification = NotificationCompat.Builder(this).apply {
            var mContentTitle = "通知のタイトル"
            var mContentText = "通知の内容"
            setSmallIcon(R.mipmap.ic_launcher)
        }.build()

        startForeground(0, notification)
        Log.d("debug", "startForeground")

        return Service.START_NOT_STICKY
    }
}

Comments

[jimbe]

ご提示のコードを AndroidStudio で Empty Activity を元にしたプロジェクトに張り付けて実行しましたが、 android:id="@+id/start" とレイアウトしたボタンを押しても
D/debug: onCreate() mainactivity
D/debug: onStart
D/debug: btnTapped
D/debug: bound = false
とログに出るだけで何も起きません。
ご提示のコード以外にどうすれば再現出来るでしょうか。

[jimbe]

失礼しました、AndroidManifest に service を入れるのを忘れていました。

Answer 1

テストしていませんので見た目だけですが、 ANR で did not then call Service.startForeground() ですので、 Thread.sleep(1000) がいけないのではないでしょうか。
onStartCommand() は時間のかかる処理はしてはいけないと思います。

---

メッセージの部分はさておき、まずは以下等は如何でしょうか。
Android - Foreground Service実行

startForegroundService()でNotification登録

startForegroundService()でサービスが実行されると、実行されたサービスは、5秒以内に startForeground()を呼び出して、サービスが実行中であるNotificaitonを登録する必要があります。もし呼び出さない場合、システムはサービスを強制的に終了させます。

Service.startForeground()は次のように Notificaiton IDとNotificaitonオブジェクトを引数として受け取ります。 注意すべき点は、IDで0を渡すと、動作がないということです。 0以外の数字をIDで渡す必要があります。

や

Android O（API 26）以上のバージョンでNotificationを登録する前に、Notificaiton Channelを登録する必要があります。

といった記述があります。

Comments

[kakashi55]

ご回答いただきありがとうございます。
Thread.sleep(1000) をコメントアウトしても同エラーが発生してしまいます。
その他の点でエラーを起こしている原因と対策方法がございましたら教えていただけないでしょうか。

[kakashi55]

IDで0を渡すと、動作がないということです。 0以外の数字をIDで渡す必要があります。
→まさしくここを改善しましたら動作しました。ありがとうございます。