kotlinでサーフェスビューに描写するメソッドを記述する場所が分からない

Question

状況・前提・実現したいこと

スマートフォンの加速度センサーをつかって画面上にボールが転がるだけの
シンプルなアプリを、参考書を見ながら作っている初心者です。

参考書に記載されているソースコードの実装はほぼ終わり、
後はサーフェスビューにボールを描くメソッド（drawCanvas）を
実装すれば良いだけだと思っているのですが、「never used」となってしまい上手く機能しません。

該当のソースコード

package com.example.accball

import android.content.Context
import android.content.pm.ActivityInfo
import android.graphics.Color
import android.graphics.Paint
import android.hardware.Sensor
import android.hardware.SensorEvent
import android.hardware.SensorEventListener
import android.hardware.SensorManager
import android.os.Bundle
import android.support.v7.app.AppCompatActivity
import android.view.SurfaceHolder
import kotlinx.android.synthetic.main.activity_main.*

class MainActivity : AppCompatActivity(), SensorEventListener, SurfaceHolder.Callback {

private var surfaceWidth: Int = 0 //サーフェスビューの幅
private var surfaceHeight: Int = 0 //サーフェスビューの高さ

private val radius = 50.0f
private val coef = 1000.0f

private var ballX: Float = 0f
private var ballY: Float = 0f
private var vx: Float = 0f
private var vy: Float = 0f
private var time: Long = 0L

override fun surfaceChanged(holder: SurfaceHolder?, format: Int, width: Int, heigth: Int) {
    surfaceWidth = width
    surfaceHeight = heigth
    ballX = (width / 2).toFloat()
    ballY = (heigth / 2).toFloat()
}

override fun surfaceDestroyed(holder: SurfaceHolder?) {
    val sensorManager = getSystemService(Context.SENSOR_SERVICE)
            as SensorManager
    sensorManager.unregisterListener(this)
}

override fun surfaceCreated(holder : SurfaceHolder?) {
    val sensorManager = getSystemService(Context.SENSOR_SERVICE)
            as SensorManager
    val accSensor = sensorManager.getDefaultSensor(Sensor.TYPE_ACCELEROMETER)
    sensorManager.registerListener(this, accSensor, SensorManager.SENSOR_DELAY_GAME)
}

override fun onAccuracyChanged(sensor :Sensor?,accuracy:Int) {
}

override fun onSensorChanged(event: SensorEvent?) {
    if (event == null) return
    if (time == 0L) time = System.currentTimeMillis()
    if (event.sensor.type == Sensor.TYPE_ACCELEROMETER){
        val x = -event.values[0]
        val y = event.values[1]

        var t = (System.currentTimeMillis() - time).toFloat()
        time = System.currentTimeMillis()
        t /= 1000.0f

        val dx = vx * t + x * t * t / 2.0f
        val dy = vy * t + y * t * t / 2.0f
        ballX += dx * coef
        ballY += dy * coef
        vx += x * t
        vy += y * t

        if (ballX - radius < 0 && vx < 0) {
            vx = -vx / 1.5f
            ballX = radius
        } else if (ballX + radius > surfaceWidth && vx > 0) {
            vx = -vx / 1.5f
            ballX = surfaceWidth - radius
        }
        if (ballY - radius < 0 && vy < 0) {
            vy = -vy / 1.5f
            ballY = radius
        } else if (ballY + radius > surfaceHeight && vy > 0) {
            vy = -vy / 1.5f
            ballY = surfaceHeight - radius
        }
    }
}

override fun onCreate(savedInstanceState: Bundle?) {
    super.onCreate(savedInstanceState)
    requestedOrientation = ActivityInfo.SCREEN_ORIENTATION_REVERSE_PORTRAIT
    setContentView(R.layout.activity_main)
    val holder = surfaceView.holder
    holder.addCallback(this)
}

private fun drawCanvas(){
    val canvas = surfaceView.holder.lockCanvas()
    canvas.drawColor(Color.YELLOW)
    canvas.drawCircle(ballX, ballY, radius, Paint().apply {
        color = Color.MAGENTA
    })
    surfaceView.holder.unlockCanvasAndPost(canvas)
}

}

試したこと

定義したメソッドがどこからも使われていないという事なので、
「surfaceChanged」メソッド、
「onSensorChanged」メソッド、
「onCreate」メソッド
の中に記述すれば、センサーに変更があった際に呼び出されるかと思いましたが
エラーを消すことが出来ませんでした。

どなたか詳しい方アドバイスをお願い致します。

Comments

[jimbe]

参考書とこのコードとの関係が分かりません.
「参考書に記載されているソースコードの実装」というのは, 参考書に載っているコードを丸写ししたということでしょうか, それとも参考書に仕様が載っていてそれに合わせて実装したということでしょうか.
そこには drawCanvas は書かれていないのでしょうか?

[jimuma]

ご質問有難うございます。
参考書にはソースコードの全てが書かれている訳ではなく、drawCanvasの部分についても
「MainActivity.ktに以下のコードを記載します。
の後に「private fun drawCanvas(){」の後の記述が書かれていて、MainActivity.ktの中のどこか、
までは明示されていません。

先ほど「SurfaceHolder.Callback{」の中に記載してみましたが、やはり同じでした。
どなたか原因と記載箇所が分かりますでしょうか。

[keicha_hrs]

drawCanvas()を呼び出す処理が存在しないので、never usedとなるのは必然です。参考物にはdrawCanvas()を呼び出す処理が存在するものと思われますが、何か見落としがあるのではないでしょうか。

[jimuma]

参考書には完全なソースコードがないのでお手本とのすりあわせは出来ないのですが、
先ほど、「surfaceCreated{」の中でdrawCanvas()を呼び出したら動いたことが確認出来ました。
有難うございました。

Answer 1

サンプルによると, onSensorChanged の

if (event.sensor.type == Sensor.TYPE_ACCELEROMETER) {
  :
  (略)
  :
  drawCanvas()
}

と入っているようです.

Comments

[jimuma]

有難うございます！
エラーが消えました！
コメント遅くなってすみません。

Answer 2

「surfaceCreated{」の中でdrawCanvas()を呼び出したら動いたことが確認出来ました。

下記を参考にさせていただきました。

https://tech.pjin.jp/blog/2014/02/28/surfaceview%E3%82%92%E7%94%A8%E3%81%84%E3%81%9F%E7%B0%A1%E5%8D%98%E3%81%AAandroid%E3%82%B5%E3%83%B3%E3%83%97%E3%83%AB%E3%82%A2%E3%83%97%E3%83%AA/