Array kopieren

[gurkenprinz]

Hallo Leute !!

Ich häng wieder mal

Ich hab ein int array mit zahlen von 1 bis 20.

Dann habe ich ein weiteres int Array das 2 mal mit 10 zufälligen Werten aus dem ersten Array befüllt werden soll. (Hoffe ich hab das richtig erklärt *gg*)

Hier wäre mal mein Lösungsansatz der leider nicht so wirklich hinhaut ... entweder bekomme ich eine ArrayIndexOutofBoundsException ...oder manche Werte kommen öfter als 2 mal in dem zufalls array vor.

public class ArrayTest {


	private int[]all = {1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20};
	private int[] random = new int[20];
	
	private int imagesLength = this.all.length;
	

	public int[] getRandomImages()
	{
		
		for(int i = 0;i<10;i++)
		{
			int img = all[this.getRandomImage()];
			random[i] = img;
			
		}
		for(int i =0;i<10;i++)
		{
			random[10+i]=random[i];
		}
		return random;
	}
	
	private int getRandomImage()
    {
		
    	int pos = (int) (Math.random() * 100) % imagesLength;

        int c = all[pos];
        //Lücke im Array schließen !!
        for (int i = pos+1; i < imagesLength; i++)
        {
            all[i - 1] = all[i];
        }
        imagesLength--;
        return c;
    }
	
	/**
	 * @param args
	 */
	public static void main(String[] args) 
	{
		ArrayTest t = new ArrayTest();
		int[] i = t.getRandomImages();
		System.out.println("Länge:"+i.length);
		for(int j = 0;j<i.length;j++)
		{
			System.out.println(j+" : "+i[j]);
		}
		
	}

}

Bitte um Hilfe !!

lg gurkenprinz

 

[SlaterB]

int pos = (int) (Math.random() * 100) % imagesLength;
liefert 0-19

dann machst du irgendwas unverständliches mit dem all-Array,
in jedem Fall wird eines der all-Elemente zurückgegeben, das kann auch 20 sein, denn 20 ist im all-Array


mit der Rückgabe greifst du dann auf das all-Array als zu und 20 in einem 20er Array -> Exception

 

[Wortraum]

Da ist auch ein Denkfehler drin. In Zeile 31 holst du einen Wert aus dem Feld, zum Beispiel die 20. In Zeile 15 dann aber nimmst Du diesen Wert als Index und versuchst, auf die Stelle 20 zuzugreifen, um noch einmal einen Wert zu holen; dabei geht das Feld von 0 bis 19. Hier hast Du also etwas mit dem Wert/Inhalt im Feld und dem Index durcheinandergebracht. Ich vermute, gemeint ist das:

for(int i = 0;i<10;i++)
{
    // int img = all[this.getRandomImage()];
    // random[i] = img;
    random[i] = this.getRandomImage();
}

Die Sache mit den doppelten Werten kann ich spontan nicht nachvollziehen, außer Du willst etwas ganz anderes, als Du programmiert hast. Derzeit nimmst Du einfach 10 Werte aus dem 20er-Feld und packst sie in zwei neue 10er-Felder. Daß diese beiden 10er-Felder gleich sind, also daß Du quasi nur eine Kopie hast, ist dann nur logisch. Vielleicht hast Du hier vergessen, 10 weitere Werte zu entnehmen, und meintest eigentlich:

public int[] getRandomImages()
{
    for(int i = 0;i<10;i++)
    {
        random[i] = this.getRandomImage();
    }
    for(int i =0;i<10;i++)
    {
        random[10+i] = this.getRandomImage();
    }
    return random;
}

Übrigens brauchst Du keine zwei Schleifen:

public int[] getRandomImages()
{
    for(int i = 0;i<10;i++)
    {
        random[i] = this.getRandomImage();
        random[10+i] = this.getRandomImage();
    }
    return random;
}

Deine Lösung ist sonst übrigens gut, allerdings hättest Du es Dir mit einem Objekt ArrayList einfacher machen können, da es dort bereits eine Methode remove(int index) gibt, die ein Element entfernt. Auch zum Kopieren eines Feldes gibt es bereits die Methode System.arraycopy(…).

 

[Landei]

Am einfachsten eine Liste machen, mit den Zahlen füllen und Collections.shuffle(liste) aufrufen.

 

[gurkenprinz]

Danke Leute für die Tipps !

habs jetzt mit Collections.shuffle gemacht.. is wohl die einfachste Möglichkeit

lg gurkenprinz