newInstance() liefert null zurück

[inv_zim]

Hallo,

ich muss in einer Software Objekte dynamisch über den Klassennamen erstellen. Die Methode dafür sieht so aus, und lief bisher immer gut:

private CommunicationObject instantiateTransferObject(String comObjectClassPath) {
        CommunicationObject comObject = null;
        try{
            @SuppressWarnings("unchecked")
            Class<? extends CommunicationObject> comClass = (Class<? extends CommunicationObject>) Class.forName(comObjectClassPath);
            comObject = comClass.newInstance();
        } catch(ClassNotFoundException e) {
            e.printStackTrace();
        } catch(InstantiationException e) {
            e.printStackTrace();
        } catch(IllegalAccessException e) {
            e.printStackTrace();
        }
        return comObject;
    }

Ich versuche in diesem Beispiel eine Objekt der Klasse UserData zu erstellen:

public class UserData implements CommunicationObject {
    private static final long serialVersionUID= -3663712107665665551L;
    
    private long id= 0;
    private String name= null;
    private RoleData role= null;
    private String hash= null;
    
    public UserData() {
        System.out.println("Constructor UserData");
    }

    public long getId() {
        return id;
    }

    public void setId(long id) {
        this.id= id;
    }

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name= name;
    }

    public RoleData getRole() {
        return role;
    }

    public void setRole(RoleData role) {
        this.role= role;
    }

    public String getHash() {
        return hash;
    }

    public void setHash(String hash) {
        this.hash= hash;
    }

    public String toString() {
        return name;
    }
}

Die Klasse "CommunicationObject" liegt in einem anderen Projekt, welches aber über Maven referenziert wird. Bevor ich das Projekt auf Maven umgestellt habe, lief es. Jetzt nicht mehr.

Der Aufruf forName liefert mir die richtige Klasse zurück, allerdings sind alle Werte der Variablen (im Debugger zu sehen) null. Wenn ich comClass.newInstance() aufrufe, bekomme ich null als returnValue.

Wenn ich mit dem Debugger einen Breakpoint im Konstruktor von UserData setze, sehe ich aber eindeutig, dass dieser aufgerufen wird. Habt ihr eine Idee, ich bin ratlos? :bahnhof:

 

[SlaterB]

sicher mit dem return-Value? Debugger sind immer zu misstrauen,
solange sie funktionieren ist ja gut, aber wenn es um unerklärliche Phänomene geht, kann System.out.prinln in gewissen Grenzen zuverlässiger sein,
sinnvoll ist z.B. eine Ausgabe direkt nach newInstance() um 100% zu garantieren, dass keine Exception geflogen ist,

Test-Programm:

public class Test
{
    public static void main(String[] args)
        throws Exception
    {
        CommunicationObject c = instantiateTransferObject(UserData.class.getName());
        System.out.println("c ist null? " + (c == null));
        System.out.println(c);
    }

    private static CommunicationObject instantiateTransferObject(String comObjectClassPath)
    {
        CommunicationObject comObject = null;
        try
        {
            @SuppressWarnings("unchecked")
            Class<? extends CommunicationObject> comClass = (Class<? extends CommunicationObject>)Class.forName(comObjectClassPath);
            comObject = comClass.newInstance();
            System.out.println("nach new Instance noch am Leben");
        }
        catch (ClassNotFoundException e)
        {
            e.printStackTrace();
        }
        catch (InstantiationException e)
        {
            e.printStackTrace();
        }
        catch (IllegalAccessException e)
        {
            e.printStackTrace();
        }
        return comObject;
    }
}


interface CommunicationObject
{
}


class UserData
    implements CommunicationObject
{
    private static final long serialVersionUID = -3663712107665665551L;

    private long id = 0;
    private String name = null;
    // private RoleData role = null;
    private String hash = null;

    public UserData()
    {
        System.out.println("Constructor UserData");
    }

    public long getId()
    {
        return id;
    }

    public void setId(long id)
    {
        this.id = id;
    }

    public String getName()
    {
        return name;
    }

    public void setName(String name)
    {
        this.name = name;
    }

    public String getHash()
    {
        return hash;
    }

    public void setHash(String hash)
    {
        this.hash = hash;
    }

    public String toString()
    {
        return name;
    }
}

Ausgabe:

Constructor UserData
nach new Instance noch am Leben
c ist null? false
Exception in thread "main" java.lang.NullPointerException
	at java.io.Writer.write(Unknown Source)
	at java.io.PrintStream.write(Unknown Source)
	at java.io.PrintStream.print(Unknown Source)
	at java.io.PrintStream.println(Unknown Source)
	at test.Test.main(Test.java:11)

false klärt hier eindeutig dass ein Objekt erzeugt wurde,
die NullPointerException danach führt vielleicht zu der irrigen Annahme?
das liegt hier daran dass println() erstaunlicherweise mit einem Objekt, welches null als toString() zurückgibt, nicht klar kommt,
wußte ich selber noch gar nicht

wirkt sich bei dir vielleicht auch irgendwie aus? der Debugger könnte allein schon "null" als Beschreibung des vorhandenen Objektes ausgeben..,
ändere testweise deine toString()-Methode oder lasse sie weg

wenn du ganz andere Fehler wie falschen Parameter für Klassenname hast, dann kann es natürlich wieder ganz anders aussehen

 

[inv_zim]

Du hast recht, das Objekt wird erstellt. War wohl eine falsche Schlussfolgerung von mir, Augen auf beim Debuggen

Ich versuche danach mit Dozer Daten von einem ins andere Objekt zu mappen und bekomme eine nicht sonderlich aussagekräftige Fehlermeldung, die hat dann wohl eine andere Ursache. Das hat mit dem Thema hier aber nichts mehr zu tun, denke ich. Danke für deine Hilfe!

Tim

 

[ARadauer]

das liegt hier daran dass println() erstaunlicherweise mit einem Objekt, welches null als toString() zurückgibt, nicht klar kommt,
wußte ich selber noch gar nicht

kann ich mir nicht vorstellen...

public class NpeTest
{
    public static void main(String[] args)
    {
        String nullString = null;
        System.out.println(nullString);
        System.out.println(new NpeTest());
    }
    
    public String toString(){
        return null;
    }
}

null
null
kein Problem

 

[SlaterB]

vielleicht bei dir in anderer Java-Version,

der interne Code lautet:

PrintStream:
public void print(Object obj) {
      write(String.valueOf(obj)); // also write((String)null);
}

private void write(String s) {
	try {
	    synchronized (this) {
		ensureOpen();
		textOut.write(s); // BufferedWriter, Writer-Klasse bekommt den Aufruf
[..]

Writer:
    /**
     * Write a string.
     *
     * @param  str  String to be written
     *
     * @exception  IOException  If an I/O error occurs
     */
    public void write(String str) throws IOException {
	write(str, 0, str.length());
    }