IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

C++ Discussion :

Mon programme tourne fou


Sujet :

C++

Vue hybride

Message précédent Message précédent   Message suivant Message suivant
  1. #1
    Membre expérimenté
    Avatar de beegees
    Homme Profil pro
    Développeur Web
    Inscrit en
    Mars 2004
    Messages
    3 610
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 50
    Localisation : Belgique

    Informations professionnelles :
    Activité : Développeur Web
    Secteur : Enseignement

    Informations forums :
    Inscription : Mars 2004
    Messages : 3 610
    Par défaut Mon programme tourne fou
    Bonjour tout le monde,

    J'essaie à partir de code existant de faire un exercice.

    Mon programme tourne fou, j'ai mis des points d'arrêt et alors qui'l ne doit lire qu'une seule ligne d'un fichier txt, après plus de 150 clic sur F5, ça continue.

    Voici la ligne sur lequel le programme tourne fou :

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    TabClassifications.Ajouter(Classifications);
    L'appel ci-dessus va sur cette méthode :

    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
    bool CTableauGenerique::Ajouter(const CGenerique& ModeleAAjouter, bool RetourEnCasDeDoublon)
    {
    	if (!ModeleAAjouter.EstValide()) return false;
    	if ( (m_Poubelle->GereUnicite()) && (Indice(ModeleAAjouter) >= 0) ) return RetourEnCasDeDoublon;
    	void* Nouveau = realloc(m_Tableau,(m_Nombre+1)*sizeof(CGenerique*));
    	if (Nouveau == NULL) return false;
    	m_Tableau = (CGenerique**)Nouveau;
    	CGenerique* AAjouter = ModeleAAjouter.Cloner();
    	if (!AAjouter->EstValide())
    	{
    		delete AAjouter;
    		return false;
    	}
    	m_Tableau[m_Nombre] = AAjouter;
    	m_Tableau[m_Nombre]->DefinirConteneur(this);
    	m_Nombre++;
    	return true;
    }
    donc il n'arrive jamais ici :

    for (n=0; n < TabClassifications.Nombre(); n++)


    Voici le code de mon main :

    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
    #include <BasicConsole.h>
    #include "vehicule.h"
     
    void main()
    {
    	long	n;
     
    	CTableauGenerique	TabVehicules((const CGenerique&)CTabVehicules());
    	for (n=1; true; n++)
    	{
    		CTabVehicules	Vehicules;
    		char			NomFichier[26];
    		sprintf(NomFichier,"vehicules.%ld.txt",n);
    		if (!Vehicules.Charger(NomFichier)) break;
    		Vehicules.Trier();
    		TabVehicules.Ajouter(Vehicules);
    	}
    	for (n=0; n < TabVehicules.Nombre(); n++)
    	{
    		printf("\n\nTableau des véhicules de \"vehicules.%ld.txt\"\n",n+1);
    		((CTabVehicules&)TabVehicules.Element(n)).Afficher();
    	}
    }
     
    /*
    void main()
    {
    	CTabVehicules	Vehicules;
     
    	if (Vehicules.Charger("vehicules.txt"))
    	{
    		Vehicules.Trier();
    		printf("Tous les véhicules...\n");
    		Vehicules.Afficher();
    		printf("\nUniquement les voitures...\n");
    		Vehicules.Afficher("Voiture");
    		printf("\nUniquement les camions...\n");
    		Vehicules.Afficher("Camion");
    		printf("\nUniquement les bus...\n");
    		Vehicules.Afficher("Bus");
    	}
    }
    */
    Voici aussi mon fichier cpp :

    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
    61
    62
    63
    64
    65
    66
    67
    68
    69
    70
    71
    72
    73
    74
    75
    76
    77
    78
    79
    80
    81
    82
    83
    84
    85
    86
    87
    88
    89
    90
    91
    92
    93
    94
    95
    96
    97
    98
    99
    100
    101
    102
    103
    104
    105
    106
    107
    108
    109
    110
    111
    112
    113
    114
    115
    116
    117
    118
    119
    120
    121
    122
    123
    124
    125
    126
    127
    128
    129
    130
    131
    132
    133
    134
    135
    136
    137
    138
    139
    140
    141
    142
    143
    144
    145
    146
    147
    148
    149
    150
    151
    152
    153
    154
    155
    156
    157
    158
    159
    160
    161
    #include <BasicConsole.h>
    #include <search.h>
    #include "chiens.h"
     
    //il faut donc encore créer les classes origines et utilisations
     
    /////////////////
    //Class CChiens//
    ////////////////
     
    CChiens::CChiens()
    {
    }
     
    CChiens::CChiens(const CChiens& Source) 
    :CGenerique(Source),m_Nom(Source.m_Nom)
    {
    }
     
    CChiens::CChiens(const char* UnNom)//OK
    {
    	Nom(UnNom);
    }
     
     
    CTabClassifications::~CTabClassifications()
    {
    }
     
    bool CChiens::Nom(const char* UnNom) //ok
    {
    	if ( (UnNom == NULL) || (strlen(UnNom) < 2) ) return false;
    	return m_Nom.Chaine(UnNom);
     
    }
     
    void CChiens::Afficher()const
    {
    	printf("- Classifications : %s\n",Nom());
    }
     
    CTabClassifications::CTabClassifications(const CTabClassifications& Source)
    :CTableauGenerique(Source)
    {
    }
     
     
    void CTabClassifications::Afficher(const char* FiltreSurType) const
    {	long ordre = 0;
    	for (long n=0; n < 10; n++)
    	{
    		if ( (FiltreSurType == NULL) || (strcmp(Element(n).Type(),FiltreSurType) == 0) )
    		{
    			ordre++;
    			printf("%s n°%ld :\n",Element(n).Type(),ordre);
    			Element(n).Afficher();
    		}
    	}
     
    }
     
    const char* CChiens::Type() const
    {
    	return null;
    }
     
    int CChiens::Comparer(const CGenerique& Autre, long TypeComparaison) const
    {
    	return 0;
    }
     
    bool CChiens::EstValide() const
    {
    	return true;
    }
     
    CGenerique* CChiens::Cloner() const
    {
    	return new CChiens(*this);
    }
     
    long CChiens::NombreClesDeTri() const
    {
    	return 1;
    }
     
    bool CChiens::GereUnicite() const
    {
    	return true;
    }
     
    CChiens::~CChiens()
    {
     
    }
     
    const char* CChiens::Nom() const
    {
    	return m_Nom.Chaine();
    }
     
     
    CChiens& CTabClassifications::Element(long UnIndice) const
    {
    	return (CChiens&)CTableauGenerique::Element(UnIndice);
    }
    ///////////////////////////////
    //Class CTabClassifications////
    ///////////////////////////////
     
    CTabClassifications::CTabClassifications()
    :CTableauGenerique(CChiens("POUBELLE"))
    {
    }
     
    long CTabClassifications::Indice(const char* UnNom) const
    {
    	//ici je ne passe que le nom et pas le compteur, on travaille ici avec le constructeur spécifique
    	//de la classe CChiens ! donc je ne passe qu'un paramètre contrairement aux véhicules
    	//donc je vérifie le nom dans le tableau, si il existe déjà j'incrémente le compteur
    	//autrement je l'ajoute et j'incrémente le compteur.
     
     
    	//dans les tableaux il ne faut pas remettre les membres !!!!
    	return CTableauGenerique::Indice(CChiens(UnNom));
     
    }
     
    bool CTabClassifications::Ajouter(const CChiens& Modele)
    {
    	return CTableauGenerique::Ajouter(Modele);
    }
     
    void CTabClassifications::Trier()
    {
    	CTableauGenerique::Trier(ComparaisonPourTri);
    }
     
    bool CTabClassifications::Charger(const char* NomFichier)
    {
    	CChargeurFichierTexte	Chargeur;
     
    	return Chargeur.Charger(NomFichier,'\t',CTabClassifications::FctnTraiterLigne,this);
    }
     
    /*static*/ bool CTabClassifications::FctnTraiterLigne(const CChargeurFichierTexte& Chargeur, void* Contexte)
    {
    	return ((CTabClassifications*)Contexte)->TraiterLigne(Chargeur);
    }
     
     
    bool CTabClassifications::TraiterLigne(const CChargeurFichierTexte& Chargeur)
     
    //le chargeur ouvre le fichier, lit ligne par ligne et éclate la ligne, il n'y a donc plus rien à faire
    {
    	if (Chargeur.NombreChamps() == 4)
    	{
    		Ajouter(CChiens(Chargeur.Champs(1)));
    	}
    	return true;
    }



    Si vous avez besoin d'autre info n'héistez pas à me le dire.

    Un tout grand merci d'avance pour votre aide.

    beegees

  2. #2
    Expert confirmé

    Inscrit en
    Août 2006
    Messages
    3 967
    Détails du profil
    Informations forums :
    Inscription : Août 2006
    Messages : 3 967
    Par défaut
    Fao,

    La boucle
    est infinie, puisque tu mets comme condition d'arrêt true, qui, étant une constante, ne changera pas.

  3. #3
    Membre expérimenté
    Avatar de beegees
    Homme Profil pro
    Développeur Web
    Inscrit en
    Mars 2004
    Messages
    3 610
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 50
    Localisation : Belgique

    Informations professionnelles :
    Activité : Développeur Web
    Secteur : Enseignement

    Informations forums :
    Inscription : Mars 2004
    Messages : 3 610
    Par défaut
    Citation Envoyé par droggo Voir le message
    Fao,

    La boucle
    est infinie, puisque tu mets comme condition d'arrêt true, qui, étant une constante, ne changera pas.
    Salut,

    Merci pour ta réponse.

    Ce qui est étrange s'est que je me base sur un autre exercice, cet autre exercice utilise cette boucle infinie qui se termine :

    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
    	CTableauGenerique	TabVehicules((const CGenerique&)CTabVehicules());
    	for (n=1; true; n++)
    	{
    		CTabVehicules	Vehicules;
    		char			NomFichier[26];
    		sprintf(NomFichier,"vehicules.%ld.txt",n);
    		if (!Vehicules.Charger(NomFichier)) break;
    		Vehicules.Trier();
    		TabVehicules.Ajouter(Vehicules);
    	}
    	for (n=0; n < TabVehicules.Nombre(); n++)
    	{
    		printf("\n\nTableau des véhicules de \"vehicules.%ld.txt\"\n",n+1);
    		((CTabVehicules&)TabVehicules.Element(n)).Afficher();
    	}
    }
    Qu'en penses-tu ?

    Merci encore.

    beegees

  4. #4
    Membre éclairé
    Avatar de Le Farfadet Spatial
    Homme Profil pro
    En cours de précision…
    Inscrit en
    Avril 2008
    Messages
    190
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Haute Garonne (Midi Pyrénées)

    Informations professionnelles :
    Activité : En cours de précision…

    Informations forums :
    Inscription : Avril 2008
    Messages : 190
    Par défaut
    Salut à tous !

    Citation Envoyé par beegees Voir le message
    Ce qui est étrange s'est que je me base sur un autre exercice, cet autre exercice utilise cette boucle infinie qui se termine :

    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
    	CTableauGenerique	TabVehicules((const CGenerique&)CTabVehicules());
    	for (n=1; true; n++)
    	{
    		CTabVehicules	Vehicules;
    		char			NomFichier[26];
    		sprintf(NomFichier,"vehicules.%ld.txt",n);
    		if (!Vehicules.Charger(NomFichier)) break;
    		Vehicules.Trier();
    		TabVehicules.Ajouter(Vehicules);
    	}
    	for (n=0; n < TabVehicules.Nombre(); n++)
    	{
    		printf("\n\nTableau des véhicules de \"vehicules.%ld.txt\"\n",n+1);
    		((CTabVehicules&)TabVehicules.Element(n)).Afficher();
    	}
    }
    Qu'en penses-tu ?
    J'en pense que l'instruction break met fin à la boucle, de sorte que cette ligne :

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    if (!Vehicules.Charger(NomFichier)) break;
    est là pour mettre fin à la boucle. Cela dit, je trouve cet exemple pas terrible, je te conseille de ne pas trop le prendre comme exemple.

    À bientôt.

    Le Farfadet Spatial

  5. #5
    Membre expérimenté
    Avatar de beegees
    Homme Profil pro
    Développeur Web
    Inscrit en
    Mars 2004
    Messages
    3 610
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 50
    Localisation : Belgique

    Informations professionnelles :
    Activité : Développeur Web
    Secteur : Enseignement

    Informations forums :
    Inscription : Mars 2004
    Messages : 3 610
    Par défaut
    Salut Le Farfadet Spatial,

    OK le problème doit venir de là car il arrive toujours à charger le fichier, s'est pas normal quand même ?

    Merci pour ton aide.

    beegees

  6. #6
    Membre Expert
    Avatar de coyotte507
    Profil pro
    Inscrit en
    Octobre 2006
    Messages
    1 327
    Détails du profil
    Informations personnelles :
    Âge : 35
    Localisation : France

    Informations forums :
    Inscription : Octobre 2006
    Messages : 1 327
    Par défaut
    Salut,

    il faut voir le code de ta fonction Vehicule::chargerFichier

+ Répondre à la discussion
Cette discussion est résolue.

Discussions similaires

  1. mon programme tourne en boucle
    Par domxaline dans le forum Langage
    Réponses: 4
    Dernier message: 12/06/2009, 18h46
  2. Mon programme Matlab tourne mal!
    Par halyins dans le forum MATLAB
    Réponses: 6
    Dernier message: 03/06/2009, 17h25
  3. Savoir sur quel os tourne mon programme
    Par clampin dans le forum Débuter
    Réponses: 4
    Dernier message: 13/09/2008, 00h28
  4. Réponses: 12
    Dernier message: 03/04/2008, 13h19
  5. Réponses: 11
    Dernier message: 17/03/2003, 10h56

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo