1

我正在尝试将动态创建的 3d 数组中的元素移动一个索引，以便每个元素[i][j][k]都应该是 on [i+1][j][k]。

这就是我的数组创建的样子

typedef struct stencil{
int ***arr;
int l;
int m;
int n;}matrix;

void createMatrix(matrix *vector){

vector->arr = (int***) malloc(sizeof(int**) * (vector->l+2));
for (int i = 0; i< vector->l+2; ++i) {
    vector->arr[i] = (int**) malloc(sizeof(int*) * (vector->m+2));
    for (int j = 0; j < vector->m+2; ++j) {
        vector->arr[i][j] = (int*) calloc((vector->n+2),sizeof(int));
    }

}
}

这基本上是我想用 memmove 实现的

for(int i = vector->l-1; i >= 0; --i){
    for(int j = vector->m; j >= 0; --j){
        for(int k = vector->n; k >= 0; --k){
            vector->arr[i+1][j][k] = vector->arr[i][j][k];
        }
    }
}

由于某种原因，memmove 移动了 2 个索引。

memmove(&(vector->arr[1][1][1]), &(vector->arr[0][1][1]), (vector->l+2)*(vector->m+2)*(vector->n)*sizeof(int*));

谁能给我一个提示？

4

3 回答 3

2

当你像这样创建一个动态多维数组时，数组内容是不连续的——每一行都是一个单独的分配。所以你不能用一个memmov().

但是你不需要复制所有的数据，只需移动顶层数组中的指针即可。

int **temp = arr[l-1]; // save last pointer, which will be overwritten
memmov(&arr[1], &arr[0], sizeof(*arr[1]));
arr[0] = temp;

我将最后一个元素移动到第一个元素，以避免两个元素指向相同的数据。您还可以释放旧的最后一个元素（包括释放它指向的数组）并创建一个新的第一个元素，但这更简单。

于 2017-12-26T22:36:23.363 回答
1

只需这样做就可以了（以 3d 数组说明）

memmove(arr[1], arr[0], Y*Z*sizeof(int));

其中Y和Z表示二维数组的其他二维。

这arr[X][Y][Z]是int数组 where X>=2。

如果是动态分配的内存，您需要一个一个地处理每个连续的块。然后它会工作。

于 2017-12-26T21:14:20.773 回答
1

使用更高的优化级别 ( -O3) 进行编译。vector->arr获取对每个数组访问的直接引用，而不是强制取消引用。

memmove假设您分配arr为连续内存，您的调用看起来是正确的。不过，既然你说的是“动态”，那我就很怀疑了。加上大小计算似乎非常错误，与sizeof(int*).

我想arr不是int arr[constexpr][constexpr][constexpr]（单一，连续分配），而是int ***arr.

在这种情况下，memmove事情就大错特错了。在将字段的int**内容移动了arr一个（实际上已经移动了）之后，它导致了堆上令人讨厌的溢出，很可能偶然也命中了int*随后的大部分分配。

看起来像一个双重移动，并留下一个完全被破坏的堆。

于 2017-12-26T21:28:00.890 回答