Datentypen Eigenen Datentyp toArray()

[madomat]

Hallo Leute,

ich hab noch ein Problem.
Ich hab mir eine eigene Dictionary-Klasse geschrieben, welche auch toArray() implementieren soll:

	@SuppressWarnings("unchecked")
	@Override
	public T[] toArray() {
		// den kram in ein array umwandeln
		ListItem active = firstitem;
		Object returnarray[] = new Object[size()];
		int counter = 0;
		while(active.nextitem != null) //bis zum letzten Element wandern
		{
			active = active.nextitem; //zeiger immer aktualisieren
			returnarray[counter] = active.element;
			counter++;
		}
		return (T[]) returnarray;
	}

Ruf ich das von der Testclass dann auf:

mydictionary<String> testlist = new mydictionary<String>();
		testlist.insert("erstens");
		testlist.insert("zweitens");
System.out.println(testlist.toArray()[1]);

Bekomme ich:

Exception in thread "main" java.lang.ClassCastException: [Ljava.lang.Object; cannot be cast to [Ljava.lang.String;
at tester.main(tester.java:14)

Was mache ich hier falsch?
Vielen Dank für eure Superhilfe :toll:

 

[Gast2]

In Zeile 6 legst du das Array ja als Object[] an und castest es am Ende einfach nach T[], du musst schon intern auch mit den generischen Typen arbeiten

Btw - wo ich es eben woanders geschrieben hatte: Klassennamen im CamelCase, z.B.:

class MyDictionary

 

[madomat]

T returnarray[] = new T[size()];

führt zu

Cannot generate a generic Array of T

Deswegen der Spaß mit dem Object.

 

[gman]

Hi,

die richtige Signatur der Methode müsste doch:

<T> T[] toArray(T[] a)

sein, oder nicht? Dann müsste der Vorschlag von fassy auch
funktionieren, da der generische Typ "T" dann auch bekannt ist.

    @SuppressWarnings("unchecked")
    @Override
    public T[] toArray(T[] a) {
        // den kram in ein array umwandeln
        ListItem active = firstitem;
        a = new T[size()];
        int counter = 0;
        while(active.nextitem != null) //bis zum letzten Element wandern
        {
            active = active.nextitem; //zeiger immer aktualisieren
            a[counter] = active.element;
            counter++;
        }
        return a;
    }

(ungetesteter Code)

 

[gman]

Hi,

ich hab mich wohl irgendwie versehen, du willst ja die andere Methode implementieren, sorry.
Und der Vorschlag:

a = new T[size()];

ist auch Mist, haste ja auch schon gemerkt Aber hier noch ein paar Tips die dir vielleicht
weiterhelfen können:

- Guck dir mal die Klassne "java.util.Arrays" und "java.lang.System" (vor allem "arraycopy") an
- in diesem Thread geht es wohl um eine ähnliche Aufgabe