Setting Text On Textview From Background Thread Anomaly

I've just started playing with Android Concurrency/Loopers/Handers, and I have just faced with strange anomaly. The code below doesn't block me from setting text on TextView from d

Solution 1:

From the very first try, your code is wrong because you created Handler on different thread. This leads to looper/handler is run on different thread. Based on your comment, I guess you know this issue and you want to understand why exception doesn't throw on first try but second.

You should beware this: Accessing UI Element on different thread except from UI Thread causes undefined behavior. This means:

• Sometime you see it works.
• sometime you don't. You will meet exception as you have seen.

That means Accessing UI Element on different thread isn't always 100% reproducible.

Why you should access all UI Elements only on UI Thread? Because processing UI Element (change internal state, draw to screen ...) is a complex process and need to synchronized between related parties. For example, you call TextView#setText(String) on 2 fragments that both visible on screen. Android doesn't do this concurrently but pushing all jobs into an UI message queue and do that sequentially. This is also true not only from your application viewpoint but also from whole Android system perspective. Updating from status bar that called by system, updating from your app that called by your application always push actions to same UI Message queue before processing.

When you access and modify UI elements on different thread, you broke that process. That means maybe two threads might access and modify an element at same state and same time. As the result, you will meet race condition at some time. That when error occurs.

Explaining for your situation is hard because not enough data for analyzing. But there are some few reasons:

• At your first try, TextView haven't displayed on screen yet. So different thread was able to make change on TextView. But on your second try, you sleep for 1 second. At this time, all view has rendered and displayed successfully on screen, so exception threw. You can try Thread.sleep(0) and hopefully your code doesn't crash.
• This is will happen in some situations but hard to guess why. That by some chance, both your thread and ui thread accesses same lock object, exception threw.

You can read more about thread issue here Android Thread

Explicit references

Many tasks on non-main threads have the end goal of updating UI objects. However, if one of these threads accesses an object in the view hierarchy, application instability can result: If a worker thread changes the properties of that object at the same time that any other thread is referencing the object, the results are undefined.

Hope this help you.

Solution 2:

Only the UI Thread can make edits to UI items. In other words, you can not make user interface edits from background threads.

So, instead of tv.setText(Thread.currentThread().getName() + " " + text);, use the following code inside your backgroundHandler :-

runOnUiThread(new Runnable() {
    public void run() {
        tv.setText(Thread.currentThread().getName() + " " + text);
    }
});