Verschachteln Sie generische Typen nicht in Elementsignaturen

c# expression-trees linq

Frage

Ich habe eine Erweiterungsklasse erworben, die die folgenden Elemente mit Signaturen implementiert, die gegen die Regel CA1006: DoNotNestGenericTypesInMemberSignatures verstoßen.

Der Code, auf den sich die Warnung bezieht, ist unten aufgeführt.

Wie sollte ich den Code umwandeln, um die CA1006-Warnung zu beheben?

Bitte denken Sie daran, dass ich mit Ausdrucksbäumen nicht sehr vertraut bin, obwohl ich ziemlich gut mit anonymen Methoden, Delegaten und Lambda vertraut bin.

Jede Hilfe wird sehr geschätzt.

    public static DataServiceQuery<TElement> Expand<TElement, TPropType>(this DataServiceQuery<TElement> source, Expression<Func<TElement, TPropType>> propertySelector) 
    {
        string includeString = BuildString(propertySelector);
        return source.Expand(includeString);
    }

    private static string BuildString(Expression propertySelector)
    {
        switch (propertySelector.NodeType)
        {
            case ExpressionType.Lambda:
                LambdaExpression lambdaExpression = (LambdaExpression)propertySelector;
                return BuildString(lambdaExpression.Body);

            case ExpressionType.Quote:
                UnaryExpression unaryExpression = (UnaryExpression)propertySelector;
                return BuildString(unaryExpression.Operand);

            case ExpressionType.MemberAccess:

                MemberExpression memberExpression = (MemberExpression)propertySelector;
                MemberInfo propertyInfo = memberExpression.Member;

                if (memberExpression.Expression is ParameterExpression)
                {
                    return propertyInfo.Name;
                }
                else
                {
                    // we've got a nested property (e.g. MyType.SomeProperty.SomeNestedProperty)
                    return BuildString(memberExpression.Expression) + "/" + propertyInfo.Name;
                }

            case ExpressionType.Call:
                MethodCallExpression methodCallExpression = (MethodCallExpression)propertySelector;
                if (IsSubInclude(methodCallExpression.Method)) // check that it's a SubInclude call
                {
                    // argument 0 is the expression to which the SubInclude is applied (this could be member access or another SubInclude)
                    // argument 1 is the expression to apply to get the included property
                    // Pass both to BuildString to get the full expression
                    return BuildString(methodCallExpression.Arguments[0]) + "/" +
                           BuildString(methodCallExpression.Arguments[1]);
                }
                // else drop out and throw
                break;
        }
        throw new InvalidOperationException("Expression must be a member expression or an SubInclude call: " + propertySelector.ToString());

    }

    private static readonly MethodInfo[] SubIncludeMethods;
    static MyExtensions()
    {
        Type type = typeof(MyExtensions);
        SubIncludeMethods = type.GetMethods().Where(mi => mi.Name == "SubExpand").ToArray();
    }

    private static bool IsSubInclude(MethodInfo methodInfo)
    {
        if (methodInfo.IsGenericMethod)
        {
            if (!methodInfo.IsGenericMethodDefinition)
            {
                methodInfo = methodInfo.GetGenericMethodDefinition();
            }
        }
        return SubIncludeMethods.Contains(methodInfo);
    }

    public static TPropType SubExpand<TSource, TPropType>(this Collection<TSource> source, Expression<Func<TSource, TPropType>> propertySelector)
        where TSource : class
        where TPropType : class
    {
        throw new InvalidOperationException("This method is only intended for use with DataServiceQueryExtensions.Expand to generate expressions trees"); // no actually using this - just want the expression! 
    }

    public static TPropType SubExpand<TSource, TPropType>(this TSource source, Expression<Func<TSource, TPropType>> propertySelector)
        where TSource : class
        where TPropType : class
    {
        throw new InvalidOperationException("This method is only intended for use with DataServiceQueryExtensions.Expand to generate expressions trees"); // no actually using this - just want the expression! 
    }

Akzeptierte Antwort

Die Warnung ist eine allgemeine Warnung, die Ihnen helfen soll, eine bessere und einfachere öffentliche Schnittstelle zu erstellen. In diesem Fall erhalten Sie eine Warnung darüber, dass in Ihrer Methode ein Expression<Func<TElement, TPropType>> -Parameter vorhanden ist. Für diese Methode hat es jedoch keinen Sinn, den Typ zu vereinfachen, stattdessen sollten Sie die Warnung mithilfe eines Attributs unterdrücken oder die Regel vollständig aus Ihrem Regelsatz entfernen.


Ein dummes Beispiel, bei dem Sie wahrscheinlich den Rat der Regel befolgen sollten, ist eine Methode wie diese:

public void F(Dictionary<String, List<Tuple<String, Int32>>> dictionary);


Lizenziert unter: CC-BY-SA with attribution
Nicht verbunden mit Stack Overflow
Ist diese KB legal? Ja, lerne warum
Lizenziert unter: CC-BY-SA with attribution
Nicht verbunden mit Stack Overflow
Ist diese KB legal? Ja, lerne warum