passing argument 1 of 'functions' makes a pointer from an integer without a cast

Question

i have to program a function which asks for an array and its size, and then prints its values. The main function must call 2 other functions, both of them take two arguments (an array and its size). The first one asks for values to fill the array, the second one prints each value.

Im new to C and to programming in general, so any tips and useful info is very welcomed. I know very little about pointers aswell.

void requestarray(int a[], int  size){       
int i;
 for (i=0;i<size;i++)
 {
  printf("Insert value for %d position of the array\n",i);
  scanf("%d\n",&a[i]);
 }
}

void printarray(int a[], int size){     
int i;

 for (i=0;i<size;i++){
 printf("%d\n",a[i]);
 }
}

int main(void){
int size;

  printf("Insert array length\n");
  scanf("%d\n",&size);

int a[size];
requestarray(&a[size],size);
printf("\n");
printf("The array is\n");
printarray(&a[size],size);

return 0;
}

the program compiles but does not work. when i debug it using gdb, i place a breakpoint after

 int a[size];

then i print the value of a and it returns an array with size length and random values (mainly big numbers). Then i place a breakpoint after

requestarray(&a[size],size);

(which should modify the values of the array) but it returns the same values as printed in the previous breakpoint.

if i change the code in the main function like this

 requestarray(a[size],size);
printf("\n");
printf("The array is\n");
printarray(a[size],size);

the debugger gives me this warning

arreglo1.c: In function ‘main’:
arreglo1.c:33:14: warning: passing argument 1 of ‘requestarray’ makes pointer from integer without a cast [-Wint-conversion]
resquestarray(a[size],size);
              ^
arreglo1.c:8:6: note: expected ‘int *’ but argument is of type ‘int’
 void requestarray(int a[], int  size){
      ^~~~~~~~~~~~
arreglo1.c:36:16: warning: passing argument 1 of ‘imprimearreglo’ makes pointer from integer without a cast [-Wint-conversion]
 printarray(a[size],size);
            ^
arreglo1.c:17:6: note: expected ‘int *’ but argument is of type ‘int’
 void printarray(int a[], int size){
      ^~~~~~~~~~~~   

Answer 1

scanf ("%d\n", &a[i]); is simply wrong (notice having to press Enter or provide more input than expected?) scanf will not process escape characters in that manner. Instead a literal '\n' causes scanf to disregard all whitespace forcing you to enter a non-whitespace character before it returns.

Whatever character you type to cause scanf to return does not match anything in your format string resulting in that character being left unread in stdin and will be used as your next attempted input.

That is why..., you must validate EVERY input by checking the return of each input function used. For instance in main(), e.g.

    printf ("Insert array length: ");
    if (scanf ("%d", &size) != 1) {     /* validate EVERY input */
        fputs ("error: invalid integer input.\n", stderr);
        exit (EXIT_FAILURE);
    }

and again in requestarray(), e.g.

    for (i = 0; i < size; i++) {
        printf ("Insert value for %d position of the array: ", i + 1);
        if (scanf ("%d", &a[i]) != 1) {     /* validate EVERY input */
            fputs ("error: invalid integer input.\n", stderr);
            exit (EXIT_FAILURE);    /* handle error condition as desired */
        }
    }

Otherwise on a matching or input failure, scanf stops extracting characters from your input buffer (stdin here), the offending character is left in your input buffer just waiting to cause the same failure on your next call to scanf.

When you pass an array, you can simply use a pointer as the array is converted to a pointer on access, see: C11 Standard - 6.3.2.1 Other Operands - Lvalues, arrays, and function designators(p3). So you can write your function as simply:

void requestarray (int *a, int  size)
{       
    int i;

    putchar ('\n');         /* tidy up with newline before looping for input */

    for (i = 0; i < size; i++) {
        printf ("Insert value for %d position of the array: ", i + 1);
        if (scanf ("%d", &a[i]) != 1) {     /* validate EVERY input */
            fputs ("error: invalid integer input.\n", stderr);
            exit (EXIT_FAILURE);    /* handle error condition as desired */
        }
    }
}

(note there is nothing wrong with passing int a[]. Also note how the code is spaced a bit wider making it easier to read (especially for older eyes..))

Putting it altogether you could do:

#include <stdio.h>
#include <stdlib.h>     /* for EXIT_FAILURE define */

void requestarray (int *a, int  size)
{       
    int i;

    putchar ('\n');         /* tidy up with newline before looping for input */

    for (i = 0; i < size; i++) {
        printf ("Insert value for %d position of the array: ", i + 1);
        if (scanf ("%d", &a[i]) != 1) {     /* validate EVERY input */
            fputs ("error: invalid integer input.\n", stderr);
            exit (EXIT_FAILURE);    /* handle error condition as desired */
        }
    }
}

void printarray (int *a, int size)
{     
    int i;

    for (i = 0; i < size; i++){
        printf ("%d\n", a[i]);
    }
}

int main(void) {

    int size;

    printf ("Insert array length: ");
    if (scanf ("%d", &size) != 1) {     /* validate EVERY input */
        fputs ("error: invalid integer input.\n", stderr);
        exit (EXIT_FAILURE);
    }

    int a[size];            /* Variable Length Array of 'size' elements */

    requestarray (a, size);

    puts ("\nThe array is\n");  /* there is no convertion, puts will do */
    printarray (a ,size);

    return 0;
}

Example Use/Output

$ ./bin/inputarray
Insert array length: 5

Insert value for 1 position of the array: 1
Insert value for 2 position of the array: 2
Insert value for 3 position of the array: 3
Insert value for 4 position of the array: 4
Insert value for 5 position of the array: 5

The array is

1
2
3
4
5

Look things over and let me know if you have further questions.