Finde den Fehler (bitte)

[Wühlhase (Gast)]

Mahlzeit

Ich sitze hier vor einem Javaprogram und sehe den Grund für die ständig 
kommende Exception nicht.
Es geht darum, zwei Arrays zu einem neuen Array zu verbinden.

DataFrame = new byte[FrameHeader.length + Data.length -2];
//Daten aus FrameHeader in DataFrame kopieren (1-4)            System.arraycopy(FrameHeader, 0, DataFrame, 0, FrameHeader.length -1);
//Daten aus Data in DataFrame kopieren (5-Data.Lenght)            System.arraycopy(Data, 0, DataFrame, FrameHeader.length, DataFrame.length -1);

Erst soll das Array 'FrameHeader' vorne reinkopiert werden, im Anschluß 
soll 'Data' dahinter reinkopiert werden. Warum kopiert Java das erste 
Array, aber ds zweite nicht und schmeißt dafür eine 
ArrayIndexOutOfBoundsException?

Sieht irgendjemand etwas, was ich nicht sehe?

[Wühlhase (Gast)]

Code nochmal vernünftig formatiert:

DataFrame = new byte[FrameHeader.length + Data.length -2];
//Daten aus FrameHeader in DataFrame kopieren
System.arraycopy(FrameHeader, 0, DataFrame, 0, FrameHeader.length -1);
//Daten aus Data in DataFrame kopieren
System.arraycopy(Data, 0, DataFrame, FrameHeader.length, DataFrame.length -1);

[K. S. (hagbart06)]

System.arraycopy(Data, 0, DataFrame, FrameHeader.length-1, DataFrame.length -1);

[Der Andere (Gast)]

1.
Reduziere den Code bis auf einen den man einfach ausführen kann. So 
sieht man z.B. nicht die Definition der Arrays die du zusammenkopieren 
willst.
2.
Du schreibst zwar ArrayIndexOutOfBounds Exception, aber warum kopierst 
du nicht den kompletten fehler hier rein?

3. Warum hat das neue Arraydie Länge Array1 + Array2 - 2
Was soll die Minus 2?

Denke darüber mal nach

Nutze den Debugger, mache dir ein einfaches Beispiel mit zwei Array aus 
z.B. nur 4 und 5 Bytes und schaue was rauskommt. Dann kannst du den 
Fehler problemlos selbst finden.

[zyxw (Gast)]

Prüfe doch mal, ob die Größe des Zielarrays wirklich ausreicht.

[K. S. (hagbart06)]

Der Andere schrieb:
> Was soll die Minus 2?

Berechtigter Einwand, bin automatisch davon ausgegangen, dass er das 
jeweils letzte Element nicht mit kopieren möchte. Da er davon aber 
nichts schrieb, gehe ich davon aus, dass der ihm der Unterschied 
zwischen Anzahl der Elemente und Index nicht klar ist...

[Wühlhase (Gast)]

Danke für die Hinweise-nein, die -2 gehören da wirklich nicht rein. Es 
macht aber keinen Unterschied, ohne funktioniert es auch nicht. Das 
DataFrame-Array ist auch lang genug.

FrameHeader.length = 4
Data.length = 2055
DataFrame.legth = 2059

[Wühlhase (Gast)]

Oh man...wie auch immer, es funktioniert jetzt.

[c.m. (Gast)]

Wühlhase schrieb:
> ArrayIndexOutOfBoundsException

wird nur geworfen wenn das zielarray zu klein ist, du also versuchst 
über das ende hinaus zu schreiben.
vielleicht magst du diese lösung:

public class JavaApplication3 {

    public static void main(String[] args) {
        byte[] arr1 = {'a', 'b', 'c'};
        byte[] arr2 = {'A', 'B', 'C', 'D'};
        byte[] target = null;
        ByteArrayOutputStream baos = new ByteArrayOutputStream(arr1.length + arr2.length); // initiallänge optional
        try {
            baos.write(arr1);
            baos.write(arr2);
            target = baos.toByteArray();
        } catch (IOException ex) {
            Logger.getLogger(JavaApplication3.class.getName()).log(Level.SEVERE, null, ex);
            System.exit(1);
        } finally {
            if (null == target) {
                System.out.println("kacke");
            } else {
                System.out.println("len: " + target.length);
                System.out.println("inh: " + new String(target));
            }
        }
    }
}

out:

run:
len: 7
inh: abcABCD
BUILD SUCCESSFUL (total time: 0 seconds)

[Wühlhase (Gast)]

Wie gesagt-jetzt läuft es.

Delete complete...

Aber trotzdem vielen Dank. Das nächste Problem kommt bestimmt. ;)

[Eric B. (beric)]

Wühlhase schrieb:
> Oh man...wie auch immer, es funktioniert jetzt.

Und ist dir klar warum es jetzt funktioniert und zuvor nicht?
Sonst kommt nicht das nächste Problem, sondern das gleiche nochmal.

[Bernd B. (Gast)]

Der Knackpunk dürfte wohl sein, dass es in der letzten Zeile Data.length 
anstatt DataFrame.length heißen muss, sonst ist der Überlauf garantiert.

[Bernd B. (Gast)]

Knackpunkt :)

[Erwin D. (Gast)]

Bernd B. schrieb:
> Knackpunkt :)

Immer noch besser, als wenn du den zweiten Buchstaben vergessen 
hättest...
:-)