C - 创建“2D阵列”时的分段错误

时间:2015-01-06 15:20:19

标签: c arrays segmentation-fault malloc multidimensional-array

我必须创建一个带有x行的“2D数组”(用户可以决定多少行),每行应该有一个随机数量的列,这些列将随机生成,所以它看起来像这样:

2 - 4 - 6

1 - 2 - 8 - 9 - 2 - 3

1 - 2

每行的列数将保存在sizes [i]中。 2d数组中的数字将随机生成。我通过stackoverflow查看了这里,并找到了一些关于动态内存分配的解决方案,但不知怎的,我总是以“分段错误”结束,我无法真正看到代码中的一个主要缺陷。所以任何帮助都会受到关注。 :)

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

int** create_array(int* sizes, int rows){
    int** array;
    array=(int**) malloc(rows*sizeof(int*));
    for(int i=0;i<rows;i++){
        array[i]=(int*) malloc((sizes[i])*sizeof(int));
    }
    for(int i=0;i<rows;i++){
        for(int j=0;j<sizes[i];j++){
            array[i][j]=(((double) rand() / (RAND_MAX))*20);
        }
    }
    return array;
}

void print_array(int** array, int* sizes){
    int rows=sizeof(sizes)/sizeof(sizes[0]);
    for(int i=0;i<rows;i++){
        for(int j=0;j<sizes[i];j++){
            printf("%d ",array[i][j]);
        }
        printf("\n");
    }
}

int main(int argc, char const *argv[])
{
    int rows = 0;
    srand(time(NULL));
    printf("Wie viele Zeilen möchten Sie erzeugen?");
    scanf("%d",&rows);
    int sizes[rows];
    for(int i=0;i<rows;i++){
        sizes[i]=(((double) rand() / (RAND_MAX))*9+1);
        printf(" %d ",sizes[i]);
    }
    int** arr;
    arr=create_array(sizes,rows);
    print_array(arr,sizes);
    return 0;
}

1 个答案:

答案 0 :(得分:1)

你有错误

for(int j=0;i<sizes[i];j++)

应该是

for(int j=0;j<sizes[i];j++)

如果你使用更多像

这样的空格,那就更容易发现了
for (int j = 0 ; i < sizes[i] ; j++)
/*               ^ see, here it's very clear now

此外,请勿忘记在使用完数据后致电free

也改变这个功能

void print_array(int** array, int* sizes){
    int rows =sizeof(sizes)/sizeof(sizes[0]);
    for(int i=0;i<rows;i++){
        for(int j=0;i<sizes[i];j++){
            printf("%d ",array[i][j]);
        }
        printf("\n");
    }
}

无法确定sizes中元素的数量,您必须将其作为参数传递

void print_array(int** array, int* sizes, int rows)
{
    for(int i=0;i<rows;i++){
        for(int j=0;i<sizes[i];j++){
            printf("%d ",array[i][j]);
        }
        printf("\n");
    }
}

注意:无需在c中投射malloc,它可以隐藏错误。