奇怪的空合并运算符自定义隐式转换行为

注意:这似乎已经修复了Roslyn

这个问题是在写我对这一个的回答时出现的,它讨论了空合并操作符的结合律。

提醒一下,空合并操作符的思想是窗体的表达式

x ?? y

首先计算x,然后:

  • 如果x的值为null,则计算y,这是表达式的最终结果
  • 如果x的值是非空的,则y是计算的,而x的值是表达式的最终结果,在必要时转换为y的编译时类型之后

现在,通常不需要转换,或者它只是从一个可空的类型到一个不可空的类型-通常类型是相同的,或者只是从(说)int?int。但是,您可以创建了自己的隐式转换运算符,并在必要时使用它们。

对于x ?? y这个简单的例子,我没有看到任何奇怪的行为。然而,对于(x ?? y) ?? z,我看到了一些令人困惑的行为。

这是一个简短但完整的测试程序-结果在评论中:

using System;


public struct A
{
public static implicit operator B(A input)
{
Console.WriteLine("A to B");
return new B();
}


public static implicit operator C(A input)
{
Console.WriteLine("A to C");
return new C();
}
}


public struct B
{
public static implicit operator C(B input)
{
Console.WriteLine("B to C");
return new C();
}
}


public struct C {}


class Test
{
static void Main()
{
A? x = new A();
B? y = new B();
C? z = new C();
C zNotNull = new C();


Console.WriteLine("First case");
// This prints
// A to B
// A to B
// B to C
C? first = (x ?? y) ?? z;


Console.WriteLine("Second case");
// This prints
// A to B
// B to C
var tmp = x ?? y;
C? second = tmp ?? z;


Console.WriteLine("Third case");
// This prints
// A to B
// B to C
C? third = (x ?? y) ?? zNotNull;
}
}

所以我们有三种自定义值类型,ABC,从A到B,从A到C,从B到C。

第二种情况和第三种情况我都能理解……但是为什么在第一种情况下有额外的A到B转换吗?特别是,我希望真的的第一种情况和第二种情况是一样的——毕竟,它只是将一个表达式提取到一个局部变量中。

有人知道发生了什么吗?当谈到c#编译器时,我非常犹豫要不要喊“bug”,但我被难住了,不知道发生了什么……

编辑:好吧,这里有一个正在发生的事情的更糟糕的例子,多亏了配置器的答案,这让我进一步有理由认为这是一个bug。编辑:示例现在甚至不需要两个空合并操作符了…

using System;


public struct A
{
public static implicit operator int(A input)
{
Console.WriteLine("A to int");
return 10;
}
}


class Test
{
static A? Foo()
{
Console.WriteLine("Foo() called");
return new A();
}


static void Main()
{
int? y = 10;


int? result = Foo() ?? y;
}
}

它的输出是:

Foo() called
Foo() called
A to int

事实上,Foo()在这里被调用了两次,这让我非常惊讶——我看不出表达式有任何理由被评估调用两次。

26625 次浏览

如果你看一下左分组情况下生成的代码,它实际上是这样做的(csc /optimize-):

C? first;
A? atemp = a;
B? btemp = (atemp.HasValue ? new B?(a.Value) : b);
if (btemp.HasValue)
{
first = new C?((atemp.HasValue ? new B?(a.Value) : b).Value);
}

另一个发现,如果你a0 first,它会生成一个快捷方式,如果ab都是空的,并返回c。然而,如果ab是非空的,它会重新计算a作为隐式转换到B的一部分,然后返回ab中哪一个是非空的。

来自c# 4.0规范,§6.1.4:

  • 如果可空转换是从S?T?:
    • 如果源值是null (HasValue属性是false),则结果是类型为T?null值。
    • 否则,转换将被评估为从S?展开到S,然后是从ST的底层转换,然后是从TT?的包装(§4.1.10)。
    • 李< / ul > < / >

这似乎解释了第二种展开-包裹组合。


c# 2008和2010编译器生成非常相似的代码,但是这看起来像是c# 2005编译器(8.00.50727.4927)的回归,后者为上述代码生成如下代码:

A? a = x;
B? b = a.HasValue ? new B?(a.GetValueOrDefault()) : y;
C? first = b.HasValue ? new C?(b.GetValueOrDefault()) : z;

我想知道这是不是由于给类型推断系统的额外魔法 ?

实际上,我现在用更清晰的示例将其称为bug。这仍然成立,但双重评估肯定不是好事。

似乎A ?? B被实现为A.HasValue ? A : B。在本例中,也有很多强制转换(遵循三元操作符?:的常规强制转换)。但如果你忽略所有这些,那么根据它的实现方式,这是有意义的:

  1. A ?? B扩展为A.HasValue ? A : B
  2. Ax ?? y。展开到x.HasValue : x ? y
  3. 替换所有出现的A -> (x.HasValue : x ? y).HasValue ? (x.HasValue : x ? y) : B

在这里您可以看到x.HasValue被选中两次,如果x ?? y需要强制转换,x将被强制转换两次。

要点:不要创建带有副作用的隐式强制转换操作符。

这似乎是围绕??如何实现的编译器错误。要点:不要让合并表达式带有副作用。

这绝对是一个bug。

public class Program {
static A? X() {
Console.WriteLine("X()");
return new A();
}
static B? Y() {
Console.WriteLine("Y()");
return new B();
}
static C? Z() {
Console.WriteLine("Z()");
return new C();
}


public static void Main() {
C? test = (X() ?? Y()) ?? Z();
}
}

这段代码将输出:

X()
X()
A to B (0)
X()
X()
A to B (0)
B to C (0)
这让我认为每个??联合表达式的第一部分被求值两次。 这段代码证明了这一点:

B? test= (X() ?? Y());

输出:

X()
X()
A to B (0)

这似乎只在表达式需要在两个可空类型之间进行转换时才会发生;我尝试了各种各样的排列,其中一边是一个字符串,没有一个导致这种行为。

我不是c#专家,你可以从我的问题历史中看到,但是,我尝试了一下,我认为这是一个bug....但作为一个新手,我不得不说我不明白这里发生的一切,所以如果我错了,我会删除我的答案。

我通过编写处理相同场景的不同版本的程序来得出这个bug的结论,但要简单得多。

我使用三个空整数属性与备份存储。我将每个设置为4,然后运行int? something2 = (A ?? B) ?? C;

(# EYZ0)

这个只读取A,其他什么都没有。

这句话在我看来应该是这样的:

  1. 从方括号开始,查看A,返回A,如果A不为空就结束。
  2. 如果A为空,则计算B,如果B不为空则结束
  3. 如果A和B为空,则计算C。

因此,由于A不为空,它只看A并结束。

在您的示例中,在First Case中放置断点表明x、y和z都不为空,因此,我希望它们与我的不太复杂的示例....相同但我担心我是一个c#新手,完全错过了这个问题的要点!

感谢所有为分析这个问题做出贡献的人。这显然是一个编译器错误。似乎只有在合并运算符左侧涉及两个可空类型的提升转换时才会发生这种情况。

我还没有确定到底哪里出了问题,但是在编译的“可空化降低”阶段的某个时候——在初始分析之后,但在代码生成之前——我们减少了表达式

result = Foo() ?? y;

从上面的例子到道德等价物:

A? temp = Foo();
result = temp.HasValue ?
new int?(A.op_implicit(Foo().Value)) :
y;

显然这是不正确的;正确的下放方式是

result = temp.HasValue ?
new int?(A.op_implicit(temp.Value)) :
y;

到目前为止,根据我的分析,我最好的猜测是可空优化器在这里偏离了轨道。我们有一个可空优化器,它可以查找我们知道可空类型的特定表达式不可能为空的情况。考虑一下下面的天真分析:我们可能首先会说

result = Foo() ?? y;

A? temp = Foo();
result = temp.HasValue ?
(int?) temp :
y;

然后我们可以说

conversionResult = (int?) temp

A? temp2 = temp;
conversionResult = temp2.HasValue ?
new int?(op_Implicit(temp2.Value)) :
(int?) null

但优化器可以介入并说:“哇,等一下,我们已经检查过temp不为空;没有必要仅仅因为我们调用了一个提升的转换运算符而第二次检查它为空”。我们让他们把它优化到

new int?(op_Implicit(temp2.Value))

我的猜测是,我们在某个地方缓存的事实是,(int?)Foo()的优化形式是new int?(op_implicit(Foo().Value)),但这实际上不是我们想要的优化形式;我们希望将Foo()的优化形式替换为-temporary-然后转换。

c#编译器中的许多错误都是错误缓存决策的结果。一言以蔽之:每次缓存一个事实以供以后使用时,如果相关的内容发生更改,就可能会造成不一致。在这种情况下,初始分析后的相关变化是,对Foo()的调用应该始终作为一个临时对象的取回来实现。

我们在c# 3.0中对可空重写传递做了大量的重组。这个错误在c# 3.0和4.0中重现,但在c# 2.0中没有,这意味着这个错误可能是我的错。对不起!

我将把一个错误输入到数据库中,我们将看看是否可以在语言的未来版本中修复这个问题。再次感谢大家的分析;这很有帮助!

更新:我重写了可空优化器从头罗斯林;它现在做得更好,避免了这些奇怪的错误。关于Roslyn中的优化器如何工作的一些想法,请参阅我的系列文章,从这里开始:https://ericlippert.com/2012/12/20/nullable-micro-optimizations-part-one/