Анализатор нашёл потенциальную ошибку в теле метода. Один из его параметров перезаписывается перед тем, как используется. Таким образом значение, пришедшее в метод, попросту теряется.
Эта ошибка имеет разные проявления. Рассмотрим пример кода:
void Foo1(int A, int B) { A = Calculate(A); B = Calculate(A); // do smt... }
Здесь допущена опечатка, из-за чего объект 'B' примет неверное значение. Исправленный код выглядит так:
void Foo1(int A, int B) { A = Calculate(A); B = Calculate(B); // do smt... }
Рассмотрим еще один пример данной ошибки:
void Foo2(List<Integer> list, int count) { list = new ArrayList<Integer>(count); for (int i = 0; i < count; ++i) list.add(MyRnd(i)); }
Данный метод должен был инициализировать список некоторыми значениями. В данном случае происходит копирование ссылки ('list'), хранящей адрес блока памяти в куче, в котором расположен список (или 'null', если память не была выделена). Поэтому, когда мы вновь выделяем память под список, адрес блока памяти записывается в локальную копию ссылки, а исходная (за пределами метода) остаётся неизменной. Таким образом выполняется лишняя работа для выделения памяти, инициализации списка и последующей уборки мусора.
Возможно, следует переписать метод следующим образом:
List<Integer> Foo2(int count) { List<Integer> list = new ArrayList<>(count); for (int i = 0; i < count; ++i) list.add(MyRnd(i)); } ... list = Foo2(count);
Взгляните на примеры ошибок, обнаруженных с помощью диагностики V6023. |