Discussion :

[Dark_Alex69]

Bonsoir tout le monde. Voilà mon programme compile sans erreurs mais plante quand je l'execute (dès que je rentre le 1er mot).
Je pense que c'est un problème d'allocation dynamique mais je ne vois pas où se trouve mon erreur ni Visual Studio d'alleurs ! J'aurais besoin d'avis d'experts...

Voici 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
 
#include "stdafx.h"
#include "stdlib.h"
#include "string.h"
 
 
int _tmain(int argc, _TCHAR* argv[])
{
	char mot[51];
	char *pmotPlusLg;
	int lgmotPlusLg, lgMotLu, cpt=0;
 
	lgmotPlusLg=-1;
	pmotPlusLg = " ";
 
	//Prototypes
	void FONCTION(char *, int *, int *, char [], int);
 
	printf("\tLecture d'une suite de mots. \nArret a la lecture d'une * en 1er caractere.\n\nVeuillez entrer le mot : ");
	gets(mot);
	do
	{
		if((strcmp(mot, pmotPlusLg))==0)
			cpt++;
		else
		{
			lgMotLu=strlen(mot);
			if(lgMotLu>lgmotPlusLg)
				FONCTION(pmotPlusLg, &cpt, &lgmotPlusLg, mot, lgMotLu);
			else
			{
				if(lgMotLu==lgmotPlusLg)
				{
					if((strcmp(mot, pmotPlusLg))<0)
						FONCTION(pmotPlusLg, &cpt, &lgmotPlusLg, mot, lgMotLu);
				}
			}
		}
		printf("Veuillez rentrer le nouveau mot : ");
		gets(mot);
	}
	while(mot[0]!='*');
	free(pmotPlusLg);
	printf("Le mot le plus long est : %s avec une longueur de : %d. Il a ete lu %d fois\n", pmotPlusLg, lgmotPlusLg, cpt);
 
	system("PAUSE");
 
	return 0;
}
 
//Fonction
 
void FONCTION(char *i, int *c, int *lgi, char mLu[], int lgmLu)
{
	free(i);
	*c=1;
	i=(char *) malloc(lgmLu + 1);
	strcpy(i, mLu);
	*lgi=lgmLu;
}

Merci d'avance.

[nicolas.sitbon]

T'abuses, on te donnes des conseils pourquoi?
[URL="http://www.developpez.net/forums/showpost.php?p=2681855&postcount=4"]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
test2.c:1:20: erreur: stdafx.h : Aucun fichier ou répertoire de ce type
test2.c:5: erreur: expected declaration specifiers or «..." before «_TCHAR"
test2.c:6: attention : no previous prototype for «_tmain"
test2.c: In function «_tmain":
test2.c:12: attention : assignment discards qualifiers from pointer target type
test2.c:15: attention : nested extern declaration of «FONCTION"
test2.c:17: erreur: implicit declaration of function «printf"
test2.c:17: attention : incompatible implicit declaration of built-in function «printf"
test2.c:18: erreur: implicit declaration of function «gets"
test2.c:18: attention : nested extern declaration of «gets"
test2.c: Hors de toute fonction :
test2.c:52: attention : no previous prototype for «FONCTION"
test2.c: In function «FONCTION":
test2.c:55: attention : passing argument 1 of «malloc" as unsigned due to prototype

[Dark_Alex69]

Je ne pense pas que ce soit un problème de gets mais bon...

[Dark_Alex69]

Avec fgets le résultat est le même !

[DavidDeharbe]

Tu peux t'arranger pour que ton code apparaisse indenté? Cela facilite la lecture donc la compréhension...

Il y a des balises pour inclure du code.

David.
--

[Fred83]

Bonjour,

On retrouve les mêmes erreurs de post en post.
Comme le suggère ton titre, il y a bien un problème d'allocation dynamique.

Ta variable pmotPlusLg est déclarée comme un pointeur de caractère.
Avant de t'en servir, il faut lui allouer de la mémoire (Malloc) ou la faire pointer sur une chaîne valide.

Ensuite ton pointeur non initialisé est passé en paramètre à une fonction qui le libère --> ????.

Pour finir, si tu veut comparer la longueur des mots, utilise donc strlen().

strcmp sert a comparer alphabétiquement des chaînes ainsi strcmp("BB","AAAAAA") retourne 1,
ce qui n'est sûrement pas le résultat que tu espérais.

Bon courage et pense aux balises codes.