Kann Logikfehler im Programm nicht finden

[Sneek]

Hi Leute, hab mal wieder ein kleines Problem.

Also wir sollen ein Programm entwickeln was eine Statistik aufstellt.
Es werden drei Würfel simuliert. Der Benutzer soll die Anzahl der Würfe eingeben. Dabei soll aufgezeichnet werden wie oft ein Pasch in der 1, 2, 3 Runde, 4-9 Runde, 10 - 49 Runde geworfen wurde.
Ich hoffe, dass ist erstmal soweit alles klar.

Ich habe das ganze so umgesetzt:

Die Variable int b stellt die Rundenanzahl da, ich denke der Rest ist aus dem Programm ersichtlich.

class StatistikWuerfeln{
  public static void main(String[] args){
    int n, runde1 = 0, runde2 = 0 , runde3 = 0, runde4_9 = 0 , runde10_49 = 0, runde50 = 0, b = 1;
    int zahl1, zahl2, zahl3;
  
    n = In.readInt();
    for(int i = 0; i < n; ++i){
      
      zahl1 = (int) (Math.random() * 6) + 1; 
      zahl2 = (int) (Math.random() * 6) + 1;
      zahl3 = (int) (Math.random() * 6) + 1;
      
      if((b == 1) && (zahl1 == zahl2) && (zahl2 == zahl3)){
        runde1 += 1;
        b = 0;
      }
    
      if((b == 2) && (zahl1 == zahl2) && (zahl2 == zahl3)){
        runde2 += 1;
        b = 0;
      }
    
      if((b == 3) && (zahl1 == zahl2) && (zahl2 == zahl3)){
        runde3 += 1;
        b = 0;
      }
    
      if((b >= 4) && (b <= 9) && (zahl1 == zahl2) && (zahl2 == zahl3)){
        runde4_9 += 1;
        b = 0;
      }
    
      if((b >= 10) && (b <= 49) && (zahl1 == zahl2) && (zahl2 == zahl3)){
        runde10_49 += 1;
        b = 0;
      }
    
      if((b > 50) && (zahl1 == zahl2) && (zahl2 == zahl3)){
        runde50 += 1;
        b = 0;
      }
      ++b;
      //Out.println("Zahl1: " + zahl1 + "Zahl2 :" + zahl2 + "zahl3: " + zahl3 + "b: " + b);
            
    }
  
    Out.println("Pasch in 1. Runde: " + runde1 + "-mal");
    Out.println("Pasch in 2. Runde: " + runde2 + "-mal");
    Out.println("Pasch in 3. Runde: " + runde3 + "-mal");
    Out.println("Pasch in 4 bis 9 Runden: " + runde4_9 + "-mal");
    Out.println("Pasch in 10 bis 49 Runden: " + runde10_49 + "-mal");
    Out.println("Pasch in 50 oder mehr Runden: " + runde50 + "-mal");
  }
}

Ich scheine irgendwas mit b falsch zu machen. Bei 100000 müsste ungefähr folgende Werte ausgespuckt werden:

Bitte Anzahl gewuenschter Versuche eingeben: 100000
Pasch in 1. Runde: 2786-mal
Pasch in 2. Runde: 2663-mal
Pasch in 3. Runde: 2657-mal
Pasch in 4 bis 9 Runden: 14379-mal
Pasch in 10 bis 49 Runden: 52372-mal
Pasch in 50 oder mehr Runden: 25143-mal

Bei mir kommen folgende Werte raus:

100000
Pasch in 1. Runde: 72-mal
Pasch in 2. Runde: 69-mal
Pasch in 3. Runde: 63-mal
Pasch in 4 bis 9 Runden: 417-mal
Pasch in 10 bis 49 Runden: 1453-mal
Pasch in 50 oder mehr Runden: 677-mal

Da stimmt irgendwas nicht, ich komm nichtmal auf die Gesamtzahl der Versuche, hab jetzt schon soviel mit b rumgetüftelt, aber es will einfach nicht klappen.

Seht ihr den Fehler? Eigentlich müsste es so doch klappen.

Danke schonmal im Vorraus

 

[SlaterB]

gar nicht schlecht, wobei der Fall b == 50 nicht berücksichtigt wird,

das Hauptproblem ist, dass bei z.B. b == 50 auch i um 50 erhöht wird,
deshalb hast du am Ende nur wenige Ergebnisse, die Verteilung kommt ja hin

hier ein Vorschlag auch mit anderen Anregungen:

public class Test {
	public static void main(String[] args) {
		int n, runde1 = 0, runde2 = 0, runde3 = 0, runde4_9 = 0, runde10_49 = 0, runde50 = 0, b = 1;
		int zahl1, zahl2, zahl3;

		n = 100000;
		while (n > 0) {

			zahl1 = (int) (Math.random() * 6) + 1;
			zahl2 = (int) (Math.random() * 6) + 1;
			zahl3 = (int) (Math.random() * 6) + 1;
			++b;

			boolean gleich = (zahl1 == zahl2) && (zahl2 == zahl3);
			if (b >= 50 || gleich) {

				if (b == 1) {
					runde1 += 1;
				}

				if (b == 2) {
					runde2 += 1;
				}

				if (b == 3) {
					runde3 += 1;
				}

				if (b >= 4) {
					runde4_9 += 1;
				}

				if (b >= 10 && b <= 49) {
					runde10_49 += 1;
				}

				if (b >= 50) {
					runde50 += 1;
				}
				b = 0;
				n--;
			}
		}

		System.out.println("Pasch in 1. Runde: " + runde1 + "-mal");
		System.out.println("Pasch in 2. Runde: " + runde2 + "-mal");
		System.out.println("Pasch in 3. Runde: " + runde3 + "-mal");
		System.out.println("Pasch in 4 bis 9 Runden: " + runde4_9 + "-mal");
		System.out.println("Pasch in 10 bis 49 Runden: " + runde10_49 + "-mal");
		System.out.println("Pasch in 50 oder mehr Runden: " + runde50 + "-mal");
	}
}

 

[Sneek]

gar nicht schlecht, wobei der Fall b == 50 nicht berücksichtigt wird,

das Hauptproblem ist, dass bei z.B. b == 50 auch i um 50 erhöht wird,
deshalb hast du am Ende nur wenige Ergebnisse, die Verteilung kommt ja hin

hier ein Vorschlag auch mit anderen Anregungen:

Danke für die Anregungen sind gute Ideen dabei, aber ich glaub so ganz stimmt der Code nicht (kann ihn gerade nicht compillieren, sitz an einem anderen PC).
Aber vorher was anderes. Was mich gerade am meisten verwirrt ist, dass wenn z.B. b==50 true ist, i auch um 50 erhöht wird? i wird doch nur im Schleifenkopf inkrementiert und sonst gar nicht weiter behandelt.

So und beim Code stimmt folgendes glaub ich nicht: Zum einen wird b ja mit 1 initialisiert und bei dir erhöht sich b schon vor dem Würfeln, also würde beim ersten Durchlauf b==1 gar nicht möglich sein, bei allen anderen haut es dann aber hin. Und wenn b = 50 ist wird es wieder auf 0 gesetzt, es sollen ja aber auch Paschs einberechnet werden die über 50 Versuchen geworfen wurden.
Lese ich das so richtig herraus? Aber das kann ich ja selber korrigieren.

Edit: Ich seh gerade, dass ich oben nicht mit angegeben habe, das Würfe über 50 auch mit berücksichtigt werden sollen.

 

[SlaterB]

> Zum einen wird b ja mit 1 initialisiert und bei dir erhöht sich b schon vor dem Würfeln, also würde beim ersten Durchlauf b==1 gar nicht möglich sein

stimmt, b sollte anfangs besser 0 sein

> es sollen ja aber auch Paschs einberechnet werden die über 50 Versuchen geworfen wurden.

bei 50 wird ja auch die entsprechende Variable erhöht, es lohnt sich nicht bis 75 zu warten und dann immer noch die gleiche Variable zu erhöhen
kannst aber gerne so machen, kommt das gleiche raus, dauert nur 3 ns länger

> Was mich gerade am meisten verwirrt ist, dass wenn z.B. b==50 true ist, i auch um 50 erhöht wird?

jedes b++ ist auch immer mit einem Schleifendurchlauf, also einem i++ verbunden,

setze doch n testweise auf einen überschaubaren Wert wie z.B. 20 oder 100 und schaue dir in jedem Schleifendurchgang an,
wie weit i ist, b, und wieviele Ergebnisse schon gezählt wurden

 

[Sneek]

Ich hab gerade mal 20 genommen (mit meiner Variante) und alles ausgeben lassen.
Ich verstehe aber immernoch nicht so ganz was du meinst. i wird ja trotzdem immer nur um eins erhöht (ich glaub ich versteh das falsch mit i wird um 50 erhöht), und b wird nach einem Treffer wieder auf 0 gesetzt, wie es bei dir ja auch der Fall ist.
Aber trotzdem haut es bei mir nicht hin, ich hab das Gefühl ich steh irgendwie auf der Leitung :bahnhof:

Edit: AHHHH ich habs War ein kleiner Denkfehler, i nach jedem Durchlauf zu erhöhen ist falsch, da i nur nach einem Pasch erhöht werden darf :toll:

Danke für deine Hilfe