Stateless Bean: NullPointerException

[Guest]

Hi zusammen,

ich habe folgende Frage zu einer NullPointerException, wobei ich nicht weiß wieso sie auftritt:


Meine Stateless-Bean "MyBean" implementiert ein RemoteInterface mit einer Methode, welche eine Entität zurückliefert:

public class MyBean implements MyRemoteInterface
{
    public MyEntity create(Integer id)
    {
        MyEntity en = null;
        
        return en;
    }
}

Wenn ich rufe nun in "MyHandler" diese Methode auf:

public void myMethod()
{
    InitialContext ctx = new InitialContext();
    (MyRemoteInterface) iface = (MyRemoteInterface)ctx.lookup("Pfad zum Interface incl. Name");
    MyEntity en = iface.create(en.getId());
}

Hierbei erhalte ich die NullPointerException, wenn ich die Methode create() aufrufe. Ändere ich diese Methode wie folgt, tritt dieses Problem nicht mehr auf:

public class MyBean implements MyRemoteInterface
{
    public MyEntity create(Integer id)
    {
        MyEntity en = new MyEntity(id);
        
        return en;
    }
}

Nun verstehe ich nicht wieso ich nicht eine "leere" Entität zurückliefern kann.
Ich habe vorher normale WindowsApplikationen in Java geschrieben. Dort wäre dies nicht vorgekommen. Ist das eien Spezifika von einer Stateless-Bean, dass dort niemals null auftreten darf?

 

[Murray]

Meine Stateless-Bean "MyBean" implementiert ein RemoteInterface mit einer Methode, welche eine Entität zurückliefert:

Tut sie eben nicht - sie liefert null zurück.

Ist das eien Spezifika von einer Stateless-Bean, dass dort niemals null auftreten darf?

Das hat m.E. nicht notwendigerweise etwas mit Beans zu tun (seien sie nun stateful oder stateless) - bei Factory-Methoden (und das ist die create-Methode ja) darf natürlich nicht null kommen, denn der Aufrufer hat die Methode ja verwendet, um eine bestimmte Instanz zu erzeugen.

 

[Guest]

Aber sowas sollte doch zulässig sein oder?:

Connection conn = null;
try
{
    conn = dataSource.getConnection
}
catch(SQLException e)
{
    e.printStackTrace();
}

 

[Murray]

Syntaktisch ist das sicher zulässig (wenn auch printStackTrace() eher eine Minimalform der Ausnahmebehandlung darstellt, die im produktiven Code so nicht mehr auftauchen sollte).

Wenn dieser Code aber z.B. in so einer Methode

public Connection createConnection() {
  Connection conn = null;
  try {
      conn = dataSource.getConnection
  } catch(SQLException e) {
      e.printStackTrace();
  }
}

steckt, dann wäre das nicht so gut, denn ein Verwender würde natürlich erwarten, dass hier ein Connection-Objekt bekommt. Wenn das auch null sein kann, dann muss der Verwender a) den Rückgabewert vor der Verwendung prüfen und b) könnte dann nur mit einer unspezifischen Fehlermeldung reagieren, da die eigentliche Exception ja abgefangen worden ist.
Besser also

public Connection createConnection() throws SQLException {
  return dataSource.getConnection
}