empiler des caracteres en pile langage c

[med126]

bonjour , je voulais empiler des caractere dans une pile en adoptant la representation contegue mais au niveau de l'exécution il ya une probleme
voici mon travaile

pile.h
void creer();
unsigned pile_vide();
void empiler(char );
char dernier();
void depiler();

pile.c
#include<assert.h>
#include"pile.h"
#define n 10

struct pile
{char cle[n];
int sommet;};

static struct pile p;

void creer_pile()
{p.sommet=-1;}

unsigned pile_vide()
{return(p.sommet==-1);}

char dernier()
{assert(!pile_vide());
return(p.cle[p.sommet]);}

void empiler(char x)
{p.sommet++;
p.cle[p.sommet]=x;}

void depiler()
{assert(!pile_vide());
 (p.sommet)--;}

test.c
#include<stdio.h>
#include"pile.cpp"
unsigned i;
void main()
{char c;
creer_pile();

for(i=0;i<10;i++)
{printf("donner c=");
scanf("%c",&c);
empiler(c);}

printf("le dernier caractere est=%c\n",dernier());

for(i=0;i<10;i++)
{printf("%c",dernier());
 depiler();} }

je travaile sur turbo c++, si un erreus existe me eclaire et merci en avance
-----

[Jack]

Le code est tellement mal indenté que ça ne donne pas envie de le lire. Essaie de le présenter comme il faut en ajoutant les balises code (voir les règles participatives de ce forum)

mais au niveau de l'exécution il ya une probleme

La moindre des choses serait d'indiquer ce qui te fait dire qu'il y a un problème.

PS : turbo C++ est complètement dépassé et ne correspond plus aux normes actuelles du C++. Tu devrais passer à quelque chose de plus moderne.

[med126]

a) c'est une probleme au niveau de l'empilement dans l'execution
b) si tu ne veux pas m'aider allez-vous en

[Jack]

b) si tu ne veux pas m'aider allez-vous en

Si tu le prends comme ça, ton vœux va être exaucé, il n'y a aucune chance pour que je t'aide désormais, vu comment tu tiens compte des remarques.

[A voir en vidéo sur Futura]

[EauPure]

Pourquoi (p.sommet)-- dans dépile
vue que p.sommet++; dans empile
p.sommet-- ça devrait peut être aller mieux
et puis c'est plus concis d'écrire p.cle[++p.sommet]=x; (en une ligne au lieu de 2)

[EauPure]

il y a une mauvaise utilisation de assert
si assert est vérifié en mode debug (option _DEBUG du compilateur) le programme s’arrête avec une erreur d’exécution
mais si tu es en mode release (sans l'option _DEBUG) ça ne fait rien
void depiler()
{
assert(!pile_vide());
(p.sommet)--;
}
donc remplace tout tes assert par des if
if (!pile_vide())
p.sommet--;

[pm42]

il y a une mauvaise utilisation de assert
si assert est vérifié en mode debug (option _DEBUG du compilateur) le programme s’arrête avec une erreur d’exécution
mais si tu es en mode release (sans l'option _DEBUG) ça ne fait rien
void depiler()
{
assert(!pile_vide());
(p.sommet)--;
}
donc remplace tout tes assert par des if
if (!pile_vide())
p.sommet--;

Ce n'est pas une mauvaise utilisation : c'est exactement ce qu'assert est supposé faire : décrire le contrat d'utilisation et pouvoir ou non être controlé au runtime suivant qu'on privilégie la fiabilité/débugging ou la performance.
Le remplacer par un if comme indiqué est une mauvaise idée : on passe de quelque chose d'immédiatement identifié comme un contrat, compilable ou pas à quelque chose qui n'a aucune de ses qualités et qui en cas d'erreur, de dépilage alors que rien n'est sur la pile va fonctionner sans rien signaler...

Enfin, _DEBUG est spécifique à certains compilateurs notamment Microsoft. La norme est NDEBUG (qui fonctionne dans l'autre sens) : https://en.wikipedia.org/wiki/Assert.h.

[Chanur]

a) c'est une probleme au niveau de l'empilement dans l'execution
b) si tu ne veux pas m'aider allez-vous en

J'ai compilé et exécuté ton programme.
Je n'ai pas compris le "#include "pile.cpp", alors je l'ai enlevé. On ne met pas un .cpp en include. C'est absurde.
Chez moi, l'empilement et le dépilement ont l'air de bien se passer.

Par contre, la saisie des caractères à empiler, c'est n'importe quoi. Mais comme, chez toi le problème est au niveau de l'empilement, ça n'a sans doute aucun rapport. Comme de toute façon tu ne nous dis pas ce que donne le programme quand tu le lances ...