Android Surface getHolderが使えない？

Question

前提

Android で加速度センサを用いた玉転がしゲームを作成しています。
それにサーフェスを使うためにいろいろ調べながら書いているのですが、調べて出てきた getHolder()　を自分のコードで書いてみたところ、そこがエラーになってしまいました。
どうしてこうなってしまったのでしょうか？正しい書き方をお教えくださいませ。

実現したいこと

サーフェスを追加するための正しい書き方を知りたい。

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

該当のソースコード

MainActivity.java

package com.example.accball;

import androidx.annotation.NonNull;
import androidx.appcompat.app.AppCompatActivity;
import androidx.core.widget.TextViewCompat;

import android.hardware.Sensor;
import android.hardware.SensorEvent;
import android.hardware.SensorEventListener;
import android.hardware.SensorManager;
import android.os.Bundle;
import android.util.Log;
import android.view.SurfaceHolder;
import android.view.SurfaceView;
import android.widget.TextView;

public class MainActivity extends AppCompatActivity implements SensorEventListener, SurfaceHolder.Callback {

    private int surfaceWidth = 0;
    private int surfaceHeight = 0;


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        //サーフェス
        SurfaceHolder holder = getHolder();
        holder.addCallback(this);
    }



    //センサーの値が変わった時に呼ばれる
    public void onSensorChanged(SensorEvent event) {
        if (event == null) {
            return;
        }

        //ログにも出力
        Log.d("MainActivity",
                "x = ${event.values[0].toString()}" +
                        "y = ${event.values[1].toString()}" +
                        "z = ${event.values[2].toString()}"
        );

    }

    //サーフェスが作成されたとき
    @Override
    public void surfaceCreated(@NonNull SurfaceHolder holder) {
        //センサーマネジャーのインスタンスを取得
        SensorManager sensorManager = (SensorManager) getSystemService(SENSOR_SERVICE);
        //加速センサを指定
        Sensor accSensor = sensorManager.getDefaultSensor(
                Sensor.TYPE_ACCELEROMETER);
        //リスナーをセット　更新頻度は3-30ms
        sensorManager.registerListener(this, accSensor, SensorManager.SENSOR_DELAY_GAME);
    }

    //サーフェスが変更されたとき
    @Override
    public void surfaceChanged(@NonNull SurfaceHolder holder, int format, int width, int height) {
        float surfaceWidth = width;
        float surfaceHeight = height;
    }

    //サーフェスがはかいされたとき
    @Override
    public void surfaceDestroyed(@NonNull SurfaceHolder holder) {
        //センサーマネジャーのインスタンスを取得
        SensorManager sensorManager = (SensorManager) getSystemService(SENSOR_SERVICE);
        // Listenerを解除してセンサーの値の監視を終了
        sensorManager.unregisterListener(this);
    }

}


    @Override
    public void onAccuracyChanged(Sensor sensor, int accuracy) {
        //これがないとエラーになるので形だけ
    }
}

activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity">

    <SurfaceView
        android:id="@+id/surfaceView"
        android:layout_width="0dp"
        android:layout_height="0dp"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent" />
</androidx.constraintlayout.widget.ConstraintLayout>

試したこと

SurfaceHolder holder = this.getHolder();
holder = getHolder();
など書いてみましたがだめでした

###追記
SurfaceHolder holder = surfaceview.getHolder();と試してみましたがcannnot resolve symbolと言われてしまいました。

extend surfaceViewと書いてみましたがCannot resolve method 'getHolder' in 'MainActivity'と言われてしまいました。

ここにより詳細な情報を記載してください。

次のように書き換えたところpublic class MainActivityのかっこが閉じるところで. class or interfacce expectedというエラーになりました

Comments

[jimbe]

イメージが見えません。コードのマークダウン中では他のマークダウンが機能しませんので、コードのマークダウンを消してみてください。
プレビューで状況が見えると思いますので、修正しながら確認してみては如何でしょうか。

[hoshi-takanori]

とりあえず SurfaceView のサブクラスを作る必要があるのでは。
https://www.hiroom2.com/2014/04/25/android%E3%81%AEsurfaceview%E3%81%AE%E4%BD%BF%E3%81%84%E6%96%B9/

[dodox86]

hoshi-takanoriさんが示唆されていますが、getHolderメソッドはMainActivityでimplementsしているSensorEventListenerやSurfaceHolderインターフェースのものではなく、
SurfaceViewのものですね。なので、MainActivityクラスのthisで呼び出そうとしてもダメです。

https://developer.android.com/reference/android/view/SurfaceView#getHolder()

「調べて出てきた getHolder()」とのことですが、恐らくどこかで誤解されているのだと思います。私自身はSurfaceViewの扱いに詳しくは無いので、以上コメントのみです。

[poipoipain]

画像が見えなくなっていたところを修正しました
また、言われたことを自分なりに反映してみましたがダメでした。。。

[jimbe]

画像ありがとうございます。

想像ですが、 getHolder() とだけ書いて SurfaceHolder を得ているコードは、 hoshi-takanori さんの言われている「SurfaceView のサブクラス」のコンストラクタではないでしょうか。
(「調べて出てきた getHolder()」という参考元のコードをご確認ください。)
それを「どこでも getHolder() で得られる」と勘違いされているように思います。
dodox86 さんの言われている通り、 getHolder() は SerfaceView のメソッドですので
>SurfaceHolder holder = surfaceview.getHolder();と試してみました
が正解に近づいていると思いますが、そこから、 SurfaceView はレイアウト xml で定義しているのですから
`SurfaceView surfaceview = findViewById(R.id.surfaceView);'
が必要であることにも気付く必要があります。

そして、ご提示のコードでは {} の対応が正しくできておりません。
具体的には、onAccuracyChanged メソッドが MainActivity のクラスの {} の外に出ている形になってしまっていますので、 MainActivity に onAccuracyChanged メソッドが無い状態になっています。
落ち着いて、 {} の対応が正しくなるように修正する必要があります。

[poipoipain]

ご回答＆細かい解説ありがとうございます。無事解決しました

Answer 1

SurfaceView にセンサーの各値を 100ms 毎に表示するようにしてみました。

import androidx.appcompat.app.AppCompatActivity;

import android.graphics.*;
import android.hardware.Sensor;
import android.hardware.SensorEvent;
import android.hardware.SensorEventListener;
import android.hardware.SensorManager;
import android.os.Bundle;
import android.util.Log;
import android.view.SurfaceHolder;
import android.view.SurfaceView;

public class MainActivity extends AppCompatActivity implements SensorEventListener, Runnable {
  private static final String LOG_TAG = "MainActivity";

  private SensorManager sensorManager;
  private SurfaceHolder holder;

  private Thread thread;

  private int[] sensorLock = new int[0];
  private float sensorX, sensorY, sensorZ;

  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    sensorManager = (SensorManager) getSystemService(SENSOR_SERVICE);

    SurfaceView surfaceView = findViewById(R.id.surfaceView);
    holder = surfaceView.getHolder();
  }

  @Override
  protected void onResume() {
    super.onResume();
    Sensor accSensor = sensorManager.getDefaultSensor(Sensor.TYPE_ACCELEROMETER);
    sensorManager.registerListener(this, accSensor, SensorManager.SENSOR_DELAY_GAME);

    thread = new Thread(this);
    thread.start();
  }

  @Override
  protected void onPause() {
    super.onPause();
    sensorManager.unregisterListener(this);

    thread.interrupt();
    thread = null;
  }

  @Override
  public void onAccuracyChanged(Sensor sensor, int accuracy) {
    //ignore
  }

  @Override
  public void onSensorChanged(SensorEvent event) {
    if (event == null) return;
    synchronized(sensorLock) {
      sensorX = event.values[0];
      sensorY = event.values[1];
      sensorZ = event.values[2];
    }
    //Log.d(LOG_TAG, "x = " + sensorX + "y = " + sensorY + "z = " + sensorZ);
  }

  @Override
  public void run() {
    Paint paint = new Paint();
    paint.setTextSize(80);
    paint.setColor(Color.BLACK);

    while(!Thread.interrupted()) {
      float x, y, z;
      synchronized(sensorLock) {
        x = sensorX;
        y = sensorY;
        z = sensorZ;
      }

      Canvas canvas = holder.lockCanvas();
      if(canvas != null) {
        canvas.drawColor(Color.WHITE);
        canvas.drawText("w=" + canvas.getWidth() + ", h=" + canvas.getHeight(), 0, 100, paint);
        canvas.drawText("x=" + x, 0, 200, paint);
        canvas.drawText("y=" + y, 0, 300, paint);
        canvas.drawText("z=" + z, 0, 400, paint);
        holder.unlockCanvasAndPost(canvas);
      }

      try {
        Thread.sleep(100L); //[ms]
      } catch(InterruptedException e) {
        break;
      }
    }
  }
}

Answer 2

解決しました。onAccuracyChamgedの関数をclass MainActivityの中に移動させたところエラーが出なくなりました。
getholdのところの書き方は修正画像ｎの書き方で合ってたようです。