Оглавление
V3086. Variables are initialized through the call to the same function. It's probably an error or un-optimized code.
Анализатор обнаружил потенциальную ошибку, найдя в коде инициализацию двух различных переменных одинаковыми выражениями. Анализатор считает опасными не все выражения, а только в которых используется вызов функций (либо слишком длинное выражение).
Рассмотрим наиболее простой случай:
x = X();
y = X();Возможны три варианта действий:
- Код содержит ошибку. Необходимо исправить ошибку, заменив 'X()' на 'Y()'.
- Код верен, но работает медленно. Если функция 'X()' требует много вычислений, то лучше написать 'y = x;'.
- Код верен и работает быстро. Или функция 'X()' читает значение из файла. Тогда чтобы избавиться от ложного срабатывания, можно использовать комментарий "//-V3086".
Теперь рассмотрим реальный пример:
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', которая там сейчас находится.
Исправленный код должен выглядеть так:
string manifestFile2 =
Path.Combine(frameworkPath2, "sdkManifest.xml");Была ли полезна эта страница документации?
