函数导致通过引用传递的变量行为异常

Question

我有一个使用SDL2 的C 程序。当PenTool 的内容不在函数中而是在调用函数的位置时，我有一个工作程序，但是将其放入函数中导致了一个我无法理解的问题。避免使用全局变量，我已经通过引用传递了我需要操作的所有变量，包括 PenTool 中的函数bresline。该代码应该通过使当前鼠标坐标为终点而前一个鼠标坐标为线的起点来从鼠标绘制一条连续的线。但是，在这个功能布局中，当前鼠标坐标被正确识别并在屏幕上绘制，但之前的鼠标坐标保持不变，导致每条线都从同一点开始绘制，而不是实际的先前鼠标位置。此外，打印功能将所有变量显示为在鼠标移动时不变，尽管正在运行的程序绘制我的鼠标所在的位置。

问题在于bresline。当我注释掉它时，打印函数显示所有变量随着鼠标移动而正确更新，所以有人可以解释为什么bresline 会阻止以前的鼠标坐标被正确分配，以及为什么它会阻止printf 显示正确的鼠标坐标.

我在 StackOverflow 和其他地方查看了有关正确通过引用传递的其他问题，据我所知，我已经正确传递了我的变量，但我不知道如何阅读 C++/Java所以这些答案没有帮助，所以我真的不知道我哪里出错了。

void bresline(SDL_Renderer *,int *, int *, int *, int *, Uint32 *);
void PenTool(int, int *, int *, SDL_Point, int *, int *, SDL_Renderer *, Uint32 *);

int main(void)
{
    int mouseX = 0, mouseY = 0, prevMouseX = 0, prevMouseY = 0;
    int penSize = 1;
...
    case SDL_MOUSEMOTION:
        if(leftMouseButtonDown)
        {
            PenTool(penSize, &mouseX, &mouseY, mouse_position, &prevMouseX, 
                &prevMouseY, renderer, pixels);
        }
...
void bresline(SDL_Renderer *renderer, int *x0, int *y0, int *xn, int *yn, Uint32 *pixels)
{
    int dx = abs(*xn-*x0), sx = *x0<*xn ? 1 : -1;
    int dy = abs(*yn-*y0), sy = *y0<*yn ? 1 : -1; 
    int error = (dx>dy ? dx : -dy)/2, e2;
    SDL_SetRenderDrawColor(renderer, 0, 0, 0, 255);
    while(1)
    {
        pixels[*y0 * 1000 + *x0] = 0;
        if(*x0==*xn && *y0==*yn) break;
        e2 = error;
        if(e2 >-dx)
        {
            error -= dy;
            *x0 += sx;
        }
        if(e2 < dy)
        {
            error += dx;
            *y0 += sy;
        }
    }
}

void PenTool(int penSize, int *mouseX, int *mouseY, SDL_Point mouse_position, 
    int *prevMouseX, int *prevMouseY, SDL_Renderer *renderer, Uint32 *pixels)
{
    if(penSize == 1)
    {
        *mouseX = mouse_position.x;
        *mouseY = mouse_position.y;
        if((*prevMouseX == 0) && (*prevMouseY == 0))
        {
            *prevMouseX = *mouseX;
            *prevMouseY = *mouseY;
        }
        bresline(renderer, mouseX, mouseY, prevMouseX, prevMouseY, pixels);
        *prevMouseX = *mouseX;
        *prevMouseY = *mouseY;
        printf("M1_x: %d, M1_y: %d, M2_x: %d, M2_y: %d \n", *mouseX, *mouseY, *prevMouseX, *prevMouseY);
    }
}

Answer 1

顺便说一句，我会将 bresline 更改为：

void bresline(SDL_Renderer *,int , int , int , int , Uint32 *);

void bresline(SDL_Renderer *renderer, int x0, int y0, int xn, int yn, Uint32 *pixels)
{
    int dx = abs(xn-x0), sx = x0<xn ? 1 : -1;
    int dy = abs(yn-y0), sy = y0<yn ? 1 : -1; 
    int error = (dx>dy ? dx : -dy)/2, e2;
    SDL_SetRenderDrawColor(renderer, 0, 0, 0, 255);
    while(1)
    {
        pixels[y0 * 1000 + x0] = 0;
        if(x0==xn && y0==yn) break;
        e2 = error;
        if(e2 >-dx)
        {
            error -= dy;
            x0 += sx;
        }
        if(e2 < dy)
        {
            error += dx;
            y0 += sy;
        }
    }
}

在您的代码中，您正在更改 Bresenham 算法函数中的值，因此，当函数退出时，MouseX 和 MouseY 将始终是 PrevMouseX 和 PrevMouseY。这可以解释您总是从同一点开始的问题。

---

并更改为此调用函数的方式：

bresline(renderer, *mouseX, *mouseY, *prevMouseX, *prevMouseY, pixels);