Choix du type d'Exception à lever

Version imprimable

20/02/2015, 21h57
vduvernet

Choix du type d'Exception à lever
Bonsoir,

suite à la levée de CA2201 lors de la compilation de mon code, je me demande comment corriger le problème.
Code:

1 2 3 4 5 if (length2 < 0) { throw new IndexOutOfRangeException(); } string Value = keyAndValue.Substring(length + 1, length2).Trim();
Le but est de remonter une exception si la longueur est nulle ou négative. Dans ce code, cette exception est réservée. Du coup, laquelle utiliser ?

Merci,

Vincent

Citation:

Envoyé par vduvernet

Code:

1
2
3
4
5
if (length2 < 0)
            {
                throw new IndexOutOfRangeException();
            }
string Value = keyAndValue.Substring(length + 1, length2).Trim();

bonjour,

pourquoi ne pas utiliser une Exception "sur mesure" ? :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
    [Serializable]
    public class NullOrNegativeException : Exception
    {
        public NullOrNegativeException() { }
        public NullOrNegativeException(string message) : base(message) { }
        public NullOrNegativeException(string message, Exception inner) : base(message, inner) { }
        protected NullOrNegativeException(
          System.Runtime.Serialization.SerializationInfo info,
          System.Runtime.Serialization.StreamingContext context)
            : base(info, context) { }
    }
 
if (length2 < 0)
            {
                throw new NullOrNegativeException ();
            }
string Value = keyAndValue.Substring(length + 1, length2).Trim();

21/02/2015, 00h08
vduvernet

je trouvais ça un peu violent. Et p-e aurais-je ré-inventé la roue
21/02/2015, 22h42
Pragmateek

Attention à l'overengineering ! :)

Il vaut mieux se payer un petit warning sans importance et conséquence (OK IndexOutOfRangeException n'est pas conçu pour ça à la base) que d'ajouter du code pour le contourner.
Quitte à le faire taire en le désactivant localement via l'attribut SuppressMessage.

Surtout que dans ce cas si length2 est un paramètre je pense que ArgumentOutOfRangeException ferait parfaitement l'affaire.