Question

I’m writing a method that receives a number l and returns a vector of size l with random numbers. I have this code, but does not work

#include <time.h>    

 int makea (int z) {
    int a1[z];
    int i;

    for (i = 0; i < tam; i++) {
        a1[i]=srand(time(0));
    }
    return a1;
 }

These are the errors that the compiler returns me

arrays1.c: In function ‘makea’:
arrays1.c:12: error: void value not ignored as it ought to be
arrays1.c:14: warning: return makes integer from pointer without a cast
arrays1.c:14: warning: function returns address of local variable

I think is a problem of pointers… but I’m not really sure

Answer 1

A few problems:

1. Your array is allocated on the stack, meaning that when your function exits, the memory you return will be invalid
2. In C, you cannot return an array from a function, it must first decay into a pointer.

So, to fix, use malloc and a pointer:

int *makea (int z) {
    int *a1 = malloc(sizeof(int) * z);
    int i;

    srand(time(NULL));
    for (i = 0; i < tam; i++) {
        a1[i]= rand();
    }

    // remember to free a1 when you are done!
    return a1;
}

Also note that using malloc can sometimes basically grant you the ‘random number’ scenario for free, negating the need to loop through the elements as the value returned from malloc is garbage (and thus random numbers).

However, also note that malloc is implementation-specific, meaning that an implementation could theoretically clear the memory for you before returning it.