c# - 如何重构这些方法以避免重复？

Question

在我们的代码库中

public ActionResult Copy(string id, string targetId)
            {
                //lot of similar code
                Copy(sourcePageRef, destinationPageRef);
                //lot of similar code
            }

和

public ActionResult Move(string id, string targetId)
        {
            //lot of similar code
            Move(sourcePageRef, destinationPageRef);
            //lot of similar code
        }

问题是，Copy 和 Move 有不同的签名:

PageRef Copy(PageRef, PageRef)

和

void Move(PageRef, PageRef)

如何重构这些方法以避免重复？谢谢

Answer 1

如果您不需要 Copy 的结果, 你仍然可以使用 Action<string, string>或任何类型:

public ActionResult Copy(string id, string targetId)
{
    CopyOrMove((x, y) => Copy(x, y));
}

public ActionResult Move(string id, string targetId)
{
    CopyOrMove(id, targetId, (x, y) => Move(x, y));
}

private void CopyOrMove(string id, string targetId,
                        Action<string, string> fileAction)
{
    // lot of similar code
    fileAction(sourcePageRef, destinationPageRef);
    // lot of similar code
}

这是一个选项。这取决于“大量相似代码”真正在做什么，以及第二个 block 是否需要第一个 block 的结果。例如，如果您可以这样做:

public ActionResult Copy(string id, string targetId)
{
    string sourcePageRef = PrepareSourceFile(id, targetId);
    string targetPageRef = PrepareTargetFile(targetId);
    Copy(sourcePageRef, targetPageRef);
    CleanUp(sourcePageRef, targetPageRef);
    return ...;
}

public ActionResult Move(string id, string targetId)
{
    string sourcePageRef = PrepareSourceFile(id, targetId);
    string targetPageRef = PrepareTargetFile(targetId);
    Move(sourcePageRef, targetPageRef);
    CleanUp(sourcePageRef, targetPageRef);
    return ...;
}

...那么这可能比使用委托(delegate)进行重构的方法更简单。