式ツリーラムダには、null伝播演算子を含めることはできません


96

質問price = co?.price ?? 0,次のコードの行は、上記のエラーを表示します。しかし、私は削除する場合?からco.?、それは罰金に動作します。私は、彼らがオンラインで使用しているこのMSDNの例に従おうとしてい?ました。select new { person.FirstName, PetName = subpet?.Name ?? String.Empty };したがって、いつ使用?するか??、いつ使用しないかを理解する必要があるようです。

エラー

式ツリーラムダには、null伝播演算子を含めることはできません

public class CustomerOrdersModelView
{
    public string CustomerID { get; set; }
    public int FY { get; set; }
    public float? price { get; set; }
    ....
    ....
}
public async Task<IActionResult> ProductAnnualReport(string rpt)
{
    var qry = from c in _context.Customers
              join ord in _context.Orders
                on c.CustomerID equals ord.CustomerID into co
              from m in co.DefaultIfEmpty()
              select new CustomerOrdersModelView
              {
                  CustomerID = c.CustomerID,
                  FY = c.FY,
                  price = co?.price ?? 0,
                  ....
                  ....
              };
    ....
    ....
 }

...エラーを投稿してください
ウィレム・ヴァンOnsem

3
C#がこれをサポートしたかった男!
nawfal 2018

回答:


150

引用元の例では、クエリ内の暗黙のラムダ式がデリゲートに変換されるLINQ to Objectsを使用していますが、クエリIQueryable<T>でラムダ式が式ツリーに変換されるEFなどを使用しています。式ツリーは、null条件演算子(またはタプル)をサポートしていません。

古い方法でやってください:

price = co == null ? 0 : (co.price ?? 0)

(式ツリーでは、null合体演算子で問題ないと思います。)


Dynamic LINQ(System.Linq.Dynamic.Core)を使用している場合は、このnp()メソッドを使用できます。github.com/StefH/System.Linq.Dynamic.Core/wiki/NullPropagation
StefHeyenrath19年

11

リンク先のコードはを使用しますList<T>List<T>実装しますIEnumerable<T>が、しませんIQueryable<T>。その場合、プロジェクションはメモリ内で実行され、?.機能します。

いくつかを使用していますがIQueryable<T>、動作が大きく異なります。の場合IQueryable<T>、プロジェクションの表現が作成され、LINQプロバイダーが実行時にそれをどのように処理するかを決定します。下位互換性の理由から、?.ここでは使用できません。

LINQプロバイダーによっては、プレーンを使用できる場合がありますが.、それでも取得できませんNullReferenceException


@hvd下位互換性のためにこれが必要な理由を説明できますか?
ジャグ2017年

1
@jagの導入前にすでに作成されていたすべてのLINQプロバイダーは、合理的な方法で?.処理?.する準備ができていなかったでしょう。

1
しかし、?.新しい演算子はありませんか?したがって、古いコードは使用?.されないため、壊れることはありません。Linqプロバイダーは、CLRメソッドなどの他の多くのものを処理する準備ができていません。
ジャグ2017

2
@jagそうです、古いLINQプロバイダーと組み合わせた古いコードは影響を受けません。古いコードはを使用しません?.。新しいコードは、(例外をスローすることによって)認識しないCLRメソッドを処理する準備ができている古いLINQプロバイダーを使用している可能性があります。これ、それらが既存の式ツリーオブジェクトモデルにうまく適合するためです。全く新しい式ツリーノード型に収まらない。

3
「我々はできるということはないと思いますので、私たちはサポートされないように使用される」 -すでにLINQプロバイダによってスローされた例外の数を考えると、ほとんど価値のあるトレードオフ思わない
NetMage

1

Jon Skeetの答えは正しかった、私の場合、私はDateTimeEntityクラスに使用していた。のように使おうとしたら

(a.DateProperty == null ? default : a.DateProperty.Date)

エラーが発生しました

Property 'System.DateTime Date' is not defined for type 'System.Nullable`1[System.DateTime]' (Parameter 'property')

そのDateTime?ため、エンティティクラスを変更する必要がありました。

(a.DateProperty == null ? default : a.DateProperty.Value.Date)

これは、帰無仮説演算子に関するものではありません。
GertArnold20年

ジョン・スキートが正しかったとあなたが言っているのが好きです。彼が間違っている可能性があることを示唆しています。いいね!
Klicker

0

式ツリーはC#6.0のnull伝播をサポートしていませんが、オペレーターが行うのと同じように、安全なnull伝播のために式ツリーを変更するビジターを作成することができます。

これが私のものです:

public class NullPropagationVisitor : ExpressionVisitor
{
    private readonly bool _recursive;

    public NullPropagationVisitor(bool recursive)
    {
        _recursive = recursive;
    }

    protected override Expression VisitUnary(UnaryExpression propertyAccess)
    {
        if (propertyAccess.Operand is MemberExpression mem)
            return VisitMember(mem);

        if (propertyAccess.Operand is MethodCallExpression met)
            return VisitMethodCall(met);

        if (propertyAccess.Operand is ConditionalExpression cond)
            return Expression.Condition(
                    test: cond.Test,
                    ifTrue: MakeNullable(Visit(cond.IfTrue)),
                    ifFalse: MakeNullable(Visit(cond.IfFalse)));

        return base.VisitUnary(propertyAccess);
    }

    protected override Expression VisitMember(MemberExpression propertyAccess)
    {
        return Common(propertyAccess.Expression, propertyAccess);
    }

    protected override Expression VisitMethodCall(MethodCallExpression propertyAccess)
    {
        if (propertyAccess.Object == null)
            return base.VisitMethodCall(propertyAccess);

        return Common(propertyAccess.Object, propertyAccess);
    }

    private BlockExpression Common(Expression instance, Expression propertyAccess)
    {
        var safe = _recursive ? base.Visit(instance) : instance;
        var caller = Expression.Variable(safe.Type, "caller");
        var assign = Expression.Assign(caller, safe);
        var acess = MakeNullable(new ExpressionReplacer(instance,
            IsNullableStruct(instance) ? caller : RemoveNullable(caller)).Visit(propertyAccess));
        var ternary = Expression.Condition(
                    test: Expression.Equal(caller, Expression.Constant(null)),
                    ifTrue: Expression.Constant(null, acess.Type),
                    ifFalse: acess);

        return Expression.Block(
            type: acess.Type,
            variables: new[]
            {
                caller,
            },
            expressions: new Expression[]
            {
                assign,
                ternary,
            });
    }

    private static Expression MakeNullable(Expression ex)
    {
        if (IsNullable(ex))
            return ex;

        return Expression.Convert(ex, typeof(Nullable<>).MakeGenericType(ex.Type));
    }

    private static bool IsNullable(Expression ex)
    {
        return !ex.Type.IsValueType || (Nullable.GetUnderlyingType(ex.Type) != null);
    }

    private static bool IsNullableStruct(Expression ex)
    {
        return ex.Type.IsValueType && (Nullable.GetUnderlyingType(ex.Type) != null);
    }

    private static Expression RemoveNullable(Expression ex)
    {
        if (IsNullableStruct(ex))
            return Expression.Convert(ex, ex.Type.GenericTypeArguments[0]);

        return ex;
    }

    private class ExpressionReplacer : ExpressionVisitor
    {
        private readonly Expression _oldEx;
        private readonly Expression _newEx;

        internal ExpressionReplacer(Expression oldEx, Expression newEx)
        {
            _oldEx = oldEx;
            _newEx = newEx;
        }

        public override Expression Visit(Expression node)
        {
            if (node == _oldEx)
                return _newEx;

            return base.Visit(node);
        }
    }
}

次のテストに合格します。

private static string Foo(string s) => s;

static void Main(string[] _)
{
    var visitor = new NullPropagationVisitor(recursive: true);

    Test1();
    Test2();
    Test3();

    void Test1()
    {
        Expression<Func<string, char?>> f = s => s == "foo" ? 'X' : Foo(s).Length.ToString()[0];

        var fBody = (Expression<Func<string, char?>>)visitor.Visit(f);

        var fFunc = fBody.Compile();

        Debug.Assert(fFunc(null) == null);
        Debug.Assert(fFunc("bar") == '3');
        Debug.Assert(fFunc("foo") == 'X');
    }

    void Test2()
    {
        Expression<Func<string, int>> y = s => s.Length;

        var yBody = visitor.Visit(y.Body);
        var yFunc = Expression.Lambda<Func<string, int?>>(
                                    body: yBody,
                                    parameters: y.Parameters)
                            .Compile();

        Debug.Assert(yFunc(null) == null);
        Debug.Assert(yFunc("bar") == 3);
    }

    void Test3()
    {
        Expression<Func<char?, string>> y = s => s.Value.ToString()[0].ToString();

        var yBody = visitor.Visit(y.Body);
        var yFunc = Expression.Lambda<Func<char?, string>>(
                                    body: yBody,
                                    parameters: y.Parameters)
                            .Compile();

        Debug.Assert(yFunc(null) == null);
        Debug.Assert(yFunc('A') == "A");
    }
}
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.