Creating a 2D string array in C

Question

So basically I am trying to read from a huge text file and need to store the data in a 2D string array in C. But I am getting a segmentation fault every time.

Here is the code I use to create the array:

Y = 3
X = 12
char ***some_array=NULL;
some_array = (char ***)malloc(Y * sizeof(char *));
    for (int i=0; i<Y; i++)
        for (int j=0; j<X; j++){
            some_array[i] = (char **)malloc(X * sizeof(char *));
            some_array[i][j] = (char *)malloc(16 * sizeof(char));
        }

So technically, I am creating a 3D char array for this means. Am I doing something wrong here?

So technically, I am creating a 3D char array... No, you're creating a one-dimensional array of pointers to multiple one-dimensional arrays of pointers to multiple one-dimensional arrays of char[16]. — Andrew Henle
– Andrew Henle, Commented Dec 26, 2020 at 14:37

dreamcrash · Accepted Answer · 2020-12-26 15:24:10Z

1

You need to move the allocation of some_array[i] = (char **)malloc(X * sizeof(char *)); from the inner most loop to the outer loop. Moreover, you should not cast the malloc return value. For the final code:

Y = 3
X = 12
char ***some_array = NULL;
some_array = malloc(Y * sizeof(char *));
    for (int i = 0; i < Y; i++){
        some_array[i] = malloc(X * sizeof(char *));
        for (int j = 0; j < X; j++){
            some_array[i][j] = malloc(16 * sizeof(char));
        }
    }

Alternatively you can create a statically allocated array:

char some_array [X][Y][16];

edited Dec 26, 2020 at 15:24

answered Dec 26, 2020 at 14:32

dreamcrash

52.5k26 gold badges114 silver badges138 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Jarvis · Accepted Answer · 2020-12-26 14:32:41Z

1

Error is in your loop:

for (int i=0; i<Y; i++) {
    some_array[i] = (char **)malloc(X * sizeof(char *));
    for (int j=0; j<X; j++){
        some_array[i][j] = (char *)malloc(16 * sizeof(char));
    }
}

You are supposed to allocate memory to some_array[i] outside the inner loop.

answered Dec 26, 2020 at 14:32

Jarvis

8,5923 gold badges33 silver badges61 bronze badges

Collectives™ on Stack Overflow

Creating a 2D string array in C

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related