Array gefüllt mit verschiedenen Werten

[Bierhumpen]

Kann mir einer sagen ob das so stimmt?! ich will einfach nur bestätigung ob es mit der Aufgabenstellung übereinstimmt oder ich es mal wieder verbaselt habe.

Aufgabe: Implementieren Sie eine Funktion static int[] constructArray49(int n). Diese be-
kommt eine int-Zahl n zwischen 1 und 49, die Lange des Arrays, übergeben und erzeugt
ein int-Array mit n paarweise verschiedenen Zahlen zwischen 1 und 49. Falls n nicht
zwischen 1 und 49 liegt, so soll eine IllegalArgumentException geworfen werden, die
den Fehler sinnvoll dokumentiert!

    static int[] constructArray49(int n) throws IllegalArgumentException {
        
        int [] array;
        
        try{
            if (n<1 || n>49){
                throw new IllegalArgumentException();
            }
            else{
                array = new int[n];
                
                for (int i=0; i<array.length; i++){
                    int zufallszahl = (int)((Math.random()*49)+1);
                    for (int j=0; j<array.length; j++){
                        while (array[j]==zufallszahl){
                            zufallszahl = (int)((Math.random()*49)+1);
                        }
                    }
                    array[i]=zufallszahl;
                }
            }
        }
        catch(IllegalArgumentException iae){
            System.out.println("Die eingegebene Zahl ist zu hoch/niedrig!\n"+
                               "Sie muessen eine Zahl zwischen 1 und 49 "+
                               "eingeben (1 und 49 eingeschlossen)");
            array = null;
        }
        
        return array;
    }

 

[SlaterB]

wo steht denn da was von Zufallszahlen, nimm doch einfach 1 bis n

----------


bei

for (int j=0; j<array.length; j++){
      while (array[j]==zufallszahl){
            zufallszahl = (int)((Math.random()*49)+1);
       }
}

gehts du falsch vor, sind die bisherigen Zahlen [3,14, 7, 9, 0, 0,0,0]
und die anfangs gewürfelte Zahl = 7, dann wird verglichen:
3 und 7, ok
14 und 7 ok
7 und 7, schlecht, neue Zufallszahl 3
7 und 3 ok
9 und 3 ok usw (es müsste übrigens nicht das ganze Array geprüft werden, nur bis zu einer bestimmten Grenze, welche?)

und am Ende wird 3 gewählt obwohl 3 schon drin ist,
überleg dir mal woran das liegt und was du ändern musst

 

[Bierhumpen]

hi, also das mit den zufallszahlen macht schon sinn, denn es soll ein lottospiel sein. denn wenn ich das nicht mit zufalls zahlen machen dann wird immer die ziehung lauten 1,2,3,4,5,6.

hmm und das andere überlege ich mir noch

 

[Bierhumpen]

sprich wenn er eine zahl gefunden hat die bereits im array drin ist geht er mit der neuen zufallszahl weiter, obwohl die davor schon gewesen sein könnte. also muss ich die schleife ändern. ich versuchs mal... hmm rekursiv? dann müsste ich aber ne neue methode schreibe und laut aufgabenstellung sollen die reichen die uns vorgegeben werden.

 

[SlaterB]

bevor eine Zahl bestätigt wird muss sie einfach mit jeder vorherigen Zahl einmal vergleichen werden,
dafür brauchst höchstens Schleifen, nix mit Rekursion

hauptsache du beginnst nach der Änderung wieder von vorne mit dem Vergleichen, nicht mittendrin

 

[Bierhumpen]

uhhh. zu einfach einfach nur ein j=0 einfügen

also

            if (n<1 || n>49){ 
                throw new IllegalArgumentException(); 
            } 
            else{ 
                array = new int[n]; 
                
                for (int i=0; i<array.length; i++){ 
                    int zufallszahl = (int)((Math.random()*49)+1); 
                    for (int j=0; j<array.length; j++){ 
                        while (array[j]==zufallszahl){ 
                            zufallszahl = (int)((Math.random()*49)+1);
                            j=0; 
                        } 
                    } 
                    array[i]=zufallszahl; 
                } 
            }

 

[SlaterB]

funktioniet wohl aber ist böse, in korrekteren Java formuliert:

boolean nochmal = true;
while (nochmal) {
    nochmal = false;
    zufallszahl = (int)((Math.random()*49)+1);

    for (int j=0; j<array.length; j++){
        if (array[j]==zufallszahl){
            nochmal = true;  
        } 
    }
}

gibt natürlich viele Möglichkeiten,
überlege auch noch nicht das ganze Array zu durchlaufen wenn es gerademal erst um die 3. von 49 Zahlen geht

------------

        try{
            if (n<1 || n>49){
                throw new IllegalArgumentException();
            }
            else{
               ........
            }
        }
        catch(IllegalArgumentException iae){
            System.out.println("Die eingegebene Zahl ist zu hoch/niedrig!\n"+
                               "Sie muessen eine Zahl zwischen 1 und 49 "+
                               "eingeben (1 und 49 eingeschlossen)");
            array = null;
        }

macht übrigens keinen Sinn, wenn dann

        if (n<1 || n>49){
            System.out.println("Die eingegebene Zahl ist zu hoch/niedrig!\n"+
                               "Sie muessen eine Zahl zwischen 1 und 49 "+
                               "eingeben (1 und 49 eingeschlossen)");
            array = null;
        }
        else{
            ........
         }

aber es soll ja eine Exception NACH AUSSEN abgegeben werden, also throw schon richtig, aber nicht direkt selber anfangen

 

[Bierhumpen]

// Edit by me //