5

我收到以下消息：

警告：字段永远不会分配给，并且始终具有其默认值 null。

我的代码看起来像（它被简化了，所以没用）：

public class MyEntity
{
    // ...
    public string MyProp { get; set; }
}

public class MyClass
{
    string dbMyProp;

    public string MyProp { get { return dbMyProp.Replace("a", "b"); } }

    public static readonly Expression<Func<MyEntity, MyClass>> FromMyEntity = e => new MyClass
    {
        dbMyProp = e.MyProp // ...
    };
}

我认为这个消息是不正确的。

它是 C# 编译器中的错误还是我错过了什么？

更新该字段是dbMyProp。它已简化，但仍会产生此警告。

UPDATE2以下代码不会产生这样的警告：

public class MyClass2
{
    string dbMyProp;

    public string MyProp { get { return dbMyProp.Replace("a", "b"); } }

    public static MyClass2 FromMyEntity(MyEntity e)
    {
        return new MyClass2 
        {
            dbMyProp = e.MyProp // ...
        };
    }
}
4

4 回答 4

6

一个Expression 不是代码。这是意图；因此，在编译器级别，确实可以说没有代码实际上曾经分配过该字段。有一个MemberAssignment(from Expression.Bind)，但那是无关的。

如果表达式被编译，那只会通过实际的字段分配。这将是运行时的反射，编译器不会尝试检测。

这是编译时的实际情况：

static MyClass()
{
    ParameterExpression CS$0$0000;
    FromMyEntity = Expression.Lambda<Func<MyEntity, MyClass>>(
      Expression.MemberInit(
        Expression.New(
          (ConstructorInfo)methodof(MyClass..ctor),
          new Expression[0]
        ),
        new MemberBinding[] {
          Expression.Bind(
            fieldof(MyClass.dbMyProp),
            Expression.Property(
              CS$0$0000 = Expression.Parameter(typeof(MyEntity), "e"),
              (MethodInfo)methodof(MyEntity.get_MyProp)
            )
          )
        }
      ),
      new ParameterExpression[] {
        CS$0$0000
      }
    );
}

请注意没有对字段进行任何分配；只需使用虚构的fieldof运算符（意​​思是：它直接嵌入字段句柄 - 它不使用string名称的 a ）。

按照同样的逻辑，如果我们将其设为委托，警告就会消失：

public static readonly Func<MyEntity, MyClass> FromMyEntity = e => new MyClass
{
    dbMyProp = e.MyProp // ...
};

现在这是代码；它不仅仅是一个显示“如果我被编译，这就是我会做的事情”的对象模型。

于 2012-09-21T09:11:22.197 回答
1

是的，这似乎是编译器的一个缺点。它显然无法看穿（不包括）表达式。
以下按预期打印“bbc”：

var exp = MyClass.FromMyEntity.Compile();
var mc = exp(new MyEntity { MyProp = "abc"});
Console.WriteLine(mc.MyProp);

我尝试了一些技巧，例如 MyClass 上的私有构造函数，但警告仍然存在。

似乎编译器应用了更保守的算法，只检查是否有任何（正常）成员分配给dbMyProp.

您会因为名称中有 2 个不同的元素而混淆事物，dbMyProp但MyEntity.MyProp这里从未写过。

于 2012-09-21T09:10:39.377 回答
0

这不是编译器中的错误。

请。看看马克·格雷维尔的答案

下面的解释是不正确的。

显示警告是因为在以下场景dbMyProp中将使用未分配的值。1. MyClass 被实例化。2.使用实例MyProp属性被调用。（这将导致NullReferenceException）

于 2012-09-21T09:09:03.053 回答
0

还是我错过了什么？

你错过了一些东西。在 lambda 表达式中，您创建了一个新实例MyClass并分配了它的dbMyProp字段。但是您从未在实例方法中分配当前dbMyProp实例的字段。

看看这个例子：

var instance = new MyClass();
var myProp = instance.MyProp; // BOOM - you never assigned a value to the dbMyProp field
于 2012-09-21T09:09:57.647 回答