Discussion :

[Kimly]

Bonjour,

Voilà mon erreur :

Code :

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

Program received signal SIGSEGV, Segmentation fault. [Switching to Thread 0x8f88340 (LWP 3494)] 0xb7e7470a in ?? () from /lib/i386-linux-gnu/libc.so.6

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 <stdlib.h>
#include <stdio.h>
#include <string.h>
 
typedef struct
{
	char * key;
	char * value;
} stringPair;
 
int main(const int argc, const char * argv[])
{
char* list1;
char *key1;
stringPair * pPair1;
 
key1="";
key1="test";
list1="";
list1="bonjour";
pPair1=malloc(sizeof(stringPair));/*malloc(strlen(key1)+strlen(list1));*/
strcpy (pPair1->key,key1);
	strcpy (pPair1->value,list1);
fprintf(stdout, "%s and %s\n",pPair1->key, pPair1->value);
free(pPair1);
}

Donc je ne comprends pas pourquoi.

Merci.

[Bktero]

Code :

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

1
2
3
pPair1=malloc(sizeof(stringPair));/*malloc(strlen(key1)+strlen(list1));*/
strcpy (pPair1->key,key1);
strcpy (pPair1->value,list1);

En faisant ça, tu alloues de la place pour la structure, mais tu n'alloues pas de place pour les zones pointées par les champs de la structure.

Code :

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

1
2
3
4
5
6
7
pPair1=malloc(sizeof(stringPair));/*malloc(strlen(key1)+strlen(list1));*/
 
pPair1->key = malloc(200);
pPair1->value = malloc(200);
 
strcpy (pPair1->key,key1);
strcpy (pPair1->value,list1);

marche beaucoup mieux.

[srvremi]

Pourquoi 200 ?

@+
Rémi

[Bktero]

C'était une taille tout à fait arbitraire, pour simplement illustrer le fait qu'il faut allouer de la mémoire et ainsi avoir un espace réservé pour y copier les chaines. J'aurais dû le préciser .
Dans la réalité, il faudrait donner une taille justifiée et libérer la mémoire après utilisation.


Je relis le code et je me rend compte de ça :

Code :

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

1
2
3
4
key1="";
key1="test";
list1="";
list1="bonjour";

Les affectations à "" ne servent à rien, puisqu'elles sont immédiatement écrasées.

[gangsoleil]

Oula, alors y'a pas mal de problemes. Mes commentaires dans le code. Pose des questions si tu ne comprends pas.

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
 
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
typedef struct
{
	char * key;
	char * value;
} stringPair;
 
int main(int argc, char * argv[]) /* pas de const dans le prototype de main */
{
char* list1;
char *key1;
stringPair * pPair1;
 
/*********************
key1="";
key1="test";
list1="";
list1="bonjour";
**********************/
/* allocation de l'espace memoire pour key1 et list1 */
list1 = key1 = pPair1 = NULL; /* initialisation a NULL pour la liberation en cas d'erreur */
 
key1 = malloc (200); /* taille a definir */
list1 = malloc (200); /* taille a definir */
pPair1=malloc(sizeof(stringPair));
 
/* verification */
if (key1 == NULL  ||  list1 == NULL  ||  pPair1 == NULL)
{
  fprintf (stderr, "Erreur d'allocation memoire\n");
  free (key1), key1 = NULL;
  free (list1), list1 = NULL;
  free (pPair1), pPair1 = NULL; 
 return EXIT_FAILURE;
}
 
/* la memoire est allouee. On peut ecrire dedans */
sprintf (key1, "test");
sprintf (list1, "bonjour");
 
/* on alloue le contenu de la structure */
pPair1->key = pPair1->value = NULL;
 
pPair1->key = malloc (300);
ppair1->value = malloc (300);
if (pPair1->key == NULL  || pPair1->value == NULL)
{
  fprintf (stderr, "Erreur d'allocation memoire\n");
  free (key1), key1 = NULL;
  free (list1), list1 = NULL;
  free (pPair1->key), pPair1->key = NULL; 
  free (pPair1->value), pPair1->value = NULL; 
  free (pPair1), pPair1 = NULL; 
  return EXIT_FAILURE;
}
 
/* on peut copier les donnees. Utiliser strncpy serait mieux */
strcpy (pPair1->key, key1);
strcpy (pPair1->value,list1);
 
/* affichage */
fprintf(stdout, "%s and %s\n",pPair1->key, pPair1->value);
 
/* liberation memoire */
  free (key1), key1 = NULL;
  free (list1), list1 = NULL;
  free (pPair1->key), pPair1->key = NULL; 
  free (pPair1->value), pPair1->value = NULL; 
  free (pPair1), pPair1 = NULL; 
 
/* fin */
  return EXIT_SUCCESS;
}

A changer dans ce code : utiliser snprintf, strncpy.

Pourquoi utilises-tu de la memoire dynamique ? Pour l'exercice, ou bien il n'y a aucune raison, et tu pourrais utiliser de la memoire statique, ce qui serait beaucoup plus simple pour toi ?

[gangsoleil]

Je relis le code et je me rend compte de ça :

Code :

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

1
2
3
4
key1="";
key1="test";
list1="";
list1="bonjour";

Les affectations à "" ne servent à rien, puisqu'elles sont immédiatement écrasées.

Je pense que tu es fatigue : les chaines de caracteres ne se copient pas comme ca

[Bktero]

Euh.... Je sais que ça ne se copie pas comme ça !

char *ptr = "developpez.com" crée un pointeur sur une chaine et comme la chaine est constante, cette chaine est dans la zone mémoire réservée aux constantes. Si a la suite, je fais ptr= "developpez.net", ptr pointe maintenant vers une nouvelle chaine constante, qui est à un autre endroit de la mémoire statique (à voir même si le compilateur n'est pas suffisamment malin pour ne garder que la seconde affectation dans le code cité à mon message précédent).

Le code suivant est valide, non ? (en tout cas, ni MinGW ni Dr Memory ne me jettent) :

Code :

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

1
2
3
4
5
6
7
8
int main(void)
{
    char *ptr = "developpez.com";
    printf("%p : %s\n", ptr, ptr);
 
    ptr = "developpez.net";
    printf("%p : %s\n", ptr, ptr);
}

[gangsoleil]

Le code suivant est valide, non ? (en tout cas, ni MinGW ni Dr Memory ne me jettent) :

Code :

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

1
2
3
4
5
6
7
8
int main(void)
{
    char *ptr = "developpez.com";
    printf("%p : %s\n", ptr, ptr);
 
    ptr = "developpez.net";
    printf("%p : %s\n", ptr, ptr);
}

Oui, nous sommes d'accord, ce code est valide :

Dans ce code, tu crees un pointeur ptr que tu fais pointer sur un const char [15], ce que te confirme gcc ou clang si tu compiles avec l'option -Wwrite-strings :

-Wwrite-strings
When compiling C, give string constants the type const char[length] so that copying the address of one into a non-const char * pointer will get a warning; when compiling C++, warn about the deprecated conversion from string constants to char *. These warnings will help you find at compile time code that can try to write into a string constant, but only if you have been very careful about using const in declarations and prototypes. Otherwise, it will just be a nuisance; this is why we did not make -Wall request these warnings.

Code :

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

1
2
3
4
5
6
7
8
% clang  -Wwrite-strings test.c 
test.c:7:11: warning: initializing 'char *' with an expression of type 'const char [15]' discards qualifiers
    char *ptr = "developpez.com";
          ^     ~~~~~~~~~~~~~~~~
test.c:10:9: warning: assigning to 'char *' from 'const char [25]' discards qualifiers
    ptr = "developpez.net_plus_long";
        ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~
2 warnings generated.

C'est juste que dans son cas, je pense que le but est de manipuler des chaines de caracteres, et qu'il serait donc preferable de prendre de bonnes habitudes tout de suite (pour ne pas le voir revenir plus tard avec une copie de chaine de caractere avec =)developpez.com

[Bktero]

Tu m'as mis un -1 et m'a traité d'endormi alors que je ne disais (presque) pas de bêtise

-Wwrite-strings pour se prémunir des gens qui voudraient utiliser un = au lieu d'un strcpy ,si je comprend bien ?

Code :

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

1
2
3
/* la memoire est allouee. On peut ecrire dedans */
sprintf (key1, "test");
sprintf (list1, "bonjour");

Pourquoi ne pas utiliser strcpy ?

[Neckara]

Code :

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

1
2
3
/* la memoire est allouee. On peut ecrire dedans */
sprintf (key1, "test");
sprintf (list1, "bonjour");

Pourquoi ne pas utiliser strcpy ?

Encore un petit peu endormi ?

Code :

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

int sprintf (char *str, const char *format, ...);

- sprintf va servir à remplir une chaîne de caractère avec des données formatée :

Code :

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

sprintf(monChar, "%s %d...%f", "toto", 5, 2.0);

- strcpy va servir à copier le contenu d'une chaîne de caractère dans une autre.
Donc si on veut juste faire une copie, on utilisera plutôt strcpy.

@PO : Rajouter le 'n' : snprintf, strncpy permettra de s'assurer qu'on ne dépasse pas la taille allouée et ainsi on se protègera d'éventuels SEGFAULT.

[Bktero]

Encore un petit peu endormi ?

Bon hey ho !

http://man.developpez.com/man3/printf.3.php

Je sais à quoi sert sprintf, merci, et justement dans ce cas, ce n'est pas franchement une chaine formatée, c'est une simple copie. Comme utiliser printf("%s", chaine) au lieu de puts(chaine), pas grand intérêt. Je cherchais une raison par rapport aux contraintes de l'option de compilation donnée par gangsoleil.

[Neckara]

Je sais à quoi sert sprintf

Je n'en doute pas, mais je préférai donner cette précision pour le PO ou d'éventuels lecteurs.


Pour revenir à ta question, je n'avais pas vu que tu citais du code de gangsoleil qui voulait juste montrer les deux solutions :

Code :

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

1
2
3
4
5
sprintf (key1, "test");
sprintf (list1, "bonjour");
//et
strcpy (pPair1->key, key1);
strcpy (pPair1->value,list1);

Mais je pense que gangsoleil a eu un petit problème de copié-collé (d'où la raison de mon précédant post)

Code :

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

1
2
sprintf (key1, "%s", "test");
sprintf (list1, "%s", "bonjour");

Il est vrai qu'on utilisera plutôt str(n)cpy mais je pense que gangsoleil voulait aussi lui montrer qu'on peut utiliser s(n)printf si jamais on a besoin d'une chaîne de caractère formatée.

[Bktero]

Je n'en doute pas

Bon ça va alors ^^

Mais je pense que gangsoleil a eu un petit problème de copié-collé

Je ne pense pas. Tu écris quoi toi ? printf("Bonjour"); ou printf("%s", "bonjour"); ? ^^

[Neckara]

Bon, les vacances ne me font pas que du bien

Je me suis tout de même trouvé une petite excuse :
Le f de printf et de scanf signifie formaté, quand on ne manipule pas des données formatée mais simplement une chaîne de caractère, on utilisera des fonctions plus rapide comme str(n)cpy, fgets, puts etc...
Donc j'ai tellement peu l'habitude de voir un simple fprintf(char *, char *) que c'en est perturbant.

Bon j'avais quand même fait un test pour ne pas dire de bêtises avant de poster :

Code :

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

1
2
3
4
5
    char c[4];
    char d[4] = "tot";

    sprintf(c, d);
    printf("%c",d);

Je commence bien la journée

[Bktero]

Et après tu me demandes si je dors encore quand je propose d'utiliser strcpy

Ca t'apprendra, on ne devrait pas avoir le droit d'être étudiant, en vacances et levé si tôt !

[gangsoleil]

Pourquoi ne pas utiliser strcpy ?

l'avantage de s(n)printf, c'est que tu es certain que la chaine est terminee par un \0, ce qui n'est pas le cas de str(n)cpy.

[Neckara]

l'avantage de s(n)printf, c'est que tu es certain que la chaine est terminee par un \0, ce qui n'est pas le cas de str(n)cpy.

On peut toujours faire :

Code :

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

1
2
3
4
 
char destination[taille];
strncpy(destination, source, taille - 1);
destination[taille - 1] = '\0';