Discussion :

[Nadawoo]

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 :

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

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.

[jabbounet]

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 :

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

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

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

Code :

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

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 :

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

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

[Invité(e)]

Bonjour,

En écrivant :

Code :

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

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 :

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

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

[ram-0000]

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 :

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

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 :

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

1
2
3
4
void afficherHello(char *chaine)
{
   strcat(chaine, " world");
}

[Nadawoo]

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 :

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

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 ?

[ram-0000]

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.

[jabbounet]

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 :

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

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

En écrivant par exemple

Code :

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

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 :

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

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

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
 
#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 :

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

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

[gl]

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 :

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

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

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

[Nadawoo]

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

[jabbounet]

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 :

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

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 :

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

1
2
3
4
5
6
 0x40201c - h -
 0x40201d - e -
 0x40201e - l -
 0x40201f - l -
 0x402020 - o -
fin