android studio あるタイミングでのsetText()について

2018/04/16 13:23

止まった時にコンソールにエラーログなどはでていませんか？また、SimulationViewのupdateは自前のThreadやTimer内で実行していますか？

2018/04/16 13:43

updateの部分を追加いたしました。止まった時エラーは出ていないと思います。

2018/04/16 13:59

main = new MainActivity() とかやってるのが原因ですよね・・・

2018/04/16 14:05 編集

SimulationView class でfindViewbyidをするとnullpointerexeptionになってしまったのでこうやってみました。　どのようにすればよいか教えていただけないでしょうか？

2018/04/16 14:05

あ、ほんとですね^^;

2018/04/16 14:25 編集

update()ってどこにあるメソッドなのですか？SimulationViewクラス？それとも、Ballクラス？

2018/04/16 14:31

Ballクラスです。

2018/04/16 15:31 編集

これは二重苦状態になっていますね。まずMainActivityのインスタンスを新たに生成している点ですが、これを参照して呼び出したメソッドの中でfindViewById()を用いても、正しい結果は得られません。そもそも、ActivityのインスタンスはAndroidフレームワークがその管理下で作成するもので、ユーザーが任意に作成してはいけないものです。それから、update()をScheduledExecutorServiceによる実装タスクの中から呼び出している点。ここで実装したRunnableは別スレッドでの動作になりますが、Activity上に描画されているコンポーネントは、Activityと同じスレッド（UIスレッドと呼ばれます）でなければ操作できません。この二重の問題のために、申し訳ないけどすぐに答えを出せません。ちょっと考えてはみますが・・・

2018/04/16 21:00

かしこまりました。お時間を割いてくださり大変ありがとうございます。

行動規範の内容に同意します

回答1件

ベストアンサー

ざっとですが、こうしたらどうかという案を考えてみました。まずコードです。変更点とその周辺のみを記述しています。

MainActivity.java

Java
1
2    @Override
3    protected void onCreate(Bundle savedInstanceState) {
4        super.onCreate(savedInstanceState);
5        setContentView(R.layout.activity_main);
6
7        // MainActivityのインスタンスを渡す処理を追加
8        SimulationView simView = findViewById(R.id.canvas);
9        simView.getMainActivity(this);
10
11        setBarText();
12    }
13
14

SimulationView.java

Java
1
2    public void init() {
3 
4
5       // main = new MainActivity(); // これはやめる
6    }
7
8    // MainActivityのインスタンスを受け取るメソッドを追加
9    void getMainActivity(MainActivity m) {
10        main = m;
11    }
12
13    public void startnow() {
14        ScheduledExecutorService executor = Executors.newSingleThreadScheduledExecutor();
15        executor.scheduleAtFixedRate(new Runnable() {
16            @Override
17            public void run() {
18                main.runOnUiThread(ball); // runOnUiThread()に変更
19                Log.d("test", "ループ");
20                draw();
21            }
22        }, 100, 100, TimeUnit.MICROSECONDS);
23    }
24    
25    // BallにRunnableを実装
26    class Ball implements Runnable {
27 
28        // run()の中でupdate()を呼ぶ
29        @Override
30        public void run() {
31            update();
32        }
33
34        void update() {
35             // この中は変更なし
36        }
37    }
38

MainActivityの持つメソッドを呼ぶために、SimulationViewにgetMainActivity()というメソッドを追加します。このメソッドを介して、MainActivityのインスタンスを受け取って保存します。newするのではなくすでに存在するインスタンスを受け取って参照することで、countUpBarText()も正常に動くはずです。

それから、scheduleAtFixedRate()の問題を解決するため、まずBallにRunnableを実装します。Ballの中にrun()をオーバーライドして、その中からupdate()を呼び出すようにします。

そして、scheduleAtFixedRate()に記述したrun()の中では、update()を直接呼ぶのではなく、runOnUiThread()を呼び出すように変更します。これは、メソッド名を呼んで字の如くで、UIスレッドで処理して欲しい内容を他スレッドから渡すためのメソッドです。これはActivityが持つメソッドなので、mainを参照して呼び出します。その引数として、Runnableを実装したBallのインスタンスを渡すようにします。

こんな感じでどうでしょうかね？テストしていない（できない）ので動く保証はできませんが・・・。

（2017/04/19追記）
kakajikaさんのご指摘を受けて書き直したものです。前のものは残しておきます。

MainActivity.java

Java
1// SimulationView.OnSimulationUpdateListenerを実装（implements）
2public class MainActivity extends AppCompatActivity implements SimulationView.OnSimulationUpdateListener {
3
4    @Override
5    protected void onCreate(Bundle savedInstanceState) {
6        super.onCreate(savedInstanceState);
7        setContentView(R.layout.activity_main);
8
9        // Listenerをセットする
10        SimulationView simview = findViewById(R.id.canvas);
11        simview.setOnSimulationUpdateListener(this);
12
13        setBarText();
14    }
15
16
17    //壁に当たった時に呼ばれる
18    @Override
19    public void countUpBarText(int key) {//textviewを変更する
20         // この中は変更なし
21    }
22}
23

SimulationView.java

Java
1public class SimulationView extends SurfaceView implements SurfaceHolder.Callback {
2
3    //private MainActivity main;  // これはやめる
4    private OnSimulationUpdateListener mListener; // 追加
5
6    void setOnSimulationUpdateListener(OnSimulationUpdateListener listener) {
7        mListener = listener;
8    }
9
10    public void startnow() {
11        ScheduledExecutorService executor = Executors.newSingleThreadScheduledExecutor();
12        executor.scheduleAtFixedRate(new Runnable() {
13            @Override
14            public void run() {
15                post(ball); // post()に変更
16                Log.d("test", "ループ");
17                draw();
18            }
19        }, 100, 100, TimeUnit.MICROSECONDS);
20    }
21
22   class Ball implements Runnable {
23
24        // run()の中でupdate()を呼ぶ
25        @Override
26        public void run() {
27            update();
28        }
29
30        void update() {
31           //右壁
32            if (x < 0) {
33                checkBar = 1;
34                mListener.countUpBarText(checkBar); // mListenerを参照
35                dx = -dx;
36            }
37
38            // 以降、同様にmainをmListenerに置き換える
39
40        }
41    }
42
43    interface OnSimulationUpdateListener {
44        void countUpBarText(int key);
45    }
46}

interfaceの解説をするのはちょっと辛いので、書籍やネットの資料を参考にしてください。やっていることの趣旨としては、前とさほど違いはありません。

＞ kakajikaさん

こんなところでどうでしょうか・・・。（あまり自信ない）

投稿2018/04/17 16:25

編集2018/04/19 13:04

総合スコア6768

2018/04/19 04:34

できればMainActivityをそのまま渡すのは密結合になってしまって避けたいので、interface（例えばOnSimulationUpdateListener）で定義してMainActivityにそれを実装する、という形がいいかなと思います。メインスレッドでの実行は、Activityを経由しなくてもView::postで可能です。

2018/04/19 13:29 編集

以前も密結合についてご指摘を受けたことがあったので、またやってしまいました。無用に混乱させて、質問者様においては申し訳ないです。

2018/04/19 14:08

回答修正ありがとうございます、いいと思います! つい気になってツッコミ入れてしまって、すみません。