do-while-Schleife mit Button weiter

bacarni · 9. Dezember 2010

Hallo!
Ich hole mir aus meiner Datenbank Fragen und Antworten und stelle sie dar. Das funktioniert auch wunderbar (über ListActivity).

Jetzt aber wollte ich gerne eine Frage zur Zeit und die Antwort dazu. Dazu habe mir die Buttons Richtig und Falsch hinzugefügt. Leider hänge ich jetzt in der while-Schleife fest. Ich bekomme zwar meine Toast-Nachricht, aber die while-Schleife geht nicht weiter.

Habt ihr eine Idee? Vllt. ist mein Ansatz auch falsch.

Der Quelltext sieht wie folgt aus:

// Check if our result was valid.
c.moveToFirst();
if (c != null) {
int i = 0;
// Loop through all Results
do {
i++;

String Question = c.getString(columnQuestion);
String Answer = c.getString(columnAnswer);

txtQuestion.setText(Question);
txtAnswer.setText(Answer);

btnCorrect.setOnClickListener(new View.OnClickListener() {
public void onClick(View v) {

Toast toast = Toast.makeText(getApplicationContext(), "Richtig", Toast.LENGTH_SHORT);
toast.show();

newquestion = 1;

}
});

} while (c.moveToNext() && newquestion == 1);
}

Thrakbad · 9. Dezember 2010

newquestion is ne member-Variable? Check mal, ob er die richtig auf 1 setzt, wie du dir das gedacht hast.

bacarni · 10. Dezember 2010

Die Variable habe ich vor OnCreate in der Klasse deklariert. Also sollte das passen.
Dann habe ich mir in der Methode onClick die Variable mittels

Java

Log.d("newquestion: ", String.valueOf(newquestion));

ausgeben lassen und sie wird auf 1 gesetzt.

Kann es vielleicht daran liegen, dass er aus der Schleife springt, sobald die Bedingung nicht erfüllt ist und gar nicht auf den Button-klick wartet?

Thrakbad · 10. Dezember 2010

Ah klar...manchmal sieht man den Wald vor lauter Bäumen nicht. Genau daran liegt das natürlich, nach dem ersten Schleifendurchlauf ist newQuestion logischerweise 0, weil so schnell kannste gar nicht klicken und dann isser raus. Da is jetz natürlich der Ansatz etwas ausgehebelt
Ich würde das ganze so ungefähr machen:

Java

class ...{ // Cursor c hier irgendwo als member 
public void onCreate(..) { btnCorrect.setOnClickListener(new View.OnClickListener() {
public void onClick(View v) {


Toast toast = Toast.makeText(getApplicationContext(), "Richtig", Toast.LENGTH_SHORT);
toast.show();


nextQuestion(); }});
} private void nextQuestion() { // c initialisieren und prüfen usw if(c.moveToNext()) { String Question = c.getString(columnQuestion);
String Answer = c.getString(columnAnswer);


txtQuestion.setText(Question);
txtAnswer.setText(Answer);
 } else { // was auch immer passieren soll, wenn die Fragen Alle sind } }}

Alles anzeigen

EDIT: voll blöd, irgendwie mag er meine Leerzeilen in dem Code nicht richtig übernehmen

bacarni · 10. Dezember 2010

Ok! Dann werde ich das mal so umsetzen Danke für die Hilfe!

Btw: Meine Leerzeichen wollte er auch nicht haben! Deshalb ist der obere Quelltext auch einfach eingefügt. Hat es immer wieder in eine Zeile geschrieben. Nicht sehr optimal

do-while-Schleife mit Button weiter

Jetzt mitmachen!

Teilen