objective-c - 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

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

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

对于 n = 0、1、2，根本不会执行。

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

那么循环条件就是i < 2 - 1ie i < 1。

我认为你需要循环条件是i < n

Answer 2

如果 points.count 为 0 或 1，您将面临一些问题，因为那时 n 为 -1 或 0，并且您访问 rhs[n-1]; 你 malloc n* 字节；

也许这可能是问题所在。您在代码中添加了一些范围检查吗？