BufferedWriter und Reader in seperaten Thread?!

Greg50007 · 25. April 2018

Guten Abend,
ich bin in den letzten Wochen weit voran gekommen in meinem derzeitigen und ersten Android Projekt... nun verzweifle ich aber doch ein einer Fehlermeldung...
Ich habe einen BufferedWriter und Reader und wenn ich die App ausführe kommt die Meldung das ich das nicht im MainThread machen darf...
Ich weiß aber nicht wie ich den Writer in einen anderern Thread packen soll/kann...
Bevor hier nun nur den Writer und Reader reinpacke und dann Stück für Stück den Rest poste, gleich einmal alles:

Habt ihr eine Idee wie ich das umsetzten kann?
Lg.
Greg

Greg50007 · 25. April 2018

Java

package com.example.niklas.medienlister;


import android.content.Context;
import android.content.DialogInterface;
import android.content.Intent;
import android.net.Uri;
import android.support.v7.app.AlertDialog;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.support.v7.view.menu.ExpandedMenuView;
import android.view.MenuItem;
import android.view.View;
import android.view.WindowManager;
import android.widget.AdapterView;
import android.widget.ListView;
import android.widget.PopupMenu;
import android.widget.SearchView;
import android.widget.TextView;
import jcifs.smb.NtlmPasswordAuthentication;
import jcifs.smb.SmbException;
import jcifs.smb.SmbFile;
import jcifs.smb.SmbFileInputStream;
import jcifs.smb.SmbFileOutputStream;




import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.InputStreamReader;
import java.io.OutputStreamWriter;
import java.io.UnsupportedEncodingException;
import java.net.MalformedURLException;
import java.net.UnknownHostException;
import java.util.ArrayList;


public class Suche extends AppCompatActivity implements SearchView.OnQueryTextListener {




    ListView list;
    ListViewAdapter adapter;
    SearchView editsearch;
    ArrayList&lt;DVD&gt; arraylist = new ArrayList&lt;DVD&gt;();
    String user = "admin";
    String pass = "****";
    public static String titel;
    public static String erscheinungsjahr;
    public static String laufzeit;
    public static String fsk;
    public static String server;


    NtlmPasswordAuthentication auth = new NtlmPasswordAuthentication(null, user, pass);








    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R .layout.activity_suche);


        getSupportActionBar().setTitle("Suche");
        getSupportActionBar().setDisplayHomeAsUpEnabled(true);
        getSupportActionBar().setDisplayShowHomeEnabled(true);
        getWindow().setSoftInputMode(WindowManager.LayoutParams.SOFT_INPUT_STATE_ALWAYS_HIDDEN);


        Menu.dvdv.dvdSortieren();
        // Locate the ListView in listview_main.xml
        list = (ListView) findViewById(R.id.listview);


        for(int i = 0; i &lt; Menu.dvdv.counter; i++) {
            arraylist.add(Menu.dvdv.dvds[i]);
        }




        // Pass results to ListViewAdapter Class
        adapter = new ListViewAdapter(this, arraylist);


        // Binds the Adapter to the ListView
        list.setAdapter(adapter);
        list.setOnItemClickListener(new AdapterView.OnItemClickListener() {
            @Override
            public void onItemClick(AdapterView&lt;?&gt; adapterView, View view, final int i, long l) {
                final String suchtitel = Menu.dvdv.dvds[i].titel;


                final String auswahl = Menu.dvdv.dvds[i].titel;
                final String sicherung = Menu.dvdv.dvds[i].erscheinungsjahr;
                final String sicherung21 = Menu.dvdv.dvds[i].laufzeit;
                String array1[] = sicherung21.split(" ");
                final String sicherung2 = array1[0];
                final String sicherung3 = Menu.dvdv.dvds[i].fsk;
                final String sicherung4 = Menu.dvdv.dvds[i].server;


                final PopupMenu popupMenu = new PopupMenu(Suche.this, list);
                popupMenu.getMenuInflater().inflate(R.menu.popupmenu, popupMenu.getMenu());
                popupMenu.setOnMenuItemClickListener(new PopupMenu.OnMenuItemClickListener() {
                    @Override
                    public boolean onMenuItemClick(MenuItem menuItem) {
                        if(menuItem.getTitle().equals("Web-Suche")){
                            String url = "https://de.wikipedia.org/wiki/" + suchtitel;
                            Intent intent = new Intent();
                            intent.setAction(Intent.ACTION_VIEW);
                            intent.setData(Uri.parse(url));
                            startActivity(intent);
                        } else if(menuItem.getTitle().equals("")){


                        }else if(menuItem.getTitle().equals("Löschen")){
                            AlertDialog.Builder ab = new AlertDialog.Builder(Suche.this);
                            ab.setMessage("Sicher das sie den Film " + suchtitel + "\nals gelöscht markieren wollen?").setCancelable(false).setPositiveButton("Ja", new DialogInterface.OnClickListener() {

Alles anzeigen

Greg50007 · 25. April 2018

Hier das Problem:

Java

@Override
                                public void onClick(DialogInterface dialogInterface, int i) {


                            try{
                                String zeilen;
                                String urlR = "smb://192.168.178.69//public//Lister//liste.txt";
                                SmbFile Rfile = new SmbFile(urlR, auth);
                                BufferedReader br = new BufferedReader(new InputStreamReader(new SmbFileInputStream(Rfile), "Cp1252"));


                                String urlW = "smb://192.168.178.69//public//Lister//test.txt";
                                SmbFile Wfile = new SmbFile(urlW, auth);
                                BufferedWriter bw = new BufferedWriter(new OutputStreamWriter(new SmbFileOutputStream(Wfile, true), "Cp1252"));




                                while ((zeilen = br.readLine()) != null) {
                                    String[] array = zeilen.split("€");
                                    titel = array[0];
                                    erscheinungsjahr = array[1];
                                    laufzeit = array[2];
                                    fsk = array[3];
                                    server = array[4];


                                    if (titel.equals(auswahl) && erscheinungsjahr.equals(sicherung)
                                            && laufzeit.equals(sicherung2) && fsk.equals(sicherung3)
                                            && server.equals(sicherung4)) {
                                        if (auswahl.contains(" *gelöscht*")) {
                                            bw.write(titel + "€" + erscheinungsjahr + "€" + laufzeit + "€" + fsk + "€" + server
                                                    + "\r\n");
                                        } else {
                                            bw.write(titel + " *gelöscht*" + "€" + erscheinungsjahr + "€" + laufzeit + "€" + fsk
                                                    + "€" + server + "\r\n");
                                        }
                                    } else {
                                        bw.write(zeilen + "\r\n");


                                    }


                                }


                                br.close();
                                bw.close();
                                //Rfile.delete();
                                //Wfile.renameTo(Rfile);




                            } catch (Exception e) {
                                e.printStackTrace();
                            }


                            for (int j = 0; j &lt; Menu.dvdv.counter; j++) {
                                if (Menu.dvdv.dvds[j].titel.equals(auswahl)
                                        && Menu.dvdv.dvds[j].erscheinungsjahr.equals(sicherung)
                                        && Menu.dvdv.dvds[j].laufzeit.equals(sicherung2)
                                        && Menu.dvdv.dvds[j].fsk.equals(sicherung3)
                                        && Menu.dvdv.dvds[j].server.equals(sicherung4)) {


                                    if (Menu.dvdv.dvds[j].titel.contains(" *gelöscht*")) {
                                        AlertDialog.Builder ab2 = new AlertDialog.Builder(Suche.this);
                                        ab2.setMessage("Dieser Film ist bereits als gelöscht markiert!").setCancelable(false).setPositiveButton("Ok", new DialogInterface.OnClickListener() {
                                            @Override
                                            public void onClick(DialogInterface dialogInterface, int i) {
                                                dialogInterface.cancel();
                                            }
                                        });
                                        AlertDialog ad2 = ab2.create();
                                        ad2.setTitle("Info");
                                        ad2.show();


                                    } else {
                                        Menu.dvdv.dvds[j].titel = (Menu.dvdv.dvds[j].titel + " *gelöscht*");
                                        list.invalidateViews();
                                    }


                                }
                            }


                                }
                            }).setNegativeButton("Nein", new DialogInterface.OnClickListener() {
                                @Override
                                public void onClick(DialogInterface dialogInterface, int i) {
                                    dialogInterface.cancel();
                                }
                            });
                            AlertDialog ad = ab.create();
                            ad.setTitle("Löschen");
                            ad.show();
                        }
                        return true;
                    }
                });
                popupMenu.show();
            }
        });


        // Locate the EditText in listview_main.xml
        editsearch = (SearchView) findViewById(R.id.search);
        editsearch.setOnQueryTextListener(this);
    }




    @Override
    public boolean onQueryTextSubmit(String query) {


        return false;
    }


    @Override
    public boolean onQueryTextChange(String newText) {
        String text = newText;
        adapter.filter(text);
        return false;
    }


    @Override
    public boolean onOptionsItemSelected(MenuItem item){
        int id = item.getItemId();
        if(id == android.R.id.home){
            Intent menu = new Intent(this, Menu.class);
            startActivity(menu);
            this.finish();
        }
        return super.onOptionsItemSelected(item);
    }
}

Alles anzeigen

nono124 · 25. April 2018

Hi ich würde sagen etwa so

Java

public class BufferThread implements Runnable {
        public void run() {
            try{
                String zeilen;
                String urlR = "smb://192.168.178.69//public//Lister//liste.txt";
                SmbFile Rfile = new SmbFile(urlR, auth);
                BufferedReader br = new BufferedReader(new InputStreamReader(new SmbFileInputStream(Rfile), "Cp1252"));
                String urlW = "smb://192.168.178.69//public//Lister//test.txt";
                SmbFile Wfile = new SmbFile(urlW, auth);
                BufferedWriter bw = new BufferedWriter(new OutputStreamWriter(new SmbFileOutputStream(Wfile, true), "Cp1252"));
                while ((zeilen = br.readLine()) != null) {
                    String[] array = zeilen.split("€");
                    titel = array[0];
                    erscheinungsjahr = array[1];
                    laufzeit = array[2];
                    fsk = array[3];
                    server = array[4];
                    if (titel.equals(auswahl) && erscheinungsjahr.equals(sicherung)
                            && laufzeit.equals(sicherung2) && fsk.equals(sicherung3)
                            && server.equals(sicherung4)) {
                        if (auswahl.contains(" *gelöscht*")) {
                            bw.write(titel + "€" + erscheinungsjahr + "€" + laufzeit + "€" + fsk + "€" + server
                                    + "\r\n");
                        } else {
                            bw.write(titel + " *gelöscht*" + "€" + erscheinungsjahr + "€" + laufzeit + "€" + fsk
                                    + "€" + server + "\r\n");
                        }
                    } else {
                        bw.write(zeilen + "\r\n");
                    }
                }
                br.close();
                bw.close();
                //Rfile.delete();
                //Wfile.renameTo(Rfile);
            } catch (Exception e) {
                e.printStackTrace();
            }
        }
    }


    @Override
    public void onClick(DialogInterface dialogInterface, int i) {




        Thread t1 = new Thread( new BufferThread() );
        t1.start();






        for (int j = 0; j &lt; Menu.dvdv.counter; j++) {
            if (Menu.dvdv.dvds[j].titel.equals(auswahl)
                    && Menu.dvdv.dvds[j].erscheinungsjahr.equals(sicherung)
                    && Menu.dvdv.dvds[j].laufzeit.equals(sicherung2)
                    && Menu.dvdv.dvds[j].fsk.equals(sicherung3)
                    && Menu.dvdv.dvds[j].server.equals(sicherung4)) {
                if (Menu.dvdv.dvds[j].titel.contains(" *gelöscht*")) {
                    AlertDialog.Builder ab2 = new AlertDialog.Builder(Suche.this);
                    ab2.setMessage("Dieser Film ist bereits als gelöscht markiert!").setCancelable(false).setPositiveButton("Ok", new DialogInterface.OnClickListener() {
                        @Override
                        public void onClick(DialogInterface dialogInterface, int i) {
                            dialogInterface.cancel();
                        }
                    });
                    AlertDialog ad2 = ab2.create();
                    ad2.setTitle("Info");
                    ad2.show();
                } else {
                    Menu.dvdv.dvds[j].titel = (Menu.dvdv.dvds[j].titel + " *gelöscht*");
                    list.invalidateViews();
                }
            }
        }
    }

Alles anzeigen

Greg50007 · 25. April 2018

Mhh also ich würde sagen Das es nicht ganz die Lösung ist...
Ich kann von Thread schonmal nicht das listview updaten. Und mit dem schreiben klappt auch nicht.
Code folgt:

nono124 · 25. April 2018

hi es sollte klar sein das die threadklasse nicht in einer Methode erstellt werden sollte. Sonder irgendwo in der acktivity Kasse.

Greg50007 · 25. April 2018

mhh ja schon klar... das schreiben klappt jetzt... aber wie kann ich vom Thread aus der/die/das listview updaten?
ich hab mal eine methode in der mainactivity geschrieben

Java: Methode

public static void listupdate(){
        list.invalidateViews();
    }

die ich im Thread aufrufe

Java: Aufruf

Suche.listupdate();

Da bekomme ich dann

Code

E/AndroidRuntime: FATAL EXCEPTION: Thread-8
                  Process: com.example.niklas.medienlister, PID: 29295
                  android.view.ViewRootImpl$CalledFromWrongThreadException: Only the original thread that created a view hierarchy can touch its views.
                      at android.view.ViewRootImpl.checkThread(ViewRootImpl.java:7769)
                      at android.view.ViewRootImpl.requestLayout(ViewRootImpl.java:1294)
                      at android.view.View.requestLayout(View.java:20970)
                      at android.view.View.requestLayout(View.java:20970)
                      at android.view.View.requestLayout(View.java:20970)
                      at android.view.View.requestLayout(View.java:20970)
                      at android.view.View.requestLayout(View.java:20970)
                      at android.view.View.requestLayout(View.java:20970)
                      at android.widget.RelativeLayout.requestLayout(RelativeLayout.java:360)
                      at android.view.View.requestLayout(View.java:20970)
                      at android.widget.AbsListView.requestLayout(AbsListView.java:2821)
                      at android.widget.AbsListView.invalidateViews(AbsListView.java:8496)
                      at com.example.niklas.medienlister.Suche.listupdate(Suche.java:183)
                      at com.example.niklas.medienlister.löschenThread.run(löschenThread.java:102)

Alles anzeigen

eine schöne fehlermeldung... habt ihr eine idee wie ich in einer if abfrage im Thread der/die/das listview aus der mainactivity updaten kann?

nono124 · 25. April 2018

von einem thread kannst du nicht so auf ein view Element zugreifen denn das läuft im UI thread. Dazu benutz man handler.
Schaue dir dazumal die klasse asynctask an.
https://www.google.de/url?sa=t…Vaw2kd0qkPmdKL6ulqJ_MDNOY

Greg50007 · 25. April 2018

Code

@Override protected Void doInBackground(Void... params) { Handler handler = new Handler(context.getMainLooper()); handler.post( new Runnable(){ public void run(){ Toast.makeText(context, "Created a server socket",Toast.LENGTH_LONG).show(); } }); }

sowas? Da muss ich mich reinlesen wo das wir hinkommt... oder ist das mehr?
(Könntest du mir vielleicht ein passendes Beispiel schicken?)
Lg. Und gute Nacht

Greg

Greg50007 · 26. April 2018

also ich hab mir das Dokument mal angeguckt... aber mir ist irgendwie noch nicht klar wie ich den Handler mit dem Thread verbinde... kannst du mir da helfen?

Greg50007 · 26. April 2018

Tut mir leid falls das jetzt langsam in richtung Spam geht
aber ich hab jetzt was gebastelt was schon mal halbwegs funktioniert... jetzt bekomme ich aber wieder die Fehlermeldung das ich nicht in der MainActivity mit dem Buffered Writer schreiben darf...

Code

final Handler handler = new Handler();
                                    new Thread(){
                                        @Override
                                        public void run(){
                                            handler.post(new Runnable() {
                                                @Override
                                                public void run() {
                                                    try {
                                                        String zeilen;
                                                        String urlR = "smb://192.168.178.69//public//Lister//liste.txt";
                                                        SmbFile Rfile = new SmbFile(urlR, auth);
                                                        BufferedReader br = new BufferedReader(new InputStreamReader(new SmbFileInputStream(Rfile), "Cp1252"));


                                                        String urlW = "smb://192.168.178.69//public//Lister//test.txt";
                                                        SmbFile Wfile = new SmbFile(urlW, auth);
                                                        BufferedWriter bw = new BufferedWriter(new OutputStreamWriter(new SmbFileOutputStream(Wfile, true), "Cp1252"));




                                                        while ((zeilen = br.readLine()) != null) {
                                                            String[] array = zeilen.split("€");
                                                            titel = array[0];
                                                            erscheinungsjahr = array[1];
                                                            laufzeit = array[2];
                                                            fsk = array[3];
                                                            server = array[4];


                                                            if (titel.equals(Suche.auswahl) && erscheinungsjahr.equals(Suche.sicherung)
                                                                    && laufzeit.equals(Suche.sicherung2) && fsk.equals(Suche.sicherung3)
                                                                    && server.equals(Suche.sicherung4)) {
                                                                if (Suche.auswahl.contains(" *gelöscht*")) {
                                                                    bw.write(titel + "€" + erscheinungsjahr + "€" + laufzeit + "€" + fsk + "€" + server
                                                                            + "\r\n");
                                                                } else {
                                                                    bw.write(titel + " *gelöscht*" + "€" + erscheinungsjahr + "€" + laufzeit + "€" + fsk
                                                                            + "€" + server + "\r\n");
                                                                }
                                                            } else {
                                                                bw.write(zeilen + "\r\n");


                                                            }


                                                        }


                                                        br.close();
                                                        bw.close();
                                                        Rfile.delete();
                                                        Wfile.renameTo(Rfile);




                                                    } catch (Exception e) {
                                                        e.printStackTrace();
                                                    }


                                                    for (int j = 0; j &lt; Menu.dvdv.counter; j++) {
                                                        if (Menu.dvdv.dvds[j].titel.equals(Suche.auswahl)
                                                                && Menu.dvdv.dvds[j].erscheinungsjahr.equals(Suche.sicherung)
                                                                && Menu.dvdv.dvds[j].laufzeit.equals(Suche.sicherung2)
                                                                && Menu.dvdv.dvds[j].fsk.equals(Suche.sicherung3)
                                                                && Menu.dvdv.dvds[j].server.equals(Suche.sicherung4)) {


                                                            if (Menu.dvdv.dvds[j].titel.contains(" *gelöscht*")) {


                                                                AlertDialog.Builder ab2 = new AlertDialog.Builder(Suche.this);
                                                                ab2.setMessage("Dieser Film ist bereits als gelöscht markiert!").setCancelable(false).setPositiveButton("Ok", new DialogInterface.OnClickListener() {
                                                                    @Override
                                                                    public void onClick(DialogInterface dialogInterface, int i) {
                                                                        dialogInterface.cancel();
                                                                    }
                                                                });
                                                                AlertDialog ad2 = ab2.create();
                                                                ad2.setTitle("Info");
                                                                ad2.show();


                                                            } else {
                                                                Menu.dvdv.dvds[j].titel = (Menu.dvdv.dvds[j].titel + " *gelöscht*");
                                                                Suche.list.invalidateViews();
                                                            }


                                                        }
                                                    }
                                                }
                                            });
                                        }
                                    }.start();




                                }
                            }).setNegativeButton("Nein", new DialogInterface.OnClickListener() {
                                @Override
                                public void onClick(DialogInterface dialogInterface, int i) {
                                    dialogInterface.cancel();
                                }
                            });
                            AlertDialog ad1 = ab1.create();
                            ad1.setTitle("Info");
                            ad1.show();

Alles anzeigen

Ich hoffe auf Lösungsideen und/oder verbesserungsvorschläge... ist das so falsch?
oder hab ich was vergessen

Lg.
Greg

nono124 · 26. April 2018

So erst mal eine zwischenantwort bin im Bus nur am Handy. Was mir schon auffällt ist das du alles im Handler also im ui thread machst. Somit ist ist die Fehlermeldung wieder kein Wunder. Schaue später vielleicht noch mal.

Greg50007 · 26. April 2018

ich bitte dich drum
Das was da raus kam ist das was ich aus deinem Dokument entnommen habe...
So wie es jetzt ist funktionieren die alertdialogs und ich kann das listview aktualisieren... dafür halt der writer nicht mehr...

Bitte hilf mir
Lg.
Greg

nono124 · 26. April 2018

Wie ich schon geschrieben habe machst du wieder alles in einen Thread.
Du brauchst nur den Try und Catch Block in einen Thread zu verschieben da du da auch keine Ausgaben auf den Bildschirm (UI Thread) machst brauchst du auch keinen Handler.
Ich habe den Block bewusst in eine innere Klasse ausgelagert. Erstens ist Runnable ein Interface und Übersichtlicher ist es auch . Nachteil ist das du nicht auf lokale Variablen die du in deinen Adapter erstellst zugreifen kannst. Dafür habe ich Globale Variablen erstellt (oder du übergibst die Werte mit dem Aufruf der Klasse ) im Adapter weise ich den Globalen deine werte zu damit sie im Thread verfügbar sind. Warum machst du alles Final?
Teste es mal so aber schreibe nicht wieder alles in einander.
Da du auch immer Klicklistner usw. hintereinander erstellet anstatt in einer eigenen Methode ist dein Quellcode sehr unübersichtlich.

Ahnungslos · 26. April 2018

Das ist zwar denk ich mal bisschen schlampig müsste aber funktionieren

Java

new Thread(new Runnable() {
            @Override
            public void run() {
                //To-Do in seperate Thread
                //...
                runOnUiThread(new Runnable() {
                    @Override
                    public void run() {
                        //to-do in UI Thread
                        //...
                    }
                });


            }
        }).start();

Alles anzeigen

Wenn du eine Methode schreiben willst müssen die Parameter final sein

Greg50007 · 26. April 2018

Hallo, erstmal an beide danke für die Antwort.
@Ahnungslos auch dir Danke, aber ich fand die Lösung von jogimuc übersichtlicher und für mich leichter zu verstehen

@jogimuc danke. Diese verbindung zwischen den ganzen Sachen hat mir irgendwie gefehlt... und irgendwie kam ich nicht auf die idee nur den try and catch block zu verschieben xP

WAs die übersicht angeht... ja da gebe ich dir absolut recht... das ist noch nicht meine stärke und ich bin froh wenn es einfach mal funktioniert

Aber vielen vielen dank für die Hilfe, nur noch 2-3 Schritte und meine APp kann in betrieb gehen
schönen abend noch:
Greg

BufferedWriter und Reader in seperaten Thread?!

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema