# V3061. Parameter 'A' is always rewritten in method body before being used.

Анализатор нашёл потенциальную ошибку в теле метода. Один из его параметров перезаписывается перед тем, как используется. Таким образом значение, пришедшее в метод, попросту теряется.

Эта ошибка имеет разные проявления. Рассмотрим пример кода:

    void Foo1(Node A, Node B)
    {
      A = SkipParenthesize(A);
      B = SkipParenthesize(A);
      // do smt...
    }

Здесь допущена опечатка, из-за чего объект 'B' примет неверное значение. Исправленный код выглядит так:

    void Foo1(Node A, Node B)
    {
      A = SkipParenthesize(A);
      B = SkipParenthesize(B);
      // do smt...
    }

Но данная ошибка может проявить себя и более интересно:

    void Foo2(List<Int32> list, Int32 count)
    {
      list = new List<Int32>(count);
      for (Int32 i = 0; i < count; ++i)
        list.Add(GetElem(i));
    }

Данный метод должен был инициализировать список некоторыми значениями. В данном случае происходит копирование ссылки ('list'), хранящей адрес блока памяти в куче, в котором расположен список (или 'null', если память не была выделена). Поэтому, когда мы вновь выделяем память под список, адрес блока памяти записывается в локальную копию ссылки, а исходная (за пределами метода) остаётся неизменной. Таким образом выполняется лишняя работа для выделения памяти, инициализации списка и последующей уборки мусора.

Ошибка заключается в пропущенном модификаторе 'out'. Корректный вариант кода:

    void Foo2(out List<Int32> list, Int32 count)
    {
      list = new List<Int32>(count);
      for (Int32 i = 0; i < count; ++i)
        list.Add(GetElem(i));
    }

Взгляните на примеры ошибок, обнаруженных с помощью диагностики V3061.