Problem with type-casting array of strings in C

Question

I am trying to read a large list of English words from a text file to array of strings. The number of words is 2016415, and maximum length of a word is 69 characters.

If I define array like "char data[2016415][70]; " then I get stack overflow when I run the program.

So I am trying to use calloc() instead, however I can't understand how should I type-cast it so that it becomes equivalent to "char data[2016415][70];".

The following program returns "passing arg 1 of `fgets' makes pointer from integer without a cast" warning during compiling. And when I execute it, it gets "Exception: STATUS_ACCESS_VIOLATION" problem.

Can you help me?

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

int main(void){
char *data;  //data[2016415][70];

int i;
FILE *fsol;

fsol = fopen("C:\\Downloads\\abc\\sol2.txt","r");

data = (char*) calloc(2016415,70);

for(i=0;i<2016415;i++){
    fgets(data[i] , 70 , fsol);
}

fclose(fsol);

return 0;

}

Have you tried using fgets(&data[i*70] , 70 , fsol); instead? — Il-Bhima
– Il-Bhima, Commented Jun 29, 2010 at 23:09
Here's a related question from earlier today: stackoverflow.com/questions/3144132/… — Cogwheel
– Cogwheel, Commented Jun 29, 2010 at 23:13
Out of curiosity, how this even compiled? There is no implicit conversion from char to char*, which would be required for the call to fgets to be valid. WTH is it a warning and not an error? — Pavel Minaev
– Pavel Minaev, Commented Jun 30, 2010 at 0:05

Anon. · Accepted Answer · 2010-06-29 23:13:07Z

1

calloc just allocates a big swath of memory - not an array of pointers to other arrays.

fgets expects a pointer to the memory location it should dump it's stuff at.

So instead of giving it the contents of data[i], you want to give it the address of data[i] so it can put its stuff there.

fgets(&data[i], 70, fsol);

You'll probably also need to adjust your loop so that it goes up by 70-odd characters at a time rather than one.

answered Jun 29, 2010 at 23:13

Anon.

60.3k9 gold badges84 silver badges86 bronze badges

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

1 Comment

Sunny88 Over a year ago

Thanks! I used fgets(&data[i*70] , 70 , fsol); and it works fine now.

Amardeep AC9MF · Accepted Answer · 2010-06-29 23:40:37Z

1

Okay, sorry about the previous suggestion. I forgot how horrible arrays can be. This one is tested with a small data set of 10 words, but it should scale to your word count. Note that fgets() seems to pull in the line endings as part of the preceding word.

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

#define MAX_WORD_CNT 2016415
#define MAX_WORD_LEN 70

int main(void)
{
    char *data;  //data[2016415][70];

    int i;
    FILE *fsol;

    fsol = fopen("C:\\Downloads\\abc\\sol2.txt","r");

    data = (char*) calloc(MAX_WORD_CNT, MAX_WORD_LEN);

    // check for valid allocation
    if (data == NULL)
    {
        return 1;
    }

    for(i=0; i<MAX_WORD_CNT; i++)
    {
        fgets(&data[i * MAX_WORD_LEN], MAX_WORD_LEN, fsol);
    }

    fclose(fsol);

    return 0;
}

edited Jun 29, 2010 at 23:40

answered Jun 29, 2010 at 23:11

Amardeep AC9MF

19.2k5 gold badges42 silver badges51 bronze badges

4 Comments

Anon. Over a year ago

This just crashes when fgets tries to dereference the null pointer you're giving it.

Amardeep AC9MF Over a year ago

It shouldn't be a NULL pointer if it gets past the if statement.

Anon. Over a year ago

Still not it. calloc fills its allocation with zeroes - hence, data[i] will always be zero. Bam, NPE.

Nyan Over a year ago

@Anon, there's nothing to BAM. relax and take your time ;)

Bwmat · Accepted Answer · 2010-06-29 23:46:55Z

0

Here's how I would allocate the array

char **data = malloc(MAX_WORD_CNT * sizeof(char *));
for(int i = 0; i < MAX_WORD_CNT; i++)
    data[i] = malloc(MAX_WORD_LEN);

you might want to add some error checking for malloc though.

answered Jun 29, 2010 at 23:46

Bwmat

4,7243 gold badges30 silver badges51 bronze badges

Comments

Paul Richter · Accepted Answer · 2010-06-30 02:11:03Z

0

data is a pointer to char (also addressable as an array of char), so data[i] is a single char. fgets expects a pointer to char but you're passing it a single char; that's why you're getting the warning, you're trying to use a char (integer) as a pointer.

When you run the program, it then takes that single char argument and interprets it as a pointer to char, hence the access violation because the value of that char is not a valid address.

So, in your loop you should pass fgets a pointer into data and increment that by 70 with each iteration. You can use the "pointer to an array element" form &data[i] and increment i, or the simple pointer form, with another pointer variable initially set to data, and itself incremented.

edited Jun 30, 2010 at 2:11

answered Jun 29, 2010 at 23:59

Paul Richter

6,3512 gold badges22 silver badges25 bronze badges

Comments

R.. GitHub STOP HELPING ICE · Accepted Answer · 2010-06-30 05:32:49Z

0

The answer is simple: you DON'T cast it. Casting the results of malloc/calloc/etc. has no purpose but can have the side-effect of hiding a major bug if you forgot to include stdlib.h. The return type of these allocation functions, which is void *, will automatically be cast to whatever you need.

If you really want to know the type, it's (char (*)[70]). But please don't actually obfuscate your program by writing that. (Unless you're actually writing C++, in which case you should have tagged your question C++ and not C, or better yet used new.)

answered Jun 30, 2010 at 5:32

R.. GitHub STOP HELPING ICE

217k36 gold badges404 silver badges744 bronze badges

Collectives™ on Stack Overflow

Problem with type-casting array of strings in C

5 Answers 5

1 Comment

4 Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

1 Comment

4 Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related