Thread: Pass array of char pointers by reference

Code:

malloc(20)

this is to cold :P
I would write it like this

Code:

malloc(sizeof(char) * 20);

on the other hand, if a pointer to the first element of the array is what is actually passed as a parameter, so don't worry about the size of the array.
On other words, no matter if the size of the array is BIG or small, a pointer is going to be passed as parameter.

Originally Posted by Joshun

I know that to pass an array of char pointers by value you would do

Code:

char *str[]

, but how would you do it by reference?

Code:

char **str[]

in a function definition, and using the & operator with the array of char pointers doesn't seem to work.

In C there is no such thing as "pass by reference". To simulate pass by reference, a pointer is passed. Additionally, when you pass an array in C, the name of the array acts as a pointer to the start of the array. Strings are just a special array where there's a gentleman's agreement that the final element shall be '\0'.

To understand, put your example into the context of a compilable program:

Code:

void fill_strings(char *str[], int nstrings)
{
    for (int i=0; i < nstrings; i++)
        strcpy(str[i], "hello world");
}

void print_strings(char *str[], int nstrings)
{
    for (int i=0; i < nstrings; i++)
        printf("string %d: %s\n", i, str[i]);
}

int main(void)
{
    char *index[10];
    int i;
    for(i=0; i<10; i++)
    {
        index[i] = malloc(20);
    }
    fill_strings(index, 10);
    print_strings(index, 10);
    
    return 0;
}

When you call fill_strings and print_strings, the parameter index just is an address of where this array happens to be stored in your memory. String copying only takes place once inside the actual function, not in the parameter passing.

1. By the way, you can write "char **str" in the parameter list for the same effect.
2. malloc(20) is fine

I am aware about that. But, in my way you don't have to think much, you just follow the same procedure for every data type

It's only a suggestion.

// As I and the c99 said, what you have there is correct

And you are checking to see if every malloc was successful and freeing the memory at the end, right?...

Code:

int main(void)
{
    char *index[10];
    int i;
    for(i=0; i<10; i++)
    {
        if ((index[i] = malloc(20 * sizeof(**index))) == NULL)
        {
            break;
        }
    }


    fill_strings(index, 10);
    print_strings(index, 10);


    while (i > 0)
    {
        free(index[--i]);
    }


    return EXIT_SUCCESS;
}