Ver Mensaje Individual
  #2  
Antiguo 09-04-2008
Avatar de escafandra
[escafandra] escafandra is offline
Miembro Premium
 
Registrado: nov 2007
Posts: 2.197
Reputación: 20
escafandra Tiene un aura espectacularescafandra Tiene un aura espectacular
Desde mi punto de vista tienes dos errores:
1) Asumes que el resultado de gets(t), es decir t, no va a cambiar. La realidad es que si cambia, por lo que apunta siempre a la última cadena introducida por teclado.

2) El bucle del printf está mal planteado, por eso obtienes un error, como el Pop lo haces después de comprobar si pila es nulo, en el ultimo elemento pila pasa a ser cero y ejecuta el printf pila->url, siendo pila NULL => ERROR.

La estructura del Push y Pop esta bien planteada.

Mira este código, basado en el tuyo.
Código:
#include <stdio.h>
#include <conio.h>
#include <string.h>
#include <stdlib.h>
typedef struct _nodo{
        char *url;
        int n;
        struct _nodo *siguiente;
        }TipoNodo;

typedef TipoNodo *pNodo;
typedef TipoNodo *Pila;

void Push(Pila *l,char *web);
char* Pop(Pila *l);

int main(){
  Pila pila= NULL;
  char *t="dentro";

do{
  printf("Introdusca la paguina web a visitar\n");
  gets(t);
  Push(&pila,t);
} while(strcmp(t,"salir"));

  while(pila->siguiente!=NULL){
  Pop(&pila);
  printf("%d\n",pila->n);
}

  printf("Final presione una tecla para continuar...");
  getch();
  return 0;  
    }

void Push(Pila *pila,char *web){
  pNodo nuevo;
  nuevo= (pNodo)malloc(sizeof(TipoNodo));
  if(*pila==0) nuevo->n=0;
  else nuevo->n=(*pila)->n+1;
  nuevo->url= web;
  nuevo->siguiente= *pila;
  *pila=nuevo;
}

char* Pop(Pila *pila){
      pNodo nodo;
      char *val;
      nodo=*pila;
      if(!nodo) return "Vacio";
      *pila= nodo->siguiente;
      val=nodo->url;
      free(nodo);
      return val;
}


Saludos.

Última edición por escafandra fecha: 11-04-2008 a las 00:49:55.
Responder Con Cita