# V3124\. Appending an element and checking for key uniqueness is performed on two different variables\.

Анализатор обнаружил подозрительный фрагмент кода, в котором производится проверка на наличие ключа в одном словаре, после чего новый элемент добавляется в другой словарь\. Возможно, допущена опечатка или логическая ошибка\.

Рассмотрим пример:

```cpp
Dictionary<string, string> dict = new Dictionary<string, string>();
Dictionary<string, string> _dict = new Dictionary<string, string>();
....
void Add(string key, string val)
{
  if (!dict.ContainsKey(key))
    _dict.Add(key, val);
}
```

Здесь, вероятно, допущены сразу две ошибки\. Первая \- добавление элемента не в тот словарь, что может исказить логику работы программы\. Вторая ошибка связана с тем, что проверка наличия ключа 'key' выполняется для словаря 'dict', а не для '\_dict'\. Если словарь '\_dict' уже содержит значение для ключа 'key', в момент выполнения '\_dict\.Add\(key, val\)' будет выброшено исключение 'ArgumentException'\. В данном случае возможны два варианта исправления данной конструкции \(в обоих случаях на наличие ключа проверяется тот словарь, куда производится добавление нового элемента\):

```cpp
Dictionary<string, string> dict = new Dictionary<string, string>();
Dictionary<string, string> _dict = new Dictionary<string, string>();
....
void Add1(string key, string val)
{
  if (!_dict.ContainsKey(key))
    _dict.Add(key, val);
}
...
void Add2(string key, string val)
{
  if (!dict.ContainsKey(key))
    dict.Add(key, val);
}
```