Fonction atoi et pointeurs

Bonsoir,

J'ai actuellement, dans un programme:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27

#include <stdio.h> #include <stdlib.h> #include <string.h> #define MAX 50   char x1_string[MAX]; char x2_string[MAX];   int main(int argc, char* argv[]){ strcpy(x1_string, argv[1]); strcpy(x2_string, argv[2]);   int x1_length=strlen(x1_string); int x2_length=strlen(x2_string);   // ...   int x1_t[MAX]={0}; int x2_t[MAX]={0};   int j=x1_length; while(j>=0){ x1_t[i]=atoi(x1_string[j]); i--; j--; }

---

Pourquoi reçois-je un avertissement ``passing argument 1 of `atoi' makes pointer from integer without a cast''?

Merci.

Bonjour,
la réponse est simple: x1_string est de type "char *" donc x1_string est de type "char" qui grâce à la promotion des entiers devient "int". Or atoi attends une chaîne de caractère donc il te dit qu'il doit convetir en entier en chaîne de caractère...
Peut-être n'as-tu pas bien compris la sémantique de atoi:

Code:

---

1 2 3 4 5 6

int i=atoi("585");// i=585 const char *s="35"; int j=atoi(s);// j=35 // MAIS int k=atoi('0');// erreur, cela va probablement faire une erreur de segmentation

---

J'espère que cela réponds à ta question.

parce que atoi() attend un pointeur sur chaine de caractère et que tu lui passe un char. Relis la documentation de la fonction.

[Edit] Grilled :aie:

Citation:

---

Envoyé par lucaordi

J'ai actuellement, dans un programme:

---

Ce code est inutilement complexe. Si tu veux récupérer une liste de paramètres numériques, il suffit de faire ceci (atoi() est obsolète, on utilise strtol()).

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37

#include <stdio.h> #include <stdlib.h>   #define MAX 50   int main (int argc, char *argv[]) { if (argc > 1 && argc < 50) { int ax[MAX]; int n;   /* extraire les arguments de la ligne de commande */ { int j = 0; int i; for (i = 1; i < argc; i++) { ax[j] = strtol (argv[i], NULL, 10); j++; } n = j; }   /* afficher les valeurs stockees. */ { int i; for (i = 0; i < n; i++) { printf ("%d ", ax[i]); } printf ("\n"); } } return 0; }

---

Ce que je souhaite faire, c'est décomposer une chaîne (donnée par l'utilisateur en paramètre au programme) en chacun des chiffres qui composent cette dernière (on la suppose ne contenant uniquement que des chiffres); de manière à faciliter les calculs, je pensais simplement, plutôt que de la transformer directement en entier puis de décomposer le nombre ainsi déterminé, lire caractère par caractère de la chaîne, et retransformer chaque caractère considéré comme faisant partie de la chaîne donnée en un chiffre, stocké dans mon tableau.

Le dernier code soumis va-t-il dans ce sens?

Merci.

Citation:

---

Envoyé par lucaordi

Ce que je souhaite faire, c'est décomposer une chaîne (donnée par l'utilisateur en paramètre au programme) en chacun des chiffres qui composent cette dernière (on la suppose ne contenant uniquement que des chiffres); de manière à faciliter les calculs,

---

Je ne vois pas en quoi ça facilite les calculs. strtol() fait la conversion "chaine numérique" -> binaire directement. Je ne vois pas ce que tu cherches à compliquer ni pourquoi...

Citation:

---

Envoyé par Emmanuel Delahaye

Je ne vois pas en quoi ça facilite les calculs. strtol() fait la conversion "chaine numérique" -> binaire directement. Je ne vois pas ce que tu cherches à compliquer ni pourquoi...

---

Pourquoi utiliser le binaire? La chaîne donnée contient des chiffres de 0 à 9, et la saisie est donc un nombre entier!

Citation:

---

Envoyé par lucaordi

Pourquoi utiliser le binaire? La chaîne donnée contient des chiffres de 0 à 9, et la saisie est donc un nombre entier!

---

Un nombre entier est codé en binaire...

Par 'binaire', on entend la valeur numérique, par opposition à sa représentation textuelle.

Citation:

---

Envoyé par Emmanuel Delahaye

Par 'binaire', on entend la valeur numérique, par opposition à sa représentation textuelle.

---

C'est un abus de langage, mais je ne le connaissais pas.

La nuance est qu'ici, pour économiser de la mémoire, et, comme je ne stocke à chaque fois qu'un chiffre, il ne me semble pas opportun d'utiliser strtol(), qui convertit en un type long. En effet, le stockage d'un (et un seul) chiffre est déjà réalisé correctement via un type short. Le fait de renvoyer un long me semble exagéré, non?

Citation:

---

Envoyé par lucaordi

La nuance est qu'ici, pour économiser de la mémoire, et, comme je ne stocke à chaque fois qu'un chiffre, il ne me semble pas opportun d'utiliser strtol(), qui convertit en un type long. En effet, le stockage d'un (et un seul) chiffre est déjà réalisé correctement via un type short. Le fait de renvoyer un long me semble exagéré, non?

---

Franchement, quelle importance ?

Ensuite, tu stockes le retour de strtol() dans ce que tu veux du moment qu'il n'y a pas de débordement.

Code:

---

int x = strtol (chaine, NULL, 10);

---

Mais si la chaine ne comporte qu'un chiffre de '0' à '9', tu peux utiliser l'astuce du " - '0' " si tu veux :

Code:

---

int x = chaine[0] - '0';

---

Citation:

---

Envoyé par Emmanuel Delahaye

Franchement, quelle importance ?

---

Je dois malheureusement rendre un travail, et l'utilisation de fonctions non ``appropriées'' est pénalisée. N'existe-t-il donc pas une autre fonction?

Merci.

Citation:

---

Envoyé par lucaordi

Je dois malheureusement rendre un travail, et l'utilisation de fonctions non ``appropriées'' est pénalisée. N'existe-t-il donc pas une autre fonction?

---

Non, mais j'ai indiqué une autre méthode si il n'y a qu'un chiffre...

Si c'est une histoire de stockage, je rappelle que ceci marche:

Code:

---

short x = (short)strtol(chaine, NULL, 10);

---

Tu peux même faire la même chose avec un unsigned char si tu y tiens...

Citation:

---

Envoyé par lucaordi

Ce que je souhaite faire, c'est décomposer une chaîne (donnée par l'utilisateur en paramètre au programme) en chacun des chiffres qui composent cette dernière (on la suppose ne contenant uniquement que des chiffres); de manière à faciliter les calculs, je pensais simplement, plutôt que de la transformer directement en entier puis de décomposer le nombre ainsi déterminé, lire caractère par caractère de la chaîne, et retransformer chaque caractère considéré comme faisant partie de la chaîne donnée en un chiffre, stocké dans mon tableau.

---

Si je comprend ta question (?), il suffit de lire la chaîne caractère par caractère jusqu'à la fin (marquée par '\0' comme toute chaîne C qui se respecte) et puisque la chaîne ne comporte que des caractères "chiffres décimaux" de transformer chaque caractère "chiffre" lu en sa valeur.
Par exemple :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

char x1_string[] = "123456"; char x1_t[MAX]; int i; for (i=0;x1_string[i]!= '\0'; i++) x1_t[i]= x1_string[i]-'0'; x1_t[i]= -1; // Par exemple, parce qu'il faut bien noter la fin du tableau // - avec quelque chose qui ne peut figurer normalement dedans // - ou en mémorisant la valeur de i en sortie de boucle // si on veut l'afficher : for (i=0;x1_t[i]!= -1; i++) printf("%d\n", x1_t[i]);

---

Oui, excepté qu'il est nécessaire de commencer par la fin de la chaîne (pour des raisons dues à l'entrée donnée et à des règles d'énoncé à respecter); par conséquent, l'idée de vérifier si l'on n'a pas encore atteint '\0' ne fonctionnera plus :cry:

Citation:

---

Oui, excepté qu'il est nécessaire de commencer par la fin de la chaîne

---

Alors, si il n'y a que ça, commence par la fin ! C'est pas difficile à adapter

Code:

---

1 2	for (i=strlen(x1_string)-1;i>=0; i--) x1_t[i]= x1_string[i]-'0';

---

Effectivement.

Merci!