Firebase überträgt Daten nicht richtig?

nono124 · 27. April 2020

dein alter code solte so auch laufen. nur bei dem neuen Lauf bin ich mir nicht sicher habe ich nicht gestestet.

Code

private void zeitBerechnung(int runde) {


        for (String e : userLastTime.keySet()) {
            zeitTabelle.put(e, userLastTime.get(e).get(runde - 1));
        }


        final Map&lt;String, Integer&gt; unsortMap = new HashMap&lt;&gt;();


        for (boolean e : checked) {
            if (e) {
                auswahlAnzahl++;
            }
        }




        for (final Map.Entry e : zeitTabelle.entrySet()) {
            if (!e.getValue().toString().equals("00:00:00")) {


                //mit dem yardstick berechnen
                mDatabase.child("users").child(e.getKey().toString()).addListenerForSingleValueEvent(new ValueEventListener() {
                    @Override
                    public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
                        int yardstickDb = Integer.parseInt(dataSnapshot.child("Yardstick").getValue().toString());


                        int zeitInSekunden;
                        int berechneteYardstickzeit;
                        String numbers = e.getValue().toString();
                        String[] split = numbers.split(":");
                        zeitInSekunden = Integer.parseInt(split[2]);
                        zeitInSekunden += Integer.parseInt(split[1]) * 60;
                        zeitInSekunden += Integer.parseInt(split[0]) * 60 * 60;
                        berechneteYardstickzeit = (zeitInSekunden * 100) / yardstickDb;
                        berechnungsZähler++;
                        unsortMap.put(e.getKey().toString(), berechneteYardstickzeit);//hilfe


                        Log.i("test", "im listner :" + e.getKey()+ " "+ berechnungsZähler+ " "+auswahlAnzahl);
                        //berechnung fertig
                        Map&lt;String, Integer&gt; sortedMap = sortByValue(unsortMap);
                            int i = 0;
                            for (String key : sortedMap.keySet()) {
                                i++;
                                if (i == 1) {
                                    alleUser.put(key, 0.);
                                }
                                if (i == 2) {
                                    alleUser.put(key, 3.);
                                }
                                if (i == 3) {
                                    alleUser.put(key, 5.7);
                                }
                                if (i == 4) {
                                    alleUser.put(key, 8.);
                                }
                                if (i == 5) {
                                    alleUser.put(key, 10.);
                                }
                                if (i == 6) {
                                    alleUser.put(key, 11.7);
                                }
                                if (i &gt;= 7) {
                                    alleUser.put(key, i + 6.);
                                }
                            }


                            //bei neue Regatta
                            Log.i("test", "if test ob auswahl Regatta :  " + auswahl);
                            if (auswahl.equals("Neue Regatta")) {
                                /*for (String key : alleUser.keySet()) {
                                    Log.i("test", "in der daten schreiben Schleife :" + key);
                                    mDatabase.child("regatten").child(Integer.toString(regatten + 1)).child("1").child(key).setValue(alleUser.get(key));
                                }*/
                                String key = (String) e.getKey();
                                    mDatabase.child("regatten").child(Integer.toString(regatten+1)).child("1").child(key).setValue(alleUser.get(key));
                                Log.i("test", "Daten geschreiben :" + key);


                            }


                            //Bei Neuer Lauf


                            else if (auswahl.equals("Neuer Lauf")) {
                                mDatabase.child("regatten").child(Integer.toString(regatten)).addListenerForSingleValueEvent(new ValueEventListener() {
                                    @Override
                                    public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
                                        lauf = 0;
                                        Iterator&lt;DataSnapshot&gt; dataSnapshots = dataSnapshot.getChildren().iterator();
                                        while (dataSnapshots.hasNext()) {
                                            DataSnapshot dataSnapshotChild = dataSnapshots.next();
                                            lauf++;
                                        }
                                        for (String key : alleUser.keySet()) {
                                            mDatabase.child("regatten").child(Integer.toString(regatten)).child(Integer.toString(lauf + 1)).child(key).setValue(alleUser.get(key));
                                        }
                                        
                                    }


                                    @Override
                                    public void onCancelled(@NonNull DatabaseError databaseError) {
                                    }
                                });
                            }


                        if (berechnungsZähler == auswahlAnzahl) {
                            Log.i("test", "regatta abbrechen  ");
                            regattaAbbrechen();
                        }


                    }


                    @Override
                    public void onCancelled(@NonNull DatabaseError databaseError) {
                    }
                });
            }
        }
    }

Alles anzeigen

nono124 · 27. April 2020

Warum wird bei nerer Lauf ein neuer listner erstellt?

das Prüfen welche listner du brauchst solltest du vor dem Erstellen des Regatta listner machen in der ersten for Schleife.

Kurmuro · 27. April 2020

Der ist dafür da um rauszufinden wie viel läufe es davor schon gegeben hat. Heißt wenn es schon zwei gibt soll der jetzige unter lauf 3 abgespeichert werden.
Ist allerdings erstmal irrelevant da es schon bei regatta nicht geht. Wie gesagt, ich werde das mit der abfrage ob alle Spieler in der Datenbank sind mal in Angriff nehmen.

Mal ne Frage abseits. Ich möchte sobald alle Teilnehmer übertragen wurden einen Toast angezeigt bekommen das es funktioniert hat. Jetzt beendet mir das System.exit allerdings direkt den Toast. Heißt er wird nicht angezeigt. Gibt es da eine möglichkeit das zu umgehen?
Hab da schon was gefunden: https://stackoverflow.com/ques…e-never-goes-away#5420929 allerdings wird der Toast bei mir trotzdem nicht angezeigt.

Am einfachsten wäre es vermutlich auf das System.exit zu verzichten. Dann könnte ich mir auch die abfrage sparen ob alle teilnehmer übertragen wurden.
Bekanntes problem: Wenn ich es weglasse habe ich ja das Problem das in der nächsten regatta die letzten werte noch vorhanden sind und ich erst die alten zeiten manuell löschen muss.
Allerdings hab ich keine Ahnung wie ich meinen Viewholder leer bekomme

nono124 · 27. April 2020

Der geänderte Code von mir geht. Ob deine Bechnung stimmt ist deine Sache . Die Listner werden alle durchlaufen.
Un d am ende mache ich auch ein exit.

nono124 · 27. April 2020

hallo eine einfache verzögerte lösung wäre in etwas so.
Bei einem Toast musst du aber aufpassen das du auch den richtgen Context hast.

Code

public void regattaAbbrechen() {
        Toast.makeText(Regatta.this, "Speichern OK!", Toast.LENGTH_LONG).show();
        stoppuhr.cancel();
        checked = null;
        timerisrunning = false;
        zeitTabelle.clear();
        TimerExit te = new TimerExit();
        te.execute();
    }


    public class TimerExit extends AsyncTask&lt;Void, Void, Void&gt; {
        @Override
        protected Void doInBackground(Void... params) {
            try {
                Thread.sleep(2000); // 1000 = eine sek.
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
            return null;
        }
        @Override
        protected void onPostExecute(Void aVoid) {
            super.onPostExecute(aVoid);
            Regatta.this.finish();
            System.exit(0);
        }
    }

Alles anzeigen

Kurmuro · 27. April 2020

Den Code von dir habe ich auch getestet allerdings funktioniert da bei mir garnichts... Einmal geht die Berechnung nichtmehr. Es trägt in die datenbank immer nur die Punktzahl 0 ein. Zudem speichert er genauso fehlerhaft wie momentan. Mal ein teilnehmer, mal zwei.. Nie alle drei.

Ich mach jetzt die einfachste lösung. Ich lasse den bestehenden Code wie er ist. Und füge (auch wenn man es nicht machen sollte) das

try{
Thread.sleep(1000);
}catch(InterruptedException e){} vor dem System.exit(0) auf... mehr als das die Activity abstürtzt kann ja nicht wirklich passieren... Unsauber aber selbst wenn würde es ja nicht stören. Dieser Fehler nervt mich jetzt schon so lang. Zumindest funktioniert es so... Never touch a runnning system

nono124 · 27. April 2020

mit meinem Asyncktask ist es sauber und du hälst nicht den UI an.

ok dann viel spass bei deiner neuen variante.

Kurmuro · 27. April 2020

Oh die Nachricht hatte ich wohl überlesen. Ja so funktioniert es tatsächlich und gibt mir sogar den Toast aus *.*
Endlich eine schöne funktionierende lösung
dankeschön

Kurmuro · 27. April 2020

Das nächste Oorschwerbleede Bier geht auf mich

Firebase überträgt Daten nicht richtig?

Jetzt mitmachen!

Teilen

Tags

Benutzer online in diesem Thema