Оглавление
V6123. Modified value of the operand is not used after the increment/decrement operation.
Анализатор обнаружил, что значение постфиксной операции не используется. Скорее всего, или операция избыточна, или вместо постфиксной операции следует использовать префиксную.
Пример:
int calculateSomething() {
int value = getSomething();
....
return value++;
}В данном примере имеется локальная переменная 'value'. Ожидается, что метод вернёт её инкрементированное значение. Однако согласно JLS:
The value of the postfix increment expression is the value of the variable before the new value is stored.
Таким образом оператор '++' никак не повлияет на значение, которое вернёт метод 'calculateSomething'. Возможный исправленный вариант:
int calculateSomething() {
int value = getSomething();
....
return ++value;
}Следующий вариант исправления кода ещё лучше подчёркивает, что следует вернуть значение на единицу больше:
int calculateSomething() {
int value = getSomething();
....
return value + 1;
}Мы рекомендуем использовать второй вариант, так как его проще понять.
Рассмотрим ещё один синтетический пример:
void foo() {
int value = getSomething();
bar(value++);
bar(value++);
bar(value++);
}Каждый раз метод 'bar' вызывается с аргументом на единицу больше. Последний инкремент не имеет смысла, так как увеличенное значение переменной далее не используется. Однако ошибки здесь нет, так как последний инкремент написан из эстетических соображений. Предупреждение не выдаётся, если переменная последовательно инкрементируется более двух раз подряд.
Однако мы всё равно рекомендуем писать следующим образом:
void foo() {
int value = getSomething();
bar(value++);
bar(value++);
bar(value);
}Или так:
void foo() {
int value = getSomething();
bar(value + 0);
bar(value + 1);
bar(value + 2);
}Данная диагностика классифицируется как:
Была ли полезна эта страница документации?
