Discussion :

[Dunkhan]

Bonjour,

Dans ce bout de code, je crée un tableau dynamique pour y stocker la chaine de caractère entrée par l'utilisateur.

J'ai des consignes d'exercices qui sont celle ci:

Écrire une fonction capable de lire au clavier une chaîne de caractères de taille quelconque.
Tant que l’utilisateur introduit des caractères, il faudra les stocker dans la chaîne de caractères.
La fin de lecture étant signalée par l’utilisateur au moyen de la touche <enter>.
Le programme appelant ne réserve pas de place mémoire. Il se contente de faire appel à la
fonction et d’afficher le résultat.

Le code fonctionne a plusieurs reprise, mais plante au bout d'un certains nombre d'essais en m'affichant cette fenêtre



Si je relance de suite après, j'ai le plantage qui revient quasi systématiquement plusieurs fois d'affilée.. puis le code refonctionne pour qlq essais..
Bizarrement, ca affiche tjrs le même nombre de caractères (4) avant de planter..

le code :

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
int main(int argc, char *argv[])
{
 
	// =======================================================================================================
	// VARIABLE(S) LOCALE
	// =======================================================================================================
	int i = 0;
	int j = 1;
	char cTemp;
	char *tTabChaine = NULL;
 
 
	// =======================================================================================================
	//  CODE
	// =======================================================================================================
 
	printf("Entrer une chaine de caractere : ");
 
	tTabChaine = malloc(sizeof(char));
	if (tTabChaine == NULL)
	{
		exit(0);
		printf("Erreur malloc!!!");
	}
 
	do
	{
 
		cTemp = fgetc(stdin);
 
 
		realloc(tTabChaine, j + 1);
		if (tTabChaine == NULL)
		{
			exit(0);
			printf("Erreur realloc!!!");
		}
 
		tTabChaine[i] = cTemp;
		printf("%c", tTabChaine[i]);
		j++;
		i++;
	}
	while (cTemp != '\n');
 
	realloc(tTabChaine, j + 1 * sizeof(char));
	if (tTabChaine == NULL)
	{
		exit(0);
		printf("Erreur realloc!!!");
	}
 
	tTabChaine[i] = '\0';
	printf("%s\n", tTabChaine);
 
 
	free(tTabChaine);
 
	return 0;
}

Est ce qlq'un a une idée de pq ca me fait cette erreur de temps en temps, mais pas à chaque fois??
Je sais que le code est un peu brut de décoffrage, mais je l’affinerais après que je sois sur qu'il fonctionne..

Merci d'avance a ceux qui prendront le temps de réponde

[ternel]

Il faut toujours réaffecter la variable avec réalloc.
réalloc peut déplacer le bloc mémoire, donc il faut mettre a jour la variable donnée en argument.

Code :

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

ptr = realloc(ptr, ...);

[Dunkhan]

Il faut toujours réaffecter la variable avec réalloc.
réalloc peut déplacer le bloc mémoire, donc il faut mettre a jour la variable donnée en argument.

Code :

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

ptr = realloc(ptr, ...);

un tout grand merci

[Médinoc]

Il faut toujours réaffecter la variable avec réalloc.
réalloc peut déplacer le bloc mémoire, donc il faut mettre a jour la variable donnée en argument.

Code :

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

ptr = realloc(ptr, ...);

Tu oublies que realloc() retourne NULL en cas d'échec, et là tu as droit à une fuite de mémoire (et ce juste au moment où tu n'en as déjà plus assez).
Personnellement, je gère ça avec des wrappers:

Code C :

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

1
2
3
4
5
6
7
_Bool realloc_char(char** pp, size_t newSize)
{
	char* pNew = realloc(*pp, newSize);
	if(pNew != NULL)
		*pp = pNew;
	return (pNew != NULL);
}

Ce qui permet de l'utiliser ainsi:

Code C :

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

1
2
3
4
 
if(realloc_char(&tTabChaine, j+1)) {
	...
}

[ternel]

J'approuve totalement ton approche medinoc.
Note: en cas d'échec, le pointeur source n'est pas libéré par realloc, l'utilisateur doit en faire quelque chose.

[Dunkhan]

Merci pour ton conseil Ternel.

On peut tout de suite voir que c'est une très bonne façon de faire.. ca évite plein d’écueil.


Par contre, étant vraiment novice, j'aurais aimer savoir ce que tu aurais mis dans les braquet du if??
Pq je ne comprend pas le pourquoi le test sur la fonction..

Code :

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

1
2
3
if(realloc_char(&tTabChaine, j+1)) {
	...
}

[ternel]

La fonction realloc peut échouer.
C'est spécifié dans la documentation, et les cas possibles sont un problème dans la nouvelle taille, et un manque de mémoire disponible pour l'opération (qui peut inclure une copie du bloc mémoire)

Comme elle peut le faire, il faut considérer qu'elle va le faire. Donc, il faut en tenir compte et définir que faire à ce moment là.

[Dunkhan]

ah ok,

donc ca correspond a ce que je faisais ??!

Code :

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

1
2
3
4
5
if (tTabChaine == NULL)
	{
		exit(0);
		printf("Erreur malloc!!!");
	}

[ternel]

presque, sauf qu'il faudrait le faire dans l'autre ordre, exit terminant le programme, et la, le problème est plus complexe, parce qu'on parle de realloc