Pile en C++

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...

//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;

}

//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

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

ç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'

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 :

void pile::afficher()
{
	Pile* p=this;

	while(p!=NULL)
	{
		printf("La valeur est %d\n", p->valeur);
		p=p->suivant;
	}
}

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;

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();
	}
}

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;
}
}

il faut bien qu'elle ma pille que j'ai constiuté si je lui donne rien comment elle affiche??

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();

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)

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

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.

salut,
Est ce que ton objet pile hérite bien de l'objet elt ?
-
Grou29
http://lienssanslien.free.fr