Android Studio でループ処理で1秒ごとにカウントを表示させたい

Question

Android stdioでループ処理で1秒ごとにカウントを表示させたいのですが、うまくいきません。

xmlでテキストビューを一つ配置しました。

実行はできるのですが一瞬で落ちてしまいます。どこが悪いのでしょうか？
詳しい方教えてください。

package net.npaka.audiorec20210927;

import android.app.Activity;
import android.os.Bundle;
import android.widget.TextView;

public class MainActivity extends Activity
{
    public TextView TEXT;
    public Thread thread;

    public int co=0;

    @Override
    protected void onCreate(Bundle savedInstanceState)
    {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        TEXT = findViewById(R.id.textView);
        TEXT.setText("アクティビティ起動時");

        
        new Thread(new Runnable()//ループ
        {
            @Override
            public void run()
            {
                // マルチスレッドにしたい処理 ここから
                try//??
                {
                    co++;
                    thread.sleep(1000);
                    TEXT.setText("ループNo." + co);

                }
                catch (InterruptedException e)//??
                {

                }

                // マルチスレッドにしたい処理 ここまで
            }
        }).start();
    }


}

Comments

[K_3578]

>Android stdio
Android Studioのtypoでは？
因みにAndroid Studioタグもあるのでそうであればそちらのタグを入れてAndroidタグを外すと良いかと。

[jimbe]

AndroidStudio のタグは、本来は AndroidStudio 自体に関する質問で使うので、単に開発環境として使っているだけの場合には適さないのではないでしょうか。
(android studio 以外の開発環境でも「実行したら落ちる」のは同じでしょう。)
変えて頂くならタイトル等の「Android st(u)dioで」という表現のほうかと思います。

[jimbe]

> 実行はできるのですが一瞬で落ちてしまいます
基本的には、落ちた時にログがどこかしらに出ているはずです。
logcat 等をご確認頂いて、具体的に何が起きているのかを調べてみては如何でしょうか。

[yasiko]

Android Studioですね。綴り間違えてました。

Answer 1

Handler#postDelayed を用いては如何でしょうか。

---

import androidx.appcompat.app.AppCompatActivity;

import android.os.*;
import android.widget.TextView;

public class MainActivity extends AppCompatActivity {
  private TextView textView;
  private int count = 0;

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

    textView = findViewById(R.id.textView);
    textView.setText("アクティビティ起動時");

    Handler handler = new Handler(getMainLooper());
    postLoop(handler, 1000); //[ms]
  }

  private void postLoop(Handler handler, int delayMillis) {
    handler.postDelayed(() -> {
      textView.setText("ループNo." + ++count);
      postLoop(handler, delayMillis);
    }, delayMillis);
  }
}

Comments

[yasiko]

解答ありがとうございます。
こちらも正常にカウントアップしました。

Answer 2

UIスレッド以外からUI操作をしたからだと思います。
setTextをUIスレッドで実行させる（Handler　を使う）等でしょうか

Comments

[yasiko]

ハンドラを使うとすぐには落ちず、一回カウントアップできました。
ループ処理はできないのでしょうか？

protected void onCreate(Bundle savedInstanceState)
{
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);
TEXT = findViewById(R.id.textView);
TEXT.setText("起動時");

//UIスレッドはメイン的なもの
Handler handler = new Handler(); // ハンドラー　　UIスレッドで実行できるようにする。

handler.post(new Runnable()
{
@Override
public void run()
{
// マルチスレッドにしたい処理 ここから
try
{
thread.sleep(1000);
co++;
TEXT.setText("ループNo." + co);

}
catch (InterruptedException e)
{

}
return;
}

} );//start();;tukaenai
｝

[退会済みユーザー]

例えばですが 以下のように　実際にUIにアクセスする部分だけを切り出すイメージです

new Thread(new Runnable()//ループ
{
@Override
public void run()
{
// マルチスレッドにしたい処理 ここから
try//??
{
co++;
thread.sleep(1000);
handler.post(new Runnable() {
@Override
public void run() {
TEXT.setText("ループNo." + co);
}
});
}
catch (InterruptedException e)//??
{

}

// マルチスレッドにしたい処理 ここまで
}
}).start();

[yasiko]

返信ありがとうございます。イメージはなんとなく掴めました。
↑のコードに変えてみたのですが、落ちてしまいました。
public class MainActivity extends Activity
{
public TextView TEXT;
public Thread thread;
private Handler handler;

public int co=0;

@Override
protected void onCreate(Bundle savedInstanceState)
{
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);
TEXT = findViewById(R.id.textView);
TEXT.setText("起動時");

new Thread(new Runnable()//ループ
{
@Override
public void run()
{
// マルチスレッドにしたい処理 ここから
try//??
{

co++;
thread.sleep(1000);

handler.post(new Runnable()
{
@Override
public void run()
{
TEXT.setText("ループNo." + co);
}

});

}
catch (InterruptedException e)//??
{

}

// マルチスレッドにしたい処理 ここまで
}

}).start();
}

}

[退会済みユーザー]

直前では

//UIスレッドはメイン的なもの
Handler handler = new Handler(); // ハンドラー　　UIスレッドで実行できるようにする。

こうされていたかと思いますが

handler変数が初期化されていないからだと思います。

private Handler handler = new Handler();

とされてみてはいかがでしょう

[yasiko]

落ちなくはなるのですが、カウントアップは一回しかしてくれませんでした。orz

public class MainActivity extends Activity
{
public TextView TEXT;
public Thread thread;
public int co=0;

@Override
protected void onCreate(Bundle savedInstanceState)
{
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);
TEXT = findViewById(R.id.textView);
TEXT.setText("起動時");

//Handler handler = new Handler();

new Thread(new Runnable()//ループ
{
private Handler handler = new Handler();

@Override
public void run()
{
// マルチスレッドにしたい処理 ここから
try//??
{

co++;
thread.sleep(1000);

handler.post(new Runnable()
{
@Override
public void run()
{
TEXT.setText("ループNo." + co);
}

});

}
catch (InterruptedException e)//??
{

}

// マルチスレッドにしたい処理 ここまで
}

}).start();
}

}

[退会済みユーザー]

原因を探す手段として、
Logを出してcoがどう変わっていくかを確認してみるのはどうでしょうか

例えば
TEXT.setText("ループNo." + co);
の下に
Log.d("MainActivity", "ループNo." + co)

を入れてみると　毎秒ここが動いていたら
AndroidStudio下の方の　Logcatというところにでてきます

[退会済みユーザー]

整形してみてみましたが、　Threadのrun内がループしてないみたいですね
runの中身を　while(True){}で囲うと繰り返し動くと思います

jimbeさんも回答されてますが、そちらのほうが実装がシンプルで良いかもしれません

[yasiko]

while(true){try.....}で正常にテキストが更新されカウントアップしました。
またlogcatで変数の中身が見れることを初めて知りました。
Log.d()も正常に動作してました。
親身にありがとうございましたorz