Discussion :

[Gat-]

Bonjour,

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

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

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 :

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

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);
 
    [...]
}

[Gat-]

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

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

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

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.

[Gat-]

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

Code :

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

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