Ce type d'exception doit être levée lorsque vous essayez d'ajouter des éléments en double pour une collection?

Code suivant doit jeter l'exception pour empêcher l'ajout double élément de la collection.

ICollection<T> collection = new List<T>();

public void Add(T item)
{
    if (collection.Contain(item))
    {
          throw new SomeExceptionType()
    }

    collection.Add(item);
}

Ce standard type d'exception est le plus approprié?

  • Plus proche d'exception, j'ai trouvé DuplicateNameException pour les bases de données ... qui est juste une mauvaise idée pour un Collection. Message à retenir: toute Personne voulant plus spécifiques pourraient rouler leur propre Exception
InformationsquelleAutor klashar | 2009-08-14
  1. 34

    Bien, Dictionary<,>.Add() jette ArgumentException si cette clé existe déjà, donc je suppose que cela pourrait être un précédent.

    InformationsquelleAutor Pavel Minaev
  2. 6

    ArgumentException serait probablement le meilleur. C'est l'exception levée lorsqu'un argument est invalide.

    InformationsquelleAutor Brandon
  3. 4

    Linq utilise deux exceptions DuplicateNameException et DuplicateKeyException vous pouvez les utiliser si vous utilisez le système.les données de l'assemblée.

    InformationsquelleAutor amarnath chatterjee
  4. 3

    Je voudrais utiliser InvalidOperationException:

    L'exception est levée lorsqu'un
    appel de la méthode n'est pas valide pour l'
    objet dans son état actuel.

    Puisque la validité de l'argument de la valeur dépend de l'état de l'objet (qui est de savoir si ou de ne pas collection.Contains(item) est vrai), je pense que c'est le meilleur exception à utiliser.

    Assurez-vous que vous ajoutez un bon message, à l'exception qu'il soit clair que le problème était à l'appelant.

    • À mon humble avis, InvalidOperationException de dérouter l'utilisateur parce qu'il a utilisé essentiellement pour montrer unapropriate oparation séquence. Par exemple, la lecture de fermé reader, etc.
    InformationsquelleAutor Andrew Hare
  5. 3

    ArgumentException serait la bonne exception (Dictionnaire utilise cette exception ainsi)

    InformationsquelleAutor cyberconte
  6. 1
    System.ArgumentException
    InformationsquelleAutor Ben M
  7. 1

    Je dirais InvalidOperationException, car il n'est pas possible d'ajouter un objet qui est déjà dans la collection

    • Si c'était un Dictionnaire de ce serait vrai, mais il ne l'est pas.
    • Il y a des collections qui ne permettent pas de doublons, ET les dictionnaires qui leur permettent... l'affiche est le cas il est évident qu'il ne veut pas de doublons, ou pourquoi veut-il lancer une exception ?
    • Désolé, j'ai formulé ce commentaire mal, j'ai plus voulu dire que si l'OP est à l'aide d'une collection qui n'autoriser les doublons, ce n'est pas vraiment une opération non valide. Ses un "il ne considère pas cela valide" chose. Aussi, je ne suis pas celui qui downvoted ce >_> je ne suis pas d'accord c'est le meilleur exception, mais je ne pense pas qu'il serait mal à l'utiliser.
    InformationsquelleAutor Thomas Levesque
  8. 1

    Je voudrais jeter un ArgumentException. C'est ce que le générique System.Collections.Generic.SortedList<> n'est dans ses Add méthode.

    De la .NET Framework 2.0:

        public void Add(TKey key, TValue value)
    {
        if (key == null)
        {
            System.ThrowHelper.ThrowArgumentNullException(System.ExceptionArgument.key);
        }
        int num = Array.BinarySearch<TKey>(this.keys, 0, this._size, key, this.comparer);
        if (num >= 0)
        {
            System.ThrowHelper.ThrowArgumentException(System.ExceptionResource.Argument_AddingDuplicate);
        }
        this.Insert(~num, key, value);
    }
    InformationsquelleAutor Paul Williams