Discussion :

[vduvernet]

Bonsoir,

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[pascalCH]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[vduvernet]

je trouvais ça un peu violent. Et p-e aurais-je ré-inventé la roue

[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.