C语言。逻辑错误：“ - ”的左操作数是垃圾值

Question

我有以下代码，但在这行代码中我有警告x[i] = (rhs[i] - x[i - 1]) / b;，编译器告诉我rhs [i]是垃圾值。为什么会这样？以及如何删除此警告？

double* getFirstControlPoints(double* rhs, const int n) {
    double *x;
    x = (double*)malloc(n * sizeof(double));
    double *tmp; // Temp workspace.
    tmp = (double*)malloc(n * sizeof(double));

    double b = 2.0;
    x[0] = rhs[0] / b;
    for (int i = 1; i < n; i++) // Decomposition and forward substitution.
    {
        tmp[i] = 1 / b;
        b = (i < n - 1 ? 4.0 : 3.5) - tmp[i];
        x[i] = (rhs[i] - x[i - 1]) / b; //The left operand of '-' is a garbage value
    }
    for (int i = 1; i < n; i++) {
        x[n - i - 1] -= tmp[n - i] * x[n - i]; // Backsubstitution.
    }
    free(tmp);
    return x;
}

您可以在屏幕截图中看到所有编译器警告和调用getFirstControlPoints。

Answer 1

您需要检查以确保points数组中至少有 4 点，因为此循环（第333行）：

for (NSInteger i = 1 ; i < n - 1 ; ++i) {
    // initialisation stuff
}

对于n = 0,1,2，

根本不会执行。

假设points中有3个对象，在第311行，您将n设置为计数 - 1，即n == 2

然后循环条件为i < 2 - 1，即i < 1。

我认为您需要将循环条件设为i < n

Answer 2

如果points.count为0或1，则表示您遇到一些问题，因为那时，n是-1或0，并且您访问rhs [n-1];你和malloc n * bytes;

也许这可能是问题所在。你把一些范围检查放在代码中了吗？