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設置為count - 1即n == 2

然后循環條件是i < 2 - 1 2-1，即i < 1 。

我認為你需要循環條件為i < n

Answer 2

如果points.count是0或1，你面臨一些問題，因為那時，n是-1或0，你訪問rhs [n-1]; 而你malloc n *字節;

也許這可能是問題所在。 你在代碼中放了一些范圍檢查？