c++ - 为什么我的 C++ 函数必须有不同的指针参数？

Question

我制作了一个带有指针传递的 C 函数，它仅在指针不同时才给出正确的结果。例如:

void dotransform(Point *pout, const Point *pin, transform mat)
{
    pout->x = mat[0][0] * pin->x + mat[1][0] * pin->y + mat[2][0] * pin->z + mat[3][0] * 1.0;
    pout->y = mat[0][1] * pin->x + mat[1][1] * pin->y + mat[2][1] * pin->z + mat[3][1] * 1.0;
    pout->z = mat[0][2] * pin->x + mat[1][2] * pin->y + mat[2][2] * pin->z + mat[3][2] * 1.0;
}

dotransform() 应该这样调用:

//...
Transform toWorldMat;
Point local;
Point world;
dotransform(&world, &local, toWorldMat );

问题是，我团队中有人这样调用它:

Point p;
dotransform(&p, &p, toWorldMat);

我只花了一个星期的时间就弄明白了为什么程序的最终输出发生了变化。

为了避免这种情况，声明这种函数的最佳风格是什么？我这样写是不是错了？

Answer 1

鉴于您已经直接传递了 transform 对象(尽管不是通过 const&:您真的应该这样做)，您有什么理由不能写代码如下:

Point dotransform(Point const& pin, transform const& mat)
{
    Point pout
    pout.x = mat[0][0] * pin.x + mat[1][0] * pin.y + mat[2][0] * pin.z + mat[3][0] * 1.0;
    pout.y = mat[0][1] * pin.x + mat[1][1] * pin.y + mat[2][1] * pin.z + mat[3][1] * 1.0;
    pout.z = mat[0][2] * pin.x + mat[1][2] * pin.y + mat[2][2] * pin.z + mat[3][2] * 1.0;
    return pout;
}

这将允许您编写如下代码:

Transform toWorldMat;
Point local;
Point world = dotransform(local, toWorldMat);

或者这个:

Point p;
p = dotransform(p, toWorldMat);

这些是编写此代码的正确、理想的 C++ 方法。