c++ - 正交变量代码重复问题

Question

我最近开始重构一些遗留代码并遇到了两个用于绘制坐标网格的函数，问题是这些函数仅在它们处理的正交变量方面有所不同，类似的东西

void DrawScaleX(HDC dc, int step, int x0, int x1, int y0, int y1)
{
    for(int x = x0; x < x1; x += step)
    {
         MoveToEx(dc, x, y0, NULL);
         LineTo(dc, x, y1);
    }
}
void DrawScaleY(HDC dc, int step, int x0, int x1, int y0, int y1)
{
    for(int y = y0; y < y1; y += step)
    {
         MoveToEx(dc, x0, y, NULL);
         LineTo(dc, x1, y);
    }
}

因此，如果我决定添加一些花哨的东西，例如抗锯齿或只是更改绘图铅笔或其他任何东西，我将不得不在两者中放入相同的代码，这是代码重复，我们都知道这是为什么。

我的问题是，您如何将这两个函数重写为一个函数来避免这个问题？

Answer 1

为什么不将 for 循环的主体提取到单独的函数中？然后你可以在提取的函数中做一些有趣的事情。

void DrawScaleX(HDC dc, int step, int x0, int x1, int y0, int y1)
{
    for(int x = x0; x < x1; x += step)
    {
        DrawScale(dc, x, y0, x, y1);
    }
}

void DrawScaleY(HDC dc, int step, int x0, int x1, int y0, int y1)
{
    for(int y = y0; y < y1; y += step)
    {
        DrawScale(dc, x0, y, x1, y);
    }
}

private void DrawScale(HDC dc, int x0, int y0, int x1, int y1)
{
    //Add funny stuff here

    MoveToEx(dc, x0, y0, NULL);
    LineTo(dc, x1, y1);

    //Add funny stuff here
}