尝试使用 malloc 做 3 维矩阵

Question

我想使用malloc 收集一个 3 维矩阵。还有矩阵打印部分，但我没有包括。

#include <stdio.h>
#include <stdlib.h>
int main()
{
   int i,j,k,m,n,o;
   float ***A;
   printf("Input dimension for matrix(m,n,o)\n");
   printf("Enter m : ");
   scanf("%d",&m);
   printf("Enter n : ");
   scanf("%d",&n);
   printf("Enter o : ");
   scanf("%d",&o);
   A = (float***)malloc(o*sizeof(float**));
   for(i=0;i<o;i++){
           A[i] = (float**)malloc(m*sizeof(float*));
           for(j=0;j<m;j++){
               A[i][j] = (float*)malloc(n*sizeof(float));
           }
   }
   for(i=0;i<o;i++){
       for(j=0;j<m;j++){
           for(k<0;k<n;k++){
               printf("Input number for (%d,%d,%d) : ",j+1,k+1,i+1);
               scanf("%f",&A[i][j][k]);
           }
       }
   }
   return 0;
}

在我完成输入 m,n,o 后程序似乎跳到最后，所以我无法输入任何矩阵值。我必须使用malloc，因为它是任务所必需的。

编辑：一切都是错误的。 m代表行，n代表列，o代表mXn矩阵的个数。

Answer 1

使用动态可变长度数组：

float (*A)[n][m] = calloc(o, sizeof *A);

并在最后释放它

free(A);

更多的人必须传递scanf-ed对象的地址，而不是它的值。应该是：

scanf("%f", &A[i][j][k])

Answer 2

这应该可行：

#include <stdio.h>
#include <stdlib.h>

int main(void) {
    float ***A;

    // n1, n2 and n3 will contain the three dimensions (i.e.: A[n1][n2][n3])
    size_t n1, n2, n3;

    printf("Insert first dimension: ");
    if (scanf("%zu", &n1) != 1) {
        printf("ERROR: cannot read number\n");
        return 1;
    }

    printf("Insert second dimension: ");
    if (scanf("%zu", &n2) != 1) {
        printf("ERROR: cannot read number\n");
        return 1;
    }

    printf("Insert third dimension: ");
    if (scanf("%zu", &n3) != 1) {
        printf("ERROR: cannot read number\n");
        return 1;
    }

    // firstly, here we alloc space for first dimension
    A = malloc(n1 * sizeof(float**));

    for (size_t i = 0; i < n1; i += 1) {
        // now we alloc space for second dimension
        A[i] = malloc(n2 * sizeof(float*));
        for (size_t j = 0; j < n2; j += 1) {
            // here we alloc space for third dimension
            A[i][j] = malloc(n3 * sizeof(float));
        }
    }

    // let's ask the user to input all elements into the array
    for (size_t i = 0; i < n1; i += 1) {
        for (size_t j = 0; j < n2; j += 1) {
            for (size_t k = 0; k < n3; k += 1) {
                printf("Insert value for A[%zu][%zu][%zu]: ", i, j, k);
                if (scanf("%f", &(A[i][j][k])) != 1) {
                    printf("ERROR: cannot read number\n");
                    return 1;
                }
            }
        }
    }

    // now let's print all the numbers that the user has input previously
    for (size_t i = 0; i < n1; i += 1) {
        for (size_t j = 0; j < n2; j += 1) {
            for (size_t k = 0; k < n3; k += 1) {
                printf("A[%zu][%zu][%zu] = %f\n", i, j, k, A[i][j][k]);
            }
        }
    }

    return 0;
}

我没有编译它，但它应该是正确的。仔细看看是如何分配的。

注意：当然，完全正确（例如：使用 free() 和避免使用 scanf()）是故意避免的。

P.S.：正如其他用户已经提到的，请注意为您的编译器启用警告（例如：GCC 上的 -Wall -Wextra），因为它会为您简化调试。

Answer 3

我认为你所做的只是在获取输入时在 for 循环中搞砸了。根据上下文，3D 矩阵，我调试了代码并提出了一个有意义的版本。

我会继续指出你犯的错误。

首先，scanf 获取变量的内存地址，而不是它的取消引用版本。

因此，将其从 scanf("%f",A[i][j][k]); 更改为 scanf("%f",&(A[i][j][k]));。这个也可以scanf("%f",A[i][j] + k);。

在这一行中，for(k<0;k<n;k++)、k 甚至没有被初始化。因此，您实际上是在对垃圾值进行操作。由于我已经对破译c pointer hell 感到困惑，我什至无法处理最终结果。更有意义的代码是 for(k=0;k<n;k++)。

之后，你切换了嵌套for循环的控制逻辑。

将此代码for(j=0;j<m;j++) 中的变量m 替换为n。 for(j=0;j<n;j++)

在下一个循环中反之亦然。将此for(k=0;k<n;k++)修改为for(k=0;k<m;k++)。

最终的代码如下所示，

#include <stdio.h>
#include <stdlib.h>
int main()
{
   int i,j,k,m,n,o;
   float ***A;
   printf("Input dimension for matrix(m,n,o)\n");
   printf("Enter m : ");
   scanf("%d",&m);
   printf("Enter n : ");
   scanf("%d",&n);
   printf("Enter o : ");
   scanf("%d",&o);
   A = (float***)malloc(o*sizeof(float**));
   for(i=0;i<o;i++){
           A[i] = (float**)malloc(n*sizeof(float*));
           for(j=0;j<n;j++){
               A[i][j] = (float*)malloc(m*sizeof(float));
           }
   }
   for(i=0;i<o;i++){
       for(j=0;j<n;j++){
           for(k=0;k<m;k++){
               printf("Input number for (%d,%d,%d) : ", i + 1, j + 1, k + 1);
               scanf("%f",&(A[i][j][k]));
           }
       }
   }
   
   
   return 0;
}

它现在可以用作 3D 矩阵。

为了理解代码，我不得不画一个凌乱的图表。我附上那个。它可能会有所帮助。

由于您已经省略了一些代码，我希望您没有忘记使用free 来防止内存泄漏。