"baz" wrote : | Does it make sense to you to write getInstance in this way? | @Unwrap | | public Object getInstance() throws ClassNotFoundException | | { | | Class clazz = Class.forName(entityClass); | | if (id==null) return null; | | return session.get(clazz, id); | | } | | mmmmmm, do you really need it to work like that? It makes it unclear what "null" means ... does null mean "no id", or does it mean "no object for the id". View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=3959653#... Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&a...