c# – 使用goto的最佳实践

在这段代码中使用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;
});

解决方法

你可以替换每个goto出口;返回null;或返回合作伙伴;如果您希望返回当前填充的列表. (我认为合作伙伴关系是一个很酷的合作伙伴？)