Стиль проверки Java: конечное назначение переменной задания параметра

Я работаю над разрешением предупреждений Checkstyle в проекте Java.Стиль проверки Java: конечное назначение переменной задания параметра

Пусть это метод я есть:

public SomeObject someObjectBehavior(SomeObject oldSomeObject, 
    SomeObject newSomeObject) { 

    if(oldSomeObject == null) { 
     oldSomeObject = newSomeObject; 
    } 
}

Когда я включить Checkstyle, он уведомляет

Параметр oldSomeObject должно быть окончательным

Параметр newSomeObject должно быть окончательным

Когда я конвертирую оба параграфа метров, как это:

public SomeObject someObjectBehavior(final SomeObject oldSomeObject, 
    final SomeObject newSomeObject) { 

    if(oldSomeObject == null) { 
     oldSomeObject = newSomeObject; //Assignment operation 
    } 
}

Итак, теперь операция присваивания не представляется возможным, есть ошибка в Java коде

Конечное локальная переменная oldSomeObject не может быть назначена. Он должен быть пустым и не использовать составное назначение.

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

Thank You

С уважением

Анкер Шарма

источник

2014-01-07 Ankur Sharma

В Java аргументы метода рассматриваются как локальные переменные, поэтому присвоение значения аргументу метода не влияет на вызывающего метода. Это только путает читателей вашего метода, который может подумать, что 'oldSomeObject' ссылается на значение, которое было передано, но вместо этого содержит значение, которое было присвоено ему в методе. –

@AnkurSharma Каков ваш прецедент для назначения значений объектов, переданных параметром? –

@Igor Ganapolsky, на самом деле это был предварительно написанный код, и нам был предоставлен код для решения только проблемных вопросов как можно больше.Я бы изменил код для вышеупомянутого назначения, но нам не разрешалось модифицировать код в то время ... так что эта ситуация возникла. –

Это считается плохим стилем переназначения параметров. Вы должны создать новую локальную переменную.

источник

2014-01-07 14:07:20

Ну, я не пишу свежий код, Это уже существующий код, поэтому мне нужно решить все проблемы в стиле чеков. –

и еще одна вещь, oldSomeObject - это просто ссылочная переменная здесь, поэтому я просто назначаю ссылку, есть ли способ? –

@AnkurSharma код работает? не меняйте его. Откройте для расширения, близкого к модификации. – nachokk

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

Лучше использовать новую локальную переменную, если вы хотите изменить значение. Во многих случаях назначение переменных только один раз делает код более читаемым. Это не всегда возможно, но часто это так.

Try что-то вроде

Object realObject; 
if (oldSomeObject != null) { 
    realObject = oldSomeObject; 
} else { 
    realObject = newSomeObject; 
}

источник

2014-01-07 14:10:06 koljaTM

Это я понимаю, но дело в том, что мне нужно передать ссылку newSomeObject на oldSomeObject в случае, когда oldSomeObject имеет значение NULL, и когда оба параметра являются окончательными. –

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

, вы возвращаете его и назначаете результат вызова метода вашей переменной. – koljaTM

Пример метода вы предоставляете

public SomeObject someObjectBehavior(final SomeObject oldSomeObject,final SomeObject newSomeObject) { 
    if(oldSomeObject == null) { 
     oldSomeObject = newSomeObject; //Assignment operation 
    } 
}

является прекрасным примером того, почему делает ваши параметры final это хорошая идея. Не создавая переменные final, вы можете не понимать, что ваш метод ничего не делает, поскольку переназначение ссылки на параметр ничего не меняет вне вашего метода.

источник

2014-01-07 14:15:27 Keppil

Ну, я понял, что это хорошая вещь, которую накладывает стиль проверки, но что, если мой oldSomeObject имеет значение null, и мне нужно назначить ссылку newSomeObject на oldSomeObject, они являются только ссылочной переменной, поэтому, когда метод завершается , и управление возвращается к названному месту, теперь oldSomeObject и newSomeObject оба указывают на то же место в памяти (когда oldSomeObject был пустым), поэтому я хочу знать, как следует соблюдать контрольный стиль, а также не нарушать java-правила :) –

@Ankur: Не имеет значения, что вы делаете с ссылкой «oldSomeObject» внутри метода, любые изменения будут отброшены после завершения метода, так как это локальная переменная. – Keppil

проблема в том, что java дает мне сообщение об ошибке, это назначение не может быть выполнено, поскольку param (oldSomeOBject) является окончательным (что очевидно) .. но я думаю, что должно быть каким-то образом, когда уведомление об уведомлении стиля не должен появляться, и я также не вижу ошибок в коде. : | –

Стиль проверки Java: конечное назначение переменной задания параметра

ответ

Смежные вопросы