Pile en C++
simachille
Messages postés
64
Date d'inscription
Statut
Membre
Dernière intervention
-
KX Messages postés 16761 Date d'inscription Statut Modérateur Dernière intervention -
KX Messages postés 16761 Date d'inscription Statut Modérateur Dernière intervention -
Bonjour,
sil vous plait je fais de la POO
j'essai sans succès d'afficher une pile en C++
Dans mon fichier .h j'ai declaré void Afficher(elt *lapile);
Dans mon fichier .cpp j'ai declaré void pile::Afficher(elt *lapile);
Dans mon Programme principal j'ecris (*une_pile).Afficher(une_pile);
quand je compile j'ai comme erreur
9 C:\Users\Achille\Documents\mes docs\c++\une_pile\unepile.cpp no matching function for call to `pile::Afficher(pile*&)'
note C:\Users\Achille\Documents\mes docs\c++\une_pile\pile.h:21 candidates are: void pile::Afficher(elt*)
MERCI de m'aider
sil vous plait je fais de la POO
j'essai sans succès d'afficher une pile en C++
Dans mon fichier .h j'ai declaré void Afficher(elt *lapile);
Dans mon fichier .cpp j'ai declaré void pile::Afficher(elt *lapile);
Dans mon Programme principal j'ecris (*une_pile).Afficher(une_pile);
quand je compile j'ai comme erreur
9 C:\Users\Achille\Documents\mes docs\c++\une_pile\unepile.cpp no matching function for call to `pile::Afficher(pile*&)'
note C:\Users\Achille\Documents\mes docs\c++\une_pile\pile.h:21 candidates are: void pile::Afficher(elt*)
MERCI de m'aider
15 réponses
Déjà il faut se poser la question de ce qu'est censé faire la méthode afficher...
Normalement, elle ne devrait pas prendre de paramètre !!!
On devrait avoir : void pile::afficher();
Normalement, elle ne devrait pas prendre de paramètre !!!
On devrait avoir : void pile::afficher();
candidates are: void pile::Afficher(elt*) -> ton compilateur accepte ta méthode
Par contre ton programme de test est bizarre :
(*une_pile).Afficher(une_pile); ???
Ton compilateur comprend que "une_pile" est une "pile*" par l'appel de la méthode, et un "elt*" par l'appel des paramètres.
En gros tu dis que une_pile est à la fois pile et elt.
La seule chance pour que ça marche c'est (comme grou29 le suggérait) qu'il y ait une relation mère-fille entre les classes elt et pile.
Par contre ton programme de test est bizarre :
(*une_pile).Afficher(une_pile); ???
Ton compilateur comprend que "une_pile" est une "pile*" par l'appel de la méthode, et un "elt*" par l'appel des paramètres.
En gros tu dis que une_pile est à la fois pile et elt.
La seule chance pour que ça marche c'est (comme grou29 le suggérait) qu'il y ait une relation mère-fille entre les classes elt et pile.
bonsoir , pas du tout j'ai fait un fichier.h dans lequel je forme le type element et je l'ai inclu j'ai pas d'heritage
Vous n’avez pas trouvé la réponse que vous recherchez ?
Posez votre question
Je precise que j'ai pas d'heritage
si je met ça une_pile.Afficher(une_pile.un_elt);
(un_elt est un attrribut de la classe pile et une_pile est un objet de la meme classe)
ça compile mais pas d'affichage (ça se plante)
si je met ça une_pile.Afficher(une_pile.un_elt);
(un_elt est un attrribut de la classe pile et une_pile est un objet de la meme classe)
ça compile mais pas d'affichage (ça se plante)
voici son code
//Methode pour Afficher
void pile :: Afficher( elt *lapile)
{ elt *p;
p=lapile;
while(p!=NULL)
{ printf("\nLa valeur est %d\n", (*p).valeur);
p=(*p).suivant;
}
}
//Methode pour Afficher
void pile :: Afficher( elt *lapile)
{ elt *p;
p=lapile;
while(p!=NULL)
{ printf("\nLa valeur est %d\n", (*p).valeur);
p=(*p).suivant;
}
}
Faut pas que tu le prenne mal mais c'est moche ;-)
class Pile
{
private Elt valeur;
private Pile* suiv;
void afficher()
{
std::cout << valeur << std::endl;
if (suiv!=NULL) suiv->afficher();
}
}
c'est la meme chose
j'ai mi private Elt valeur;
private Pile* suiv;
dans un fichier elt.h
dans ma classe j'ai donc elt un_elt;
j'ai mi private Elt valeur;
private Pile* suiv;
dans un fichier elt.h
dans ma classe j'ai donc elt un_elt;
Oui mais c'est ta fonction afficher() qui n'est pas bonne !
Si vraiment tu veux la faire avec un while() ce qui n'est pas vraiment le mieux tu aurais :
Si vraiment tu veux la faire avec un while() ce qui n'est pas vraiment le mieux tu aurais :
void pile::afficher()
{
Pile* p=this;
while(p!=NULL)
{
printf("La valeur est %d\n", p->valeur);
p=p->suivant;
}
}
ça me donne deux erreurs
56C:\Users\...\Documents\mes docs\c++\une_pile\pile.cpp 'class pile' has no member named 'valeur'
57 C:\Users\...\Documents\mes docs\c++\une_pile\pile.cpp 'class pile' has no member named 'suivant'
56C:\Users\...\Documents\mes docs\c++\une_pile\pile.cpp 'class pile' has no member named 'valeur'
57 C:\Users\...\Documents\mes docs\c++\une_pile\pile.cpp 'class pile' has no member named 'suivant'
Moi je les ai mis dans ma déclaration post 10, et tu me dis l'avoir mis post 11...
Mais peut-être as-tu oublié de les mettre dans ton .h
Mais peut-être as-tu oublié de les mettre dans ton .h
//Definition (implementation) d'un élément qui formera la pile
#ifndef _ELT_H_
#define _ELT_H_
typedef struct elet
{
int valeur;
elet *suivant;
}elt;
#endif
#ifndef _ELT_H_
#define _ELT_H_
typedef struct elet
{
int valeur;
elet *suivant;
}elt;
#endif
//Fichier .h de la classe pile
#ifndef _PILE_H_
#define _PILE_H_
#include "elt.h"
class pile
{
elt *un_elt;
//Methodes
public:
//Methode pour depiler
elt* empiler(elt *lapile,int a);
//Methode pour lire les valeurs
void lire();
//Methode pour depiler
elt* depiler(elt *lapile);
//fonction pour Afficher
void Afficher();
};
#endif
#include <iostream>
#include "elt.h"
#include "pile.h"
using namespace std;
//Methode pour Empiler
elt* pile :: empiler (elt *lapile,int val)
{ elt *p,*elm;
if(lapile==NULL)
{lapile=(elt*)malloc(sizeof(elt));
(*lapile).valeur=val;
(*lapile).suivant=NULL;
}
else
{
elm=(elt*)malloc(sizeof(elt));
(*elm).valeur=val;
(*elm).suivant=lapile;
lapile=elm;
}
return lapile ;
}
//Methode pour lire les valeurs
void pile::lire()
{elt *pile;
int i,n,valeur;
cout<<"\nDonner la taille de la pile\n";
cin>>n;
pile=NULL;
for(i=1;i<=n;i++)
{cout<<"entrer la valeur \t"<<i;
cin>>valeur;
pile=pile::empiler(pile,valeur);
}
}
////Methode pour Depiler
//elt* pile ::Depiler (elt *lapile)
// { elt *p;
// p=lapile;
// lapile=(*lapile).suivant;
// free(p);
// return lapile;
//}
//Methode pour Afficher
void pile::Afficher()
{
pile* p=this;
while(p!=NULL)
{
cout<<"\nLa valeur est \n"<< (*p).valeur;
p=(*p).suivant;
}
}
// void pile :: Afficher( elt *lapile)
// { elt *p;
// p=lapile;
// while(p!=NULL)
// { cout<<"\nLa valeur est \n"<< (*p).valeur;
// p=(*p).suivant;
// }
// }
Prog principal
#include <iostream>
#include "elt.h"
#include "pile.h"
using namespace std;
main()
{
pile *une_pile=new pile();int s;
(*une_pile).lire();
(*une_pile).Afficher();
//cin>>s;
}
#ifndef _PILE_H_
#define _PILE_H_
#include "elt.h"
class pile
{
elt *un_elt;
//Methodes
public:
//Methode pour depiler
elt* empiler(elt *lapile,int a);
//Methode pour lire les valeurs
void lire();
//Methode pour depiler
elt* depiler(elt *lapile);
//fonction pour Afficher
void Afficher();
};
#endif
#include <iostream>
#include "elt.h"
#include "pile.h"
using namespace std;
//Methode pour Empiler
elt* pile :: empiler (elt *lapile,int val)
{ elt *p,*elm;
if(lapile==NULL)
{lapile=(elt*)malloc(sizeof(elt));
(*lapile).valeur=val;
(*lapile).suivant=NULL;
}
else
{
elm=(elt*)malloc(sizeof(elt));
(*elm).valeur=val;
(*elm).suivant=lapile;
lapile=elm;
}
return lapile ;
}
//Methode pour lire les valeurs
void pile::lire()
{elt *pile;
int i,n,valeur;
cout<<"\nDonner la taille de la pile\n";
cin>>n;
pile=NULL;
for(i=1;i<=n;i++)
{cout<<"entrer la valeur \t"<<i;
cin>>valeur;
pile=pile::empiler(pile,valeur);
}
}
////Methode pour Depiler
//elt* pile ::Depiler (elt *lapile)
// { elt *p;
// p=lapile;
// lapile=(*lapile).suivant;
// free(p);
// return lapile;
//}
//Methode pour Afficher
void pile::Afficher()
{
pile* p=this;
while(p!=NULL)
{
cout<<"\nLa valeur est \n"<< (*p).valeur;
p=(*p).suivant;
}
}
// void pile :: Afficher( elt *lapile)
// { elt *p;
// p=lapile;
// while(p!=NULL)
// { cout<<"\nLa valeur est \n"<< (*p).valeur;
// p=(*p).suivant;
// }
// }
Prog principal
#include <iostream>
#include "elt.h"
#include "pile.h"
using namespace std;
main()
{
pile *une_pile=new pile();int s;
(*une_pile).lire();
(*une_pile).Afficher();
//cin>>s;
}
Ce n'est pas vraiment ça que j'appelle utiliser les balises de code !
Pourquoi vouloir mélanger le C et le C++ ?
Dans ton .h tu ne définis pas un type elt, mais une struct elet dont elt est une instance !
Le mieux c'est de faire une classe Elt...
Mais en fait ta classe Elt telle que tu l'as définis ici c'est déjà ta classe Pile !
Remarque : (un_type*).qqch <=> un_type->qqch !!!
De plus normalement tu n'as pas besoin de faire de malloc, l'allocation mémoire se fait dans ton constructeur et la désallocation dans ton destructeur mais tu n'as fait aucun des deux !
Bref : si tu veux faire du C++, oublie le C (pas totalement mais quand même)
Regarde le site du zéro, c'est un bon site pour commencer. En particulier regarde les cours sur les classes même si un peu avant ne devrait pas te faire de mal...
Pourquoi vouloir mélanger le C et le C++ ?
Dans ton .h tu ne définis pas un type elt, mais une struct elet dont elt est une instance !
Le mieux c'est de faire une classe Elt...
Mais en fait ta classe Elt telle que tu l'as définis ici c'est déjà ta classe Pile !
Remarque : (un_type*).qqch <=> un_type->qqch !!!
De plus normalement tu n'as pas besoin de faire de malloc, l'allocation mémoire se fait dans ton constructeur et la désallocation dans ton destructeur mais tu n'as fait aucun des deux !
Bref : si tu veux faire du C++, oublie le C (pas totalement mais quand même)
Regarde le site du zéro, c'est un bon site pour commencer. En particulier regarde les cours sur les classes même si un peu avant ne devrait pas te faire de mal...
