Errore di compilazione GCC: il formato '% c' si aspetta argomento di tipo 'char *', ma l'argomento 2 ha tipo 'int' [-Wformat]

Question

Ok, sono un noob con C, ma penso che il codice sia di base e semplice. Questo programma è per un incarico universitario e si suppone che abbia la funzione 'isdigit()' in esso. Ecco il codice

//by Nyxm 
#include <stdio.h> 
#include <ctype.h> 

main() 
{ 
    char userChar; 
    int userNum, randNum; 
    srand(clock()); 
    printf("\nThis program will generate a random number between 0 and 9 for a user to guess.\n"); 
    /*I changed it from '1 to 10' to '0 to 9' to be able to use the isdigit() function which 
    will only let me use a 1 digit character for an argument*/ 
    printf("Please enter a digit from 0 to 9 as your guess: "); 
    scanf("%c", userChar); 
    if (isdigit(userChar)) 
    { 
      userNum = userChar - '0'; 
      randNum = (rand() % 10); 
      if (userNum == randNum) 
      { 
        printf("Good guess! It was the random number.\n"); 
      } 
      else 
      { 
        printf("Sorry, the random number was %d.\n", randNum); 
      } 
    } 
    else 
    { 
      printf("Sorry, you did not enter a digit between 0 and 9. Please try to run the program again.\$ 
    } 
} 

Quando provo a compilare, ottengo il seguente errore

week3work1.c: In function ‘main’: 
week3work1.c:14:2: warning: format ‘%c’ expects argument of type ‘char *’, but argument 2 has type ‘int’ [-Wformat] 

Cosa diavolo sta succedendo? Sono alla disperata ricerca di aiuto. Qualsiasi aiuto. Sto seriamente sul punto di rinunciare a questo programma. Perché sta dicendo che si aspetta un argomento di "char *" quando il mio libro di testo mostra che "% c" è per il normale ole "char"? Sto usando nano, gcc e Ubuntu se questo fa la differenza.

Answer 1

Per scanf(), è necessario passare un puntatore a un char, altrimenti non ha alcun modo per memorizzare il carattere, dal momento che ha detto char sarebbe passato in termini di valore. Quindi hai bisogno di &userChar.

Diciamo che userChar è 0 prima della chiamata. Con il vostro codice attuale, si sta praticamente facendo questo (per quanto riguarda l'utilità va):

scanf("%c", 0); 

quello che vuoi è questo:

scanf("%c", some-location-to-put-a-char); 

Che è &userChar.

Il man pagina dedicata agli scanf menzioni questo:

c  Matches a sequence of characters whose length is specified by 
      the maximum field width (default 1); the next pointer must be a 
      pointer to char, and there must be enough room for all the char‐ 
      acters (no terminating null byte is added). The usual skip of 
      leading white space is suppressed. To skip white space first, 
      use an explicit space in the format. 

Answer 2

sostituire scanf("%c", userChar); con scanf("%c", &userChar);.

Answer 3

È necessario passare in un puntatore al posto del valore del char.

scanf("%c",&userChar);