Android «Только исходный поток, создавший иерархию представлений, может касаться его представлений».

942

Я построил простой музыкальный проигрыватель в 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()- и это не вызывает у меня жалобы.

herpderp
источник

Ответы:

1897

Вы должны переместить часть фоновой задачи, которая обновляет пользовательский интерфейс, в основной поток. Для этого есть простой фрагмент кода:

runOnUiThread(new Runnable() {

    @Override
    public void run() {

        // Stuff that updates the UI

    }
});

Документация для Activity.runOnUiThread.

Просто вложите это в метод, работающий в фоновом режиме, а затем скопируйте и вставьте код, который реализует любые обновления в середине блока. Включите только наименьшее возможное количество кода, иначе вы начнете побеждать назначение фонового потока.

провидение
источник
5
работал как шарм. для меня единственная проблема здесь в том, что я хотел сделать error.setText(res.toString());метод run () внутри, но я не мог использовать res, потому что он не был окончательным ... очень плохо
noloman
64
Один краткий комментарий по этому поводу. У меня был отдельный поток, который пытался изменить пользовательский интерфейс, и приведенный выше код работал, но у меня был вызов runOnUiThread из объекта Activity. Я должен был сделать что-то вроде myActivityObject.runOnUiThread(etc)
Кирби
1
@Kirby Спасибо за эту ссылку. Вы можете просто сделать 'MainActivity.this', и он должен работать так же, чтобы вам не приходилось хранить ссылки на ваш класс активности.
JRomero
24
Мне понадобилось время, чтобы понять, что runOnUiThread()это метод Деятельности. Я выполнял свой код во фрагменте. Я закончил тем, что делал, getActivity().runOnUiThread(etc)и это работало. Фантастический!;
lejonl
Можем ли мы остановить выполнение задачи, записанной в теле метода runOnUiThread?
Каран Шарма
143

Я решил это, вставив 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();
Гюнай Гюльтекин
источник
2
Этот потряс. Спасибо за информацию, это также может быть использовано внутри любого другого потока.
Набин
Спасибо, очень грустно создавать поток, чтобы вернуться к потоку пользовательского интерфейса, но только это решение спасло мое дело.
Пьер Мауи
2
Одним из важных аспектов является то, что wait(5000);он не находится внутри Runnable, иначе ваш пользовательский интерфейс будет зависать в течение периода ожидания. Вы должны рассмотреть возможность использования AsyncTaskвместо Thread для таких операций.
Мартин
это так плохо для утечки памяти
Рафаэль Лима
Зачем возиться с синхронизированным блоком? Код внутри него выглядит достаточно потокобезопасным (хотя я полностью готов к употреблению своих слов).
Дэвид
69

Мое решение этого:

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

Вызовите этот метод в фоновом потоке.

Анджело Анжелес
источник
Ошибка: (73, 67) ошибка: на набор нестатических методов (String) нельзя ссылаться из статического контекста
1
У меня та же проблема с моими тестовыми классами. Это сработало как обаяние для меня. Однако замена runOnUiThreadна runTestOnUiThread. Спасибо
DaddyMoe
28

Обычно любое действие, связанное с пользовательским интерфейсом, должно выполняться в основном потоке или потоке пользовательского интерфейса, то есть в том, в котором onCreate()выполняется обработка событий. Один из способов убедиться в этом - использовать runOnUiThread () , другой - использовать обработчики.

ProgressBar.setProgress() имеет механизм, который он всегда будет выполнять в основном потоке, поэтому он работал.

См. Безболезненная нить .

bigstones
источник
В статье «Безболезненная нить» по этой ссылке теперь 404. Вот ссылка на (более старую?) Статью в
Тони Адамс,
20

Я был в этой ситуации, но я нашел решение с помощью объекта Handler.

В моем случае я хочу обновить ProgressDialog с помощью шаблона наблюдателя . Мой взгляд реализует наблюдателя и переопределяет метод обновления.

Итак, мой основной поток создает представление, а другой поток вызывает метод update, который обновляет ProgressDialop и ....:

Только исходный поток, создавший иерархию представлений, может касаться его представлений.

Возможно решить проблему с Объектом Обработчика.

Ниже приведены различные части моего кода:

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);
    }
}

Это объяснение можно найти на этой странице , и вы должны прочитать «Пример ProgressDialog со вторым потоком».

Джонатан
источник
10

Вы можете использовать Handler для удаления вида, не нарушая основной поток пользовательского интерфейса. Вот пример кода

new Handler(Looper.getMainLooper()).post(new Runnable() {
                                                        @Override
                                                        public void run() {
                                                           //do stuff like remove view etc
                                                            adapter.remove(selecteditem);
                                                        }
                                                    });
Билал Мустафа
источник
7

Я вижу, что вы приняли ответ @ провидения. На всякий случай, вы также можете использовать обработчик тоже! Сначала сделайте поля 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

У меня была похожая проблема, и мое решение некрасиво, но оно работает:

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
}
Błażej
источник
2
@ R.jzadeh это приятно слышать. С того момента, как я написал этот ответ, вероятно, теперь вы можете сделать это лучше :)
Błażej
6

Я использую Handlerс Looper.getMainLooper(). Это работало хорошо для меня.

    Handler handler = new Handler(Looper.getMainLooper()) {
        @Override
        public void handleMessage(Message msg) {
              // Any UI task, example
              textView.setText("your text");
        }
    };
    handler.sendEmptyMessage(1);
Санкар Бехера
источник
5

Используйте этот код, и не нужно 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();
}
Хамид
источник
5

Это явно выдает ошибку. В нем говорится, какой поток создал вид, только тот, который может касаться его. Это потому, что созданный вид находится внутри пространства этого потока. Создание представления (GUI) происходит в потоке пользовательского интерфейса (основного). Таким образом, вы всегда используете поток пользовательского интерфейса для доступа к этим методам.

Введите описание изображения здесь

На рисунке выше переменная прогресса находится внутри пространства потока пользовательского интерфейса. Таким образом, только поток пользовательского интерфейса может получить доступ к этой переменной. Здесь вы получаете доступ к прогрессу через новую Thread (), и именно поэтому вы получили ошибку.

Уддхав Гаутам
источник
4

Это случилось с моим , когда я назвал для изменения пользовательского интерфейса из doInBackgroundиз Asynctaskвместо того , чтобы использоватьonPostExecute .

Работа с пользовательским интерфейсом onPostExecuteрешает мою проблему.

Джонатан душ Сантуш
источник
1
Спасибо Джонатан. Это тоже была моя проблема, но мне пришлось немного почитать, чтобы понять, что вы имели в виду. Для всех остальных onPostExecuteэто также метод, AsyncTaskно он работает в потоке пользовательского интерфейса. Смотрите здесь: blog.teamtreehouse.com/all-about-android-asynctasks
ciaranodc
4

Сопрограммы Kotlin могут сделать ваш код более кратким и читабельным, например:

MainScope().launch {
    withContext(Dispatchers.Default) {
        //TODO("Background processing...")
    }
    TODO("Update UI here!")
}

Или наоборот:

GlobalScope.launch {
    //TODO("Background processing...")
    withContext(Dispatchers.Main) {
        // TODO("Update UI here!")
    }
    TODO("Continue background processing...")
}
Kenichi
источник
3

Я работал с классом, который не содержал ссылку на контекст. Так что я не мог использовать, 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);
IFTA
источник
Что такое аналогия audioMessageи tvPlayDurationс кодом вопросы?
получил
audioMessageявляется объектом-держателем текстового представления. tvPlayDurationэто текстовое представление, которое мы хотим обновить из не-пользовательского интерфейса. В приведенном выше вопросе currentTimeтекстовое представление, но у него нет объекта-держателя.
Ифта
3

При использовании AsyncTask обновите интерфейс в методе onPostExecute

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

     }
Дипак Катария
источник
это случилось со мной. я обновлял пользовательский интерфейс в doinbackground задачи asynk.
mehmoodnisar125
3

Я столкнулся с подобной проблемой, и ни один из методов, упомянутых выше, не помог мне. В конце концов, это помогло мне:

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

Я нашел этот драгоценный камень здесь .

Хагбарда
источник
2

Это трассировка стека упомянутого исключения

        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)

Так что, если вы идете копать, то вы узнаете,

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();

Все, что я хочу сказать, это то, что когда мы создавали конкретное представление, мы создавали его в потоке пользовательского интерфейса, а затем пытались изменить его в рабочем потоке.

Мы можем проверить это через приведенный ниже фрагмент кода

Thread.currentThread().getName()

когда мы раздуваем макет и позже, где вы получаете исключение.

Амит Ядав
источник
2

Если вы не хотите использовать runOnUiThreadAPI, вы можете реализовать AsynTaskоперации, выполнение которых занимает несколько секунд. Но в этом случае, также после обработки вашей работы doinBackground(), вам нужно вернуть законченный вид в onPostExecute(). Реализация Android позволяет только основным потокам пользовательского интерфейса взаимодействовать с представлениями.

Сэм
источник
2

Если вы просто хотите сделать недействительной (вызвать функцию перерисовки / перерисовки) из вашей ветки, не связанной с пользовательским интерфейсом, используйте postInvalidate ()

myView.postInvalidate();

Это опубликует недействительный запрос в UI-потоке.

Для получения дополнительной информации: что-делает-postinvalidate-do

Налин
источник
1

Для меня проблема была в том, что я звонил onProgressUpdate()явно из своего кода. Это не должно быть сделано. Я звонил publishProgress()вместо этого, и это решило ошибку.

телепат
источник
1

В моем случае у меня есть EditText в адаптере, и это уже в потоке пользовательского интерфейса. Однако при загрузке этого действия происходит сбой с этой ошибкой.

Мое решение - мне нужно удалить <requestFocus />из EditText в XML.

Sruit A.Suk
источник
1

Для людей, борющихся в Котлине, это работает так:

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

Решено: Просто поместите этот метод в класс doInBackround ... и передайте сообщение

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

В моем случае вызывающий вызов слишком много раз за короткое время получит эту ошибку, я просто помещаю проверку истекшего времени, чтобы ничего не делать, если она слишком короткая, например игнорировать, если функция вызывается менее чем за 0,5 секунды:

    private long mLastClickTime = 0;

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

        //... do ui update
    }
Фрукты
источник
Лучшим решением было бы отключить кнопку при нажатии и включить ее снова после завершения действия.
lsrom
@lsrom В моем случае это не так просто, потому что вызывающая сторона - сторонняя библиотека, внутренняя и вне моего контроля.
Фрукты
0

Если вы не смогли найти UIThread, вы можете использовать этот способ.

yourcurrentcontext означает, что вам нужно проанализировать текущий контекст

 new Thread(new Runnable() {
        public void run() {
            while (true) {
                (Activity) yourcurrentcontext).runOnUiThread(new Runnable() {
                    public void run() { 
                        Log.d("Thread Log","I am from UI Thread");
                    }
                });
                try {
                    Thread.sleep(1000);
                } catch (Exception ex) {

                }
            }
        }
    }).start();
Удара Касун
источник
0

Котлин Ответ

Мы должны использовать UI Thread для правильной работы. Мы можем использовать UI Thread в Kotlin:

runOnUiThread(Runnable {
   //TODO: Your job is here..!
})

@canerkaseler

canerkaseler
источник
0

В Kotlin просто поместите ваш код в метод активности runOnUiThread

runOnUiThread{
    // write your code here, for example
    val task = Runnable {
            Handler().postDelayed({
                var smzHtcList = mDb?.smzHtcReferralDao()?.getAll()
                tv_showSmzHtcList.text = smzHtcList.toString()
            }, 10)

        }
    mDbWorkerThread.postTask(task)
}
Рахил Хан
источник