[CustomException] FxCop indique une erreur étrange

Version imprimable

Bonjour,

J'ai testé l'utilisation d'une Expression (System.Linq.Expressions) dans le cas d'une exception personnalisée :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 [Serializable] public PropertyException<T> : Exception, ISerializable { public PropertyException(Expression<Func<T, object>> propertySelector) : base(GetMessage(propertySelector)) { } static string GetMessage(Expression<Func<T, object>> propertySelector) { MemberExpression memberExpr = (MemberExpression)propertySelector.Body; return String.Format(CultureInfo.InvariantCulture, MyResources.PropertyErrorMessage, memberExpr.Member.Name); // propertyName } [...] }
Cela compile mais FxCop m'indique une erreur critique "DoNotNestGenericTypesInMemberSignatures".
La description m'indique le texte suivant :

Citation:

Resolution : "Consider a design where
'PropertyException<T>.PropertyException(Expression<Func<T, object>>)'
doesn't nest generic type 'Expression<Func<T, object>>'."

Info : "Avoid API that require users to instantiate a generic
type with another generic type as type argument. The
syntax gets too complex."

Je sais qu'FxCop n'est là que pour aider/conseiller lors du developpement.
Mais je ne comprends pas en quoi la syntaxe est trop complexe ?

A l'utilisation on obtient un code plutôt simple je trouve :
Code:

1 2 3 4 5 6 7 8 9 10 public string DoSomething(MyClass myObject) { if (myObject == null) throw new ArgumentNullException("myObject"); if (myObject.Id == null) throw new PropertyNullException<MyClass>(o => o.Id); [...] }

Pour retirer l'erreur critique, j'ai modifié le code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
[Serializable]
public PropertyNullException<TDelegate> : Exception, ISerializable
{
    public PropertyNullException(Expression<TDelegate> propertySelector)
        : base(GetMessage(propertySelector)) { }
 
    static string GetMessage(Expression<TDelegate> propertySelector)
    {
        MemberExpression memberExpr = (MemberExpression)propertySelector.Body;
 
        return String.Format(CultureInfo.InvariantCulture,
                             MyResources.PropertyErrorMessage,
                             memberExpr.Member.Name); // propertyName
    }
    [...]
}

Mais à l'utilisation, je trouve ça justement plus complexe qu'auparavant :

Code:

1
2
3
4
5
6
7
8
9
10
public string DoSomething(MyClass myObject)
{
    if (myObject == null)
        throw new ArgumentNullException("myObject");
 
    if (myObject.Id == null)
        throw new PropertyNullException<Func<MyClass, Guid>>(o => o.Id);
 
    [...]
}

Bref voiloù, c'est résolu.

Je ne sais pas si c'est utile mais j'ai rajouté une condition au type TDelegate :
Code:

1 2 3 4 5 6 [Serializable] public PropertyNullException<TDelegate> : Exception, ISerializable where TDelegate : LambdaExpression { [...] }