zweifach Laden Bug

Moin moin!

Ich komm direkt zum Problem, ich schlage mich jetzt seit mehreren Wochen mit einem, wie ich ursprünglich dachte, kleinen Bug herum, der dafür sorgt, dass ich bei einem swipe refresh layout immer mehrfach swipen muss, bis der neue Wert in meiner App ankommt.

Als ich diesen Bug nun vor einigen Tagen beheben wollte stellte ich fest, dass er immer auftritt, wenn Daten überschrieben werden, also gar nicht so klein ist wie ich dachte.

Als kleine Information vorweg, ich habe mir das programmieren von Apps bisher aus verschiedenen Guides und Erklärungen alleine beigebracht, manche Codebereiche werden also sicher nicht "dem Standard" entsprechen und auch mit manchen Begrifflichkeiten werde ich wahrscheinlich nichts anfangen können, seid also bitte lieb zu mir

Ich werde einfach mal eine Beispielklasse dazupacken, vllt fällt ja jemandem etwas auf
vielen Dank schonmal lg Neggo

PS: Ich habe Kommentare, die nur für mich interessant sind entfernt und an einigen Stellen Notizen gemacht. Die Notizen würde ich erweitern, je nach dem was ich bereits ausprobiert habe.

PPS: Wie ich bei meinen letzten Tests feststellte, scheint die Reihenfolge, mit der das Programm abgearbeitet wird ein Problem zu sein, möglicherweise fehlt mir hier eine Grundlage die mir hier ein Bein stellt :/

import android.app.Activity;
import android.content.Context;
import android.content.Intent;
import android.content.SharedPreferences;
import android.os.Bundle;

import com.android.volley.Request;
import com.android.volley.RequestQueue;
import com.android.volley.Response;
import com.android.volley.VolleyError;
import com.android.volley.toolbox.StringRequest;
import com.android.volley.toolbox.Volley;
import com.google.android.material.floatingactionbutton.FloatingActionButton;
import com.google.android.material.snackbar.Snackbar;

import androidx.appcompat.app.AppCompatActivity;
import androidx.appcompat.widget.Toolbar;
import androidx.recyclerview.widget.LinearLayoutManager;
import androidx.recyclerview.widget.RecyclerView;
import androidx.swiperefreshlayout.widget.SwipeRefreshLayout;

import android.preference.PreferenceManager;
import android.util.Log;
import android.view.Menu;
import android.view.MenuItem;
import android.view.View;
import android.widget.EditText;
import android.widget.TextView;
import android.widget.Toast;

import org.json.JSONArray;
import org.json.JSONException;
import org.json.JSONObject;

import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.prefs.PreferenceChangeEvent;

// TrainingsabschnittActivity
public class TaskListActivity extends AppCompatActivity {

int loggedInUserID;
int loggedInUserscore;
int refreshedUserscore;
int refreshedUserTear;
int loggedInUserTear;
String loggedInUsername;




private TaskListAdapter adapter;
private List<Task> taskList;

TextView currentUserscoreTV;
TextView currentUserTearTV;
RecyclerView taskRecyclerView;
SwipeRefreshLayout swipeRefreshLayout;

RequestQueue queue;



@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_task_list);
    Toolbar toolbar = findViewById(R.id.toolbar);
    setSupportActionBar(toolbar);

 // hier werden die Daten vom Server geholt, durch das Debuggen weiß ich, dass die richtigen Werte enthalten sind.
    loggedInUserID = PreferenceManager.getDefaultSharedPreferences(this).getInt("kundenID", -1);
    loggedInUserscore = PreferenceManager.getDefaultSharedPreferences(this).getInt("userScore", -1);
    loggedInUsername = PreferenceManager.getDefaultSharedPreferences(this).getString("username", "-1");
    loggedInUserTear = PreferenceManager.getDefaultSharedPreferences(this).getInt("userTear", -1);

    currentUserscoreTV = (TextView) findViewById(R.id.userScoreTV);
    currentUserTearTV = (TextView) findViewById(R.id.userTearTV);

    taskList = new ArrayList<>();
    adapter = new TaskListAdapter(this, taskList);
    taskRecyclerView = (RecyclerView) findViewById(R.id.taskListRecyclerView);
    taskRecyclerView.setHasFixedSize(true);
    taskRecyclerView.setLayoutManager(new LinearLayoutManager(this));
    taskRecyclerView.setAdapter(adapter);

    queue = Volley.newRequestQueue(this);

// swipeRefreshLayout = (SwipeRefreshLayout) findViewById(R.id.swipeRefreshLayout);
// swipeRefreshLayout.setOnRefreshListener(new SwipeRefreshLayout.OnRefreshListener() {
// @Override
// public void onRefresh() {
// // legt fest, was beim swipen refresht wird
// loadUserScore();
//
//
// // beendet die optische Laderückgabe - also den sich drehenden Preil
// swipeRefreshLayout.setRefreshing(false);
// }
// });

    loadUserScore();
    loadTaskList();
}



public void loadUserScore() {

    String create_user_url = getString(R.string.XAMPP) + "/ScoreRefresh.php";

    StringRequest postRequest = new StringRequest(Request.Method.POST, create_user_url,
            new Response.Listener<String>() {
                @Override
                public void onResponse(String response) {


                    Log.i("response", response);

                    try {
                        JSONObject jsonResponse = new JSONObject(response);

                        int success = Integer.parseInt(jsonResponse.get("success").toString());
                        if (success == 1) {

                           PreferenceManager.getDefaultSharedPreferences(TaskListActivity.this).edit().putInt("userScore", jsonResponse.getInt("refreshed_score")).apply();
                            PreferenceManager.getDefaultSharedPreferences(TaskListActivity.this).edit().putInt("userTear", jsonResponse.getInt("user_tear")).apply();

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



                }
            }, new Response.ErrorListener() {
        @Override
        public void onErrorResponse(VolleyError error) {

        }
    }){
        @Override
        protected Map<String, String> getParams() {
            Map<String, String> params = new HashMap<>();
            params.put("aUsername", loggedInUsername);

            return params;
        }
    };

// Ich weiß, dass die Shared prefs nicht richtig aufgebaut sind, da sie aber Grundsätzlich tun was sie sollen, habe ich dieses Problem ersteinmal hinten angehängt
refreshedUserscore = PreferenceManager.getDefaultSharedPreferences(this).getInt("userScore", -1);
refreshedUserTear = PreferenceManager.getDefaultSharedPreferences(this).getInt("userTear", -1);

    SharedPreferences SPRefreshedUserscore = getSharedPreferences(String.valueOf(refreshedUserscore), Activity.MODE_PRIVATE);

    String setUserscore = SPRefreshedUserscore.getString("", String.valueOf(refreshedUserscore));

    SharedPreferences SPRefreshedUserTear = getSharedPreferences(String.valueOf(refreshedUserTear), Activity.MODE_PRIVATE);
    String setUserTear = SPRefreshedUserTear.getString("", String.valueOf(refreshedUserTear));

// hier liegt das aktuelle Problem, die Daten die hier hereingeschrieben werden sollten, kommen aus der oben ausgeführten RefreshScore.php, diese liefert auch brav die Daten in der App ab, allerdings erst nachdem die App fertig geladen ist, ich muss also nocheinmal swipen bzw laden bis die Daten auch angezeigt werden

    currentUserscoreTV.setText(setUserscore);
    currentUserTearTV.setText(setUserTear);


    queue.add(postRequest);

}


public void loadTaskList() {

    taskList.clear();
    String create_user_url = getString(R.string.XAMPP) + "/GetAllTasks.php";

    StringRequest postRequest = new StringRequest(Request.Method.POST, create_user_url,
            new Response.Listener<String>() {
                @Override
                public void onResponse(String response) {


                    Log.i("response", response);

                    try {
                        JSONObject jsonResponse = new JSONObject(response);
                        JSONArray taskArray = (JSONArray) jsonResponse.get("task");


                        for (int taskObjekte = 0; taskObjekte < taskArray.length(); taskObjekte++) {
                            JSONObject taskJson = taskArray.getJSONObject(taskObjekte);

                            taskList.add(new Task(taskJson.getString("taskname")));
                        }

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

                    adapter.notifyDataSetChanged();


                }
            }, new Response.ErrorListener() {
        @Override
        public void onErrorResponse(VolleyError error) {

        }
    }){
        @Override
        protected Map<String, String> getParams() {
            Map<String, String> params = new HashMap<>();
            params.put("usertear", String.valueOf(loggedInUserTear));

            return params;
        }
    };

    queue.add(postRequest);
}


@Override
public boolean onCreateOptionsMenu(Menu menu) {
    getMenuInflater().inflate(R.menu.menu_task_list, menu);
    return true;
}
@Override
public boolean onOptionsItemSelected(MenuItem item) {
    int id = item.getItemId();
    if(id == R.id.profile) {
        Intent i = new Intent(getApplicationContext(), ProfileActivity.class);
        startActivity(i);
    }
    return super.onOptionsItemSelected(item);
}

}

also die Daten werden geladen, wenn ich mich sozusagen frisch einlogge, dann bekomme ich alle daten genau so angezeigt wie ich sie haben möchte, sowohl den Score als auch den tear. Erst wenn sich dei Daten auf dem Server danach ändern und man die Daten durch den swipe refreschen will, tritt der bug auf. Also das Anzeigen der Daten an sich funktioniert.

Ok, das mit dem Ablauf hatte ich befürchtet, ich dachte das Die einzelnen Methoden nacheinander ausgeführt werden.

"Wenn du die Daten aus dem Netz auch auf dem Bildschirm anzeigen willst musst du das auch in dem Listener machen. Von wo sie kommen." - Da weiß ich leider nicht genau was du meinst, kannst du das vllt genauer erklären?

"Die beiden Zeilen führst du aus noch bevor du überhaupt deinen Request abschickst.
Kann nicht gehen . Denke zeitlich. Parallel." - das ist eine sehr gute Information, das hatte ich zwar schon beobachtet aber nicht verstanden... Da sich die beiden Zeilen ja noch in der Methode befinden klingt es sofort logisch, da der request aber schon abgeschlossen ist, also so rein von den Klammern her - sorry wenn ich das so Leihenhaft ausdrücke - dachte ich auch diese beiden Zeilen würden nach dem Request ausgeführt werden.
Dann stellt sich mir die Frage, wo muss ich die beiden Zeilen denn unterbringen, dass sie in der Methode, aber nach dem Request verarbeitet werden?

"Ps. das ist kein Bug von Android sondern ein Fehler deinerseits. Multithreading." - das habe ich auch nie behauptet

Ich sage schonmal danke für die schnelle Antwort Ich lasse diese Infos bis morgen mal auf mich wirken, vllt macht es dann sofort klick. Wenn noch Jemand ne Idee hat, würde ich mich auch über weitere Ideen oder Anmerkungen freuen

Lg Neggo

Ich danke dir vielmals für diese genaue Erklärung

Ich habe den Codeschnipsel, der dafür sorgt, dass die Daten der Preferences in den TextViews landen in den Try-Catch Bereich verschoben, nun möchte er da den Context haben.

Den Context "this" scheint er nicht zu mögen und auch die Klasse.this wie im Response der Context definiert wird will hier nicht passen, weißt du wieso das so ist, bzw. was ich dort eintragen muss?

     refreshedUserscore = PreferenceManager.getDefaultSharedPreferences(this).getInt("userScore", -1);

Ich hoffe du entschuldigst diese Fragen, ich habe versucht das Problem über Bücher und Google zu lösen, das Problem ist dass ich nicht recht weiß wonach ich suchen soll :/

Aber Ich danke vielmals, dass Ihr euch mit meinem Problem beschäftigt

Lg Neggo

— geändert am 30.10.2020, 13:15:59

Ich poste einfach nochmal kurz die Methode, wie sie aktuell aussieht, und mache ne Markierung dran, wo er meckert, vllt ist es dann offensichtlicher

Edit: Wenn statt "this" - "TaskListActivity.this" eingetragen wird, kommt als Rückmeldung "this is not available", Ich versuche diese Info zu googeln, aber bisher hatte ich leider nur mäßig Erfolg :/, vllt hilft euch diese Info ja weiter

Edit2: Ich habe gerade festgestellt, das das, was ich da unten versuche, "TaskListActivity.this" - funktioniert... Ich habe es die ganze Zeit im Debugger versucht und nur auf die "Fehler" geachtet und nicht auf die Ausgabe, die , wie ich gerade feststellen durfte, funktioniert.
Nun ist die Frage, seht ihr da immernoch Fehler, also funktioniert es nur durch Zufall und als 2. Frage, wieso sagt er mir das "refreshedUserscore = this is not available" im Debugmode?

Ja das mir Grundlagen fehlen hatte ich ja gesagt, aber ich arbeite dran

public void loadUserScore() {

    String create_user_url = getString(R.string.XAMPP) + "/ScoreRefresh.php";

    StringRequest postRequest = new StringRequest(Request.Method.POST, create_user_url,
            new Response.Listener<String>() {
                @Override
                public void onResponse(String response) {


                    Log.i("response", response);

                    try {
                        JSONObject jsonResponse = new JSONObject(response);


                        int success = Integer.parseInt(jsonResponse.get("success").toString());
                        if (success == 1) {


                            PreferenceManager.getDefaultSharedPreferences(TaskListActivity.this).edit().putInt("userScore", jsonResponse.getInt("refreshed_score")).apply();
                            PreferenceManager.getDefaultSharedPreferences(TaskListActivity.this).edit().putInt("userTear", jsonResponse.getInt("refreshed_tear")).apply();

------------------------------------------------------------------------

    refreshedUserscore = PreferenceManager.getDefaultSharedPreferences(this).getInt("userScore", -1);

------------------------------------------------------------------------

                         refreshedUserTear = PreferenceManager.getDefaultSharedPreferences(this).getInt("userTear", -1);





                            SharedPreferences SPRefreshedUserscore = getSharedPreferences(String.valueOf(refreshedUserscore), Activity.MODE_PRIVATE);

                            String setUserscore = SPRefreshedUserscore.getString("", String.valueOf(refreshedUserscore));

                            SharedPreferences SPRefreshedUserTear = getSharedPreferences(String.valueOf(refreshedUserTear), Activity.MODE_PRIVATE);
                            String setUserTear = SPRefreshedUserTear.getString("", String.valueOf(refreshedUserTear));

                            currentUserscoreTV.setText(setUserscore);
                            currentUserTearTV.setText(setUserTear);


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



                }
            }, new Response.ErrorListener() {
        @Override
        public void onErrorResponse(VolleyError error) {

        }
    }){
        @Override
        protected Map<String, String> getParams() {
            Map<String, String> params = new HashMap<>();
            params.put("aUsername", loggedInUsername);

            return params;
        }
    };


    queue.add(postRequest);

}

— geändert am 30.10.2020, 15:39:17

Ich werd mir auf jedenfall einige Grundlagen ansehen, aber danke für all eure Antworten =)

Öh ne. ich hab mir das aus verschiedenen Guides und Threads aus dem Netz zusammen gebastelt, jenachdem was ich einbauen wollte habe ich Erklärungen gesucht und eingebaut. Aber das da veraltete Teile drin sind ist mir schon öfter aufgefallen, ich behebe stück für stück, wenn ich bessere Lösungen finde^^

— geändert am 30.10.2020, 17:16:00

• ja, steht auf meiner Todo.

• Mir ist durchaus bewusst, dass der Code da doppelt gemoppelt ist, bisher habe ich es aber nicht hinbekommen es kürzer zufassen, da meist Fehler in der Syntax auftraten, daher also danke für das Beispiel, ich werde mir mal ansehen ob das funktioniert.

Es klingt als würdest du da viel anders aufziehen.

Ich bringe mir den Kram gerade selber bei, da ist eben noch nicht alles sauber wie aus dem Lehrbuch. An welcher Stelle würdest du denn ansetzen um da Ordnung rein zu bringen?

— geändert am 30.10.2020, 18:58:16

joar, danke für die Tipps.
Dein Tonfall und manche Sprüche hättest du dir kneifen können. Aber es ist Freitag, vllt warst du betrunken oder hattest einen scheiß Tag, kann mal passieren.

Ich wünsch noch nen schönen Abend. Man schreibt sich