Frage zu ||

[Guest]

Also mit den || und && steh ich ja so ein bissel auf Kriegsfuß.

Ich habe jetzt dieses Programmstück hier:

if ( runde == 1 || runde == 2 || runde == 3 || runde == 4 ||
             runde == 5 || runde == 7 || runde == 8 || runde == 9 ||
             runde == 10 )
        falscheAntwort();

Es soll wenn runde 1,2,3,4,5,7,8,9 oder 10 ist (also eins von denen) die Methode falscheAntwort aufrufen...mache dann noch eine 2. Abfrage die bei runde 6 die Methode richtigeAntwort aufruft...wird das so klappen ?

 

[Destiny1985]

Sorry, Post war von mir, hatte mich nicht eingeloggt :<

 

[Stefan1200]

Ich denke schon, aber mach doch in diesem Fall folgendes:

if (runde == 6)
        richtigeAntwort();
else if ((runde >= 1) && (runde <= 10)) 
        falscheAntwort();

Habe ich jetzt nicht getestet.
Ich hoffe dich jetzt nicht noch mehr verwirrt zu haben ;-).

 

[Destiny1985]

Nee is ok das kapier ich

schreibe gerade ein Quiz, wo man jede Runde geld setzen kann usw....wenn das fertig ist poste ich den Code mal

*ist gerade stolz das sein programm schon so gut läuft*

[edit]

führt der code nicht zu irgendnem fehler bzw is das sauberer stil ? weil der fall das runde = 6 ist tritt ja trotzdem bei beiden auf ?!

[/edit]

 

[Stefan1200]

Naja, deswegen wird ja nach runde == 6 vorher gefragt.
Natürlich darf man die Reihenfolge der Abfrage nicht ändern.

Wenn es ganz sicher sein muss:

if (runde == 6) 
        richtigeAntwort(); 
else if (((runde >= 1) && (runde <= 5)) || ((runde >= 7) && (runde <= 10))) 
        falscheAntwort();

Aber funktionieren wird beides.

 

[DP]

wobei ich bei int-werten switch nehmen würde:

        switch (runde)
        {
          case 6:
            richtigeAntwort();
            break;
          default:
            falscheAntwort();
            break;
        }

 

[Dreezard]

Ich würde auch zu switch greifen.
Um diese noch zu komplettieren:

switch (runde) 
        { 
          case 6: 
            richtigeAntwort(); 
            break; 
          case 1:
          case 2:
          case 3:
          case 4:
          case 5:
          case 7:
          case 8:
          case 9:
          case 10:
            falscheAntwort(); 
            break; 
        }

 

[thE_29]

Das ist aber schwachsinn ^^

Da kannst gleich ein if nehmen!

Was foobar gepostet sollte man nehmen....

Alles was keine falschen Antworten sind, kommen in ein case, der Rest was falsch ist ins default!

 

[mic_checker]

wobei ich bei int-werten switch nehmen würde:

        switch (runde)
        {
          case 6:
            richtigeAntwort();
            break;
          default:
            falscheAntwort();
            break;
        }

Das "Problem" dabei ist allerdings, das auch wenn der Spieler in einer andern Runde ist (nicht 1...10) der default Fall aufgerufen wird, also die Methode falscheAntwort(). Kommt natürlich drauf an, ob sicher gestellt ist , dass man sich in einer der vorgesehenen Runden befindet (Dein Code stimmt natürlich, ist nur die Frage ob das Verhalten dann so gewünscht ist).

Ansonsten kannst du Stefans Variante nehmen.

Allgemein gilt bei den Operatoren || und && folgendes:
Es sind beides logische Operatoren. Wenn du hast a || b, so genügt es wenn eine der beiden Aussagen a , b wahr ist, damit der Ausdruck wahr ist. Sobald a wahr ist, wird b gar nicht erst ausgewertet, da der Ausdruck auf jeden Fall wahr ist.
Bei a && b gilt: Beide Ausdrücke a und b müssen wahr sein, damit der Ausdruck wahr ist. Ist der Ausdruck a bereits false, so wird b gar nicht mehr ausgewertet, da somit automatisch der ganze Ausdruck false ist.