Discussion :

[lucaordi]

Bonsoir,

J'ai actuellement, dans un programme:

Code :

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
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.

[amaury pouly]

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 :

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

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.

[ram-0000]

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

[Emmanuel Delahaye]

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 :

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
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;
}

[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.

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

Merci.

[Emmanuel Delahaye]

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...

[lucaordi]

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!

[Emmanuel Delahaye]

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.

[lucaordi]

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?

[Emmanuel Delahaye]

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 :

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

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 :

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

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

[lucaordi]

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.

[Emmanuel Delahaye]

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...

[Médinoc]

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

Code C :

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

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

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

[diogene]

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 :

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

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]);

[lucaordi]

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

[diogene]

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 :

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

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

[lucaordi]

Effectivement.

Merci!