﻿# V688\. The 'foo' local variable has the same name as one of class members\. This can result in confusion\.

Анализатор обнаружил, что имя локальной переменной совпадает с именем члена класса\. Часто, это не ошибка\. Однако, такой код ОЧЕНЬ опасен\. В процессе рефакторинга такого кода, в нём очень легко возникают ошибки\. Программист думает, что работает с членом класса, а на самом деле использует локальную переменную\.

Пример подозрительного кода:

```cpp
class M
{
  int x;
  void F() { int x = 1; foo(x); }
  ....
};
```

Класс содержит член с именем 'x'\. С таким же именем объявляется локальная переменная в функции F\(\)\.

Когда функция маленькая как в этом примере, всё просто и понятно\. И диагностика V688 кажется неинтересной\. Но когда функции большие, неудачное именование переменных может доставить массу головной боли разработчикам, сопровождающим код\.

Можно исправить ситуацию, используя другое имя для локальной переменной:

```cpp
class M
{
  int x;
  void F() { int value = 1; foo(value); }
  ....
};
```

Или можно использовать прифекс 'm\_' для обозначения членов класса:

```cpp
class M
{
  int m_x;
  void F() { int x = 1; foo(x); }
  ....
};
```

Анализатор выдаёт предупреждение не всегда\. Он использует некоторые эвристики, чтобы предотвратить ложные срабатывания\. Например, этот код анализатор посчитает безопасным:

```cpp
class M
{
  int value;
  void SetValue(int value) { this->value = value; }
  ....
};
```