已解决

Question

我参考以下URL来练习2D动态数组： https://thispointer.com/allocating-and-deallocating-2d-arrays-dynamically-in-c-and-c/

我的代码：

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

int** create_2d_arr(int row_size,int colum_size)
{
    int** array = (int**)malloc(sizeof(int*)*row_size);
    for (int i = 0; i < row_size; i++)
        array[i] = (int*)malloc(sizeof(int)*colum_size);  
    return array;
}

void free_2d_arr(int** matrix,int row_size, int colum_size) {
    for (int i = 0; i < row_size; i++) {
        free(matrix[i]);
    }
    free(matrix);
}

int main(int argc, char const *argv[])
{
    int row=3,cloum=2;
    int** arr_2d = create_2d_arr(row,cloum);

    arr_2d[0,0]=4;
    arr_2d[0,1]=5;

    arr_2d[1,0]=6;
    arr_2d[1,1]=7;

    arr_2d[2,0]=8;
    arr_2d[2,1]=9;

    for(int i=0;i<row;i++)
    for(int j=0;j<cloum;j++)
    printf("arr_2d[%d,%d] = %d \n",i,j,arr_2d[i,j]);

    free_2d_arr(arr_2d,row,cloum);

    return 0;
}

但是，编译后执行时会出错：

arr_2d[0,0] = 8 
arr_2d[0,1] = 9 
arr_2d[1,0] = 8 
arr_2d[1,1] = 9 
arr_2d[2,0] = 8 
arr_2d[2,1] = 9 
[1]    9300 segmentation fault (core dumped)  ./t

只有arr_2d [2,0] = 8 arr_2d [2,1] = 9是正确的。我不明白我的代码在哪里错误。有人帮我吗？

续订

感谢您的答复。但在我将arr_2d[2,0]=8修改为rr_2d[2][0]=8之后...

result of printf is
arr_2d[0][0] = -267545984
arr_2d[0][1] = -267545952
arr_2d[1][0] = -267545984

...

编译器警告

t.c:38:47: warning: expression result unused [-Wunused-value]
    printf("arr_2d[%d,%d] = %d \n",i,j,arr_2d[i,j]);
                                              ^
t.c:38:40: warning: format specifies type 'int' but the argument has type
      'int *' [-Wformat]
    printf("arr_2d[%d,%d] = %d \n",i,j,arr_2d[i,j]);
                            ~~         ^~~~~~~~~~~
2 warnings generated.

即使我使用gcc，我的编译器也是clang

=========

已解决

修改后：

printf("arr_2d[%d,%d] = %d \n",i,j,arr_2d[i,j]);

=>

printf("arr_2d[%d,%d] = %d \n",i,j,arr_2d[i],[j]);

它正常工作。非常感谢大家。

Answer 1

您不能像这样访问数组的值

arr_2d[x,y];

您必须使用这种语法

arr_2d[x][y];

Answer 2

arr_2d[x,y]等同于arr_2d[y]

在分配和访问中使用arr_2d[x][y]

在C（或C ++）a,b,c .., z中，先计算 a 然后 b ..然后 z 并返回的值> z

所以：

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

int** create_2d_arr(int row_size,int colum_size)
{
    int** array = (int**)malloc(sizeof(int*)*row_size);
    int i;
    for ( i = 0; i < row_size; i++)
        array[i] = (int*)malloc(sizeof(int)*colum_size);  
    return array;
}

void free_2d_arr(int** matrix,int row_size, int colum_size) {
  int i;
    for ( i = 0; i < row_size; i++) {
        free(matrix[i]);
    }
    free(matrix);
}

int main(int argc, char const *argv[])
{
    int row=3,cloum=2;
    int** arr_2d = create_2d_arr(row,cloum);

    arr_2d[0][0]=4;
    arr_2d[0][1]=5;

    arr_2d[1][0]=6;
    arr_2d[1][1]=7;

    arr_2d[2][0]=8;
    arr_2d[2][1]=9;

    int i,j;
    for( i=0;i<row;i++)
    for( j=0;j<cloum;j++)
    printf("arr_2d[%d][%d] = %d \n",i,j,arr_2d[i][j]);

    free_2d_arr(arr_2d,row,cloum);

    return 0;
}

结果是：

arr_2d[0][0] = 4 
arr_2d[0][1] = 5 
arr_2d[1][0] = 6 
arr_2d[1][1] = 7 
arr_2d[2][0] = 8 
arr_2d[2][1] = 9

Answer 3

正确的语法是

arr_2d[i][j] = n;

我从来没有见过像arr_2d[i,j]这样的东西，如果有人向我们解释它会很满足。

我的猜测是它等同于arr_2d[j]，但我不确定。

编辑：精度：您应将所有arr_2d[i,j]（如arr_2d[0,0]）更改为新语法

Answer 4

以下建议的代码：

干净地编译
执行所需的功能
将我的评论纳入OP问题
出于可读性考虑，在C运算符周围插入适当的空格：括号内，括号内，分号后，逗号后，
使用size_t而不是int来避免编译器警告有关int和size_t之间的转换，尤其是在对calloc()和{{1}的调用中}

现在，建议的代码：

malloc()

运行建议的代码将导致：

#include <stdio.h>
#include <stdlib.h>   // exit(), EXIT_FAILURE


// prototypes
void free_2d_arr( int** matrix, size_t row_size, size_t colum_size );


int** create_2d_arr( size_t row_size, size_t colum_size )
{
    int** array = calloc( row_size, sizeof( int* ) );
    if( !array )
    {
        perror( "malloc failed" );
        exit( EXIT_FAILURE );
    }

    // implied else, malloc successful

    for ( size_t i = 0; i < row_size; i++ )
    {
        array[ i ] = malloc( sizeof(int) * colum_size );  
        if( !array[ i ] )
        {
            perror( "malloc failed" );
            free_2d_arr( array, row_size, colum_size ); 
            exit( EXIT_FAILURE );
        }
    }
    return array;
}


void free_2d_arr( int** matrix, size_t row_size, size_t colum_size ) 
{
    (void)colum_size;
    for ( size_t i = 0; i < row_size; i++ ) 
    {
        free( matrix[ i ] );
    }
    free( matrix );
}


int main( void )
{
    size_t row = 3;
    size_t cloum = 2;
    int** arr_2d = create_2d_arr( row, cloum );

    arr_2d[ 0 ][ 0 ] = 4;
    arr_2d[ 0 ][ 1 ] = 5;

    arr_2d[ 1 ][ 0 ] = 6;
    arr_2d[ 1 ][ 1 ] = 7;

    arr_2d[ 2 ][ 0 ] = 8;
    arr_2d[ 2 ][ 1 ] = 9;

    for( size_t i=0; i < row; i++ )
    {
        for( size_t j=0; j < cloum; j++ )
        {
            printf( "arr_2d[%lu,%lu] = %d \n", i, j, arr_2d[ i ][ j ] );
        }
    }

    free_2d_arr( arr_2d, row, cloum );

    return 0;
}

但是请注意，以下语句：

arr_2d[0,0] = 4 
arr_2d[0,1] = 5 
arr_2d[1,0] = 6 
arr_2d[1,1] = 7 
arr_2d[2,0] = 8 
arr_2d[2,1] = 9

确实是程序的参数，size_t row = 3; size_t cloum = 2;和2是常量和“魔术”数字。 “魔术”数字是没有基础的数字。 “魔术”数字使代码更难以理解，调试等。建议使用3语句或#define语句为这些“魔术”数字赋予有意义的名称，然后在代码中使用这些有意义的名称

使用函数在C中创建和释放2D数组

续订

...

编译器警告

已解决

4 个答案: