Discussion :

[stmastma]

Petite question :

J'ai un code très simple dont voici l'extrait :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
int main() {
char inputuser;
int inputuserINT;
int i;
 
printf("enter the number \n");
scanf("%s", &inputuser);
 
intputuserINT = atoi(&inputuser);
printf("%d \n", inputuserINT);
return 0;
}

Pourquoi le fait de déclarer char *inputuser me donne une erreur (argument 1 of 'atoi' from incompatible pointer type...) ?
Finalement ici j'ai fait une allocation de valeur dans une variable mais ce n'est pas vraiment un pointeur, aurais je dû déclarer un pointeur ?
merci

[Obsidian]

Bonsoir,

C'est celle du dessus qui pose problème : tu as déclaré « char inputuser » et non « char * inputuser ». Ta variable « inputuser » est donc faire pour contenir un caractère, et pas un pointeur vers un caractère (ou une chaîne de caractères) comme on s'y attend.

Bon courage.

[Sve@r]

Petite question :

J'ai un code très simple dont voici l'extrait :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
int main() {
char inputuser;
int inputuserINT;
int i;
 
printf("enter the number \n");
scanf("%s", &inputuser);
 
intputuserINT = atoi(&inputuser);
printf("%d \n", inputuserINT);
return 0;
}

Pourquoi le fait de déclarer char *inputuser me donne une erreur (argument 1 of 'atoi' from incompatible pointer type...) ?

Bonjour

Parce qu'atoi() veut comme paramètre un char * (char étoile = adresse d'un caractère). Or, si tu déclares "char *inputuser" alors inputuser est un char *. Mais &inputuser (l'adresse d'un char étoile) est donc un char ** (char étoile étoile = adresse de l'adresse d'un caractère). Et l'adresse d'une adresse n'est certainement pas l'adresse d'un char...

En fait, le gros pb c'est que tu veux stocker un nombre (donc composé de plusieurs caractères) dans une variable prévue pour n'en stocker qu'un seul (un char). Là, ça ne peut que partir dans la semoule...

Code c :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
int main() {
    char inputuser[100];
    int inputuserINT;
 
    printf("enter the number \n");
    scanf("%s", inputuser);
 
    intputuserINT = atoi(inputuser);
    printf("%d \n", inputuserINT);
    return 0;
}

[Steph_ng8]

Bonjour.

Si tu n'as jamais besoin du nombre entré sous forme de chaîne de caractères, tu n'as pas besoin de le lire en tant que chaîne de caractères pour le convertir en nombre.
Il te suffit de le lire directement en tant que nombre.
Tu es sûr d'avoir correctement lu la documentation sur la fonction scanf ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
int inputuser;
scanf("%d", &inputuser);

Sinon, la on n'est pas obligé de passer l'adresse de la variable dans laquelle on stocke les données lues en paramètre à la fonction scanf.
Si c'est déjà un pointeur, on peut l'utiliser directement :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
int *pi;
scanf("%d", pi);
 
float *px;
scanf("%f", px);
 
char *pc;
scanf("%c", pc);
 
/* etc. */

Il va de soi qu'au moment de l'appel à scanf, les pointeurs pointent sur une zone mémoire valide (allouée statiquement ou dynamiquement).


Autre chose, si tu veux lire une chaîne de caractères depuis un flux, il est conseillé d'utiliser les fonctions de la famille gets plutôt que scanf("%s", str);.
FAQ C: Comment lire une ligne de manière securisée ?

[Obsidian]

Il va de soi qu'au moment de l'appel à scanf, les pointeurs pointent sur une zone mémoire valide (allouée statiquement ou dynamiquement).

«… que les pointeurs DOIVENT pointer sur une zone mémoire valide (allouée statiquement ou dynamiquement). ».

Autrement, présenté comme ça, on pourrait croire qu'il suffit de reproduire ton programme tel quel pour que ça marche automatiquement.

[Steph_ng8]

@Obsidian Merci de la précision.
C'est ce que je voulais exprimer, mais j'ai écrit trop vite...

@stmastma Au temps pour moi...
FAQ C: Pourquoi est-il déconseillé d'utiliser scanf() ?
On dira que ça fait longtemps que je n'ai pas fait de C...

[stmastma]

merci à tous.