Discussion :

[abdou monta]

bonjour, j'ai le code suivant:

Code :

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

1
2
3
4
 
ch = m[l][c].possibilities;
p = atoi (ch);
m[l][c].cnt = p;

m est une matrice de cellule.
p entier.
cellule est une structure de 2 champs, possibilities est une chaine, cnt est un entier.

je veux convertir la chaine possibilities en un entier p et l'affecter à cnt
mais je trouve l'avertissement suivant:
warning: passing argument 1 of 'atoi' makes pointer from integer without a cast [enabled by default]|

[Médinoc]

Quel est le type de ch?

[abdou monta]

ch est une chaine

[fred1599]

ch est une chaine

Alors tu n'as pas le droit de faire cela, utilise strcpy pour faire ce que tu veux.

Edit: T'es vraiment sûr que c'est une chaîne de caractères ?

[Médinoc]

ch est une chaine

Menteur.

Montre-nous sa déclaration.

...

Je parie que c'est char ch;, qui déclare UN caractère, et non pas une chaîne.

[Bktero]

On ne la fait pas à l'envers à Médinoc

[Elijha]

Bonjour,

warning: passing argument 1 of 'atoi' makes pointer from integer without a cast [enabled by default]

Est ce que ch un pointeur d'unsigneg char ?

Si c'est le cas, soit tu changes son type en char*, soit tu castes lors de l'appel

Code :

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

p = atoi ((const char*)ch) ;

Nota: Attention avec le cast, si ch n'est ni char* ou unsigneg char*, ça peut être dangeruex et de se trouver avec un segfault.

Bonne continuation.

[jlliagre]

ch est une chaine

Il n'existe pas de type de variable pour les chaînes de caractères en C.
Il faut nous montrer la déclaration de ta structure et en particulier du champs possibilities.

[Obsidian]

Si c'est le cas, soit tu changes son type en char*, soit tu castes lors de l'appel

Code :

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

p = atoi ((const char*)ch) ;

Nota: Attention avec le cast, si ch n'est ni char* ou unsigneg char*, ça peut être dangeruex et de se trouver avec un segfault.

C'est pourquoi, dans cette situation, il ne faut pas le faire du tout.

Soit ch est déjà un « char * » (ce dont nous doutons) et dans ce cas, il n'y a pas de message d'erreur, soit ce n'en est pas un et là, le transtypage ne servira à rien puisqu'à la base, la variable ne contient rien de sensé.

On sait qu'il ne s'agit même pas d'un pointeur mais d'un entier (au vu du message d'erreur) mais, même si c'était un pointeur, cela n'aurait de sens que s'il s'agissait d'un « void * » et là, la norme C précise qu'il n'y a pas à se plaindre non plus (contrairement au C++).

[abdou monta]

Alors tu n'as pas le droit de faire cela, utilise strcpy pour faire ce que tu veux.

Edit: T'es vraiment sûr que c'est une chaîne de caractères ?

merci pour votre réponse.
j'ai déclaré ch de la manière suivante:

Code :

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

char ch[10];

Menteur.

Montre-nous sa déclaration.

...

Je parie que c'est char ch;, qui déclare UN caractère, et non pas une chaîne.

Pardons? Vous me nommer un menteur? Je ne sais pas pourquoi certaines personnes aiment ce que tu as dit. je parie que si un membre normale à dit ce que tu as déjà dis, il sera blocké, mais, et puisque tu es un "Expert Confirmé Sénior", les autres ont que d'aimer ton parole.

[Invité]

Bonsoir,

Étant donnée l'erreur que tu nous donnes :

warning: passing argument 1 of 'atoi' makes pointer from integer without a cast [enabled by default]|

qui nous dit clairement que tu ne lui passes ni un pointeur ni un tableau, et le fait que tu nous dises que tu utilises char ch[10]; qui est tout à fait acceptable par la fonction atoi... oui il y a une erreur quelque part.

Peut-être le message ne concerne pas la partie que tu nous montres, peut-être as-tu corrigé entre temps, peut-être caches-tu ch par un autre plus local qui lui n'est pas un tableau de char...

Toujours est-il, comme te l'a suggéré fred1599 qu'il te faut utiliser strcpy sur la ligne précédente.

[jlliagre]

Pardons? Vous me nommer un menteur?

Du calme, on n'est pas dans la cour de récréation mais dans un forum de professionnels de l'informatique. "menteur" était bien sûr à prendre au deuxième degré.

Pour revenir au sujet, si ch est déclaré comme çà:

Code :

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

char ch[10];

comment est déclaré la structure qui contient le champs "possibilities" ? Parce que:

Code :

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

ch = m[l][c].possibilities;

C'est plutôt louche ...

[Obsidian]

merci pour votre réponse.
j'ai déclaré ch de la manière suivante:

Code :

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

char ch[10];

C'est une manière valable de le faire, et tu ne devrais pas avoir de message d'erreur (il se peut que ça plante ensuite pour d'autres raisons, comme un terminateur manquant, mais c'est une autre histoire).

+1 pour Winjérome : « ch » doit être déclaré deux fois et ton tableau doit être masqué par une variable plus locale.

Dépose ton code entier ici (avec les balises [code] et [/code]) qu'on puisse y jeter un œil. C'est de cette façon que l'on perdra le moins de temps.

[abdou monta]

Du calme, on n'est pas dans la cour de récréation mais dans un forum de professionnels de l'informatique. "menteur" était bien sûr à prendre au deuxième degré.

C'est plutôt louche ...

Ne me dit pas de calmer! Je pense qu'il n'y a pas un deuxième degré dans un forum de "professionnels de l'informatique".

[abdou monta]

C'est une manière valable de le faire, et tu ne devrais pas avoir de message d'erreur (il se peut que ça plante ensuite pour d'autres raisons, comme un terminateur manquant, mais c'est une autre histoire).

+1 pour Winjérome : « ch » doit être déclaré deux fois et ton tableau doit être masqué par une variable plus locale.

Dépose ton code entier ici (avec les balises [code] et [/code]) qu'on puisse y jeter un œil. C'est de cette façon que l'on perdra le moins de temps.

Merci en tout cas, mais je ne trouve plus la passion pour consulter ce forum.

[Obsidian]

Merci en tout cas, mais je ne trouve plus la passion pour consulter ce forum.

Dans ce cas, bonne continuation. Mais sache que tu risqueras d'être confronté tôt ou tard à ce genre de remarques sur à peu près tous les forums. Il faudra apprendre à faire avec (jusqu'à un certain point quand même) si tu veux parvenir à tes fins.

[Médinoc]

Ce code minimal montre que la déclaration char ch[10]; ne pose pas de problème avec atoi()

Code C :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
/*
http://www.developpez.net/forums/d1400684/c-cpp/c/debuter/probleme-fonction-atoi/
*/
 
#include <stdio.h>
#include <stdlib.h>
 
int main(void)
{
	int a;
	char ch[10];
	ch[0] = '8';
	ch[1] = '\0';
	a = atoi(ch);
	printf("a = %d\n", a);
	return 0;
}

ce code compile sans warning sous Code::Blocks.

[jlliagre]

Ce code minimal montre que la déclaration char ch[10]; ne pose pas de problème avec atoi()
...

Certes, mais il est évident pour (à peu près) tout le monde que ch n'est pas déclaré avec ce type car sinon, ce n'est pas un warning à la ligne 3 mais une erreur à la ligne 2 que l'on aurait, un tableau n'étant pas affectable en C.