Methoden Rückgabe Arrays aufangen

[Ro0kie]

Liebes Java-forum.org!

Ich habe eine Methode die mir ein Array zurück gibt nur wenn ich es mit einem anderen Array aufangen will kriege ich eine IndexOutOfBoundsEXception.

/**
 * Testklasse für die Klassen NWuerfel und Wuerfel, die nebenbei auch noch eine 
 * Statistik der gewürfelten Augen erstellt.
 *
 * @author Martin S
 * @version 2011-05-22
 */
public class Wuerfelstatistik {
    public static void main(String [] args){
        int[][] testarray = new int[100][100];
        //Befüllen des Arrays
        for(int l =0;l<testarray.length;l++){
           for(int k=0; k<testarray[l].length;k++){
               testarray = NWuerfel.nwuerfeln(1,1);
           }
        }

        //Ausgeben des Arrays
        for(int i =0;i<testarray.length;i++){
           for(int j=0; j<testarray[i].length;j++){
             System.out.println(testarray[i][j]);
           }
        }
    }
}

/**
 * Klasse welche das würfeln mit meheren Würflen simuliert.
 *
 * @author Martin S
 * @version 2011-05-22
 */
public class NWuerfel {
    public static int[][] nwuerfeln(int nwuerfel,int wurfanzahl){
        int[][] ngewuerfelt = new int [nwuerfel][wurfanzahl];
        for(int i =nwuerfel;i>0;i--){
            for(int j =0;j<wurfanzahl;i++){
                ngewuerfelt[i][j]=Wuerfel.wuerfeln();

            }
        }
        return ngewuerfelt;
    }

}

/**
 * Klasse,welche einen Würfel simuliert.
 * 
 * @author Martin S
 * @version 2011-05-22
 */
public class Wuerfel {
    /**
     * Methode,welche das würflen eines Würfels mit 6 Augen simuliert.
     *
     * @return gewuerfelt Gibt die gewürfelte Augenzahl zurück.
     */
    public static int wuerfeln(){
        int gewuerfelt =(int) (Math.random()*6+1);
        return gewuerfelt;
    }

}

Hat jemand eine Idee ?
Danke im voraus !

MfG Ro0kie

 

[nrg]

// ...
        for(int i =nwuerfel-1;i>0;i--){
// ...

 

[Ro0kie]

// ...
        for(int i =nwuerfel-1;i>0;i--){
// ...

Erklärung wäre nett
Will ja auch was lernen dabei

 

[nrg]

das array beginnt bei index 0. der letzte index ist demnach bei array.length-1.

 

[jgh]

du definierst ein int-Array mit der Länge [1,1] =>

int[][] ngewuerfelt = new int[nwuerfel][wurfanzahl];

=> du kannst nur auf das eine Element zugreifen [c]ngewuerfelt[0][0] [/c] , alles andere wirft dir halt die java.lang.ArrayIndexOutOfBoundsException
und du versuchst mit dem Index 1 auf das zweite, nicht vorhandene Element zuzugreifen.

 

[Ro0kie]

Das löst mein Problem noch immer nicht, es liegt am aufangen des Rückagabe Array wie kann ich das in einem Array aufangen sprich alle werte in ein Array übertragen ?:rtfm:

 

[nrg]

dann wäre die fehlermeldung hilfreich

edit: gibt es dazu eine aufgabenstellung? zugegeben, ich hab mich auch nicht gerade bemüht ihn zu finden aber mir ich frage mich noch nach dem sinn

 

[eRaaaa]

        for(int i =nwuerfel;i>0;i--){
            for(int j =0;j<wurfanzahl;i++){

Willst du bei der zweiten nicht j hochzählen?

Und nwuerfeln liefert doch schon ein 2-Dim Array, wieso willst dudas also nochmals in einer/bzw zwei Schleife befüllen?
->

        //Befüllen des Arrays
        for(int l =0;l<testarray.length;l++){
           for(int k=0; k<testarray[l].length;k++){
               testarray = NWuerfel.nwuerfeln(1,1);
           }
        }

???:L

 

[Ro0kie]

dann wäre die fehlermeldung hilfreich

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 100
at NWuerfel.nwuerfeln(NWuerfel.java:13)
at Wuerfelstatistik.main(Wuerfelstatistik.java:14)
Java Result: 1
BUILD SUCCESSFUL (total time: 0 seconds)

Bitte

 

[nrg]

den letzten fehler in deinen for-scheifen müsste dir dann era gesagt haben.

nochmal zum Sinn:

public class Wuerfelstatistik {
    public static void main(String [] args){
        int[][] testarray = NWuerfel.nwuerfeln(100,100);
        //Ausgeben des Arrays
        for(int i =0;i<testarray.length;i++){
           for(int j=0; j<testarray[i].length;j++){
             System.out.println(testarray[i][j]);
           }
        }
    }
}

so würde es für mich denke mehr Sinn machen. achja, die Methode nwuerfeln passt wunderbar in die Klasse Wuerfel. dafür eine neue Klasse zu machen, halte ich für unnötig

 

[Ro0kie]

dann wäre die fehlermeldung hilfreich

edit: gibt es dazu eine aufgabenstellung? zugegeben, ich hab mich auch nicht gerade bemüht ihn zu finden aber mir ich frage mich noch nach dem sinn

Aufgabenstellung ist wie immer eine Programmierhausübung Zitat

Erstelle die Klassen Wuierfel, NWuerfel und eine Applikation die eine Statistik über die erwürfelte Auganzahl anzeigt.

Willst du bei der zweiten nicht j hochzählen?

Und nwuerfeln liefert doch schon ein 2-Dim Array, wieso willst dudas also nochmals in einer/bzw zwei Schleife befüllen?

Das mit dem j war eib Kopierfehler hab ich behoben.

hab den code erneuert :

/**
 * Testklasse für die Klassen NWuerfel und Wuerfel, die nebenbei auch noch eine 
 * Statistik der gewürfelten Augen erstellt.
 *
 * @author Martin Suschny
 * @version 2011-05-22
 */
public class Wuerfelstatistik {
    public static void main(String [] args){
        int[][] testarray = NWuerfel.nwuerfeln(1,2);
        
        
        
        

        //Ausgeben des Arrays
        for(int i =0;i<testarray.length;i++){
           for(int j=0; j<testarray[i].length;j++){
             System.out.println(testarray[i][j]);
           }
        }
    }
}

 

[nrg]

Erstelle die Klassen Wuierfel, NWuerfel und eine Applikation die eine Statistik über die erwürfelte Auganzahl anzeigt.

naja dann vergiss das und mach es lieber nach aufgabenstellung. wobei ich es echt für überflüssig halte. zudem erschließt sich mir noch nicht ganz, warum du überhaupt ein 2d array nimmst

 

[Ro0kie]

zudem erschließt sich mir noch nicht ganz, warum du überhaupt ein 2d array nimmst

Dammit ich eine Zusatzfunktion einabauen kann die erkennt in welchen Würfelrunde was gewürfelt worden ist

 

[nrg]

Dammit ich eine Zusatzfunktion einabauen kann die erkennt in welchen Würfelrunde was gewürfelt worden ist

wenn ich das jetzt richtig verstanden habe, würde ich mir das dann wieder an einer anderen stelle merken. aber gut, das ist jetzt offtopic. wenn deine code jetzt funktioniert, ist das thema ja erledigt

 

[Ro0kie]

Wie fängt man denn jz Array im allgemeinem auf ?

 

[nrg]

du fängst es nicht auf sondern weisst einfach das ergebnis einer methode einer variable zu.

int[][] testarray = NWuerfel.nwuerfeln(1,2);

 

[Ro0kie]

ich kapiers nicht ich hab alles richtig gemacht aber es funktioniert noch immer nicht !
:autsch::wuerg:

 

[nrg]

fehlermeldung?

 

[Ro0kie]

fehlermeldung?

Noch immer die selbe :
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 1
at NWuerfel.nwuerfeln(NWuerfel.java:19)
at Wuerfelstatistik.main(Wuerfelstatistik.java:10)

 

[nrg]

dann wäre dein aktueller code auch noch interessant...

 

[Ro0kie]

dann wäre dein aktueller code auch noch interessant...

/**
 * Testklasse für die Klassen NWuerfel und Wuerfel, die nebenbei auch noch eine 
 * Statistik der gewürfelten Augen erstellt.
 *
 * @author Martin S
 * @version 2011-05-22
 */
public class Wuerfelstatistik {
    public static void main(String [] args){
        int[][] testarray = NWuerfel.nwuerfeln(1,2);
        
        
        
        

      
    }
}

/**
 * Klasse welche das würfeln mit meheren Würflen simuliert.
 *
 * @author Martin S
 * @version 2011-05-22
 */
public class NWuerfel {
    /**
     *
     * @param nwuerfel
     * @param wurfanzahl
     * @return
     */
    public static int[][] nwuerfeln(int nwuerfel,int wurfanzahl){
        int[][] ngewuerfelt = new int [nwuerfel][wurfanzahl];
        for(int i =nwuerfel;i>0;i--){
            for(int j =0;j<wurfanzahl;j++){
                ngewuerfelt[i][j]=Wuerfel.wuerfeln();

            }
        }
        return ngewuerfelt;
    }

}

/**
 * Klasse,welche einen Würfel simuliert.
 * 
 * @author Martin S
 * @version 2011-05-22
 */
public class Wuerfel {
    /**
     * Methode,welche das würflen eines Würfels mit 6 Augen simuliert.
     *
     * @return gewuerfelt Gibt die gewürfelte Augenzahl zurück.
     */
    public static int wuerfeln(){
        int gewuerfelt =(int) (Math.random()*6+1);
        return gewuerfelt;
    }

}

 

[nrg]

hast du meinen ersten post in dem thread auch gelesen??

 

[Ro0kie]

hast du meinen ersten post in dem thread auch gelesen??

ja ber wie soll ich das jz anwenden ?

 

[nrg]

ok dann nochmal in ruhe:
wenn du ein array intialisiert gibst du die größe des arrays in den eckigen klammern an. also bei

int[] array = new int[5];

erhälst du ein array mit der größe 5. die größe wird dabei als final attribute

length

gespeichert. das heisst also array.length liefert dir 5. nachdem aber der index bei 0 anfängt zu zählen, hast du nur die indizes 0, 1, 2, 3, 4 zur verfügung. d.h. dein erstes element ist die 0 und dein letztes in diesem fall die 4 oder in jedem fall dynamisch

array.length-1

edit: und um das nochmal auf dein problem zu referenzieren: du versucht eine leiter herunterzuklettern und greifst als erstes an die 6te sprosse, wenn die leiter aber nur 5 hat > du fällst

 

[Ro0kie]

Aso ja das meinst du

da aber in meiner Testklasse

int[][] testarray = NWuerfel.nwuerfeln(1,2);

, wird ja ein 2D Array mit 1 length und 2 Arrays drinen erzeugt und das wiederum ist ja dann gleich testarray, hab ich recht ?
wenn ja wieso krieg ich dann einen IndexOutofbounds-Fehler ?

 

[nrg]

also gerade stehst du schon gewaltig auf dem schlauch oder?

for(int i = nwuerfel; i > 0 ; i--) {

in dieser zeile intialisiert du

i

mit

nwuerfel

.

nwuerfel

ist in dem fall das gleiche ist wie

array.length

. ich habe dir jetzt 3 mal erklärt, dass es diesen index nicht gibt und der letzte index in einem array immer arrayName.length-1 ist.

aber probier es doch einfach mal aus....

for(int i = nwuerfel-1; i > 0 ; i--) {

achja: ich frage mich eh, warum du da rückwärts iterieren möchtest....

 

[Ro0kie]

Ich war nur so verwierrt weils ich es andauernd in der testklasse probiert habe hättest du gleich gesagt in der NWuerfel Klasse :lol:

 

[nrg]

hab doch gleich am anfang den schleifenkopf gepostet und darin nur -1 hinzugefügt .
ausserdem wenn du dir den stacktrace der exception anschaust, siehst du doch auch, dass es überhaupt nichts mit der main zu tun hat. es wird nur über die main durchgereicht/geworfen aber dein verursacher ist:

NWuerfel.nwuerfeln(NWuerfel.java:19)

also Methode nwuerfeln in der Klasse NWuerfel in der Zeile 19.

 

[Ro0kie]

/**
 * Klasse welche das würfeln mit meheren Würflen simuliert.
 *
 * @author Martin S
 */
public class NWuerfel {
    /**
     *
     * @param nwuerfel
     * @param wurfanzahl
     * @return
     */
    public static int[][] nwuerfeln(int nwuerfel,int wurfanzahl){
        int[][] ngewuerfelt = new int [nwuerfel][wurfanzahl];
        for(int i = nwuerfel-1; i > 0 ; i--) {
            for(int j =0;j<wurfanzahl;j++){
                ngewuerfelt[i][j]=Wuerfel.wuerfeln();

            }
        }
        return ngewuerfelt;
    }

}

/**
 * Testklasse für die Klassen NWuerfel und Wuerfel, die nebenbei auch noch eine 
 * Statistik der gewürfelten Augen erstellt.
 *
 * @author Martin S
 * @version 2011-05-22
 */
public class Wuerfelstatistik {
    public static void main(String [] args){
        int[][] testarray = NWuerfel.nwuerfeln(1,2);

        for(int i = 0; i < testarray.length-1 ; i++) {
            for(int j =0;j<testarray[j].length;j++){
                System.out.println(testarray[i][j]);
                
            }
        }
        
        
        
        

      
    }
}

Komisch Ausgabe ist nichts ? ???:L

 

[nrg]

// ...
        for(int i = 0; i < testarray.length-1 ; i++) {
            for(int j =0;j<testarray[j].length;j++){
                System.out.println(testarray[i][j]);
                
            }
        }
// ...

Komisch Ausgabe ist nichts ? ???:L

du musst meine posts schon lesen und auch verstehen. deine äußere schleife geht solange, wie i kleiner testarray.length-1. das ist falsch... verstehe auch nicht warum du es dann in der inneren schleife richtig machts und selbst nie auf die idee kommst, dir auch nur 2 minuten gedanken über deinen code und den postings hier zu machen...
nachdem ich lange erklärungen aufgegeben habe:

ein 2d array iteriert man so:

for (int i = 0; i < array.length; i++) {
   for (int j = 0; j < array[i].length; j++) {
      // array[i][j]
   }
}

oder mit einer foreach (z.b. bei einem int array):

for (int[] a : array) {
   for (int i : a) {
      // i
   }
}

edit: um es noch vollständig zu machen.
rückwärts iteriert man so:

for (int i = array.length-1; i >= 0; i--) {
   for (int j = array[i].length-1; j >= 0; j--) {
      // array[i][j]
   }
}

weil das ist ja auch noch falsch in deiner schleife... hatte ich aber beim ersten post auch nicht gesehn