Do while Befehl verkürzen?

[LiVo]

Nabend liebe leute
Habe da so ein Lotto Spiel gemacht. Funktioniert toll, ich habe es sogar geschafft das die varible zahl nicht doppelt vorkommt.
Allerdings ist es sehr sehr aufwändig gewesen, und wenn ich mir vorstelle das es mehr als 6 variblen sind die sich gleichen können wird es eine gewaltige Tipperei.

Gibt es eine Möglichkeit, diesen do while Befehl zu verkürzen, oder ihn durch einen "simpleren" zu ersetzen?

import java.util.Random;
import java.util.Arrays;

public class Lotto6aus49 {

	public static void main (String[] args) {
		int[] zahl = new int [6];
		Random  rnd = new Random();
		
		
do{
		for (int i=0 ; i<6; i++) {
			zahl[i] = rnd.nextInt(49)+1;
			
		}
 
		Arrays.sort(zahl);}
		
while (zahl[0]==zahl[1] || zahl[1]==zahl[2] || zahl[2]==zahl[3] || zahl[3]==zahl[4] || zahl[4]==zahl[5]);
		
    System.out.println ("Die Lottozahlen lauten: " );
    for (int y=0; y<6; y++ ) {
    	System.out.print(zahl[y]+" ");   
    
    
    			}
    		}
	}

Danke!

 

[Gast2]

Du könntest das so lösen:

    public static void main(String[] args) {
        List<Integer> numbers = getLottozahlen(49, 6);

        for (int i : numbers) {
            System.out.println(i);
        }
    }

    private static List<Integer> getLottozahlen(int maxNr, int count) {
        if (count > maxNr) {
            throw new IllegalArgumentException("count > maxNr");
        }

        // fill array list
        List<Integer> numbers = new ArrayList<Integer>();
        for (int i = 1; i <= maxNr; i++) {
            numbers.add(i);
        }

        // shuffle
        Collections.shuffle(numbers);

        List<Integer> result = numbers.subList(0, count);
        return result;
    }

 

[Antoras]

Oder so:

[B]while[/B] length of Array <= 6
  i := new random number
  newNum := true
  [B]foreach[/B] e := next Array element
    [B]if[/B] i equals e then newNum := false
  [B]end[/B]
  [B]if[/B] newNum equals true then add i to Array
[B]end[/B]
print Array

 

[Gast2]

Im schlimmsten fall läuft der code aber unendlich lange :bae:

 

[Antoras]

Spielverderberin ueh:
Aber stimmt schon. Je nach dem wie gut der Code für die Erzeugung der Zufallszahlen ist, könnte das schon ziemlich ins Auge gehen...

 

[Landei]

Ein shuffle, egal ob das vorgrammierte aus Collections (wie oben bei EikeB) oder ein selbst geschriebenes, ist für solche Probleme die eleganteste Lösung.

Es gibt auch eine Lösung mit Zufallszahlen ziehen, aber die ist schwieriger zu implementieren:
Man hat ein Array der Größe 6. Dann zieht man eine Zahl von 1 bis 49 und packt sie ins erste Feld. Dann ziehlt man eine Zahl von 1 bis 48. Ist sie größer/gleich der ersten Zahl, addiert man 1. Dann packt man sie ins zweite Feld. Dann zieht man eine Zahl von 1 bis 47. Ist sie größer als die erste, addiert man 1. Ist sie danach größer als die zweite, addiert man nochmal 1. Dann kommt sie ins dritte Feld. Nach diesem Schema zieht man alle Zahlen, die man braucht.

 

[Gast2]

Die idee ist auch nicht schlecht, aber ich frage mich ob man dann nicht die Wahrscheinlichkeit für die oberen Zahlen reduziert.
Hat man z.b. noch keine 49 gezogen, wird man das ab der dritten Ziehung auch nichtmehr können.

 

[Landei]

Wieso? Wenn du bei der zweiten Ziehung eine 48 ziehst, wird diese auf 49 erhöht, genauso wenn du bei der 3. Ziehung eine 47 ziehst u.s.w (du muss jedesmal +1 rechnen, wenn die Zahl größer gleich einer vorangegangenen ist, wenn du also schon 3,7,4,2 hast und die 45 ziehst, erhöhst du 4 mal und landest bei der 49)

 

[Gast2]

Ah ok, habs nicht gründlich genug gelesen
Dann erscheint mir dass doch ne gute lösung, vor allem bei großer maxNr.

 

[r.w.]

Wieso? Wenn du bei der zweiten Ziehung eine 48 ziehst, wird diese auf 49 erhöht, genauso wenn du bei der 3. Ziehung eine 47 ziehst u.s.w (du muss jedesmal +1 rechnen, wenn die Zahl größer gleich einer vorangegangenen ist, wenn du also schon 3,7,4,2 hast und die 45 ziehst, erhöhst du 4 mal und landest bei der 49)

Wenn ich das richtig sehe, ist dann aber der Wert der noch zu ziehenden Zahl nicht mehr
unbedingt ein zufälliger aus der Menge der noch zur Verfügung stehenden Zahlen, sondern wird auch
durch die bereits gezogenen beeinflusst. Oder hab' ich da etwas falsch verstanden?

 

[Gast2]

Der Wert ist noch zufällig, aber du kannst so keinen Wert doppelt ziehen. Nachdem du einen Wert gezogen hast wird alles was größer ist einfach auf +1 gemappt... sozusagen...

 

[Landei]

Nein, es wird dadurch nicht beeinflusst. Sagen wir, du hast eine 17 gezogen. Nächstes mal willst du aus allen Zahlen außer 17 gleichverteilt die nächste ziehen. Nun kannst du schlecht sagen: "Gib mir eine Zufallszahl von 1 bis 16 oder 18 bis 49". Es ist einfach praktischer, die nächste Zahl von 1 bis 48 zu ziehen und dann den Wertebereich zu korrigieren, so daß die 17 nicht mehr vorkommen kann, aber dafür die 49. Trotzdem wird gleichverteilt gezogen. Wenn du das nicht glaubst, probiere es doch einfach mit einer genügend großen Anzahl Ziehungen aus.

 

[LiVo]

könntet ihr mir vll auch noch sagen was die codes genau bedeuten?

 

[Gast2]

Hmm, sollte eigentlich alles aus den Kommentaren und einen Blick in die API klar werden. Oder hast du ne konkrete frage?

 

[LiVo]

Naja ^^ ja...
Ich verstehs noch nich so ganz. was sicherlich daran liegt das ich mit java Programmierung erst seit letzter woche was zu tun habe.
Was ist die API und wie guck ich da rein? gibts da nen link zu?
Oder anders.
Wie müsste ich eure /deinen code, bei mir einfügen/ersetzen?

 

[Gast2]

Die API ist deine Bibel
Java Platform SE 6

Ich erklär dir trotzdem mal meinen Code.

    private static List<Integer> getLottozahlen(int maxNr, int count) {

Statische Methode die eine List<Integer> zurückgibt, da stehen dann deine 6 Zahlen drin.

        if (count > maxNr) {
            throw new IllegalArgumentException("count > maxNr");
        }

Wenn count > maxNr ist wird ne Exception geworfen.

        // fill array list
        List<Integer> numbers = new ArrayList<Integer>();
        for (int i = 1; i <= maxNr; i++) {
            numbers.add(i);
        }

Füllt die List numbers mit den Zahlen 1..maxNr, die Zahlen sind jetzt noch sortiert.

        // shuffle
        Collections.shuffle(numbers);

Jetzt werden die Zahlen in der Liste gemischt.

        List<Integer> result = numbers.subList(0, count);
        return result;

Kopiert den Kopf der Liste (übergibst du für count 6, werden die ersten 6 Zahlen kopiert) und gibt die Liste zurück.

 

[Marco13]

Nebenbei ein kurzer Link auf http://www.java-forum.org/java-basi...ablen-miteinander-vergleichen.html#post612775