Zufallszahlen generieren

[Hallo1234]

Hallo,
bei diesem Programm sollen zufällige Zahlen von 1-6 "gewürfelt" werden.

import java.util.Scanner;
public class Wuerfel {

    public static void main(String[] args) {
        Scanner s = new Scanner(System.in);

        boolean quit = false;
        while(quit==false) {

            System.out.println("Wie viele Runden möchten Sie würfeln?: ");

            int runden = s.nextInt();

            if(runden < 3) {
                System.out.println("Sie müssen mindestens drei Runden würfeln! Eingabe wiederholen.");
                quit = false;
            }else if(runden > 20) {
                System.out.println("Sie dürfen nur bis zu 20 Runden würfeln.");
            }else if(runden > 3 && runden <= 20) {
                quit = true;
            }

            for(int i=1; i<=runden; i++) {
                System.out.println("Runde "+i+": ");
            
                int[] zufall = new int[runden];
                for(int j = 0; j<zufall.length; j++) {
                    zufall[j] = ermittleZufallsZahl(1,6);
                }
            }
        }
    }

    public static int ermittleZufallsZahl(int high, int low) {
        int zufallszahl = (int) (Math.random() * (high - low + 1)) + low;
        return zufallszahl;
    }

Aber es klappt nicht. Weiß jemand, woran das liegt?
Vielen Dank

 

[temi]

Was klappt denn nicht?

 

[Hallo1234]

Mir wurde keine Zufallszahl ausgegeben. Bin aber jetzt dahinter gekommen. Ich hatte kein syso xD
Allerdings ist die Ausgabe trotzdem verdächtig:

Kann das sein? Ich hatte schon bei mehreren Würfen keine 1 oder 2. Dafür aber sehr häufig 5er oder 3er.

 

[Robert Zenz]

zufall[j] = ermittleZufallsZahl(1,6);
// ...
public static int ermittleZufallsZahl(int high, int low) {

Klingt verkehrt herum. Achte deswegen immer darauf dass du Parameter und aehnliches in einer "logischen" Reihe angibst (in diesem Fall "Min, Max"), weil solche Fehler passieren schnell und koennen recht schwer zu suchen sein.

Abgesehen davon gibt es Random.nextInt(int) welches dir bereits eine Zahl zwischen "0" und "bound" (exklusive) erzeugt. Sollte deine Logik also relativ vereinfachen.

Ansonsten sind deine Variablennamen schon ganz gut, ich wuerde nur vorschlagen dass du die Logik von der Hauptschleife umdrehst, also im Moment pruefst du darauf das kein Abbruch vorliegt, sondern stattdessen pruefen ob man laufen soll:

boolen running = true;

while (running) {
    if (user wants to end the application) {
        running = false;
    }
}

Das ist etwas leichter zum nachvollziehen als die umgekehrte Logik.

 

[Hallo1234]

Vielen Dank für die Tipps, werde das so umsetzen.

 

[M.L.]

umsetzen

Weiterhin könnte man sich an einem funktionierenden Beispiel orientieren, z.B. https://howtodoinjava.com/java/generate-random-numbers/