C ++大型2D阵列访问冲突

Question

因此，我有一个为找到最佳的二进制搜索树而创建的程序，当数据集大约为100时，它的效果很好，但是当我尝试使用> 1000的数据集时，当weight [i] [i ] =频率[i]; 在computeOBST函数内部被调用。 我不确定该数据集是否很大。 我被困住了，不知道还有什么可以尝试的帮助会很棒。

int* keys = new int[numKeys];                   
int* keyLevel = new int[numKeys];               
int* frequency = new int[numKeys];              
int** weight = new int*[numKeys+2];             
int** cost = new int*[numKeys];                 
int** root = new int*[numKeys];                 

void allocateArraySpace(int n){
    int i;
    // Allocate space for the 2-dim'l cost array
    for (i = 0; i < numKeys + 2; i++) {
        cost[i] = new int[numKeys + 2];
    }

    for (i = 0; i < numKeys + 1; i++) {
        keyLevel[i] = numKeys + 1;
    }

    // Allocate space for the 2-dim'l root array
    for (i = 0; i < numKeys + 1; i++) {
        root[i] = new int[numKeys + 1];
    }

     //Allocate space for the 2-dim'l weight array
    for (i = 0; i <= numKeys + 2; i++) {
        weight[i] = new int[numKeys + 2];
    }

}

void computeOBST(int n) {
    numKeys = n;
    int i, j, k, h, m;

    allocateArraySpace(numKeys);

    //creating weight matrix
    for (int i = 1; i <= numKeys + 1; i++)
    {
        weight[i][i] = frequency[i];
        for (j = i + 1; j <= numKeys; j++)
            weight[i][j] = weight[i][j - 1] + frequency[j];
    }

    //
    for (i = 1; i <= numKeys; i++)
        for (j = i + 1; j <= numKeys + 1; j++)
            cost[i][j] = INT_MAX;

    //
    for (i = 1; i <= numKeys + 1; i++)
        cost[i][i - 1] = 0;

    //
    for (i = 1; i <= numKeys; i++) {
        cost[i][i] = weight[i][i];
        root[i][i] = i;
    }

Answer 1

cost只是numKeys的大小，但是在您的allocateArraySpace循环中，您访问的是numKeys+2元素吗？ 其他阵列也有类似的问题。

请记住，C ++中的数组是基于0的，所以如果您执行cost = new int *[numKeys] ，则cost[numKeys-1]可以，但是cost[numKeys]和cost[numKeys+1]超出范围。

Answer 2

int* frequency = new int[numKeys]

然后

for (int i = 1; i <= numKeys + 1; i++) { weight[i][i] = frequency[i]; ...

您将越界。 未定义的行为 。 碰巧此UB的小值不会导致分段错误，而对于numKeys高值numKeys导致分段错误。

这只是一个例子，在许多平台上您都有相同的错误。 再次检查所有循环和数组范围，并设置正确的限制。

通常，当您跨过一个大小为numKeys的数组时，可以通过以下方式进行扫描（请记住，C数组是从零开始的）：

for (int i = 0; i < numKeys; i++) // first index is 0, last is numKeys-1