kann fehler nicht finden

[Cyph3rlink]

Ich will eine Reihe von Zahlen, der Länge array.length. Dabei sollen nur ganze Zahlen von 0 bis array.length -1 auftreten, und keine doppelt. Die Zahlen sollen in der Reihenfolge in der sie auftreten in den Array reihe für die spätere Verwendung gespeichert werden. Allerdings funktioniert das Ablegen der Zahlen Array reihe überhaupt nicht.

Hier der Quellcode, mit println()-Anweisungen zur Kontrolle versehen:

			BitSet b = new BitSet();
			while (cnt < array.length)
			{
				int num = Math.abs(r.nextInt()) % array.length;
				System.out.println(cnt+"   :     "+num);
				if (!b.get(num))
				{
					b.set(num);
					System.out.println("Count: "+cnt);
					reihe[cnt] = num;
					cnt++;
					System.out.println("\n SOLL IN ARRAY AUFGENOMMEN:  ------------>>>>>"+ num);
					System.out.println("\n ARRAY ABER ZEIGT:"+ reihe[cnt]);
				}
			}

Ich erhalte zum Beispiel diese beim Ausführen.

0   :     1
Count: 0

 SOLL IN ARRAY AUFGENOMMEN:  ------------>>>>>1

 ARRAY ABER ZEIGT:0
1   :     0
Count: 1

 SOLL IN ARRAY AUFGENOMMEN:  ------------>>>>>0

 ARRAY ABER ZEIGT:0
2   :     3
Count: 2

 SOLL IN ARRAY AUFGENOMMEN:  ------------>>>>>3

 ARRAY ABER ZEIGT:0
3   :     2
Count: 3

 SOLL IN ARRAY AUFGENOMMEN:  ------------>>>>>2
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 4
	at TestCoop.main(TestCoop.java:45)

Die Zahlen werden also richtig ermittelt, nur kommt totaler Blödsinn im Array an.

 

[SlaterB]

merkwürdiges Vorgehen, als erstes würde ich ja an deiner Stelle das GESAMTE Array ausgeben,
irgendwohin müssen die Zahlen ja kommen

dann sieht man schnell woran es liegt: du schreibst die Zahl in Feld cnt, vor der Ausgabe von Feld cnt erhöst du aber cnt um 1

dass am Ende eine ArrayIndexOutOfBoundsException kommt ist noch mal ein sehr deutlicher Hinweis darauf,

übrigens wäre es sehr nett, um deinen Code noch die Array- und sonstige Deklaration + einen einfache Testklasse mit main-Operation drumherum zu packen, das ist ne gute Übung und erleichtert anderen das testen

import java.lang.reflect.Array;
import java.util.BitSet;
import java.util.Random;

public class Test {

    public static void main(String[] args) throws Exception {
        System.out.println("start");

        Random r = new Random();
        int cnt = 0;
        int[] reihe = new int[6];
        BitSet b = new BitSet();
        int tries = 0;
        while (( tries < 1000 ) && ( cnt < reihe.length )) {
            int num = Math.abs(r.nextInt()) % reihe.length;
            System.out.println(cnt + "   :     " + num);
            tries++;
            if (!b.get(num)) {
                b.set(num);

                System.out.println("Count: " + cnt);
                reihe[cnt] = num;
                cnt++;
                System.out.println("SOLL IN ARRAY AUFGENOMMEN:  ------------>>>>>" + num);
                System.out.println("ARRAY ABER ZEIGT:" + toString(reihe));
            }
        }
        System.out.println("end");
    }

   
    private static String toString(Object arr) {
        if (arr == null) {
            return "null";
        }
        try {
            // is object an array?
            Array.getLength(arr);
        } catch (IllegalArgumentException e) {
            return arr.toString();
        }
        StringBuilder s = new StringBuilder();
        for (int i = 0; i < Array.getLength(arr); i++) {
            s.append("_" + toString(Array.get(arr, i)));
        }
        return s.toString();
    }
}

 

[Cyph3rlink]

Danke. Ich hab des wirklich übersehen *schäm*.

 

[Cyph3rlink]

Tut mir leid dass ich hier nochmal fragen muss, wahrscheinlich ist es wieder was ganz einfaches was ich nur übersehe.

Wenn man jetzt das von oben in eine for-Schleife setzt, und mehrfahch durchlaufen lässt, warum hat dann der Array immer die gleichen Werte, und nicht jedesmal andere?

import java.lang.reflect.Array; 
import java.util.BitSet; 
import java.util.Random; 

public class Test { 

    public static void main(String[] args) throws Exception { 
        System.out.println("start"); 

        Random r = new Random(); 
        int cnt = 0; 
        int[] reihe = new int[6]; 
		for (int i =0; i< 10; i++)
		{
	        int tries = 0; 
			BitSet b = new BitSet(); 
	        while (( tries < 1000 ) && ( cnt < reihe.length )) 
	        { 
	        	int num = Math.abs(r.nextInt()) % reihe.length;  
	            tries++; 
	            if (!b.get(num)) 
	            { 
	                b.set(num); 

	                reihe[cnt] = num; 
	                cnt++;  
	            } 
	        }
	        b = null;
	        System.out.println("ARRAY HAT ZUM INHALT:" + toString(reihe));

		}
                System.out.println("end"); 
    } 

    
    private static String toString(Object arr) { 
        if (arr == null) { 
            return "null"; 
        } 
        try { 
            // is object an array? 
            Array.getLength(arr); 
        } catch (IllegalArgumentException e) { 
            return arr.toString(); 
        } 
        StringBuilder s = new StringBuilder(); 
        for (int i = 0; i < Array.getLength(arr); i++) { 
            s.append("_" + toString(Array.get(arr, i))); 
        } 
        return s.toString(); 
    } 
}

 

[SlaterB]

> Danke. Ich hab des wirklich übersehen *schäm*.

ich übersehe sowas auch 10x täglich,
die Kunst ist nicht (unbedingt) das zu sehen,
sondern Fehler Schritt für Schritt aufzuspüren,

das erfordert kein Können und keine Erfahrung,
sondern nur Fleiß wie die Ausgabe des ganzen Arrays (einfacher per for-Schleife)

 

[SlaterB]

für Fragen muss man sich nicht entschuldigen,

aber schade, wohl noch nichts gelernt, ich bemühe mich doch immer so

ich selber habe deinen Code gesehen und erstmal auch nicht gewußt woran es liegt,
weiß ich doch aus meiner Erfahrung, dass Random mehr als 6 Werte kennt und anscheinend kontinuierlich aufgerufen wird,

was mache ich da?
erstmal zur Vereinfachung das Array auf Länge 1, und dann jeden Random-Wert mit System.out ausgeben,
siehe da, es wird nur einmal Random aufgerufen,

dann muss man entweder weiterforschen woran das liegt oder denkt sich dass das mit irgendwelchen if/ while-Bedingungen zu tun hat
-> ach ja, cnt wird gar nicht auf 0 zurückgesetzt, daher das ganze Array nur einmal initialisiert

import java.lang.reflect.Array;
import java.util.BitSet;
import java.util.Random;

public class Test {

	public static void main(String[] args) throws Exception {
		System.out.println("start");

		Random r = new Random();
		int cnt = 0;
		int[] reihe = new int[1];
		for (int i = 0; i < 10; i++) {
			int tries = 0;
			BitSet b = new BitSet();
			while ((tries < 1000) && (cnt < reihe.length)) {
				int a = r.nextInt();
				System.out.println(a);
				int num = Math.abs(a) % reihe.length;
				tries++;
				if (!b.get(num)) {
					b.set(num);

					reihe[cnt] = num;
					cnt++;
				}
			}
			b = null;
			System.out.println("ARRAY HAT ZUM INHALT:" + toString(reihe));

		}
		System.out.println("end");
	}

	private static String toString(Object arr) {
		if (arr == null) {
			return "null";
		}
		try {
			// is object an array?
			Array.getLength(arr);
		} catch (IllegalArgumentException e) {
			return arr.toString();
		}
		StringBuffer s = new StringBuffer();
		for (int i = 0; i < Array.getLength(arr); i++) {
			s.append("_" + toString(Array.get(arr, i)));
		}
		return s.toString();
	}
}

(mit StringBuffer statt StringBuilder, da zu Hause kein Java 1.5 )

 

[Cyph3rlink]

Danke für die freundliche und ausführliche Hilfe