Discussion :

[jesse100]

J'ai une mission. Le programme consiste à imprimer la somme de tous les arguments de ligne de commande en C comme je l'ai lu dans cet article. J'ai essayé ce code, il compile mais génère une erreur après avoir passé des arguments dans la console. Ci-dessous le code.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
/* Printing sum of all command line arguments */
#include <stdio.h>
 
int main(int argc, char *argv[]) {
    int sum = 0, counter;
 
    for (counter = 1; counter <= argc; counter++) {
       sum = atoi(sum) + atoi(argv[counter]);
    }
    printf("Sum of %d command line arguments is: %d\n", argc, sum);
}

Après la compilation, il génère une erreur de segmentation (core dumped). Votre expérience peut résoudre mon problème.

[foetus]

Réfléchis 1 peu (<- je n'ai pas compilé, je peux me tromper )

La fonction atoi est obsolète : dans la documentation de atoi, il conseille d'utiliser strtol (<- 2 liens cplusplus.com en anglais)

ligne 8: /* ... */ atoi(sum) /* ... */ donc la variable sum est 1 "C-string beginning with the representation of an integral number", en gros 1 pointeur

[Sve@r]

Votre expérience peut résoudre mon problème.

Et ton cerveau? Il peut pas le résoudre???
sum est déjà un nombre. Qu'est-ce que tu vas le passer par atoi()??? Ou alors tu n'as rien pigé à cette fonction...

[dalfab]

Et en plus de l'erreur ligne 8 (passer au premier atoi() un paramètre qui n'est pas du tout une chaîne, d'ailleurs pourquoi ce atoi?);
il y a aussi une erreur ligne 7, qui fait que l'on va trop loin, et ça va faire planter le second atoi() de la ligne 8.

[Bktero]

Lors de son analyse, le compilateur peut émettre des erreurs si le code est invalide, mais aussi des warnings si le code lui semble "bizarre". Il ne faut (presque) jamais ignoré les warnings en C. Il faut les comprendre et résoudre les problèmes qui les causent.

Pour ton code, tu as du voir qq chose comme :

Code :

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

1
2
3
4
5
6
7
8
9
10
prog.c: In function 'main':
prog.c:9:19: warning: passing argument 1 of 'atoi' makes pointer from integer without a cast [-Wint-conversion]
    9 |        sum = atoi(sum) + atoi(argv[counter]);
      |                   ^~~
      |                   |
      |                   int
In file included from prog.c:3:
/usr/include/stdlib.h:104:30: note: expected 'const char *' but argument is of type 'int'
  104 | extern int atoi (const char *__nptr)
      |                  ~~~~~~~~~~~~^~~~~~

Ici, on voit que tu passes int au lieu d'un const char*. La solution ici n'est bien sûr pas de caster pour créer un pointeur (comme pourrait le laisser croire le premier message), mais bien de passer un pointeur...... ou peut-être de revoir l'algorithme