今天看到了这样一段代码，跟大家分享一下

[GordianZ]

if (A==null)
{
    doA();
}
else if (A!=null)
{
    if(B==0)
    {
        if(A==null)
        {
            doA();
        }
....
    }
}

我整个人都是懵逼的……

[EyreFree]

多线程条件下，进入 if (A!=null) 分支后在内层的 if(A==null) 执行前 A 变为 null 的话这段代码大概是有意义的吧...

当然我觉得这段代码的作者基本上不会有这样诡异的本意...

2333

[asd567800]

@EyreFree 对的，不过应该加锁啊

[beginor]

@EyreFree 这解释真高明！

[SoulSleep]

@EyreFree 你这个想法又涉及到线程安全问题.......显然能写出这种代码的人也应该想不到有这种情况(⊙o⊙)…

[bengxy]

估计是忘了注释吧 23333

[NxiJSiOS]

代码诡异，就往多线程想吗？ 233

[introom]

都 doA 了，else if 再检查一下也是没问题的。不过总体很怪异就是了。

[m939594960]

估计以前 代码逻辑可能是类似这样的，后来业务变动，删掉了那行吧。。。。
```
if (A==null)
{
doA();
}
else if (A!=null)
{
if(B==0)
{
a= fuck() < ----------------------
if(A==null)
{
doA();
}
....
}
}
```

[changwei]

我突然发现所有诡异的代码逻辑都可以用多线程来解释了

[sagaxu]

多线程环境下这种代码往往也是错的

[karia]

想起了 https://en.wikipedia.org/wiki/Double-checked_locking

[agostop]

估计 8 楼说的差不多

[deadEgg]

多线程这依然是竞态环境

不加锁依然赌命？

多线程不合理啊

[EyreFree]

我在讲段子啊，你萌为什么都当真...QWQ...

逃...

[rbe]

@m939594960 这都能分析出来，很强

[domty]

这种看起来无比诡异的代码主要就俩原因。
1.写代码的人是个坑
2.这段代码被删删改改了很多次而没有被认真重构过。

[seancheer]

多线程这种判断也是不能够保证的。。同意楼上的观点，要么写代码的人坑，要么这段代码被改来改去，改的人没仔细看其他的逻辑。

[GordianZ]

其实是当天写出来的，而且无关多线程，只是代码烂而已……
明天应该不用来上班了 😂