在这段代码中使用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;或返回合作伙伴;如果您希望返回当前填充的列表. (我认为合作伙伴关系是一个很酷的合作伙伴?)