为什么 SynchronizationContext 检查为空？

Question

InstallIfNeeded method 的 WindowsFormsSynchronizationContext 类，在 try 块中：

SynchronizationContext currentContext = AsyncOperationManager.SynchronizationContext;
//Make sure we either have no [....] context
//or that we have one of type SynchronizationContext

if (currentContext == null || currentContext.GetType() == typeof(SynchronizationContext))
{
...

首先，调用AsyncOperationManager.SynchronizationContext getter，并检查其返回值是否为空。这里需要检查 null 吗？

AsyncOperationManager.SynchronizationContext 的代码如下。它首先检查当前同步上下文是否为空，如果是，则创建一个新的。所以可能这个 getter 永远不会返回 null。

public static SynchronizationContext SynchronizationContext 
{
get 
{
    if (SynchronizationContext.Current == null) 
    {
        SynchronizationContext.SetSynchronizationContext(new SynchronizationContext());
    }
    return SynchronizationContext.Current;
}

Answer 1

这里有必要检查 null 吗？

我之前的回答忽略了SynchronizationContext.Current 实际上将基于线程而不是跨多个线程创建的事实（感谢@PerSerAl 指出这一点）。很明显你可以在代码中看到：

// Get the current SynchronizationContext on the current thread
public static SynchronizationContext Current 
{
    get      
    {
        return Thread.CurrentThread.GetExecutionContextReader().SynchronizationContext ??
               GetThreadLocalContext();
    }
}

实际上，这确实使null 检查变得多余，但它本身不会受到将来可能发生的任何未来更改的影响，以围绕SynchronizationContext 实现进行编码。

Answer 2

使用现在的确切代码，你是对的。

但是，请查看代码的意图。关键是安装winforms上下文是安全的唯一一点是没有其他同步上下文。这对应于具有null 同步上下文或默认SynchronizationContext。

同步上下文永远可能是null 在当前代码的精确版本中这一事实在很大程度上是无关紧要的。诚然，这可能不会改变，因为它是一个公共静态接口，但取决于它是完全不必要的依赖。您希望您的代码尽可能清楚地表明其意图，而此检查正是这样做的 - 同时还避免直接依赖于公共接口的内部行为。

Answer 3

这就是我所看到的（前两位与您的问题相同）：

SynchronizationContext currentContext = AsyncOperationManager.SynchronizationContext;
            //Make sure we either have no [....] context or that we have one of type SynchronizationContext
            if (currentContext == null || currentContext.GetType() == typeof(SynchronizationContext)) {

然后转到：

public static SynchronizationContext SynchronizationContext {
        get {
            if (SynchronizationContext.Current == null) {
                SynchronizationContext.SetSynchronizationContext(new SynchronizationContext());
            }

            return SynchronizationContext.Current;
        }

然后导致：in the line (SynchronizationContext.SetSynchronizationContext(new SynchronizationContext());

public static void SetSynchronizationContext(SynchronizationContext syncContext)
    {
        ExecutionContext ec = Thread.CurrentThread.GetMutableExecutionContext();
        ec.SynchronizationContext = syncContext;
        ec.SynchronizationContextNoFlow = syncContext;
    }

然后它返回SynchronizationContext.Current

public static SynchronizationContext Current 
    {
        get      
        {
            return Thread.CurrentThread.GetExecutionContextReader().SynchronizationContext ?? GetThreadLocalContext();
        }
    }

哪些可能在这里结束。

private static SynchronizationContext GetThreadLocalContext()
    {
        SynchronizationContext context = null;

#if FEATURE_APPX
        if (context == null && Environment.IsWinRTSupported)
            context = GetWinRTContext();
#endif

        return context;
    }

如您所见，如果IsWinRTSupported 返回 false，context 可以为 null。

根据我所看到的，空检查可能是个好主意。我不知道第一个条件 Thread.CurrentThread.GetExecutionContextReader().SynchronizationContext 是否可以返回 null 但考虑到 ?? 运算符，这是有可能的。