Как я могу сделать этот код лучше

Question

Я поля, такие как

variable1, variable2, variable3 .... и т.д.

и мои условия

if(variable1 != null && variable1 != 0) 
    myobject.setFirstValue(variable1); 

if(variable2 != null && variable2 != 0) 
myobject.setSecondValue(variable2); 

if(variable3 != null && variable3 != 0) 
myobject.setThirdValue(variable3); 

Как это я имеют почти 15 к рассмотрению. Есть ли способ написать все эти условия простым способом. Имена переменных не совсем то же, что и я. Они отличаются от того, что я разместил.

Answer 1

Вы могли бы начать с ...

if (notNullOrZer0(variable1)) { 
    ... 
} 

private boolean notNullOrZero(Integer x) { 
    return x != null && x != 0; 
} 

Тогда, возможно, положить все эти переменные в List<Integer> ... это зависит от того, как устроена остальная часть кода ...

Answer 2

Вы можете положить все переменные в массиве и использовать цикл для проверки состояния и установите значения в myobject.setFirstValue(array[index]);

Answer 3

вы можете создать метод isNotNullorequalsZero, который проверяет, не нулевой, а не равен нулю

private boolean isNotNullorequalsZero(Integer value){ 
    return value != null && value != 0; 
} 

и вы используете что -

if(isNotNullorequalsZero(variable1)) 
    myobject.setFirstValue(variable1); 

if(isNotNullorequalsZero(variable2)) 
myobject.setFirstValue(variable2); 

if(isNotNullorequalsZero(variable3)) 
myobject.setFirstValue(variable3); 

Answer 4

либо модифицировать setFirstValue метод как

void setFirstValue(Integer var){ 
if(var !=null && var !=0) { 
//current logic of setFirstValue 
} 
} 

или если нет возможности изменить его, создать оболочку вызова в качестве (логического типа возврата уведомить если отказ в вызове)

boolean checkAndSetFirstValue(Integer var) { 
    if(var !=null && var !=0){ 
     setFirstValue(var); 
     return true; 
    } 
    return false; 
} 

Answer 5

Вы могли бы сделать метод под названием что-то вроде copyIfNonZero, и это, вероятно, выглядеть следующим образом:

public void copyIfNonZero(MyObject myObject, Integer value) { 
    if (value != null && value != 0) { 
     myObject.setFirstValue(value); 
    } 
} 

И текущий код будет сокращен до

copyIfNonZero(myobject, variable1); 
copyIfNonZero(myobject, variable2); 
copyIfNonZero(myobject, variable3); 
... 

или даже поместить переменные в Collection и итерация над этим:

Collection<Integer> values = ..... 
for(Integer value : values) { 
    copyIfNonZero(myObject, value); 
} 

Answer 6

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

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

public static void setPropertyIfNonNullNorZero(Object target, String propertyName, Integer value){ 

    //use some reflection tooling here, or write the whole thing yourself... I am using commons.beanutils here 

    if(value != null && value != 0){ 
     BeanUtils.setProperty(target, propertyName, value); 
    } 

} 

Этот код предполагает, что ваш целевой объект (MyObject) соблюдает соглашения о бобах (в основном, с именами получателей и сеттеров), и значение всегда является целым.

Затем использовать его как это:

Tools.setPropertyIfNonNullNorZero(myobject, "thirdValue", variable3);