如何处理具有不同参数的相同代码路径？

Question

我遇到过一种情况，尝试不必修改底层源代码（它在该级别上并不是真正的“我的”代码；否则，我很想修改它），我有两个几乎相同的代码路径，但是我正在使用的不同类型的数据。

忽略您可能可以相当容易地将一个对象转换为另一个对象，类似的示例将是一个“Circle”对象。

一方面，我有一个 Point 对象和一个半径。另一方面，我有一个 Circle 对象。这两者都可以描述相同的实际圆，但我无法将其转换为另一个。

然后，在我的代码中，我有：

void Run(Circle circle)
{
    if(AllNegative(circle))
    {
        // ...
        // Do unrelated stuff
        // ...
        ColorCircle(circle);
        // ...
    }
}

void Run(Point pt, uint radius)
{
    if(AllNegative(pt, radius))
    {
        // ...
        // Do unrelated stuff
        // ...
        ColorCircle(pt, radius);
        // ...
    }
}

bool AllNegative(Circle circle) { return (circle.AllNegative); }
bool AllNegative(Point pt, uint radius) { return ((pt.X + radius) < 0) && ((pt.Y + radius) < 0); }

void ColorCircle(Circle circle) { /* ... */ }
void ColorCircle(Point pt, uint radius) { /* ... */ }

当然，我在 Run 中的代码比本示例中的代码多。

如何将 Run 合并到单个函数中以最大程度地减少代码重复？

Answer 1

在这种情况下，您可以采取两条路径：

1. 您可以将其保留为现在的样子 - 重载方法。
2. 将重复的代码移至新方法。如果在不相关部分中计算圆的直径（无论是否真实），请创建一个方法calcDiameter(radius)。

在这种情况下，如果只有几行代码确实无法使新方法变得实用，那么重复的代码并没有本质上的错误。

Answer 2

一个泛型的例子：

public interface ICircle
{
    Point Point{get;}
    uint Radius{get;}
    ... add whatever you need
}

public class MyCircle: ICircle
{
   private Circle _circle;
   ... implement interface
}

public class MyCircle2: ICircle
{
   private Point _point;
   private uint _radius;
   ... implement interface
}

void Run<T>(T circle) where T: ICircle
{
    if(AllNegative(circle))
    {
        ColorCircle(circle);
    }
}

Answer 3

至少，您可以在单独的方法中执行不相关的操作，并从每个Run 中调用它。

Answer 4

您可以为它们创建一个公共接口，如下所示：

public interface IMyShape
{
    public bool IsNegative();
    public void Color();
}

然后您只需要一种接受 IMyShape 引用的方法：

void Run(IMyShape shape)
{
    if(shape.AllNegative())
    {
        // ...
        // Do unrelated stuff
        // ...
        shape.Color();
        // ...
    }
}