web-dev-qa-db-ja.com

Android "ビュー階層を作成した元のスレッドだけがそのそのビューに触れることができます。"

私はAndroidでシンプルな音楽プレーヤーを作りました。各曲のビューには、次のように実装されたSeekBarが含まれています。

public class Song extends Activity implements OnClickListener,Runnable {
    private SeekBar progress;
    private MediaPlayer mp;

    // ...

    private ServiceConnection onService = new ServiceConnection() {
          public void onServiceConnected(ComponentName className,
            IBinder rawBinder) {
              appService = ((MPService.LocalBinder)rawBinder).getService(); // service that handles the MediaPlayer
              progress.setVisibility(SeekBar.VISIBLE);
              progress.setProgress(0);
              mp = appService.getMP();
              appService.playSong(title);
              progress.setMax(mp.getDuration());
              new Thread(Song.this).start();
          }
          public void onServiceDisconnected(ComponentName classname) {
              appService = null;
          }
    };

    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.song);

        // ...

        progress = (SeekBar) findViewById(R.id.progress);

        // ...
    }

    public void run() {
    int pos = 0;
    int total = mp.getDuration();
    while (mp != null && pos<total) {
        try {
            Thread.sleep(1000);
            pos = appService.getSongPosition();
        } catch (InterruptedException e) {
            return;
        } catch (Exception e) {
            return;
        }
        progress.setProgress(pos);
    }
}

これはうまくいきます。今私は、曲の進行の秒/分を数えるタイマーが欲しいです。それで、私はレイアウトにTextViewを入れ、findViewById()onCreate()でそれを得て、そしてこれをrun()の後のprogress.setProgress(pos)に入れます:

String time = String.format("%d:%d",
            TimeUnit.MILLISECONDS.toMinutes(pos),
            TimeUnit.MILLISECONDS.toSeconds(pos),
            TimeUnit.MINUTES.toSeconds(TimeUnit.MILLISECONDS.toMinutes(
                    pos))
            );
currentTime.setText(time);  // currentTime = (TextView) findViewById(R.id.current_time);

しかし、その最後の行は私に例外を与えます:

Android.view.ViewRoot $ CalledFromWrongThreadException:ビュー階層を作成した元のスレッドだけがそのビューに触れることができます。

それでも、ここでは基本的にSeekBarで行っているのと同じことを行っています - onCreateでビューを作成してからrun()でそれに触れる - そしてそれは私にこの苦情を与えません。

819
herpderp

バックグラウンドタスクのUIを更新する部分をメインスレッドに移動する必要があります。これには簡単なコードがあります。

runOnUiThread(new Runnable() {

    @Override
    public void run() {

        // Stuff that updates the UI

    }
});

Activity.runOnUiThread のドキュメント。

これをバックグラウンドで実行されているメソッド内に入れ子にして、更新を実装するコードをブロックの途中でコピーペーストします。できるだけ少ないコードを含めるようにしてください。そうしないと、バックグラウンドスレッドの目的に反することになります。

1706
providence

私はrunOnUiThread( new Runnable(){ ..run()の中に入れることでこれを解決しました:

thread = new Thread(){
        @Override
        public void run() {
            try {
                synchronized (this) {
                    wait(5000);

                    runOnUiThread(new Runnable() {
                        @Override
                        public void run() {
                            dbloadingInfo.setVisibility(View.VISIBLE);
                            bar.setVisibility(View.INVISIBLE);
                            loadingText.setVisibility(View.INVISIBLE);
                        }
                    });

                }
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
            Intent mainActivity = new Intent(getApplicationContext(),MainActivity.class);
            startActivity(mainActivity);
        };
    };  
    thread.start();
124

これに対する私の解決策:

private void setText(final TextView text,final String value){
    runOnUiThread(new Runnable() {
        @Override
        public void run() {
            text.setText(value);
        }
    });
}

このメソッドはバックグラウンドスレッドで呼び出します。

52
Angelo Angeles

通常、ユーザーインターフェイスを含むすべてのアクションはメインスレッドまたはUIスレッド、つまりonCreate()およびイベント処理が実行されるアクションで行われる必要があります。それを確実にする一つの方法は runOnUiThread() を使うことです、もう一つはハンドラを使うことです。

ProgressBar.setProgress()は常にメインスレッド上で実行されるメカニズムを持っているので、それがうまくいった理由です。

無痛スレッディングを参照してください

25
bigstones

私はこのような状況にありましたが、Handler Objectを使った解決策を見つけました。

私の場合は、 オブザーバパターン でProgressDialogを更新します。私の見解はオブザーバを実装し、updateメソッドをオーバーライドします。

だから、私のメインスレッドはビューを作成し、別のスレッドはProgressDialopと....を更新するupdateメソッドを呼び出します。

ビュー階層を作成した元のスレッドだけがそのビューに触れることができます。

Handlerオブジェクトに関する問題を解決することは可能です。

以下は、私のコードのさまざまな部分です。

public class ViewExecution extends Activity implements Observer{

    static final int PROGRESS_DIALOG = 0;
    ProgressDialog progressDialog;
    int currentNumber;

    public void onCreate(Bundle savedInstanceState) {

        currentNumber = 0;
        final Button launchPolicyButton =  ((Button) this.findViewById(R.id.launchButton));
        launchPolicyButton.setOnClickListener(new OnClickListener() {

            @Override
            public void onClick(View v) {
                showDialog(PROGRESS_DIALOG);
            }
        });
    }

    @Override
    protected Dialog onCreateDialog(int id) {
        switch(id) {
        case PROGRESS_DIALOG:
            progressDialog = new ProgressDialog(this);
            progressDialog.setProgressStyle(ProgressDialog.STYLE_HORIZONTAL);
            progressDialog.setMessage("Loading");
            progressDialog.setCancelable(true);
            return progressDialog;
        default:
            return null;
        }
    }

    @Override
    protected void onPrepareDialog(int id, Dialog dialog) {
        switch(id) {
        case PROGRESS_DIALOG:
            progressDialog.setProgress(0);
        }

    }

    // Define the Handler that receives messages from the thread and update the progress
    final Handler handler = new Handler() {
        public void handleMessage(Message msg) {
            int current = msg.arg1;
            progressDialog.setProgress(current);
            if (current >= 100){
                removeDialog (PROGRESS_DIALOG);
            }
        }
    };

    // The method called by the observer (the second thread)
    @Override
    public void update(Observable obs, Object arg1) {

        Message msg = handler.obtainMessage();
        msg.arg1 = ++currentPluginNumber;
        handler.sendMessage(msg);
    }
}

この説明は このページ で見つけることができます、そして、あなたは「2番目のスレッドでのProgressDialogの例」を読んでください。

20
Jonathan

あなたは@摂理の答えを受け入れたようです。念のために、あなたもハンドラを使用することができます!まず、intフィールドを実行します。

    private static final int SHOW_LOG = 1;
    private static final int HIDE_LOG = 0;

次に、フィールドとしてハンドラインスタンスを作成します。

    //TODO __________[ Handler ]__________
    @SuppressLint("HandlerLeak")
    protected Handler handler = new Handler()
    {
        @Override
        public void handleMessage(Message msg)
        {
            // Put code here...

            // Set a switch statement to toggle it on or off.
            switch(msg.what)
            {
            case SHOW_LOG:
            {
                ads.setVisibility(View.VISIBLE);
                break;
            }
            case HIDE_LOG:
            {
                ads.setVisibility(View.GONE);
                break;
            }
            }
        }
    };

方法を作る。

//TODO __________[ Callbacks ]__________
@Override
public void showHandler(boolean show)
{
    handler.sendEmptyMessage(show ? SHOW_LOG : HIDE_LOG);
}

最後に、これをonCreate()メソッドに置きます。

showHandler(true);
7
David Dimalanta

私は同様の問題を抱えていた、そして私の解決策は醜いが、それは動作します:

void showCode() {
    hideRegisterMessage(); // Hides view 
    final Handler handler = new Handler();
    handler.postDelayed(new Runnable() {
        @Override
        public void run() {
            showRegisterMessage(); // Shows view
        }
    }, 3000); // After 3 seconds
}
7
Błażej

メインのUIスレッドを乱すことなく、Handlerを使用してビューを削除できます。これがコード例です

new Handler(Looper.getMainLooper()).post(new Runnable() {
                                                        @Override
                                                        public void run() {
                                                           //do stuff like remove view etc
                                                            adapter.remove(selecteditem);
                                                        }
                                                    });
7
Bilal Mustafa

HandlerLooper.getMainLooper()を使います。私にとってはうまくいった。

    Handler handler = new Handler(Looper.getMainLooper()) {
        @Override
        public void handleMessage(Message msg) {
              // Any UI task, example
              textView.setText("your text");
        }
    };
    handler.sendEmptyMessage(1);
6
Sankar Behera

これは明示的にエラーを投げています。どちらのスレッドがビューを作成したかは関係ありません。これは、作成されたビューがそのスレッドのスペース内にあるためです。ビュー作成(GUI)はUI(メイン)スレッドで行われます。そのため、これらのメソッドにアクセスするには常にUIスレッドを使用します。

Enter image description here

上の図では、progress変数はUIスレッドのスペース内にあります。そのため、UIスレッドだけがこの変数にアクセスできます。ここでは、新しいThread()を介して進行状況にアクセスしているため、エラーが発生します。

5
Uddhav Gautam

これは私がdoInBackgroundを使う代わりにAsynctaskからonPostExecuteへのUI変更を要求したときに起こりました。

onPostExecuteでUIを扱うことで私の問題は解決しました。

4
Jonathan

このコードを使用すれば、runOnUiThread関数は必要ありません。

private Handler handler;
private Runnable handlerTask;

void StartTimer(){
    handler = new Handler();   
    handlerTask = new Runnable()
    {
        @Override 
        public void run() { 
            // do something  
            textView.setText("some text");
            handler.postDelayed(handlerTask, 1000);    
        }
    };
    handlerTask.run();
}
4
Hamid

AsyncTask を使用する場合 onPostExecute methodでUIを更新します。

    @Override
    protected void onPostExecute(String s) {
   // Update UI here

     }
3
Deepak Kataria

runOnUiThread APIを使用したくない場合は、実際には完了までに数秒かかる操作に対してAsynTaskを実装できます。しかしその場合、doinBackground()であなたの仕事を処理した後でも、あなたはonPostExecute()で完成したビューを返す必要があります。 Androidの実装では、メインのUIスレッドだけがビューと対話することができます。

2
Sam

これは前述の例外のスタックトレースです

        at Android.view.ViewRootImpl.checkThread(ViewRootImpl.Java:6149)
        at Android.view.ViewRootImpl.requestLayout(ViewRootImpl.Java:843)
        at Android.view.View.requestLayout(View.Java:16474)
        at Android.view.View.requestLayout(View.Java:16474)
        at Android.view.View.requestLayout(View.Java:16474)
        at Android.view.View.requestLayout(View.Java:16474)
        at Android.widget.RelativeLayout.requestLayout(RelativeLayout.Java:352)
        at Android.view.View.requestLayout(View.Java:16474)
        at Android.widget.RelativeLayout.requestLayout(RelativeLayout.Java:352)
        at Android.view.View.setFlags(View.Java:8938)
        at Android.view.View.setVisibility(View.Java:6066)

それであなたが行ってDigをするなら、あなたは知るようになる

void checkThread() {
    if (mThread != Thread.currentThread()) {
        throw new CalledFromWrongThreadException(
                "Only the original thread that created a view hierarchy can touch its views.");
    }
}

mThread は、以下のようにコンストラクタで初期化されます。

mThread = Thread.currentThread();

すべて特定のビューを作成したときにUI Threadで作成し、後でWorker Threadで変更しようとしたことを意味します。

以下のコードスニペットで確認できます。

Thread.currentThread().getName()

私たちがレイアウトを膨らませるとき、そして後であなたが例外を受けているところ。

2
Amit Yadav

コンテキストへの参照を含まないクラスで作業していました。だから私はrunOnUIThread();を使うことができませんでした私はview.post();を使い、それは解決されました。

timer.scheduleAtFixedRate(new TimerTask() {

    @Override
    public void run() {
        final int currentPosition = mediaPlayer.getCurrentPosition();
        audioMessage.seekBar.setProgress(currentPosition / 1000);
        audioMessage.tvPlayDuration.post(new Runnable() {
            @Override
            public void run() {
                audioMessage.tvPlayDuration.setText(ChatDateTimeFormatter.getDuration(currentPosition));
            }
        });
    }
}, 0, 1000);
2
Ifta

私の場合は、呼び出し側が短時間に何度も呼び出しを行うとこのエラーが発生します。関数が0.5秒未満呼び出された場合は無視します。

    private long mLastClickTime = 0;

    public boolean foo() {
        if ( (SystemClock.elapsedRealtime() - mLastClickTime) < 500) {
            return false;
        }
        mLastClickTime = SystemClock.elapsedRealtime();

        //... do ui update
    }
1
林果皞

私にとって問題なのは、コードから明示的にonProgressUpdate()を呼び出していたことです。これはしてはいけません。代わりにpublishProgress()を呼び出して、エラーを解決しました。

1
mindreader

あなただけのあなたの非UIスレッドから無効にしたい(呼び出し再描画/再描画関数)したい場合は、postInvalidate()を使用してください。

myView.postInvalidate();

これにより、UIスレッドに無効化要求が送信されます。

より多くの情報のために: what-do-postinvalidate-do

1
Nalin

私は同様の問題に直面していました、そして、上記の方法のどれも私のために働きませんでした。結局、これが私にとってのトリックでした。

Device.BeginInvokeOnMainThread(() =>
    {
        myMethod();
    });

私はこの宝石を見つけました ここ

1
Hagbard

私の場合、私はAdapterにEditTextを持っています、そしてそれはすでにUIスレッドにあります。ただし、このアクティビティが読み込まれると、このエラーでクラッシュします。

私の解決策は、XMLのEditTextから<requestFocus />を削除する必要があることです。

1
Sruit A.Suk

コトリンで苦しんでいる人々にとって、それはこのように働きます:

lateinit var runnable: Runnable //global variable

 runOnUiThread { //Lambda
            runnable = Runnable {

                //do something here

                runDelayedHandler(5000)
            }
        }

        runnable.run()

 //you need to keep the handler outside the runnable body to work in kotlin
 fun runDelayedHandler(timeToWait: Long) {

        //Keep it running
        val handler = Handler()
        handler.postDelayed(runnable, timeToWait)
    }
0
Tarun Kumar

解決済み:このメソッドをdoInBackround Class ...に入れて、メッセージを渡すだけ

public void setProgressText(final String progressText){
        Handler handler = new Handler(Looper.getMainLooper()) {
            @Override
            public void handleMessage(Message msg) {
                // Any UI task, example
                progressDialog.setMessage(progressText);
            }
        };
        handler.sendEmptyMessage(1);

    }
0
Kaushal Sachan