Problème avec "fclose"

Bonjours, cas fais 2 heures que je cherche mon bug et je ne trouve pas! C'est frustrant!
J'ai fais pas mal de test et je ne trouve rien.
Je met donc mon code sur le poste (cas fais longtemps que je n'ai pas fais de C, 4/5 ans, et là vraiment je ne m'en sort pas!)

#include <cstdlib>
#include <iostream>
#include <string.h>
#include <stdlib.h>
typedef unsigned long ErreurCode;

ErreurCode ChercheTextDansFichier (FILE * Fichier, char * TextAChercher, unsigned long * Position)
{
char * Tempon;
unsigned long PositionFichier = 0;
unsigned long PositionChaine = 0;
unsigned long TailleFichier;
    fseek (Fichier, 0, SEEK_END);
    TailleFichier = ftell(Fichier);
    fseek (Fichier, 0, SEEK_SET);
    Tempon = (char *)malloc(strlen(TextAChercher));
    while (!(feof(Fichier)) && (PositionChaine < strlen(TextAChercher)))
    {
        PositionFichier++;
        Tempon[PositionChaine] = (char)fgetc (Fichier);
        PositionChaine++;
        Tempon[PositionChaine] = '\0';
        if (strncmp(TextAChercher,Tempon,PositionChaine) != 0)
        {
             PositionChaine = 0;
        }
    }
    if (feof(Fichier))
    {
        *Position = 0;
    }
    else
    {
        *Position = PositionFichier - strlen(TextAChercher);
    }
    free (Tempon);
    return 0;
}

ErreurCode LitDonneeBaliseFichier (FILE * Fichier, char * Balise, char ** Donnee)
{
unsigned long PositionDepart;
unsigned long PositionFin;
char * Tempon;

    Tempon = (char *)malloc (strlen(Balise) + 3);
    Tempon[0] = '<';
    Tempon[1] = '\0';
    strcat (Tempon, Balise);
    strcat (Tempon,">");
    ChercheTextDansFichier (Fichier, Tempon, &PositionDepart);
    PositionDepart += (strlen(Tempon)); //Position de début de balise (donc carac '<') - carac de fin de chaine
    Tempon[1] = '/';
    Tempon[2] = '\0';
    strcat (Tempon, Balise);
    strcat (Tempon,">");
    ChercheTextDansFichier (Fichier, Tempon, &PositionFin); //position de début de balise donc OK
    *Donnee = (char *)malloc (PositionFin - PositionDepart + 2); //+1 car carac de fin de chaine + 1 (fichier commance à 0 et malloc à 1)
    *Donnee[0] = '\0';
    fseek (Fichier, PositionDepart, SEEK_SET);
    PositionFin -= PositionDepart;
    PositionDepart = 0;
    while (PositionDepart < PositionFin)
    {
        (*Donnee)[PositionDepart] = (char)fgetc (Fichier);
        PositionDepart ++;
    }
    (*Donnee)[PositionDepart] = '\0';
    free (Tempon);
    return 0;
}

int main(int argc, char *argv[])
{
FILE * FichierXML;
char * Texte;
    FichierXML = fopen("D:\\schema_bodacc_Bilan_V2006.01.xml","r");
    LitDonneeBaliseFichier (FichierXML,"versionSchema",&Texte);
    printf ("Fin de test valeur :\n%s\n",Texte);
    free (Texte);
    system("PAUSE");
    fclose(FichierXML);
    system("PAUSE");
    return 0;
}


PS : Je compile avec "Dev-C++".
PS² : Si vous avez une meilleur syntax (pour autre chose que le "fclose") vous pouvez bien sur me la faire parvenir.
Merci.
D'oh! Nuts!
Mmmmm...
 DONUTS

Pourquoi passer un char ** a LitDonneBaliseFichier() ?

LitDonneeBaliseFichier (FichierXML,"versionSchema",Texte);

ErreurCode LitDonneeBaliseFichier (FILE * Fichier, char * Balise, char * Donnee)
{
...
Donnee = (char *)malloc (PositionFin - PositionDepart + 2);
...
}

Et pour tout acces a Donnee, il suffit de faire un Donnee[].

Si tu bosses sous windows, regarde du coté des API CreateFile(), CloseFile()..etc

+2(p - n)

Pense au but à atteindre, ça t'évitera d'amuser le processeur avec des instruction inutiles.
Tempon[0] = '<';
Tempon[1] = '\0';
strcat(Tempon, Balise);

deviendrait
Tempon[0] = '<';
strcpy(Tempon + 1, Balise);
etc...

ciao...
BruNews, MVP VC++

Heeee, AlexMAN, je passe un "char **" car c'est un pointeur sur pointeur. Si non comment je fais pour changée l'adrs contenu par mon pointeur "Texte"??? En passant un "char *" et "texte" ma fonction appelante (main) ne pourra pas accéder à mes données puisque je passe à ma fonction (LitDonneeBaliseFichier) l'adrs contenu dans "texte" et pas l'adrs de "texte"??? A moins que je ne fasse erreur???

Oui BruNews effectivement je vais tenter d'optimiser un peut plus mes modifications/copies de chaine.

Merci à vous deux pour vos réponses mais j'ai toujours mon problème. Mon code plante au "fclose" (Windows m'affiche une fenêtre "Projet1.exe a rencontré un problème et doit fermer. Nous vous prions de nous excuser pour le désagrément encouru.") et je ne comprend vraiment pas pourquoi.

D'oh! Nuts!
Mmmmm...
 DONUTS

Salut,
une question me taraude : pourquoi les types de retours de toutes tes fonctions (sauf le main) sont des unsigned long (typedefé en ErreurCode) est qu'a chaque fois tu fais un return 0 ? Tes fonctions devraient être de type void !!!

TuRn3r

Sinon pour le fclose quel est son code de retour ?
As-tu testé le code de retour de fopen (pour être sur que ton fichier est bien ouvert par exemple) !

TuRn3r

Toute mes fonction on un ErreurCode comme type de retour car je vais faire une remonté d'erreur (qui n'est pas encors en place bien sur mais prévue). Oui j'ai tester mon fopen et il me renvoie une ards (j'avais pensé aussi que ca pouvais venir de cas mais c'est une bonne remarque, je vais tester la valeur de retour du "fopen" et ne rien executer si c'est 0).

D'oh! Nuts!
Mmmmm...
 DONUTS

Je confirme que le fichier est bien ouvert (et en plus j'affiche bien la donnée de ma balise, alors...)

A oui autre remarque : mon programme ne plante pas si je n'exécute pas LitDonneeBaliseFichier ce que j'avais tester :
           Allouer de la mémoire à texte
           Lire les 20 premiers carac de mon fichier (les metre dans "texte")
           Se déplace dans le fichier
           Afficher "texte"
           Libérer la mémoire de texte
           Fermer le fichier
Et là tout ce passe pour le mieux!!! Je ne comprends vraiment pas et cas vas finir par me rendre (tout du moins encor plus que maintenant)

D'oh! Nuts!
Mmmmm...
 DONUTS et

Peux tu trouver la ligne qui provoque l'erreur en mettant des printf un peu partout.
Sinon, dans ton cas ca peux passer, mais pense toujours dans tes malloc a mettre des sizeof, je m'explique:
Tempon = (char *)malloc(strlen(TextAChercher));
doit devenir:
Tempon = (char *)malloc(strlen(TextAChercher) *sizeof(char));

Tu as de la chance que ca passe car sur un Windows 32Bits (surquoi tu dois etre a mon avis) sizeof(char) == 1. Mais pense toujours à le mettre ca te sauveras la mise un jour

TuRn3r

oui j'y avais pensé mais comme un char vaux 1 oct pas de problème. Ce n'est pas le cas sur tout les system???????? Merci de l'info!!!!!

D'oh! Nuts!
Mmmmm...
 DONUTS