Java Generics for unmarshall Объект JAXB

Question

У меня есть код, который нужно поместить в unmarshall xml в объекты Java. Я хотел бы посмотреть, можно ли улучшить этот код с помощью Java Generics вместо использования типа Object в качестве возвращаемого значения.

protected static <T> Object unmarshall(String xml, Class<T> clazz) 
     throws JAXBException { 
    JAXBContext jc = JAXBContext.newInstance(clazz); 
    Unmarshaller unmarshaller = jc.createUnmarshaller(); 
    Object obj = unmarshaller.unmarshal(new StringReader(xml)); 
    return obj; 
} 

Любые предложения.

Answer 1

Да, вы можете улучшить ваш код немного:

protected static <T> T unmarshall(String xml, Class<T> clazz) 
     throws JAXBException { 
    JAXBContext jc = JAXBContext.newInstance(clazz); 
    Unmarshaller unmarshaller = jc.createUnmarshaller(); 
    T obj = clazz.cast(unmarshaller.unmarshal(new StringReader(xml))); 
    return obj; 
} 

Answer 2

Если вам не нужно хотите повторно использовать JAXBContext, нет необходимости его создания. Существует метод удобства, который (почти) имеет подписи метода, который вы хотите. Он выбрасывает RuntimeException (многие предпочитают это).

protected static <T> T unmarshall(String xml, Class<T> clazz) { 
    return JAXB.unmarshal(new StringReader(xml), clazz); 
} 

Но imo, просто используйте метод непосредственно без обертки.

Повторное использование JAXBContext делает (un) более ранним в долгосрочной перспективе. Еще одним улучшением является clazz.cast(), чтобы избежать неприятного непроверенного литья.

private static final ConcurrentMap<Class<?>, JAXBContext> CONTEXT = new ConcurrentHashMap<>(); 

protected static <T> T unmarshall(String xml, Class<T> clazz) 
     throws JAXBException { 
    JAXBContext context = CONTEXT.get(clazz); 
    if (context == null){ 
     context = JAXBContext.newInstance(clazz); 
     CONTEXT.putIfAbsent(clazz, context); 
    } 

    Unmarshaller unmarshaller = context.createUnmarshaller(); 
    Object obj = unmarshaller.unmarshal(new StringReader(xml)); 
    if (clazz.isInstance(obj)){ 
     return clazz.cast(obj); 
    } 
    throw new IllegalArgumentException("XML does not represent an instance of type:" + clazz.getName()); 
}