在我正在编写的一个C#中,我需要以相同的方式处理具有相同签名的多个方法.此外,将来可能会有更多这些方法.我没有反复重复同样的逻辑,而是想到了以下几点:
private delegate bool cleanStep(BuildData bd, out String strFailure); Liststeps = new List (); steps.Add(WriteReadme); steps.Add(DeleteFiles); steps.Add(TFSHelper.DeleteLabel); steps.Add(TFSHelper.DeleteBuild); List .Enumerator enumerator = steps.GetEnumerator(); bool result = true; while (result && enumerator.MoveNext()) { result = enumerator.Current.Invoke(build, out strFailure); if (!result) { logger.Write(LogTypes.Error, strFailure); } }
我认为这有一些很好的功能,但它也有点过于引擎和混淆.
你能感谢一个更好的方法吗?
BTW:
它不需要是交易的.
strFailure不会隐藏在必要时将其完全包装起来的异常
谢谢.
为什么不使用foreach循环而只是休息?(我已经改名cleanStep
到CleanStep
这里套子-我建议你这样做.)
foreach(CleanStep step in steps) { string failureText; if (!step(build, out failureText)) { logger.Write(LogTypes.Error, strFailure); break; } }
请注意,这也遵守IEnumerator
您当前代码所不具备的合同- foreach
自动调用Dispose
和IEnumerator
实现IDisposable
.在这种情况下它不会成为问题,但是对于迭代器块,处理用于执行finally
块.