Избегайте «Если (параметр == null)» оператор

Question

Вот моя проблема. У меня есть функция, называемая маршрутом, который работает правильно! Однако я хотел бы факторизовать контрольный блок параметров.

Позвольте мне объяснить. Когда пользователь вводит URL-адрес для доступа к функции, он может поместить некоторые необязательные параметры (всего 6). По крайней мере один из этих параметров необходим для продолжения. Моя структура настроена так, чтобы присваивать нулевое значение параметрам, которые не были проинформированы пользователем.

Чтобы проверить, какие параметры были проинформированы, и проверить их, у меня есть блок:

public Result edit(String param1, String param2, String param3, String param4, String param5, String param6) { 

    Map<String, Object> parameters = new HashMap<>(); 
    if (param1 != null) { 
     // Checking function depending on data type (URL, Boolean, ..), return a clean param or throw an InvalidParamException 
     // Variable param depends on type returned by checkParamType1 
     param = checkParamType1(param1); 
     parameters.put("param1", param); 
    } 
    if (param2 != null) { 
     param = checkParamType1(param2); 
     parameters.put("param2", param); 
    } 
    if (param3 != null) { 
     param = checkParamType2(param3); 
     parameters.put("param3", param); 
    } 
    if (param4 != null) { 
     param = checkParamType2(param4); 
     parameters.put("param4", param); 
    } 
    if (param5 != null) { 
     param = checkParamType3(param5); 
     parameters.put("param5", param); 
    } 
    if (param6 != null) { 
     param = checkParamType3(param6); 
     parameters.put("param6", param); 
    } 
    assert(parameters.size() > 0, "At least one parameter required"); 

    // [TREATMENT] 
} 

Мой вопрос, по вашему мнению, можно факторизовать этот блок?

работает мой проект на JAVA 8.

Спасибо :)

Answer 1

Лучшее, что вы можете сделать IMHO, - это извлечь метод и использовать ссылку на метод для вашей настраиваемой проверки. Что-то вроде этого:

private void checkAndAssign(Map<String, Object> map, String paramName, String paramValue, Function<String, Object> checker){ 
    if(paramValue!=null){ 
    Object param = checker.apply(paramValue); 
    map.put(paramName, param); 
    } 
} 

public Result edit(String param1, String param2, String param3, String param4, String param5, String param6) { 

    Map<String, Object> parameters = new HashMap<>(); 
    checkAndAssign(map, "param1", param1, MyClass::checkParam1); 
    checkAndAssign(map, "param2", param2, MyClass::checkParam2); 
    checkAndAssign(map, "param3", param3, MyClass::checkParam3); 
    checkAndAssign(map, "param4", param4, MyClass::checkParam4); 
    checkAndAssign(map, "param5", param5, MyClass::checkParam5); 
    checkAndAssign(map, "param6", param6, MyClass::checkParam6); 
    assert(parameters.size() > 0, "At least one parameter required"); 

    // [TREATMENT] 
} 

MyClass :: checkParam1 - это ссылка на метод. Компилятор позволит вам заменить такую ​​ссылку на функциональный интерфейс, например Function.

См. Method References в учебнике по Java 8.

Answer 2

Java 8 введена новая функция, чтобы избежать пустых чеков. Посмотрите: http://winterbe.com/posts/2015/03/15/avoid-null-checks-in-java/

Answer 3

Сначала вы можете использовать операторы case switch. Но в ОО-языках, было бы лучше использовать шаблон проектирования, как цепь ответственности ...

Answer 4

В Java-можно заменить

if (param1 != null) { 
    param = checkParamType1(param1); 
    parameters.put("param1", param); 
} 

С чем-то вроде:

Optional.ofNullable(param1) 
    .ifPresent((param) -> parameters.put("param1", checkParamType1(param))); 

Answer 5

, если количество аргументов не ограничено, попробуйте использовать параметр var-args и цикл по результирующему массиву.

public Result edit(String... params) { 
    Map<String, Object> parameters = new HashMap<>(); 
    IntStream.range(0, params.length).forEach(idx -> { 
     String param = params[idx]; 
     if (param != null) { 
      Object p = checkParamType(param); 
      parameters.put("param" + idx, p); 
     } 
    }); 
}