[?]Eliminar nodo

[lerg96]

Buenas, es la primera vez que posteo en estos foros así que no me amonesten si cometo faltas dentro de ellos, o no tan fuerte. Mi duda es la siguiente: Necesito que mi programa elimine un nodo en especifico a partir de un identificador, cuando coloco el identificador el programa si elimina el nodo. Pero ocurre un problema que es cuando el nodo que intento eliminar no es el primero y luego lo intento mostrar.




Tengo entendido que si el nodo esta al final de la lista(? (no estoy seguro si se dice así) se debe apuntar el nodo anterior a NULL, lo único es que no sé como hacerlo

Además de que tengo otro problema que es cuando el identificador que intento eliminar no lo encuentra, el programa



Estuve buscando ejemplos que pueda ver en el foro, pero no encuentro uno que pueda entender al 100%. Si pudieran ayudarme, se los agradeceria. Además de que puedan explicarme como es este proceso de eliminar (Como leer el código en sí)

Código (cpp) [Seleccionar] Expandir

[
#include <stdio.h>
#include <stdlib.h>
struct nodo
{
int dato;
char nombre[10],telefono[7];
struct nodo *puntero;
};
typedef nodo lista;
lista *nuevo,*inicio,*fin;
void crear_nodos()
{
nuevo=new(nodo);
printf("\nNuevo elemento.\n");
printf("Identificador: ");
scanf("%d",&nuevo->dato);
printf("Nombre: ");
fflush(stdin);
fgets(nuevo->nombre,10,stdin);
printf("Telefono: ");
fgets(nuevo->telefono,7,stdin);
nuevo->puntero= NULL;
if (inicio==NULL)
{
printf("Primer elemento\n");
inicio = nuevo;
fin = nuevo;
}
else
{
fin->puntero = nuevo;
/* hacemos que el nuevo sea ahora el último */
fin = nuevo;
}
}
void mostrar_nodos()
{
lista *auxiliar;
auxiliar=inicio;
while(auxiliar!=NULL)
{
printf("###################\n");
printf("Identificador: %d\n",auxiliar->dato);
printf("Nombre: %s\n",auxiliar->nombre);
printf("Telefono: %s\n",auxiliar->telefono);
auxiliar=auxiliar->puntero;
}
printf("\n");
}
void borrarNodo()
{
    system("cls");
nuevo = inicio;
    if(nuevo != NULL)
    {

        nodo *aux = NULL, *ant = NULL;
        ant = new nodo;  
        aux = new nodo;
        printf("INTRODUZCA EL ID A ELIMINAR: ");
scanf("%d",&aux->dato);
while(aux->dato != nuevo->dato)
          {
  ant = nuevo;
  nuevo = nuevo->puntero;
          }
if(nuevo == inicio)
          {          
  inicio = inicio->puntero;
             free(ant);
             free(aux);
             printf("ELEMENTO 1 ELIMINADO");
}else if(aux->dato == nuevo->dato)
               {                  
ant = nuevo;
nuevo = nuevo->puntero;                
                  free(ant);
                  free(aux);
                  printf("ELEMENTO x ELIMINADO");
               }                              
   }else  
      printf("NO HAY ELEMENTOS PARA ELIMINAR");
}
void menu()
{
int op;
system("cls");
printf (" Menu Principal\n");
printf(" 1.- Insertar Nodos \n");
printf(" 2.- Mostrar Nodos\n");
printf(" 3.- Eliminar Nodos \n");
printf(" 4.- Salir \n");
printf(" Seleccione opcion: ");
scanf("%d",&op);
switch (op)
{
case 1: system("cls");
printf("Ingresa los nodos a la lista");
crear_nodos();
system("pause");
menu();
case 2: system("cls");
printf("Muestra los nodos de la lista\n");
mostrar_nodos();
system("pause");
menu();
case 3: system("cls");
printf("Elimina nodos\n");
borrarNodo();
system("pause");
menu();
case 4: exit(0);
default: printf("Opcion no valida");
system("pause");
menu();
}
}
void main()
{
menu();
}

Sé que no debo usar fflush(stdin); pero aún no sé una manera de pedir una variable entera sin usar scanf para que fgets no se salte xD

lo único que se me ocurre es poner

Código (cpp) [Seleccionar] Expandir


ant->puntero= NULL;
fin->puntero = ant;
fin = ant;

pero nada me funciona

[rir3760]

En tu programa utilizas solo la biblioteca estándar de C cuando, por el uso de new, se trata de C++. Si estas aprendiendo C++ enfocate en su biblioteca estándar. En cuanto al uso intercalado de scanf y fgets (el problema similar en C++ aparece con el uso del operador ">>" y getline) hay un montón de temas en la base de datos de los foros, solo es cuestión de utilizar el motor de búsqueda.

También por favor lee el tema |Lo que no hay que hacer en C/C++. Nivel basico|.

----

Los problemas en su mayoría se encuentran en la función "borrarNodo", el que mencionas se genera si el nodo a eliminar no existe debido a este bucle:

Código (cpp) [Seleccionar] Expandir

printf("INTRODUZCA EL ID A ELIMINAR: ");
scanf("%d", &aux->dato);
while (aux->dato != nuevo->dato){
   ant = nuevo;
   nuevo = nuevo->puntero;
}
En el se itera una y otra vez hasta encontrar un nodo con el valor buscado, si eso no sucede se tratara de procesar el nodo después del ultimo (NULL) causando el error que mencionas al realizar la operación (NULL)->puntero.

----

Mira que pensaba poner un listado de los errores en esa función pero, y recalco que no quiero sonar grosero, son demasiados: reservas memoria cuando no es necesario, no revisas si se alcanza el final de la lista, los nombres de las variables son malos ("nuevo" es el puntero para iterar la lista), hay que tener en cuenta variables "globales" que no son necesarias, etc..

La base para eliminar un nodo es:

Código (cpp) [Seleccionar] Expandir

void borrarNodo()
{
   if (inicio == NULL)
      puts("Lista vacia");
   else {
      nodo *aux;
      int dato;
     
      puts("Dato a eliminar:");
      scanf("%d", &dato);
     
      if (inicio->dato == dato){
         // Eliminamos el primero
         aux = inicio;
         inicio = inicio->puntero;
         free(aux);
      }else {
         // Eliminamos (si aplica) algun nodo despues del primero
         nodo *p = inicio;
         
         while (p->puntero != NULL && p->puntero->dato != dato)
         p = p->puntero;
         
         // ...
      }
   }
}

Por supuesto no esta completo, falta la parte mas importante y difícil (eliminar un nodo después del primero). Inténtalo tu solo y si tienes problemas te ayudamos.

Un saludo

[avesudra]

Hola lerg96 a parte de lo que te ha comentado rir3760 te comento que:

• En la función menu, si escoges una opcion vuelves a llamar a menu recursivamente, esto se va acumulando en la pila de llamadas y no es bueno.
• La función main debe tener un tipo de retorno entero, no puede ser void.
• No deberías mezclar C++ con C, en C++ las bibliotecas adecuadas son cstdlib y cstdio, aunque ésta última es útil en casos muy concretos y su trabajo lo hace iostream prácticamente.
• El typedef está mal puesto, tienes puesto:

Código (cpp) [Seleccionar] Expandir

typedef nodo lista;
Cuando lo correcto es:

Código (cpp) [Seleccionar] Expandir

typedef struct nodo lista;

• En cuanto a lo del fflush(stdin), deberías usar fgets + sscanf(en C) o cin directamente en C++.


• No debes usar system("pause") pues no funciona en todas las plataformas, lo puedes sustituir por un getchar(), o un cin.get()
• No deberías usar system("cls"), podrías poner una directiva de preprocesador pero como trabajas sobre Visual Studio no sé como va en dicho compilador.

Lo del menu lo puedes arreglar con un do - while o con while simplemente.

Aunque como dice rir3760 el problema principal está en esa función.

[lerg96]

Mira que pensaba poner un listado de los errores en esa función pero, y recalco que no quiero sonar grosero, son demasiados: reservas memoria cuando no es necesario, no revisas si se alcanza el final de la lista, los nombres de las variables son malos ("nuevo" es el puntero para iterar la lista), hay que tener en cuenta variables "globales" que no son necesarias, etc..

Cuando estaba haciendo el procedimiento para borrar un nodo (Que por cierto, ni idea de lo que hacía en ese momento), me fije bastante de un tema que estaba aquí, básicamente copie, pegue y cambie algunas cosas hasta que me funcionara. Esa misma noche duré bastante buscando solución a esos problemas (Luego de haberlo posteado) hasta que di por fin con todos por mi cuenta.

Hola lerg96 a parte de lo que te ha comentado rir3760 te comento que:

• En la función menu, si escoges una opcion vuelves a llamar a menu recursivamente, esto se va acumulando en la pila de llamadas y no es bueno.
• La función main debe tener un tipo de retorno entero, no puede ser void.
• No deberías mezclar C++ con C, en C++ las bibliotecas adecuadas son cstdlib y cstdio, aunque ésta última es útil en casos muy concretos y su trabajo lo hace iostream prácticamente.
• El typedef está mal puesto, tienes puesto:

Código (cpp) [Seleccionar] Expandir

typedef nodo lista;
Cuando lo correcto es:

Código (cpp) [Seleccionar] Expandir

typedef struct nodo lista;

• En cuanto a lo del fflush(stdin), deberías usar fgets + sscanf(en C) o cin directamente en C++.


• No debes usar system("pause") pues no funciona en todas las plataformas, lo puedes sustituir por un getchar(), o un cin.get()
• No deberías usar system("cls"), podrías poner una directiva de preprocesador pero como trabajas sobre Visual Studio no sé como va en dicho compilador.

Lo del menu lo puedes arreglar con un do - while o con while simplemente.

Aunque como dice rir3760 el problema principal está en esa función.

Sobre el resto del código, sé que hay demasiados errores. Esto es debido a los conocimientos que me imparte mi profesora de algoritmos. El tema que me recomendaste "|Lo que no hay que hacer en C/C++. Nivel básico|." Lo había leído antes, a mitad del trimestre, imagínate mi cara al darme cuenta que la mayoría de las cosas que me enseñan no sirven para hacer algo profesional, la última vez que intente usar algo más adecuado casi pierdo la nota total de un examen T_T  


a la final el procedimiento para borrar quedo de esta manera:

Código (cpp) [Seleccionar] Expandir

    if(inicio != NULL)
    {
int dato;
        nodo *aux = NULL, *ant = NULL;
        ant = new nodo;  
        aux = new nodo;
aux = inicio;
        printf("INTRODUZCA EL ID A ELIMINAR: ");
scanf("%d",&dato);
while(dato != aux->dato)
          {
  ant = aux;
  aux = aux->puntero;
  if (aux==NULL) break;
          }
if(aux==NULL)
{
printf("IDENTIFICADOR NO ENCONTRADO\n");
system("pause");

}
else

{
if(aux == inicio)
          {          
  inicio = inicio->puntero;
             free(ant);
             free(aux);
             printf("ELEMENTO 1 ELIMINADO");
}
else if((ant == inicio)&&(aux==fin))
               {      
ant->puntero= NULL;
inicio = ant;
fin = ant;
                  printf("ELEMENTO ULTIMO ELIMINADO");
               }
else if(ant != NULL)
               {      
ant->puntero=aux->puntero;
free(aux);
                  printf("ELEMENTO x ELIMINADO");
               }
   }
}
else  
      printf("NO HAY ELEMENTOS PARA ELIMINAR");

Pero eso fue lo que hice esa noche, que me ayudo bastante al entendimiento de este. Si creen que puede hacerse mejor o de otra manera, lo agradeceria bastante. Ya me encuentro probando con el codigo de rir3760, intentando completar las partes que mencionaste.

Muchas gracias por su atención, realmente mi intención es aprender, y hacer todo de la manera correcta. Un abrazo!!