CODEWITHSUNDEEP
carraysinitializer-list
Arnold
Arnold

Reputation: 185

Error in initializing array in C

Below is my code:

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


    typedef int index_key;  
    typedef char text_t;          

    text_t *create_text()
{
    //text_t text[SIZ];
    text_t *c;
    text_t text[]="fl";
    c= text;
    return c;
}


    int main()
    {

        text_t * create();


        return 0;

    }

I get an error - expected expression before ‘]’ token. Why is this error occuring? Isn't text[] a global declaration and I can access it anywhere? What is wrong in this program and how should I correct it.

Upvotes: -1

Views: 167

Answers (3)

R Sahu
R Sahu

Reputation: 206567

I see the following problems:

text_t text[];

is a declaration, not a definition. You have to add a line that defines text, such as:

text_t text[100];

The line

text[]="sldk";

is wrong on two accounts.

  1. You cannot assign to an array using the = operator.

  2. You cannot use text[] to access the array.

Here's a fixed version of your program.

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

typedef int index_key;  
typedef char text_t;

// This is a forward declaration of the array.
text_t text[];

text_t *create()
{
    text_t *c;

    // One way to get some string into a char array.
    strcpy(text,"sldk");

    c = text;
    return c;
}

// This is the definition of the array.
text_t text[100];

int main()
{
    text_t * create();
    return 0;
}

Update

Regarding your updated create_text function...

It should work since string literals are stored in read-only memory of the program. It will lead to problems as soon as you try to change anything in it. It's better to use a safer approach, such as:

text_t *create_text()
{
    text_t *c = malloc(20);
    strcpy(c, "fl");
    return c;
}

Upvotes: 1

Ravi Chandra
Ravi Chandra

Reputation: 687

The C language does not have a built in string data type unlike C++, Java or C#. The strings are represented as arrays of characters in C. So any array in C should have some size.

int numbers[10];
char string[50];

The C function which operate on these types of strings ( like strlen which calculates the length of the strings expects a 'null' or '\0' character at the end of the array.

Initializing a character array with string literal like "test" will automatically insert a null character at the end.

char str[10] = "test";
characters stored {'t','e','s','t','\0'}

If you are initializing it with comma separated characters, you need to explicitly specify this character.

char str[10] = {'t','e','s','t', '\0'}; //explicit null character

So, generally the array size is one more than the maximum size of the string you want to store. So it is quite common to see declaration like the following. 

char name[MAXLEN+1];

If you are using C++, you can use a build-in data type called string.

string str;

Hope this helps

Upvotes: 0

Sourav Ghosh
Sourav Ghosh

Reputation: 134286

You cannot have an array definition like 

text_t text[];

Either specify the size,

#define SIZ 256   //arbitary value

text_t text[SIZ];

or use initializer.

text_t text[] = {`A`, `B`, `C`};

EDIT:

As per the latest addition, please be informed that "sldk" (as you've iused) and {'s', 'd', 'l', 'k'} (as i've suggested) are not the same. The former is a string literal while the later being initalizer list of chars. You can use the second in your case, not the first one.

EDIT 2

That said, your create_text() function is wrong. Effectively, you're returning the address of a local variable text. Incorrect. Will invoke UB.

Upvotes: 1

Related Questions