使用 goto 的最佳实践

Question

这段代码中使用goto是否正确？还有其他选择吗？

return ExecuteReader(cmd, reader =>
{
    List<BEPartnership> partnerhip = null;

    //Partnership
    if (!((SqlDataReader) reader).HasRows)
        goto exit;

    partnerhip = 
        new List<BEPartnership>{new BEPartnership().GetFromReader(reader)};

    //Customers
    if (!reader.NextResult() && !((SqlDataReader) reader).HasRows)
        goto exit;

    foreach (BEPartnership p in partnerhip)
        p.Partner = new BECustomer().GetFromReader(reader);

    //Contracts
    if (!reader.NextResult() && !((SqlDataReader) reader).HasRows)
        goto exit;

    List<BEContractB2B> contracts = new List<BEContractB2B>();
    contracts.Add(new BEContractB2B().GetFromReader(reader));
    // contracts = new BEContractB2B().GetFromReader2(reader).ToList();

    exit:
    return partnerhip;
});

Answer 1

如果您希望返回当前填充的列表，则可以将每个 goto exit; 替换为 return null; 或 returnpartnership;。 （我认为合伙人是一个很酷的合伙人？）

Answer 2

我会说不。

我从 2001 年起就开始使用 C# 编程，从未使用过 goto！

如果您希望代码中出现“短路”退出，为什么不将其替换

goto exit:

为

return partnership

Answer 3

在我看来，goto 和“最佳实践”是相互排斥的（可能/希望大多数其他人也是如此）。需要 goto 表明代码设计有缺陷。就您而言，解决方案似乎很简单：我认为您只需将 goto exit 替换为 returnpartnership 并删除标签 exit: >。 （应该读“伙伴关系”而不是“伙伴关系”吗？）

Answer 4

您最后要做的就是从阅读器加载合同（如果存在）。如果你用一个简单的 if 语句明确表达这个意图，那么读起来会更好。

将结尾更改为：

if (reader.NextResult() || ((SqlDataReader) reader).HasRows)
{
    List<BEContractB2B> contracts = new List<BEContractB2B>();
    contracts.Add(new BEContractB2B().GetFromReader(reader));
}

return partnerhip;

虽然看起来您只是忽略了合同列表...但它没有做任何事情。除非创建一个新的 BEContractB2B 类有一些全局副作用（坏消息），否则您可以将其全部删除。

将第一个 goto 更改为

if (!((SqlDataReader) reader).HasRows)
    return null;

因为这就是您正在做的事情，所以您应该明显地表明您将返回 null。

Answer 5

我发现以下情况 goto 可能有点用： When To Use Goto When Programming in C。但是“永远不要使用 GOTO”是我在大学学到的第一件事，所以我真的从未使用过它（至少在 C、C++、C#、Java 等中没有使用过）。

GOTO 最大的问题是，如果你阅读一段方法，你看不到它是从哪里被调用的。例如：

int a = 1;
division:
int b = 4 / a;

...听起来不错。但是你写了 0-除法崩溃，如果除法块后面有以下 GOTO：

int a = 1;
division:
int b = 4 / a;
// ... hundreds of lines ...
a = 0;
goto division;

... 或者如果除法块之前有 GOTO 则空异常崩溃：

goto division;
// ... hundreds of lines ...
int a = 1;
division:
int b = 4 / a;

... 这只是一个例子，GOTO 会导致更多争议情况。因此，请忘记 GOTO，人们（包括您）在阅读您的代码时会更高兴。

使用“返回合作伙伴关系；”而不是你的 goto。