String von AsyncTask ohne Activity an Activity übertragen

T5D6 · 3. Januar 2016

Hallo liebe Android-Developers Gemeinde,

bin erst seit kurzem an der Android Entwicklung dran und bin nun leider auf ein für mich unlösbares Problem gestoßen.

Zur Problemstellung:

Ich habe eine GUI aus mehreren Activitys und einen AsnyncTask welcher die TCP Verbindungen verarbeitet. Innerhalb meines AsyncTasks empfange ich Strings via TCP Verbindung. Nun möchte ich die empfangenen Strings irgendwie in meine Activitys bekommen. Hab nun schon vergeblich gegoogelt, mit onPost, interface und handler rum probiert aber ich bekomms nicht zum laufen

Ich wäre über jede Hilfe sehr dankbar.

AsyncTask:

JavaScript

public class connectTask extends AsyncTask&lt;String,String,TCPClient&gt;
{
    private TCPClient mTcpClient;
    public ArrayList&lt;String&gt; arrayList;
 //   private Context context;
    @Override
    protected TCPClient doInBackground(String... message)
    {
        //we create a TCPClient object and
        mTcpClient = new TCPClient(new TCPClient.OnMessageReceived()
        {
            @Override
            //here the messageReceived method is implemented
            public void messageReceived(String message)
            {
                //this method calls the onProgressUpdate
                publishProgress(message);
            }
        });


        if (TCPHolder.getInstance(mTcpClient) == null)
        {
            TCPHolder.getInstance(mTcpClient);
        }


        TCPHolder.getInstance(mTcpClient).getTcp().run();
        return null;
    }


    @Override
    protected void onProgressUpdate(String[] values)
    {
        super.onProgressUpdate(values);


        arrayList.add(values[0]);
        //Log
        Log.i("onProgressUpdate: ", values[0]);


// Wert values soll an Activity übergeben werden.
    }
}

Alles anzeigen

und hier meine Activity:

JavaScript

ublic class ControlActivity extends AppCompatActivity {




    private TCPClient mTcpClient;
    private connectTask connectTask;








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


        Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
        setSupportActionBar(toolbar);


        final SeekBar setgrow = (SeekBar) findViewById(R.id.setgrow);


        final SeekBar sethealth = (SeekBar) findViewById(R.id.sethealth);


        final SeekBar setflow = (SeekBar) findViewById(R.id.setflow);


        String[] values = new String[10];


        new connectTask().execute("");


// Hier muss ich den String aus dem AsyncTask empfangen können...


        }

Alles anzeigen

Bin wirklich für jede Hilfe mehr als dankbar

Marco Feltmann · 4. Januar 2016

Zunächst einmal passt der Ansatz nicht ganz.
Du möchtest in der OnCreate den String aus dem Async Task empfangen. Da der Task asynchron läuft, ist das in 99,99% der Fälle nicht möglich, weil der String leer ist. (In der unglaublich kurzen Zeit bekommt Dein Task noch keine Info zurück und kann dementsprechend nix anzeigen. Und die OnCreate wartet auch nicht auf Deinen Task, er ist ja asynchron.)

Sinnvoller wäre es, Du definierst eine Callback Methode in Deiner Activity, die Dein AsyncTask im onProgressUpdate aufruft.

Am Besten ein Interface definieren:

Java

interface StringDownloadDelegate
{
    public void taskReceivedStringComponents(AsyncTask sender, String[] components);
}


public class connectTask extends AsyncTask&lt;String,String,TCPClient&gt;
{
    private StringDownloadDelegate delegate;
    
    // Setter dazu schreiben...


    // Rest der Klasse einfügen...


    protected void onProgressUpdate(String[] values)
    {
        super.onProgressUpdate(values);
        // NULL Check etc needed
        delegate.taskReceivedStringComponents(self, values);
    }
}

Alles anzeigen

Jede Activity muss dann dieses Interface implementieren.

Java

public class ControlActivity extends AppCompatActivity implements StringDownloadDelegate {


  // Krams wie gehabt einpflegen.
  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    // Eigene Vorbereitungen und so einbinden...


    connectTask  = new connectTask();
    connectTask.setDelegate(self);
    connectTask.execute("");
  }


  public void taskReceivedStringComponents(AsyncTask sender, String[] components) {
    /* Hier kommt dann der GUI Krams rein.
     * Obacht: Kann (und wird vermutlich auch) mehrfach hintereinander aufgerufen werden.
     * Einerseits also darauf achten, dass das alles hier nicht zu lange dauert und andererseits die Werte zwischenspeichern.
     * Also am Besten eine andere Methode der Activity aufrufen, die meinetwegen die Strings kombiniert und dann die Methode zur Anzeige aufruft.
     */
  }
}

Alles anzeigen

Ben · 17. Februar 2016

primär würde ich dir empfehlen dich über Synchron und Asynchron zu informieren. Dazu gibts einige gute Erklärungen im Netz.

Marco hat dir die richtige Lösung bereits gepostet, aber du musst sie ja noch verstehen. In dem Zusammenhang lohnt es auch sich über funktionsweise von selbstgeschriebenen Listenern zu informieren.

colibri · 17. Februar 2016

Da du dein TCPClient-Objekt, welches deine doInBackground-Methode zurückgeben könnte, ohnehin nicht zurück gibst ("return null;"), könntest du stattdessen deine Methode wie folgt deklarieren:

Java

protected String[] doInBackground(String... strings)

Und dann kannst du diesea zurückgegebene String-Array in onPostExecute weiter verarbeiten:

Java

protected void onPostExecute(String[] strings)

Ich habe diese Infos bei http://www.programmierenlernen…se-threads-und-asynctask/ gefunden - vielleicht hilft dir das ja insgesamt auch weiter. Ich meine, dass das Ergebnis des dort verwendeten AsyncTask-Objekts auch in einer späteren Lektion noch weiterverarbeitet wird. Leider finde ich das gerade nicht mehr.

String von AsyncTask ohne Activity an Activity übertragen

Jetzt mitmachen!

Teilen

Tags