# V3086\. Variables are initialized through the call to the same function\. It's probably an error or un\-optimized code\.

Анализатор обнаружил потенциальную ошибку, найдя в коде инициализацию двух различных переменных одинаковыми выражениями\. Анализатор считает опасными не все выражения, а только в которых используется вызов функций \(либо слишком длинное выражение\)\.

Рассмотрим наиболее простой случай:

```cpp
x = X();
y = X();
```

Возможны три варианта действий:

1. Код содержит ошибку\. Необходимо исправить ошибку, заменив 'X\(\)' на 'Y\(\)'\.
1. Код верен, но работает медленно\. Если функция 'X\(\)' требует много вычислений, то лучше написать 'y \= x;'\.
1. Код верен и работает быстро\. Или функция 'X\(\)' читает значение из файла\. Тогда чтобы избавиться от ложного срабатывания, можно использовать комментарий "//\-V3086"\.

Теперь рассмотрим реальный пример:

```cpp
string frameworkPath = 
  Path.Combine(tmpRootDirectory, frameworkPathPattern);
string manifestFile = 
  Path.Combine(frameworkPath, "sdkManifest.xml");

string frameworkPath2 = 
  Path.Combine(tmpRootDirectory, frameworkPathPattern2);
string manifestFile2 = 
  Path.Combine(frameworkPath, "sdkManifest.xml");
```

В данном коде закралась ошибка copy\-paste, которую сложно заметить на первый взгляд\. На самом деле, при получении сроки 'manifestFile2', ошибочно передали в функцию 'Path\.Combine' первую часть пути\. Согласно логике, в коде должна использоваться переменная 'frameworkPath2', а не 'frameworkPath', которая там сейчас находится\.

Исправленный код должен выглядеть так:

```cpp
string manifestFile2 = 
  Path.Combine(frameworkPath2, "sdkManifest.xml");
```