Problème de libération de l'éspace dynamique alloué
Résolu
boualiasma
Messages postés
393
Date d'inscription
lundi 22 juin 2009
Statut
Membre
Dernière intervention
23 décembre 2011
-
17 août 2009 à 09:54
cs_rt15 Messages postés 3874 Date d'inscription mardi 8 mars 2005 Statut Modérateur Dernière intervention 7 novembre 2014 - 21 août 2009 à 09:52
cs_rt15 Messages postés 3874 Date d'inscription mardi 8 mars 2005 Statut Modérateur Dernière intervention 7 novembre 2014 - 21 août 2009 à 09:52
A voir également:
- Problème de libération de l'éspace dynamique alloué
- Python supprimer espace ✓ - Forum Python
- Supprimer les espaces dans une chaîne de caractère ✓ - Forum Python
- Supprimer les espaces dans une chaine ✓ - Forum C# / .NET
- Vba supprimer espace ✓ - Forum Visual Basic
- Espace pile insuffisant vba - Forum Visual Basic 6
1 réponse
cs_rt15
Messages postés
3874
Date d'inscription
mardi 8 mars 2005
Statut
Modérateur
Dernière intervention
7 novembre 2014
13
21 août 2009 à 09:52
21 août 2009 à 09:52
Salut,
Heu... C'est quoi ton compilo ? Tu l'as trouvé dans un paquet de lessive ?
Il y a plein de trucs bizarres dans ton code qui n'ont aucune chance de compiler (Par exemple "pFTrie" non déclaré, fopen avec un seul argument).
1) Là :
free(chaine);
fclose(fp);
return 1;
free(chaine);
fclose(fp);
return 0;
Il y a redondance du code d'une part, et d'autre part il est déconseiller d'utiliser deux "return" dans une même fonction. On risque en effet par exemple d'oublier de mettre un fclose aux deux endroits par exemple.
2) Code uniquement en anglais. Ca évite le mélange français/anglais avec les fonctions standards et on a pas de problèmes avec les accents.
3) Tu manipule a la fois des chaînes de taille fixe et des chaînes de taille quelconque. Si tu connais la taille de toutes tes chaînes, tu n'as pas besoin de faire de malloc (Opération couteuse).
4) Dans les fonction type scanf (Contrairement aux fonctions type printf), il faut passer les entier par adresse, de manière à ce que la fonction puisse les affecter.
5) Pas la peine d'initialiser une variable qui est affectée systématiquement plus tard (trouve et objet). Perte de temps à l'exécution.
6) Pourquoi ouvrir le fichier deux fois ? Pourquoi ne pas passer s à "vérifier" ?
7) Il faut tester le retour d'un fopen. Il peut arriver que le fichier soit absent ou impossible à ouvrir.
8) Et finalement, le souci... strlen renvoie la taille de la chaîne sans compter le zéro terminal. Il faut donc allouer une zone de taille + 1. Un tableau de taille n est indicé de 0 à n - 1.
Voilà le source partiellement corrigé :
Heu... C'est quoi ton compilo ? Tu l'as trouvé dans un paquet de lessive ?
Il y a plein de trucs bizarres dans ton code qui n'ont aucune chance de compiler (Par exemple "pFTrie" non déclaré, fopen avec un seul argument).
1) Là :
free(chaine);
fclose(fp);
return 1;
free(chaine);
fclose(fp);
return 0;
Il y a redondance du code d'une part, et d'autre part il est déconseiller d'utiliser deux "return" dans une même fonction. On risque en effet par exemple d'oublier de mettre un fclose aux deux endroits par exemple.
2) Code uniquement en anglais. Ca évite le mélange français/anglais avec les fonctions standards et on a pas de problèmes avec les accents.
3) Tu manipule a la fois des chaînes de taille fixe et des chaînes de taille quelconque. Si tu connais la taille de toutes tes chaînes, tu n'as pas besoin de faire de malloc (Opération couteuse).
4) Dans les fonction type scanf (Contrairement aux fonctions type printf), il faut passer les entier par adresse, de manière à ce que la fonction puisse les affecter.
5) Pas la peine d'initialiser une variable qui est affectée systématiquement plus tard (trouve et objet). Perte de temps à l'exécution.
6) Pourquoi ouvrir le fichier deux fois ? Pourquoi ne pas passer s à "vérifier" ?
7) Il faut tester le retour d'un fopen. Il peut arriver que le fichier soit absent ou impossible à ouvrir.
8) Et finalement, le souci... strlen renvoie la taille de la chaîne sans compter le zéro terminal. Il faut donc allouer une zone de taille + 1. Un tableau de taille n est indicé de 0 à n - 1.
Voilà le source partiellement corrigé :
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int verifier(int taille)
{
FILE *fp;
char *chaine;
int i;
char s[100];
int objet;
char attribut[32];
int nResult;
nResult = 0;
/* Initialisation d'une chaîne de taille "taille" avec des '1' */
chaine = (char*)malloc(taille + 1);
for(i = 0; i < taille; i++)
chaine[i] = '1';
chaine[taille] = '\0';
printf("chaine = %s\n", chaine);
fp = fopen("exemple.txt", "r");
while (fgets(s, 100, fp))
{
sscanf(s, "%d%s", &objet, attribut);
printf("attribut = %s\n", attribut);
if (memcmp(attribut, chaine, taille))
nResult = 1;
}
free(chaine);
fclose(fp);
return nResult;
}
int main()
{
FILE *fp;
char s[100];
int objet;
char attribut[32];
int taille;
int trouve;
fp = fopen("exemple.txt", "r");
while (fgets(s, 100, fp))
sscanf(s, "%d%s", &objet, attribut);
taille = strlen(attribut);
trouve = verifier(taille);
printf("valeur de retour %d\n", trouve);
return 0;
}
