Beiträge von nono124

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

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

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.

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

Wenn du nicht weist was diese Zeile in Kotlin alles macht.

class Field (context: Context, attrs: AttributeSet)

Solltest du dir Kotlin genauer anschauen, vor allen wie die ausführliche Schreibweise aussieht .

In Java ist das Ganze mit getter und setter noch mehr Schreibarbeitdas hat Kotlin alles schon für dich gemacht.

Das was in dem komprimierten Konstruktor ist sind Klassen Variablen.

nono124 · 27. April 2020

Nein du hast nicht aufgepasst den Context hast du schon wird dir in der Klasse im primer Kostrucktor mit übergeben.

class Field (context: Context, attrs: AttributeSet): View(context, attrs)

die abgeleitete View „Field“ hast du bestimmt in deinem Layout benutz. Und das bindest du in der Activity mittels setContentView ein und genau da wird die Instanz der Kalssse erstellt und ihr wird der Context der Activity übergeben.

Wo du das machst bleibt dir überlassen .
var sharedPreferences: SharedPreferences = context.getSharedPreferences("highscore",Context.MODE_PRIVATE)

nono124 · 27. April 2020

Alle Instanzen der User löschen was soll das heißen.

Beim beenden exit 0 werden alle Instanzen der activity gelöscht zurückgeben an den gc

nono124 · 27. April 2020

Hallo mit exit (0) beendest du die activity und somit auch die erstellten listner. Wenn der erste Lister fertig ist und das exit macht haben die anderen kein Möglichkeit mehr den Code auszuführen und die Daten in die DB zuschreiben.
Deshalb schreiben nicht im Listen.

Außerdem wozu brauchst du 3 oder auch 10 listner (pro User einen) wenn du es sowieso erst am Ende der regatta nach der Dialog box Abfrage machst.
Dann kannst du zu diesen zeitbunkt die Daten in die DB schreiben.
Es würde ein listner reichen der dir alle Daten der User holt und zwar alle die an der Regatta teil genommen haben auf einmal. Nun kannst du rechnen und jedem seine Punkte dazu zählen und wider Speichen. Mache das alles in einem listner. Danach kannst du die activity verlassen beenden.

Oder du lässt dir etwas einfallen woran du erkennst das alle Lister fertig waren aufgerufen wurden gespeichert haben und nur dann kannst du die activity beenden.

Da sind wir wider bei meinen schon genannten if. Was du nicht beachten wolltest.

Denn gleich nach dem Speichen der Daten im listner beendest du die activity. Somit gaben die anderen listner keine Chance mehr dann zu kommen. Weil die activity ist nach dem ersten listner beendet. Ein berechnen der akten Punkte mit den neuen Punkten der anderen User findet dann nicht mehr statt. Da die activity beendet und alle Instanzen der listner löscht sind.

nono124 · 26. April 2020

ok wie du meist für mich ist jetzt schluss

nono124 · 26. April 2020

so wir das auch nicht gehen.
lesen aus der DB geht nur im listner.

schaue dir das mit der if an und benzute den alten code.

nono124 · 26. April 2020

hast du das Github verändert.

nono124 · 26. April 2020

so wie dein code aufgebaut ist musst du in allen drei listnern schreiben nur findert das durch diese if abfrage nicht in allen stat.

sehe ich auch in meinen Log.

nono124 · 26. April 2020

ich denke das Problem ligt hier

Code

if (berechnungsZähler &gt;= auswahlAnzahl){
                                Map&lt;String, Integer&gt; sortedMap = sortByValue(unsortMap);

in den if zweig geht er nur in einem der drei listner rein und somit wird auch nur in dem einem Listner etwas geschrieben.

nono124 · 26. April 2020

Code

for(final Map.Entry e : zeitTabelle.entrySet()){
                if(!e.getValue().toString().equals("00:00:00")) {
                    Log.i( "test","zeitberechnen for Schleife:"+ e.getKey());


                    //mit dem yardstick berechnen
                    mDatabase.child("users").child(e.getKey().toString()).addListenerForSingleValueEvent(new ValueEventListener() {
                        @Override
                        public void onDataChange(@NonNull DataSnapshot dataSnapshot) {

die die du hier erstellst. drei user sind es drei oder bei 10 user eben 10.

nono124 · 26. April 2020

Auch wenn du das Regatta abbrechen weg lest wir immer nur ein Listner aufgerufen. Die überschneiden sich.

Entweder müssen die alle einzeln in Threads oder du macht nur einen Listner der dir alle daten der User in den dataSnapshot liefert und du kannst somit auch alle daten gleichzeitig berechnen.
du machst das sowieso erst am ende wo alles gelaufen ist.

nono124 · 26. April 2020

Im Debugger mit einem Brake im listner geht es richtig.

Also existiren auch die drei listner.

Bei einem normalen durchlauf wird das Handy beschäftigt sein wenn Firebas einen Lister aufruftweil es noch in einem andern Listner ist. Somit geht die daten Übergabe ins lehre . Bein Debugger geht es langsamer und ich bekomme auchalle logs.

Du musst dein Konzept überarbeiten.

denke es ist jetzt genug.

nono124 · 26. April 2020

mal erlich was soll das du wisst am anfang deines Threads auf SharedPreference zugreifen. da finde ich nichts in deinem code ausser eine Klasse die von View abgelleitet wird.

MainActivity = MainActivity()
private var sharedPreferences: SharedPreferences = main.getSharedPreferences("highscore",Context.MODE_PRIVATE)

war der Anfang

Du hast in der view Klasse doch denn Kontext der activity.
Den nimmst Du um auf die sheredpref zu zugreifen.
Das ist Deine Instanz der activity.
Eine neues Objekt der activity zu erstellen ist falsch. OOP

nono124 · 26. April 2020

warscheimlich kannst du nur einen
addListenerForSingleValueEvent setzen wuste ich nleider auch nicht der wird dann in der For schleif Überschrieben und nur der letzte bleibt.

nono124 · 26. April 2020

Ich habe ein paar logs ein gefügt. Ist ein merkwürdiges verhalten.

Eigentlich dachte ich das drei Eventlistner erstellt werden. aufgerufen wird aber nur einer .

Siehe Bild .

Da bin ich im Monet etwas überfragt.

Wenn die listner nicht aufgerufen werden oder gar nicht Vorhanden sind kann deine Berechnungauch nicht stimmen . denke da waren wir beide auf dem falschen Weg.

nono124 · 26. April 2020

ich möchte meinme Tel nicht weiter geben

Beiträge von nono124

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

High score in aus externem View speichern

High score in aus externem View speichern

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

High score in aus externem View speichern

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?

Firebase überträgt Daten nicht richtig?