SurfaceViewを使用しているのですが、変数が反映されないです。かつ、描画内容が表示されなくなりました。

Question

###前提・実現したいこと
SurfaceViewを用いてUIスレッドと非同期の描画処理を行おうとしています。
そんな中で、以下の問題が発生しました。

###発生している問題・エラーメッセージ
エラーメッセージはありません。
変数が変わっているはずなのに反映されないです。
具体的には、privateな変数を用意して、Activityクラスから随時setで値を受け取りそれを反映させるというものなのですが、以前のUIスレッドに対して同期の処理をさせていたものから非同期のものに書き換えると変数が反映されなくなりました。

###修正していく中で新たに出た問題（追記）
SurfaceViewの記事を見て描画メソッドをrun()に、holderを渡さないように変えました（下のコードでholderを受け渡ししているのが古いもの）。すると、描画されなくなりました。run()はpublicです。

###該当のソースコード
Threadを使用している部分を抜粋しました。

@Override
    public void surfaceCreated(SurfaceHolder holder) {
        isSurfaceStanby = true;
        //test
        thread = new Thread();
        thread.start();
        //test
        //doDraw(holder);///test
    }

public void todraw() {
        while (thread != null) {///test
            doDraw(getHolder());///test
        }///test
        //test
        try {
            Thread.sleep(50);
        }catch (Exception e){}
        //test
    }

・追記分

 public C(Context context) {
        super(context);
        holder = getHolder();
        init();
    }
    public C(Context context, AttributeSet attrs) {
        super(context, attrs);
        holder = getHolder();
        init();
    }
    public C(Context context, AttributeSet attrs, int defStyleAttr) {
        super(context, attrs, defStyleAttr);
        holder = getHolder();
        init();
    }
    public C(Context context, AttributeSet attrs, int defStyleAttr, int defStyleRes) {
        super(context, attrs, defStyleAttr, defStyleRes);
        holder = getHolder();
        init();
    }

public void run() {
    Canvas canvas;
    while (thread != null) {
    //描画処理
    try {
        Thread.sleep(100);
    }catch (Exception e) {
    }
    }
}

@Override
    public void surfaceCreated(SurfaceHolder holder) {
        isSurfaceStanby = true;
        //test
        thread = new Thread();
        thread.start();
        //test
    }

###試したこと
引数で値を渡したり、getを使ったりしてみたのですが結果は同じでした。

###補足情報
同時に加速度センサーを使用しているのですが、センサー精度が変化するときに呼ばれるonAccuracyChangedが常に呼び出される不具合が発生しています。
※上記の不具合は追記時の書き方を変えることによってなくなりました。
//testの記述は目印です、特に今回の質問とは関係ありません。

Answer 1

ただのThreadはstartしても**何もしません。**コード中で書いてる

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

は、無駄にThreadを作っただけで本当に何もしない無駄コードです。
Threadが非同期処理として機能するのは、Threadを継承したクラスでrunをオーバーライドしたクラスのインスタンスか、ThreadのコンストラクタにRunnableオブジェクトを渡した場合のみです。

つまり今回の場合、runの処理を非同期で行おうとしたら、このクラスにRunnableをimplementsして、Threadのコンストラクタにthisを渡してやる必要があります。

Comments

[ko20vonobird]

回答ありがとうございます。
swordoneさんのおっしゃる通りに直させていただいたところ描画がされるようになりました。また、変数が反映されないという初期の問題も偶然解決することができました。
今回もありがとうございました。