Android Auf ImageView aus einem anderen Thread zugreifen liefert Fehlermeldung (App stürzt ab)

[insidERR]

Hallo zusammen.
Bin noch recht neu in der Java/Android Welt.

Ich erstelle eine App in der ein ImageView (Bild mit einem roten WiFi Symbol) über einem anderen ImageView (Fotoanzeige) liegt.
Da die App auf WLAN angewiesen ist, will ich ein Hintegrundthread erstellen, welcher alle 3 Sekunden den aktuellen WiFiStatus abruft und bei unterschreitung eines Pegels ein ImageView (Bild mit einem roten WiFi Symbol) einblendet.
Ich starte aus "onCreate" den Hintegrundthread (WiFiUeberwachung), welcher in der Endlosschleife den Status abruft.
Je nach Wert, will ich das besagte ImageView ein bzw. ausblenden.
Dieser Thread blendet beim Start das ImageView aus. Wenn es aber wieder eingeblendet werden soll, stürzt die App ab und meldet
"android.view.ViewRootImpl$CalledFromWrongThreadException: Only the original thread that created a view hierarchy can touch its views."
Bei Stackoverflow gibt es verschiedene Lösungsansätze wie z.B. den hier.
Da komme ich aber nicht weiter. Wenn dort die while(true) Schleife einbaue, friert die App schon beim Start ein.
Hat jemand ne Idee, wie man den folgenden (meinen) Code anpassen kann.

    .
    .
    .
    public ImageView WiFiWarning;
    
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
.
.
        WiFiWarning=findViewById(R.id.WiFi_Warning);
        
        //einmalige Prüfung der WiFi Verbindung (klappt perfekt)
        if(CheckWiFiQuality()<4){WiFiWarning.setVisibility(View.VISIBLE);}else{WiFiWarning.setVisibility(View.INVISIBLE);}
        
        //alle 3 Sekunden den Status prüfen und bei Bedarf WiFiWarning ein/ausblenden
        WiFiUeberwachung(3000,4);

    //Thread für eine kontinuierliche Überwachung der WiFi Verbindung
    public void WiFiUeberwachung(long warteZeit, int minSignalstaerke){
        AsyncTask.execute(() -> {
            try {
                while(true) {
                    Thread.sleep(warteZeit); //warte X Sekunden
                    //in der folgenden Zeile schmiert die App ab, wenn das ImageView(WiFiWarning) eingeblendet werden soll. Ausblenden klappt wie gewollt.
                    if(CheckWiFiQuality()<minSignalstaerke){WiFiWarning.setVisibility(View.VISIBLE);}else{WiFiWarning.setVisibility(View.INVISIBLE);}
                }
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        });
    }

    //Liefert als Antwort eine Zahl zwischen 1 und 5 (1=ganz schlecht, 5 sehr gut)
    private int CheckWiFiQuality()
    {
        WifiManager wifiManager = (WifiManager) this.getSystemService(Context.WIFI_SERVICE);

        // Level of a Scan Result
        List<ScanResult> wifiList = wifiManager.getScanResults();
        for (ScanResult scanResult : wifiList) {
            int level = WifiManager.calculateSignalLevel(scanResult.level, 5);
            System.out.println("Level is " + level + " out of 5");
            return level;
        }

        return level;
    }

Hat jemmand ne Idee?
Danke

 

[LimDul]

Vorneweg, ich hab von Android keine Ahnung, aber generell gilt bei allen UI-Frameworks:

* Auf UI Elemente darf nur aus dem zentralen UI Thread (in Swing z.B. der Event Dispatching Thread) zugegriffen werden.

Ansonsten kann es zu vielen Problemen keinen, weil zwei Threads gleichzeitig UI-Elemente ändern. Je nach UI Framework wird das mehr oder weniger stark forciert.

Hier gibt es auf Stackoverflow ein Beispiel: https://stackoverflow.com/questions/5185015/updating-android-ui-using-threads, was auf folgende Methode verweist: https://developer.android.com/reference/android/app/Activity#runOnUiThread(java.lang.Runnable)

 

[Jw456]

Du kannst nicht aus einem Thread auf view Element zugreifen.
Benutze dazu runonuithraed.
Um im Thread auf den UI zukommen.

Eine weitere Möglichkeit ist. Handler zubenutzen

 

[Jw456]

Bemerken möchte ich noch das AsyncTask veraltet ist sollte nicht mehr benutzt werden.

 

[insidERR]

Du kannst nicht aus einem Thread auf view Element zugreifen.
Benutze dazu runonuithraed.
Um im Thread auf den UI zukommen.

Eine weitere Möglichkeit ist. Handler zubenutzen

Hallo Jw456,
vielen Dank für den Hinweiß mit dem "RunOnUIthread".
Meistens sind einem die Befehle unbekannt und man hat einfach keinen Plan, wonach man suchen soll.
Der erste Suchtreffer brachte mich auf diese Seite mit dem Beispielcode.
Habe jetzt meinen Code etwas angepasst und nun läufts wie gewollt.

    //Thread für eine kontinuierliche Überwachung der WiFi Verbindung
    public void WiFiUeberwachung(long warteZeit, int minSignalstaerke){
        new Thread() {
            public void run() {
                while (true) {
                    try {
                        runOnUiThread(new Runnable() {
                            @Override
                            public void run() {
                                if(CheckWiFiQuality()<minSignalstaerke){
                                    WiFiWarning.setVisibility(View.VISIBLE);
                                }else{
                                    WiFiWarning.setVisibility(View.INVISIBLE);}
                            }
                        });
                        Thread.sleep(warteZeit);
                    } catch (InterruptedException e) {
                        e.printStackTrace();
                    }
                }
            }
        }.start();
    }

@LimDul dir auch danke für deine Rückmeldung

 

[KonradN]

Evtl. nur ein kleiner Hinweis: Dieses new Runnable() { ist seit Java 8 so eher unüblich. Der Code ist zu aufgebläht und schwer lesbar.

Hier kommen Lambda Expressions und Methoden Referenzen ins Spiel: Runnable ist ein funktionales Interface, d.h. es wird nur eine Methode gefordert. Somit reicht es, dass Du eine Methode mit passender Signatur hast. Das hat den Vorteil, dass der Code, den Du da geschrieben hast, auch gleich eine Beschreibung bekommt. Was macht er? Sowas wie changeWiFiWarningVisibility.

Du hast also eine Methode:

public void changeWiFiWarningVisibility() {
    if(CheckWiFiQuality()<minSignalstaerke) {
        WiFiWarning.setVisibility(View.VISIBLE);
    } else {
        WiFiWarning.setVisibility(View.INVISIBLE);
    }
}

Und der Aufruf wäre hier dann einfach: runOnUiThread(this::changeWiFiWarningVisibility);

Oder - wenn man es als Lambda Expression haben möchte:
runOnUiThread( () -> changeWiFiWarningVisibility());
(Das ist üblich, wenn da Parameter unterschiedlich sind und daher keine Methodenreferenz gehen würde)