Discussion :

[vitoubien]

Hello,

Je suis tout juste débutant et mon pitoyable code doit être bourré d'erreurs. pardon d'avance.

Mais voila, après plusieurs tatonnements ce code semble fonctionner à première vue et pourtant quand je décommente la boucle, ça plante sur une "segmentation fault".

Quelqu'un pourrait-il m'aider à comprendre le problème et m'indiquer d'autres erreurs éventuelles. Vive le C, mais qu'est-ce que c'est difficile

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
 
 
	int i, j;
	char **tabchaine;
 
	tabchaine = (char**) malloc(10 * sizeof(char*));
 
	for (i = 0; i < 5; i++)
		tabchaine[i] = (char*) malloc(7 * sizeof(char));
 
//	for (i = 0; i < 10; i++)
//		strcpy(tabchaine[i], "aaaaaa");
 
	strcpy(tabchaine[1], "zzzzz");
 
	for (i = 0; i < 10; i++)
		printf ("tabchaine[%d] : %s\n", i, tabchaine[i]);

Ce n'est pas du tout comme ça qu'on fait ?

[zul]

C'est pas mal, mais tu as du changer la taille de ton tableau à un moment. Tu initialise seulement ton tableau de 0 à 5, et ensuite tu parcours de 0 à 10 (et tu alloue un tableau de 10 éléments donc ça parait cohérent).

Je te conseille d'utiliser un const size_t pour définir la taille de ton tableau une seule fois, et d'utiliser cette variable dans toutes tes boucles, ça évitera ce genre d'erreur.

[ram-0000]

Aller, c'est presque bon, l'erreur est ici :

Code :

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

1
2
3
4
5
	for (i = 0; i < 5; i++)
		tabchaine[i] = (char*) malloc(7 * sizeof(char));
 
	for (i = 0; i < 10; i++)
		strcpy(tabchaine[i], "aaaaaa");

Tu alloues 5 chaines de 7 caractères dans ton tableau et tu en écrit 10

Il faudrait utiliser les constantes pour supprimer ce genre d'erreur

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
 
#define SIZE_ARRAY 10
#define LEN_STRING 7
 
	int i, j;
	char **tabchaine;
 
	tabchaine = (char**) malloc(SIZE_ARRAY * sizeof(char*));
 
	for (i = 0; i < SIZE_ARRAY; i++)
		tabchaine[i] = (char*) malloc(LEN_STRING * sizeof(char));
 
	for (i = 0; i < SIZE_ARRAY; i++)
		strcpy(tabchaine[i], "aaaaaa");
 
	strcpy(tabchaine[1], "zzzzz");
 
	for (i = 0; i < SIZE_ARRAY; i++)
		printf ("tabchaine[%d] : %s\n", i, tabchaine[i]);

Attention tu n'alloure que 7 octets pour ta chaine de caractères (donc 6 caractères + \0).

Lorsque tu fais le strcpy(tabchaine[i], "aaaaaa"), tu es dangereusement proche de la limite.

[Edit] Grilled mais dans la même minute

[vitoubien]

Merci !

[vitoubien]

Euh pour confirmation, pas de problème avec les casts, ni mes sizeof ?
Mon compilateur sous Kdevelop me parait très permissif sur ces questions.

Et je ne suis pas sûr d'avoir encore bien compris la logique.

[vitoubien]

Autres questions de base et j'en aurai fini pour le moment:

1/ Ai-je le droit de réallouer un seul emplacement de cette façon:

Code :

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

1
2
3
4
5
6
 
	realloc(tabchaine[4], 16 * sizeof(char));
	strcpy(tabchaine[4], "123451234512345");
...
	for (i = 0; i < 10; i++)
		printf ("tabchaine[%d] : %s\n", i, tabchaine[i]);

Ce qui semble parfaitement fonctionner mais j'ai comme un doute.

2/ Ce code fonctionnerait-il si j'avais utilisé calloc au lieu de malloc pour l'initialisation ?
Là aussi, il semble que oui mais j'ai qd même un gros doute.

[ram-0000]

Autres questions de base et j'en aurai fini pour le moment:

1/ Ai-je le droit de réallouer un seul emplacement de cette façon:

realloc(tabchaine[4], 16 * sizeof(char));
strcpy(tabchaine[4], "123451234512345");
...
for (i = 0; i < 10; i++)
printf ("tabchaine[%d] : %s\n", i, tabchaine[i]);

Ce qui semble parfaitement fonctionner mais j'ai comme un doute.

Oui, c'est tout bon

realloc essaye d'agrandir le bloc mémoire précédemment alloué par malloc ou realloc. S'il n'y a pas assez de place pour agrandir, il alloue un nouveau bloc, copie l'ancien bloc dans le nouveau bloc et libère l'ancien bloc.

2/ Ce code fonctionnerait-il si j'avais utilisé calloc au lieu de malloc pour l'initialisation ?
Là aussi, il semble que oui mais j'ai qd même un gros doute.

Je ne sais pas, je n'utilise jamais calloc (mais c'est une habitude personnelle)