sprintf dans une fonction

Bonjour,

j'ai un problème qui va sans doute vous paraître tout bête mais je n'ai pas trouvé la réponse dans la FAQ.

Je veux afficher un "hello world". J'ai une chaîne initiale qui contient "hello" et je veux lui ajouter "world" à la fin via un sprintf dans une fonction.

Code:

---

1 2 3 4 5 6 7 8

void afficherHello(char *chaine[100]) { sprintf(*chaine, "%s world", *chaine); }   char *chaine = "Hello"; afficherHello(&chaine);

---

Avec ce code, la compilation réussit mais le programme plante. Quel est le problème ? Je pense qu'il y a un souci au niveau des pointeurs, mais lequel ?

Merci d'avance.

les paramètres de la procédure que tu déclare n'ont rien à voir avec ceux que tu veux passer.

ici tu indique un tableau de 100 pointeur sur char

Code:

---

1 2	void afficherHello(char *chaine[100])

---

là tu définit un char * contenant 5 caractères.

Code:

---

1 2	char *chaine = "Hello";

---

si tu utilise gcc comme compilateur tu peux activer les flags de compilations suivants:
-Wall (active tous les principaux warning de compilation)
-Wextra (ajoute des warning supplémentaires)
-Werror (traite les warning comme des erreurs, bloque la compilation)

Ceci me parait mieux:

Code:

---

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

#include <stdio.h>   void afficherHello(char s[100]) { sprintf(s,"%s world",s); }   int main() { char hello[100] = {'h','e','l','l','o',0}; afficherHello(hello); printf("--> %s",hello); return 0; }

---

Bonjour,

En écrivant :

Code:

---

char *chaine = "Hello";

---

Tu n'alloues pas de mémoire pour la variable "chaine", elle pointe juste une chaine constante.

Une bonne pratique pour ce que tu veux faire est :

Code:

---

1 2 3 4 5 6 7 8 9 10

void ajouterWorld(char *chaine) { strcat(chaine, " world"); }   void test (void) { char chaine[256] = "Hello"; ajouterWorld(chaine); printf(chaine); }

---

Ce que tu veux faire, avec ton exemple n'est pas possible.

Tu essaye d'ajouter quelque chose à la fin d'une chaine de caractères constantes allouée par le compilateur, tu n'as pas le droit d'y toucher, ce n'est pas ta chaine.

Il faut passer par une chaine de caractères qui t'appartient :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

int main(() { char chaine[256]; strcpy(chaine, "Hello"); afficherHello(chaine); }   void afficherHello(char *chaine) { sprintf(&chaine[strlen(chaine)], " world"); }

---

Toujours dans ton exemple, strcat() est probablement plus efficace que sprintf()

Code:

---

1 2 3 4

void afficherHello(char *chaine) { strcat(chaine, " world"); }

---

Merci pour ces réponses rapides ! :)

Vos codes fonctionnent mais j'avoue ne pas comprendre exactement pourquoi. Comment le code de Jabbounet marche-t-il sans faire appel aux pointeurs ? Pour faire la même chose avec une variable numérique on a besoin des pointeurs :

Code:

---

1 2 3 4 5 6 7

void incrementer(int *nombre) { *nombre++; }   int nombreAIncrementer = 5; incrementer(&nombreAIncrementer);

---

Pourquoi ne peut-on pas procéder de la même façon pour les chaînes ?

Citation:

---

Envoyé par Nadawoo

Pourquoi ne peut-on pas procéder de la même façon pour les chaînes ?

---

Lis ce post : Question : L'adresse du pointeur sur un tableau, tu comprendras peut être mieux la subtilité de la chose.

Citation:

---

Comment le code de Jabbounet marche-t-il sans faire appel aux pointeurs ?

---

Cela marche car je déclare un tableau de 100 char que j'initialise directement avec la valeur "hello\0", au lieu de déclarer un pointeur sur une chaine constante.

Code:

---

1 2	char hello[100] = {'h','e','l','l','o',0};

---

En écrivant par exemple

Code:

---

1 2 3

char *hello = "hello"; hello[0]='H';

---

A l'exécution, il y'a segmentation fault car le programme essaie de modifier "hello" qui est une constante du point de vue du compilateur.

Alors que le code suivant marchera, le tableau est alloué sur la pile, et sa valeur est initialisé en recopiant le contenu de la chaine {'h','e','l','l','o',0} dedans.

Code:

---

1 2 3

char hello[100] = {'h','e','l','l','o',0}; hello[0]='H';

---

Ensuite, tant que le tableau passé en paramètre contiens suffisamment de place pour insérer " world\0" à la fin cela marchera ("hello world\0" fait 12 caractères), sinon :aie:.....
Exemple:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

#include <stdio.h> #include <string.h>     void afficherHello(char s[100]) { printf("--> %5d %s\n", strlen(s),s); sprintf(s,"%s world",s); }   int main() { int i = 0; char hello[12] = {'h','e','l','l','o',0}; /* tableau trop petit */ for(i = 0; i < 100 ; i++) { afficherHello(hello); printf("--> %5d %5d %s\n",i, strlen(hello),hello); } return 0; }

---

Me donne à l'exécution:

Code:

---

1 2 3 4 5 6 7 8 9

--> 5 hello --> 0 11 hello world --> 11 hello world --> 1 17 hello world world --> 17 hello world world --> 2 23 hello world world world --> 23 hello world world world --> 100 29 hello world world world world

---

au lieu d'avoir une boucle de 0 à 100

Citation:

---

Envoyé par jabbounet

Cela marche car je déclare un tableau de 100 char que j'initialise directement avec la valeur "hello\0", au lieu de déclarer un pointeur sur une chaine constante.

Code:

---

1 2	char hello[100] = {'h','e','l','l','o',0};

---

---

Note que char hello[100] = "hello"; fonctionnerait tout aussi bien.

Ah OK, je pense avoir à peu près compris. J'ai un peu de mal avec les pointeurs. :)

Petit truc amusant pour aller plus loin.

tu ne peux pas modifier la contante sur laquelle pointe hello dans ce cas, mais rien ne t'empêche de modifier le pointeur.

Code:

---

1 2 3 4 5 6 7 8 9 10 11

#include <stdio.h>   int main() { char *hello = "hello"; while (*hello != 0) { printf(" %p - %c -\n",hello, *hello); hello++; } printf("fin\n"); return 0; }

---

Code:

---

1 2 3 4 5 6

0x40201c - h - 0x40201d - e - 0x40201e - l - 0x40201f - l - 0x402020 - o - fin

---