我是Func / Action / Predicate的新手,想知道重构代码的最佳方法是什么,因为我可能会有很多重复的代码。
我创建了2个扩展方法,它们接受一个Func参数(总是返回bool):
public static void MyExtension<T>(this T obj, Func<T, bool> predicate)
{
do
{
//function code
} while (predicate(obj));
}
public static void MyExtension<T1, T2>(this T1 obj, T2 OtherObject, Func<T1, T2, bool> predicate)
{
do
{
//function code
} while (predicate(obj, OtherObject));
}
我想要的是,只对do / while循环进行一次编码,因为我可能有很多函数只能生成不同的Func但是都希望运行相同的do / while循环。
我在想这样的事情,它接受任何Func参数并在每个循环上运行它。循环中的代码并不关心传递给Func的类型。
public static void DoLoop(Func predicate)
{
do
{
//function code
} while (predicate);
}
但这显然不起作用。有谁知道如何解决这个问题,还是我做了一些根本错误的事情?
最终结果是我想调用如下代码:
var Obj = new MyObj();
Obj.MyExtension((x) => x.Prop1.Contains("string"));
var OtherObj = new MyObj();
Obj.MyExtension(OtherObj, (x,y) => x.Prop1.Contains("string") && y.Prop1.Contains("other"));
感谢。
答案 0 :(得分:1)
由于您的泛型方法是在不同数量的泛型类型上进行参数化的,因此您需要保留所有函数签名,以便编译调用代码。但是,您可以将实现移动到公共帮助器方法中以减少重复的代码:
// This is your single implementation
private static void DoLoop(Func<bool> predicate) {
do {
//function code
} while (predicate());
}
// These are the wrappers
public static void MyExtension<T>(this T obj, Func<T, bool> predicate) {
DoLoop(() => predicate(obj));
}
public static void MyExtension<T1, T2>(this T1 obj, T2 OtherObject, Func<T1, T2, bool> predicate) {
DoLoop(() => predicate(obj, OtherObject));
}
Wrappers构造无参数谓词,并将其传递给提供实现的私有DoLoop
方法。