Проверьте ошибку стиля. Атрибут параметра не разрешен.

Question

Да, я знаю, что есть другой вопрос, похожий, но моя проблема в том, что мои параметры являются объектами.

public void serStuff(OBJ x, OBJ y) 

if (x== null) 
{ 
    x = ANOTHER_VALUE; 
} 
if(y == null) 
{ 
    y = ANOTHER_VALUE2; 
} 

, так что мне не нравится назначать параметры, но как бы исправить это? я бы просто локально поставил другой объект

Answer 1

Самый простой способ исправить это - просто прекратить назначать параметры, так как после возвращения из этого метода он никак не влияет.

Но иногда это еще полезно, чтобы сделать это, например:

public void doSomething(String s){ 
    if (s == null){ 
     s = ""; 
    } 
    s.callSomeMethodsAvoidingNullPointerException(); 
} 

Если вы не можете изменить Checkstyle конфигурации, вы можете просто создать еще одну переменную:

public void doSomething(String s){ 
    String sNotNull = s; 
    if (sNotNull == null){ 
     sNotNull = ""; 
    } 
    sNotNull.callSomeMethodsAvoidingNullPointerException(); 
} 

это выглядит как-то неуклюжий. Я бы предпочел изменить конфигурацию, если этот шаблон встречается часто. Но это также может быть очень субъективным взглядом.

Answer 2

Используйте локальную переменную, чтобы ее проверить. Нравится:

OBJ localX = (x == null) ? ANOTHER_VALUE : x; 

Затем обработайте новую переменную.

Значит, вы сделаете localX точкой для того же предмета или нового ANOTHER_VALUE.

Если вам нужно только проверить, не задан ли заданный параметр, вы можете использовать assert(x != null).

С другой стороны, если вам иногда необходимо убедиться, что параметр не будет изменен, используйте final. Это (примерно!) Эквивалент C const, компилятор «кусает» вас, если вы попытаетесь его изменить.