рефакторинг обязательный map.get код Java

Question

Пожалуйста, какие-либо идеи о том, как рефакторировать этот код Java, поэтому мне не нужно повторять эти кодовые блоки 3 (или более) раза?

Integer id = null; 
try { 
    id = (Integer)extradata.get("id"); 
} 
catch(Exception e) { 
    logger.error(e); 
} 

if (id == null) { 
    logger.error("no id set"); 
    task.setStatus(Status.ERROR); 
    DBService.updateTaskStatus(conn, false, task); 
    conn.commit(); 
    return; 
} 

String name = null; 
try { 
    name = (String)extradata.get("name"); 
} 
catch(Exception e) { 
    logger.error(e); 
} 

if (name == null) { 
    logger.error("no name set"); 
    task.setStatus(Status.ERROR); 
    DBService.updateTaskStatus(conn, false, task); 
    conn.commit(); 
    return; 
} 

String city = null; 
try { 
    city = (String)extradata.get("city"); 
} 
catch(Exception e) { 
    logger.error(e); 
} 

if (city == null) { 
    logger.error("no city set"); 
    task.setStatus(Status.ERROR); 
    DBService.updateTaskStatus(conn, false, task); 
    conn.commit(); 
    return; 
} 

Answer 1

Integer id = null; 
String name = null; 
String city = null; 

try { 
    id = (Integer)extradata.get("id"); 
    name = (String)extradata.get("name"); 
    city = (String)extradata.get("city"); 
catch(Exception e) { 
    logger.error(e); 
} 

if (id == null || name == null || city == null) { 
    String msg = (id == null ? "id" : name == null ? "name" : "city"); 
    logger.error("no "+msg+" set"); 
    task.setStatus(Status.ERROR); 
    DBService.updateTaskStatus(conn, false, task); 
    conn.commit(); 
    return; 
} 

Answer 2

Вы можете определить метод, который проверяет строку для null, и обновляет статус, как это:

private static boolean isValid(
    Object obj 
, String msg 
, Connection conn 
, Task task) { 
    if (obj != null) return true; 
    logger.error(msg); 
    task.setStatus(Status.ERROR); 
    DBService.updateTaskStatus(conn, false, task); 
    conn.commit(); 
    return false; 
} 

Теперь вы можете изменить основной метод для вызова checkValidString неоднократно:

Integer id = null; 
try { 
    id = (Integer)extradata.get("id"); 
} catch(Exception e) { 
    logger.error(e); 
} 
if (!isValid(id, "no id set", conn, task)) return; 

String name = null; 
try { 
    name = (String)extradata.get("name"); 
} catch(Exception e) { 
    logger.error(e); 
} 
if (!isValid(name, "no name set", conn, task)) return; 

String city = null; 
try { 
    city = (String)extradata.get("city"); 
} 
catch(Exception e) { 
    logger.error(e); 
} 
if (!isValid(city, "no city set", conn, task)) return; 

Answer 3

Попробуйте следующее:

Integer id = null; 
    try { 
     id = (Integer)extradata.get("id"); 
    } 
    catch(Exception e) { 
     logger.error(e); 
    } 

    if (id == null) { 
     handleNullVariable("id"); 
     return; 
    } 

    String name = null; 
    try { 
     name = (String)extradata.get("name"); 
    } 
    catch(Exception e) { 
     logger.error(e); 
    } 

    if (name == null) { 
     handleNullVariable("name"); 
     return; 
    } 

    String city = null; 
    try { 
     city = (String)extradata.get("city"); 
    } 
    catch(Exception e) { 
     logger.error(e); 
    } 

    if (city == null) { 
     handleNullVariable("city"); 
     return; 
    } 

    void handleNullVariable(String variableName) 
    { 
    logger.error("no "+ variableName + " set"); 
    task.setStatus(Status.ERROR); 
     DBService.updateTaskStatus(conn, false, task); 
     conn.commit(); 
     } 

Answer 4

Вы можете использовать статический универсальный метод:

private static final <T> T getValue(Map<String, Object> extradata, String key, Class<T> clazz) { 
    Object val = extradata.get(key); 
    if (val == null) { 
     handleNullVariable("name"); 
     return null; 
    }   
    return clazz.cast(val); 
} 

Тогда вы можете назвать его:

Integer id = getValue(extradata, "id", Integer.class); 
String name = getValue(extradata, "name", String.class); 

Answer 5

class NotFoundException extends Exception { 
} 
... 
Object getValue(String name) { 
    Object res; 
    try { 
     res=extradata.get(name); 
    } catch(Exception e) { 
     logger.error(e); 
    } 
    if (res == null) { 
     logger.error("no "+name+" set"); 
     task.setStatus(Status.ERROR); 
     DBService.updateTaskStatus(conn, false, task); 
     conn.commit(); 
     throw new NotFoundException(); 
    } 
} 
... 
try { 
    Integer id = (Integer)getValue("id"); 
    String name = (String)getValue("name"); 
    String city = (String)getValue("city"); 
} catch(Exception e) { 
    return; 
} 

Answer 6

public static <E> E getData(String key) 
{ 
    E ret = null; 
    try { 
     ret = (E)extradata.get(key); 
    } 
    catch(Exception e) { 
     logger.error(e); 
    } 

    if (ret == null) { 
     throw new Exception(key); 
    } 
    return ret; 
} 

Назовите это с

try 
{ 
    Integer id = getData("id"); 
    String name = getData("name"); 
    String city = getData("city"); 
} 
catch(Exception e) 
{ 
    logger.error("no " + e.getMessagE() + " set"); 
    task.setStatus(Status.ERROR); 
    DBService.updateTaskStatus(conn, false, task); 
    conn.commit(); 
    return; 
}