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

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

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

    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'. В данном случае возможны два варианта исправления данной конструкции (в обоих случаях на наличие ключа проверяется тот словарь, куда производится добавление нового элемента):

    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);
    }