Object.wait()メソッドを使うと処理が続けて実行されない。

Question

テキストビューに配列の要素であるString文字列を一つずつセットしています。
一文字ずつセットされているのが分かるようにObject.wait()メソッドを使って、一文字追加されるたびにミリ秒待つ処理を入れています。
テキストビューに配列の文字列がミリ秒毎にセットされていく筈なのですが、一時、間があった後に全て同時にテキストビューにセットされるといった動き方をします。
一体何故このような動きをするのでしょうか？
メソッドを直そうにも原因が分からず困っています。
よろしくお願いします。

public void textmessageset(){
		String str = texts[0][0].toString();
		char data[] = str.toCharArray();
		textmessage.setText("");
		for(int i = 0; i < data.length; i++){
            //ミリ秒毎にテキストビューにセット
			sleep(50);
			String a = textmessage.getText().toString();
			String tt = a.toString() + String.valueOf(data[i]).toString();
			textmessage.setText(tt.toString());
		}
	}

//ミリ秒待つメソッド
public synchronized void sleep(long msec)
	{
		try
		{
			wait(msec);
		}catch(InterruptedException e){
		}
	}

Answer 1

まず、Object.wait()は、synchronized内で別スレッドを待機するのに使う機能です。
スリープ機能として使うのには適していません。
Javaでスリープをする一般的な方法は、Thread.sleep(long)です。

Androidで、今回のケースのような場合は、AsyncTaskとHandlerを組み合わせるのが一般的のようです。
また、ご質問とは関係ありませんが、TextViewに文字列を追加する箇所はTextView#append(String)が使えますよ。

import android.app.Activity;
import android.os.AsyncTask;
import android.os.Bundle;
import android.os.Handler;
import android.view.Menu;
import android.view.MenuItem;
import android.widget.TextView;

public class MainActivity extends Activity {

    Handler handler = new Handler();

    private class MessageTask extends AsyncTask<String, Void, Integer> {

        @Override
        protected Integer doInBackground(String... args) {
            // 非UIスレッドで実行
            String[] texts = { /* ... テキストの配列 ... */ };
            for (String text : texts) {
                char[] a = text.toCharArray();
                for (int i = 0; i < a.length; i++) {
                    appendText(String.valueOf(a[i]));
                    try {
                        Thread.sleep(100L);
                    } catch (InterruptedException e) {
                        throw new RuntimeException(e);
                    }
                }
            }
            return 0;
        }

    }

    void appendText(final String text) {
        handler.post(new Runnable() { // UIスレッドで実行
            @Override
            public void run() {
                TextView view = (TextView) findViewById(R.id.textView);
                view.append(text);
            }
        });
    }

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

    // 以下略

}

---

（2015/01/16 01:03, 01:09, 01:32のコメントへの回答）

サンプルコードは、ご質問の問題に焦点を絞って単独で試せるように書いているので、細かいところは応用して使っていただければと思います。
サンプルコードの変数viewはtextmessageに読み替えてください。

クラスの外にあるtextsにはMessageTaskからはMainActivity.this.textsでアクセスできると思います。(static classにしているとこれではできませんがstaticでないので)
それよりも、MainActivityにgetClickedTextメソッドを作ってそれを呼ぶようにした方が良いかもしれませんね。
具体的には、こんな感じです。

// MainActivity
String getClickedText() {
    return texts[event][click].toString();
}

// MessageTask#doInBackground
String text = getClickedText();
char[] a = text.toCharArray();
for (int i = 0; i < a.length; i++) {
    appendText(String.valueOf(a[i]));
    try {
        Thread.sleep(100L);
    } catch (InterruptedException e) {
        throw new RuntimeException(e);
    }
}

textmessage.setTextにつきましては、サンプルコードではview.append(text)としています。textShowにnew MessageTask().execute()を書いてみるとどうなりますか？

Comments

[mop]

解答有難う御座います。
現在実装する為にプログラムを書いていたのですが、
MessageTaskクラスの
String[] texts = { /* ... テキストの配列 ... */ };
のところで分からないことがあります。
textsという配列がクラスの外にあるためMessageTaskクラスの方で参照できません。
メインアクティビティにあるのはメインアクティビティにあるメソッドでもテキストの配列を参照しているからなのですが。
後、二次元配列の為、コンパイルエラーが出るようなのですが、どのように記述すればよいのでしょうか？

[mop]

済みません追記です。
MainActivity mai = new MainActivity();
String[] texts = (String[]) mai.texts[event][click];
このように記述してみたのですが宜しいでしょうか？

[mop]

何度も済みません。
new MessageTask().execute();
はOnCreate()に書くとしまして、テキストビューに文字をセットする際、今まではtextShow()というメソッドの中にtextmessage.setText(配列)のようにしていたのですが、御回答されましたコードの場合どのようにしてテキストビューに文字をセットするのでしょうか？

[argius]

追記しました。いまコードを書ける環境でないので、分かりにくかったらすみません。

[argius]

追記を修正しました。

[mop]

返信、有難う御座います。
AsyncTaskとHandlerを組み合わせるなんて発想が自分には出来ませんでした。
期待以上のコードを教えていただいて今後のプログラム制作にも、一層やる気が増して来ました。
解答以外の知識までとても参考になりました。

[argius]

UIとの非同期処理は良く使いますが少し難易度が高いので、
初心者の方には最初の難関かも知れません。
自由に使えるようになるには時間がかかると思いますがめげずに頑張って下さい。

素敵なアプリができると良いですね。

[argius]

補足：KSwordOfHasteさんのSwingの例は、AndroidのUIスレッドとの非同期処理という観点と（GUIの非同期処理という点については）本質的には同じです。AndroidでないGUIをお使いの方々にもご理解いただければ幸いです。

[mop]

有難う御座います。励みになります。

Answer 2

setTextの結果が実際にビジュアルに反映される(JTextFieldなどの表示に反映される)のはtextmessagesetメソッドの実行が終わり、イベントディスパッチャーへ戻った後に(JTextFieldが更新されたために起動される再描画制御の中で)JTextField.paintComponent()が呼ばれたタイミングになると思います。

つまりsetText()の延長上でpaintComponent()が呼ばれるわけではないので、最終的な内容しか画面上に見えないことになります。もしアニメーション的なことがしたければ、別スレッドで待ちをはさみながらsetText要求をEDTへ投げるということをする必要があると思います。

void じわじわ設定(String message) {
  new Thread() {
    @Override
    public void run() {
      try {
        for (int i = 0; i <= message.length(); i++) {
          Thread.sleep(50);
          String s = message.substring(0, i);
          SwingUtilities.invokeLater(()->textmessage.setText(s));
        }
      } catch (InterruptedException e) {}
    }
  }.start();
}

追記: 本件にObject#waitは無関係な気がするので普通にThread#sleepで書いています。

Comments

[mop]

SwingUtilities.invokeLater(()->textmessage.setText(s));
は何をしているところでしょうか？コンパイルエラーが出てしまうのですが。
最初の方の説明が難しくてもう少し自分の勉強が必要に感じました。
とりあえず、別のスレッドで実行をする方法を模索するなり回答者さんに聞くなりしてみたいと思います。
解答有難う御座いました。

[KSwordOfHaste]

前提環境がandroidなのを見落としてました！自分のコメントは一般OS用のswingでの話しです。混乱させてしまい申し訳ないです。
argiusさんのコメントを拝見するとUIスレッドとは別のスレッドでコントロールを更新するという考え方自体はandroid/swingともに共通のようですが、実装方法が違います。argiusさんのコメントを参照ください。失礼しました。

[mop]

swingのことは分かりませんが、共通の形で実装できることを理解できれば、自分なりに工夫できたと思っています。
そもそも、質問には、直しようがないので原因を知りたいという風に書いていますので、解答自体にはかなり大きな意義がありました。
有難う御座いました。

Answer 3

メインスレッド上でwaitしていませんか？
別スレッドでwaitしてHandlerなのでポストしてみてください。

Comments

[mop]

public void textmessageset() throws InterruptedException{
String str = texts[event][click].toString();
char data[] = str.toCharArray();
textmessage.setText("");
Thread timeWait = new Thread();
for(int i = 0; i < data.length; i++){
timeWait.start();
try{
timeWait.join();
String a = textmessage.getText().toString();
String tt = a.toString() + String.valueOf(data[i]).toString();
textmessage.setText(tt.toString());
}catch(InterruptedException e){

}
}

}
public void run(){
sleep(50);
}

このような感じでやってみたのですが、上手くいきませんでした。
この場合の別スレッドでwait()させるにはどうすればよいでしょうか？