Sujet Précédent Sujet Suivant

Mon fopen ne fonctionne pas

daviddubois Messages postés 92 Date d'inscription vendredi 19 mars 2004 Statut Membre Dernière intervention 6 janvier 2012 - 11 mai 2008 à 08:01
daviddubois Messages postés 92 Date d'inscription vendredi 19 mars 2004 Statut Membre Dernière intervention 6 janvier 2012 - 11 mai 2008 à 11:03
Bonjour tout le monde,

Mon fopen ne fonctionne pas et je suis bloqué depuis hier soir.

    Fichier = fopen(NomFichier,"rt");
    if (Fichier == NULL) return false;

arrivé sur la ligne if (Fichier==NULL) return false, il renvoie false car Fichier vaut bien NULL alors que je suis sûr que mon fichier se trouve au bon endroit et que le nom est correct.

Je l'appelle via la main :

    if(Produit.Charger("produit.txt"))printf("OK");

voici une partie de mon code :

#include
#include "CarambouilleDD1.h"

////////////////
// CProduits //
///////////////

CProduit::CProduit()
:m_PP(0) // j'initialise à zéro au cas où le modificateur ne sait pas affecter une valeur.
{
}

//le constructeur par copie est appelé avec le constructeur par copie
CProduit::CProduit(const CProduit& Source)
:CGenerique(Source),m_NomProduit(Source.m_NomProduit),m_PP(Source.m_PP)
{
}

//le constructeur spécifique est appelé avec la méthode d'ajout
CProduit::CProduit(const char* UnNom,int UnPrix)
:m_PP(0) //on recopie la même chose que dans le constructeur spécifique
{
    Nom(UnNom);
    PP(UnPrix);
}

CProduit::~CProduit()
{
}

CGenerique* CProduit::Cloner() const
{
    return new CProduit(*this);
}

bool CProduit::Nom(const char* UnNom)
{
    if((strlen(UnNom)<1)&&(UnNom==NULL))return false;
    return m_NomProduit.Chaine(UnNom);   
}

bool CProduit::PP(int UnPrix)
{
    if(UnPrix < 1) return false;
    m_PP = UnPrix; //on peut ici transformer la valeur d'un membre privé car s'est sa propre classe
    return true;
}

const char* CProduit::Nom() const
{
    return m_NomProduit.Chaine;
}

int CProduit::PP() const
{
    return m_PP;
}

bool CProduit::EstValide() const
{
    return (strlen(Nom())>1 && (PP() > 1));
}

bool CProduit::GereUnicite() const
{
    return true;
}

int CProduit::Comparer(const CGenerique& Autre, long TypeComparaison) const
{
    const CProduit& P = (const CProduit&)Autre;
    switch (TypeComparaison)
    {
    case 0:
        return m_NomProduit.Comparer(P.m_NomProduit,Comparaison_TriCroissantNonStrict);
   
    }
    return 0;
}

long CProduit::NombreClesDeTri() const
{
    return 1;
}

//////////////////
// CTabProduits //
//////////////////

CTabProduits::CTabProduits()
:CTableauGenerique(CProduit())
{
}

CTabProduits::CTabProduits(const CTabProduits& Source)
:CTableauGenerique(Source)
{
}

CTabProduits::~CTabProduits()
{
}

CGenerique* CTabProduits::Cloner() const
{
    return new CTabProduits(*this);
}

CProduit& CTabProduits::Element(long Indice) const
{
    return (CProduit&)CTableauGenerique::Element(Indice);
}

long CTabProduits::Indice(const char* UnNom) const
{
    return CTableauGenerique::Indice(CProduit(UnNom,1));
}

bool CTabProduits::Charger(const char* NomFichier)
{
    CChargeurFichierTexte    Chargeur;

    return Chargeur.Charger(NomFichier,'\t',FctnTraiterLigne,this);
}

bool CTabProduits::FctnTraiterLigne(const CChargeurFichierTexte& Chargeur, void* Contexte)
{
    return ((CTabProduits*)Contexte)->TraiterLigne(Chargeur);
}

bool CTabProduits::TraiterLigne(const CChargeurFichierTexte& Chargeur)
{
    if (Chargeur.NombreChamps() == 2) Ajouter(CProduit(Chargeur.Champs(0),atol(Chargeur.Champs(1))));
    return true;
}

bool CTabProduits::Ajouter(const CProduit& Modele)
{
    return CTableauGenerique::Ajouter(Modele);
}

et le fichier .h

#ifndef CARAMBOUILLE_H
#define CARAMBOUILLE_H

#include "chaine.h"
#include "generique.h"

class CProduit;
class CTabProduits;

class CProduit : public CGenerique
{
public:
    CProduit();
    CProduit(const CProduit& Source);
    CProduit(const char* UnNom, int UnPrix);
    ~CProduit();

    CGenerique*    Cloner() const;
    const char*    Nom() const;
    int            PP() const;
    bool        Nom(const char* UnNom);
    bool        PP(int UnPrix);
    bool        EstValide() const;
    bool        GereUnicite() const;
    long        NombreClesDeTri() const;
    int            Comparer(const CGenerique& Autre, long TypeComparaison) const;

private :
    CChaine        m_NomProduit;
    int            m_PP; //Prix Produit
};

class CTabProduits : public CTableauGenerique
{
public:
    CTabProduits();
    CTabProduits(const CTabProduits& Source);
    ~CTabProduits();
   
    CGenerique*        Cloner() const;
    CProduit&        Element(long Indice) const;
    long            Indice(const char* UnNom) const;
    bool            Charger(const char* NomFichier);
    bool            Ajouter(const CProduit& Modele);

private:
    static bool    FctnTraiterLigne(const CChargeurFichierTexte& Chargeur, void* Contexte);
    bool        TraiterLigne(const CChargeurFichierTexte& Chargeur);

};
#endif

et le .cpp pour le chargement :

#include
#include "CarambouilleDD1.h"

////////////////
// CProduits //
///////////////

CProduit::CProduit()
:m_PP(0) // j'initialise à zéro au cas où le modificateur ne sait pas affecter une valeur.
{
}

//le constructeur par copie est appelé avec le constructeur par copie
CProduit::CProduit(const CProduit& Source)
:CGenerique(Source),m_NomProduit(Source.m_NomProduit),m_PP(Source.m_PP)
{
}

//le constructeur spécifique est appelé avec la méthode d'ajout
CProduit::CProduit(const char* UnNom,int UnPrix)
:m_PP(0) //on recopie la même chose que dans le constructeur spécifique
{
    Nom(UnNom);
    PP(UnPrix);
}

CProduit::~CProduit()
{
}

CGenerique* CProduit::Cloner() const
{
    return new CProduit(*this);
}

bool CProduit::Nom(const char* UnNom)
{
    if((strlen(UnNom)<1)&&(UnNom==NULL))return false;
    return m_NomProduit.Chaine(UnNom);   
}

bool CProduit::PP(int UnPrix)
{
    if(UnPrix < 1) return false;
    m_PP = UnPrix; //on peut ici transformer la valeur d'un membre privé car s'est sa propre classe
    return true;
}

const char* CProduit::Nom() const
{
    return m_NomProduit.Chaine;
}

int CProduit::PP() const
{
    return m_PP;
}

bool CProduit::EstValide() const
{
    return (strlen(Nom())>1 && (PP() > 1));
}

bool CProduit::GereUnicite() const
{
    return true;
}

int CProduit::Comparer(const CGenerique& Autre, long TypeComparaison) const
{
    const CProduit& P = (const CProduit&)Autre;
    switch (TypeComparaison)
    {
    case 0:
        return m_NomProduit.Comparer(P.m_NomProduit,Comparaison_TriCroissantNonStrict);
   
    }
    return 0;
}

long CProduit::NombreClesDeTri() const
{
    return 1;
}

//////////////////
// CTabProduits //
//////////////////

CTabProduits::CTabProduits()
:CTableauGenerique(CProduit())
{
}

CTabProduits::CTabProduits(const CTabProduits& Source)
:CTableauGenerique(Source)
{
}

CTabProduits::~CTabProduits()
{
}

CGenerique* CTabProduits::Cloner() const
{
    return new CTabProduits(*this);
}

CProduit& CTabProduits::Element(long Indice) const
{
    return (CProduit&)CTableauGenerique::Element(Indice);
}

long CTabProduits::Indice(const char* UnNom) const
{
    return CTableauGenerique::Indice(CProduit(UnNom,1));
}

bool CTabProduits::Charger(const char* NomFichier)
{
    CChargeurFichierTexte    Chargeur;

    return Chargeur.Charger(NomFichier,'\t',FctnTraiterLigne,this);
}

bool CTabProduits::FctnTraiterLigne(const CChargeurFichierTexte& Chargeur, void* Contexte)
{
    return ((CTabProduits*)Contexte)->TraiterLigne(Chargeur);
}

bool CTabProduits::TraiterLigne(const CChargeurFichierTexte& Chargeur)
{
    if (Chargeur.NombreChamps() == 2) Ajouter(CProduit(Chargeur.Champs(0),atol(Chargeur.Champs(1))));
    return true;
}

bool CTabProduits::Ajouter(const CProduit& Modele)
{
    return CTableauGenerique::Ajouter(Modele);
}

Merci d'avance pour votre aide.

2 réponses

Réponse 1 / 2
cs_jfrancois Messages postés 482 Date d'inscription vendredi 26 août 2005 Statut Membre Dernière intervention 5 décembre 2009 2
11 mai 2008 à 10:59
Bonjour,

1) Les 2 instructions suivantes sont tout à fait correctes donc le problème est certainement au niveau du fichier ! Il faudrait initialiser avec un nom complet pour tester : "C:\\Temp\\Fichier.txt" (par ex.).

Fichier = fopen(NomFichier,"rt");
if (Fichier == NULL) return false;

2) strlen(UnNom) avec UnNom = NULL ça plante !



bool CProduit::Nom(const char* UnNom)
{
   if((strlen(UnNom)<1)&&(UnNom==NULL))return false;
   return m_NomProduit.Chaine(UnNom);   
}

Il vaudrait mieux mettre (si pointeur nul ou si longueur nulle) :
   if (UnNom NULL  ||  *UnNom 0) return false;






Jean-François
0
Réponse 2 / 2
daviddubois Messages postés 92 Date d'inscription vendredi 19 mars 2004 Statut Membre Dernière intervention 6 janvier 2012
11 mai 2008 à 11:03
Bonjour Jean-François,

Un tout grand merci pour ta réponse.

Je viens juste de trouver le problème !

J'avais juste oublié de mettre ( ) dans :

const char* CProduit::Nom() const
{
    return m_NomProduit.Chaine;
}

Maintenant ça fonctionne.

Merci encore.
0

Discussions similaires

fopen(...,"a") ne crée pas le fichier
guiltyspark -
hl037 -

14 réponses
fopen https
nildv -
argentino -

11 réponses
fopen
Screderman -
Gendal67 -

2 réponses
Chemin de fichier avec fopen en tant que variable (gros débutant)
jason_red2 -
jason_red2 -

5 réponses
Application.Visible=False
pipive -
jack -

3 réponses
Rejoignez-nous