同时从方法返回对象引用和异常

Question

我只是尝试在 WinForms 中实现单例模式，以便在应用程序生命中仅保留一个表单实例，但我面临一个困难，

如果单例实例存在并同时返回相同的实例引用，我想抛出异常。

SingletonForm.cs

public class SingletonForm : BaseFormcs
{
    private static SingletonForm _instance;
    //To stop new keyword from instantiation 
    private SingletonForm()
    { }
    public static SingletonForm GetInstance()
    {
        if (_instance == null)
            return _instance = new SingletonForm();

        else
        {
            throw new Exception("Form already exists"); // execution returns from here
            return _instance; // Warning : Unreachable code detected
            //I also want to return instance reference.
        }

    }
}

Answer 1

您要么抛出异常，要么返回一个实例。对于单例，不应抛出异常，只需返回实例（如果存在）。

Singleton 模式不应该阻止您（甚至警告您）多次调用 GetInstance。它应该只返回第一次创建的相同实例。

我只是想也许有时我可能需要同时使用两者
这就是我问的原因

“抛出异常立即从函数返回”，因为这意味着发生了意外错误。在另一种情况下，您可能想抛出异常，但前提是某些条件为真（例如参数验证失败）。否则，您将返回一个值。这是一个例子：

public int SomeFunction(String someArgument)
{
    if (someArgument == null) throw new ArgumentNullException("someArgument");
    int retVal = 0;
    //Some code here
    return retVal;
}

Answer 2

从设计的角度来看，不，你不需要。异常应该代表系统需要恢复的严重的、意外的错误。您正在谈论将其用作错误代码。如果您打算这样做，请不要抛出异常 - 通过在单例上设置标志或返回 null 或其他内容来指示问题。

但是，在您的具体情况下，只需删除异常逻辑即可。单例设计模式旨在成为全局变量的存储库，因此预计公共静态实例将被多次调用（事实上，单例的许多用户倾向于在其代码库中的几乎每个类中使用它）。

Answer 3

我正在假设您有一种情况，您实际上需要知道它是否是一个新实例，并且它将在返回后更改您的执行路径。一种方法是（尽管我不敢输入这个）使用异常（选项 1）。但您更有可能希望使用选项 2 来简单地根据返回值进行分支。

public class SingletonForm : BaseFormcs
{
    private static SingletonForm _instance;
    //To stop new keyword from instantiation 
    private SingletonForm()
    { }
    // ------- Option #1
    // Use an OUT parameter for the instance, so it's set before the exception
    public static void GetInstance(out SingletonForm form)
    {
        if (_instance == null)
        {
            _instance = new SingletonForm();
            form = _instance;
            return;
        }
        form = _instance;
        throw new Exception("Form already exists"); // execution returns from here
        // return isn't needed, since you threw an exception.
        // You really, really shouldn't do this. Consider instead...
    }

    // -------- Option #2
    // Same as above, but the return value tells you whether it's shiny and new
    public static bool GetInstance(out SingletonForm form)
    {
        if (_instance == null)
        {
            _instance = new SingletonForm();
            form = _instance;
            return true; // yes, you created a new one
        }
        form = _instance;
        return false; // no, you used an extant one
    }
}

第二个选项可能是您最好的选择，因为它更符合您在 Dictionary.TryGetValue(KEY key, out VALUE value) 中看到的内容。