Petit programme élémentaire

Version imprimable

Bonjour,

Je me mets au C sous MacOs X. J'ai écrit le petit programme suivant :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 #include <stdio.h> main() { int y,z; char *s; printf("Bonjour, ceci est mon premier programme.\n"); printf("Entrez deux nombres entiers separes par un espace : "); scanf("%d %d",&y,&z); printf("Le premier entier que vous avez entre est %d. \n",y); printf("Le deuxieme entier que vous avez entre est %d. \n",z); printf("La somme de ces entiers est %d. \n",y+z); printf("Entrez une chaîne de caractères : "); scanf("%s",&s); printf("Vous avez entre : "); puts(s); printf("\n"); }
La compilation se déroule sans problème, mais l'exécution de l'instruction "puts" produit une Segmentation fault. Pourquoi ?

Merci d'avance de m'aider...
jydegos

29/07/2011, 19h25
Pouet_forever

La déclaration de 's' ne concorde pas avec son utilisation. Soi tu alloues dynamiquement la mémoire, soit tu le déclare en tant que tableau. Le scanf est faux aussi, il ne faut pas envoyer l'adresse de ce pointeur mais bien le pointeur lui-même.

On préfèrera la syntaxe 'int main(void)' pour le main.
La fonction main DOIT retourner quelque chose, 0 ou EXIT_SUCCESS pour une terminaison correcte, une autre valeur (dépendante de l'environnement) pour dire qu'il y a eu une anomalie.
Autre chose, il y a des balises de code (le petit # au dessus du cadre de saisie de texte) utilises les.

@+
29/07/2011, 19h25
jeroman
Salut

Il y a deux problèmes majeurs :

Code:

scanf("%s",&s);

Là, scanf va écraser une partie de la pile.

Un pointeur est un objet situé quelque part en mémoire (ici dans la pile). Et sa valeur est une adresse... vers laquelle on peut pointer (c'est tout l'intérêt d'un pointeur).

La fonction scanf a besoin de connaitre l'adresse où elle va écrire les données.

Or, là, tu lui envoies non pas l'adresse pointée par le pointeur, mais l'adresse du pointeur ! Du coup, elle va écraser une partie de la pile, c'est-à-dire que tu vas modifier s (qui pointera encore n'importe où), ainsi que d'autres variables.
Code:

1 2 3 4 5 char *s; (1) ... scanf("%s",&s); (2) ... puts(s); (3)
1) s n'est pas initialisé : il pointe n'importe où.
2) La mauvaise utilisation de scanf ici modifie la valeur de s... qui continuera encore à pointer n'importe où.
3) Tu veux afficher une chaine situé à une adresse bidon (i.e. : interdite)... Ca ne peut que crasher.

Citation:
Envoyé par jeroman
Salut

Il y a deux problèmes majeurs :

Code:

scanf("%s",&s);

Là, scanf va écraser une partie de la pile.

Un pointeur est un objet situé quelque part en mémoire (ici dans la pile). Et sa valeur est une adresse... vers laquelle on peut pointer (c'est tout l'intérêt d'un pointeur).

La fonction scanf a besoin de connaitre l'adresse où elle va écrire les données.

Or, là, tu lui envoies non pas l'adresse pointée par le pointeur, mais l'adresse du pointeur ! Du coup, elle va écraser une partie de la pile, c'est-à-dire que tu vas modifier s (qui pointera encore n'importe où), ainsi que d'autres variables.
Code:

1 2 3 4 5 char *s; (1) ... scanf("%s",&s); (2) ... puts(s); (3)
1) s n'est pas initialisé : il pointe n'importe où.
2) La mauvaise utilisation de scanf ici modifie la valeur de s... qui continuera encore à pointer n'importe où.
3) Tu veux afficher une chaine situé à une adresse bidon (i.e. : interdite)... Ca ne peut que crasher.
J'ai remplacé scanf("%s",&s); par scanf("%s",s); changé le prototype de la fonction main, et ajouter un return 0; symbolique à la fin du main. Il n'y a pas d'erreur de compilation avec l'option gdb. Cette fois j'obtiens "Bus error" à l'exécution.

Nouveau code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 #include <stdio.h> #include <stdlib.h> int main(void) { int y,z; char *s; printf("Bonjour, ceci est mon premier programme.\n"); printf("Entrez deux nombres entiers separes par un espace : "); scanf("%d %d",&y,&z); printf("Le premier entier que vous avez entre est %d. \n",y); printf("Le deuxieme entier que vous avez entre est %d. \n",z); printf("La somme de ces entiers est %d. \n",y+z); printf("Entrez une chaîne de caractères : "); scanf("%s",s); printf("Bonjour. Jusqu'ici, pas d'erreur"); printf("Vous avez entre : "); puts(s); printf("\n"); return 0; }
Je ne veux bien initialiser s, mais à quoi ???

Merci encore...

29/07/2011, 20h47
jeroman
Citation:

Je ne veux bien initialiser s, mais à quoi ???

Il faut bien écrire la chaine quelque part en mémoire, à une adresse valide et connue. Pas n'importe où.

Il faut donc créer un tableau :

* soit statique :
Code:

1 2 char s[...]; ...
* soit dynamique :
Code:

1 2 3 4 5 6 7 8 9 char * s = NULL; ... s = malloc( ... ); if (s != NULL) { ... free(s) , s = NULL; } ...
NULL est une valeur spéciale, qui indique que l'adresse n'est pas valide.
Il est recommandé (voire capital dans certains programmes) d'initialiser les pointeurs avec cette valeur.
Pour ton programme, qui est un programme très simple, ce n'est pas capital de le faire, mais ça permet de garder les bonnes habitudes de programmation. De même, ici, il n'est pas essentiel de remettre le pointeur à NULL après le free.

Car un pointeur non initialisé peut provoquer des bugs dans certains programmes : en effet, si le pointeur a une valeur bidon mais pas NULL, comment savoir si cette valeur correspond bien à une adresse retournée par malloc ? comment savoir on peut l'utiliser pour lire ou écrire ? comment savoir si on peut l'utiliser avec free ? En initialisant à NULL, il n'y a donc plus de doutes. Et remettre à NULL après un free indique par la suite que le pointeur ne pointe plus à une adresse valide.
29/07/2011, 20h50
Obsidian
Il y a un principe en C que tu peux pratiquement considérer comme fondamental : c'est toi qui gères la mémoire !

Si tu fais un scanf, tu demandes au système de te lire une chaîne de caractères… qu'il va bien falloir mettre quelque part. C'est un peu comme quand tu te fais livrer un canapé. Le transporteur veut bien te l'amener jusque chez toi mais il t'appartient quand même de lui faire de la place.

Ton pointeur « s » est donc là pour cela. Il est censé contenir l'adresse en mémoire d'un emplacement quelconque, en l'occurrence de la place pour déposer notre chaîne. C'est donc à toi d'initialiser ce pointeur en y mettant une adresse valide.

Maintenant, pour réserver de la place en mémoire et connaître son emplacement, il y a deux solutions :
- Soit tu déclares un tableau, comme si tu déclarais une variable locale ordinaire. Le compilateur va alors réserver cette place directement dans la pile, ce qui est acceptable à condition que ce tableau ne soit pas trop gros et que tu le libères rapidement. Comme la taille et l'emplacement de ce tableau sont, par définition, connus à la compilation, tu peux passer directement le nom de ce tableau à scanf(). Un nom de tableau utilisé seul correspond justement à son emplacement, et plus précisément à l'adresse où il commence. Il s'agit donc d'une expression de type « pointeur sur [le-type-de-données-qu'il-contient] » ;
- Soit tu utilises malloc() (Memory Allocation) pour demander au système — et à l'exécution — de t'en réserver un peu, à l'emplacement de son choix. La fonction malloc() te renverra donc un pointeur qu'il te faudra enregistrer quelque part : dans « s ». Éventuellement, malloc() peut te renvoyer « NULL » si elle n'a pas réussi à allouer la place demandée (soit parce que c'est trop grand, soit parce qu'il n'y a plus de place). Il t'appartient donc de vérifier si « s » est bien différent de « NULL » avant de la passer à scanf().

Bon... je crois que je ne suis pas encore mûr pour les chaînes de tailles variables.

Donc j'ai essayé le code suivant :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 #include <stdio.h> #include <stdlib.h> int main(void) { char s[255]="Ceci est un exemple de chaîne de caractères très longue"; printf("Bonjour, ceci est mon premier programme.\n"); puts(s); /* (1) */ printf("Entrez une chaîne de caractères : "); scanf("%s",s); printf("Bonjour. Jusqu'ici, pas d'erreur.\n"); printf("Vous avez entré : "); puts(s); /* (2) */ printf("\n"); return 0; }
Il n'y a plus d'erreur d'exécution mais le résultat est inattendu : l'instruction (1) s'exécute correctement, tandis que l'instruction (2) n'affiche que la partie précédent le premier espace de ce que j'ai saisi ?!?!

29/07/2011, 21h52
jeroman

Citation:

tandis que l'instruction (2) n'affiche que la partie précédent le premier espace de ce que j'ai saisi ?!?!

C'est le but de la fonction. scanf analyse l'entrée en s'appuyant sur le format donné.
Si tu lui donnes "%s", la fonction lit l'entrée standard et remplit le buffer.

Un cours complet au sujet de scanf : http://xrenault.developpez.com/tutoriels/c/scanf/
29/07/2011, 22h06
jydegos

Donc, en fait si je comprends bien, pour saisir "Bonjour toto", il faut saisir les caractères un par un, et créer soi-même la chaîne : pourquoi faire simple quand on peut faire compliqué ?

Allez, je m'en retourne aux mathématiques.
29/07/2011, 22h46
jeroman

La fonction scanf est une fonction compliquée à utiliser, surtout quand on est débutant.

Citation:

Donc, en fait si je comprends bien, pour saisir "Bonjour toto", il faut saisir les caractères un par un, et créer soi-même la chaîne : pourquoi faire simple quand on peut faire compliqué ?

Bien sûr que non ! Si tu veux faire au plus simple, tu peux toujours utiliser fgets à la place de scanf...
Tu saisis ta ligne, tu appuies sur ENTREE, et hop...
30/07/2011, 12h31
Melem

Tutoriel : Les entrées sorties en langage C.
30/07/2011, 13h56
Sve@r

Citation:

Envoyé par jydegos

Il n'y a plus d'erreur d'exécution mais le résultat est inattendu : l'instruction (1) s'exécute correctement, tandis que l'instruction (2) n'affiche que la partie précédent le premier espace de ce que j'ai saisi ?!?!

Salut
C'est normal. scanf("%s") s'arrête au premier espace rencontré et laisse le reste dans le buffer clavier. Rajoute un scanf() et tu auras la fin de ta saisie.

Citation:

Envoyé par jydegos

Donc, en fait si je comprends bien, pour saisir "Bonjour toto", il faut saisir les caractères un par un, et créer soi-même la chaîne

Pas forcément. Tu peux utiliser fgets(s, 80, stdin) qui récupèrera au plus 79 caractères dans stdin (le clavier) pour le stocker dans "s" et y rajoutera le '\0' final afin d'en faire une chaine.

Citation:

Envoyé par jydegos

: pourquoi faire simple quand on peut faire compliqué ?

Personne ne t'oblige à faire du C si tu n'en as pas envie. Mais contrairement à ce que tu t'imagines, le C est très simple quand on en connait les règles. Et l'une d'elle dit qu'on ne fait pas faire de la saisie "formatée" à un utilisateur (car par définition ce qu'il tape au clavier est tout sauf "formaté")

Citation:

Envoyé par jydegos

Allez, je m'en retourne aux mathématiques.

Tchao. Reviens quand tu auras mûri...