Блог

Итак, я попытался создать некоторые базовые методы расширения для List . По сути, у меня есть UniqueAdd и UniqueAddRange. Он проверит наличие значения перед добавлением, и если оно уже есть в списке, оно его не добавит. Вот код:

 public static class ListExtensions
{
    /// <summary>
    /// Adds only the values in the 'values' collection that do not already exist in the list. Uses list.Contains() to determine existence of
    /// previous values.
    /// </summary>
    /// <param name="list"></param>
    /// <param name="values"></param>
    public static void UniqueAddRange<T>(this List<T> list, IEnumerable<T> values)
    {
        foreach (T value in values)
        {
            list.UniqueAdd(value);
        }
    }

    /// <summary>
    /// Adds the value to the list only if it does not already exist in the list. Uses list.Contains() to determine existence of previos values.
    /// </summary>
    /// <typeparam name="T"></typeparam>
    /// <param name="list"></param>
    /// <param name="value"></param>
    public static void UniqueAdd<T>(this List<T> list, T value)
    {
        if (!list.Contains(value))
        {
            list.Add(value);
        }
    }
}
  

И я получаю следующую ошибку при сборке:

 CA0001 : Rule=Microsoft.Maintainability#CA1506, Target=Some.Namespace.ListExtensions : Collection was modified; enumeration operation may not execute.
  

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

Попробуйте изменить тип или метод, чтобы уменьшить количество типов, с которыми он связан.

Кто-нибудь знает, почему я получаю эту ошибку и как исправить мои методы расширения, чтобы они не нарушали это правило?

Спасибо!

PS: Прежде чем кто-либо упомянет об этом, я рассмотрел возможность использования HashSet, но HashSet не существует в compact framework.

Я думаю, что ваш код вызвал ошибку в FxCop, «Коллекция была изменена» — это классический упс. Затем он решил, что его ошибка была вашей проблемой, стиль catch (исключение).

Ищите обновление. Тот, который я использую, не жалуется на ваш код (версия VS2010).

Он сообщает вам, что вы меняете список при его перечислении. Это ясно из вашего кода (вы добавляете в список одновременно с перечислением).

Как насчет:

 public static void UniqueAddRange<T>(this List<T> list, IEnumerable<T> values)
{
    list.AddRange(values.Except(list));
}
  

Или, если интерфейс подходит для ваших нужд, используйте Hashset . Он делает то, что вы хотите, прямо из коробки.