2

El programa complila pero tira la siguiente advertencia

warning: assignment from incompatible pointer type

y no entiendo por qué.

Este es el código:

#include <stdio.h>
#include <stdlib.h>

typedef struct nodo
{
    int dato;
    struct Nodo *siguiente;
} Nodo;

typedef Nodo *list;
typedef Nodo *pnodo;

//Nodo *inicio,*inicio2,*ultimo,*ultimo2;

void agregar_lista1(list *inicio,list *ultimo);
void agregar_lista2(list *inicio2,list *ultimo2);
void mostrar_lista(list *inicio);

int main ()
{
    int op;
    list inicio = NULL,inicio2 = NULL,ultimo = NULL,ultimo2 = NULL;
    do
    {
        printf ("MENU");
        printf ("\nOp1 Ingresar nodo en la lista 1");
        printf ("\nOp2 Ingresar nodo en la lista 2");
        printf ("\nOp3 Mostar los datos almacenados en la lista");
        printf ("\nOp4 Mostrar suma de las listas anteriores\n\n");
        scanf ("%d",&op);
        switch(op)
        {
        case 1:
            agregar_lista1(&inicio,&ultimo);
            break;
        case 2:
            agregar_lista1(&inicio2,&ultimo2);
            break;
        case 3:
            mostrar_lista(&inicio);
            break;
        case 4:
            break;
        }
    }
    while (op!=4);

    return 0;
}

void agregar_lista1(list *inicio,list *ultimo)
{
    Nodo *nuevo;
    nuevo = malloc(sizeof(Nodo));
    if (nuevo==NULL)
    {
        printf ("\nNo se pudo crear el nodo\n");
    }

    printf ("Ingrese un numero entero positivo: ");
    scanf ("%d",&nuevo->dato);

    if (inicio == NULL)
    {
        nuevo->siguiente = (*inicio);
        *inicio = nuevo;
        *ultimo  = nuevo;
    }
    else
    {
        /* en la linea de abajo es donde me da el error */
        (*ultimo)->siguiente = nuevo;
        *ultimo = nuevo;
        nuevo->siguiente = NULL;
    }
}

void mostrar_lista(list *inicio)
{
    pnodo ptr;
    ptr = *inicio;
    if (inicio != NULL)
    {
        while (ptr != NULL)
        {
            printf ("\nDato %d \n",ptr->dato);
            ptr = ptr->siguiente;
        }
    }
    else
    {
        printf ("\n\nLa lista esta vacia\n\n");
    }
}
1
  • La definición de mostrar_lista arriba y la implementación abajo no coinciden. En la definición no tiene parámetros y en la implementación sí. Además, si recibe un list*, no le puedes pasar la variable inicio en la llamada que haces en el switch porque inicio es del tipo list y no del tipo list*. Además, tienes definida una función agregar_lista2 que luego ni llamas ni implementas. No me he parado a revisar el resto del código, pero el warning parece que es por cómo llamas a mostrar_lista.
    – SuperG280
    Commented el 23 ago. 2019 a las 5:46

2 respuestas 2

2

El lenguaje C es sensitivo a mayúsculas y minúsculas. Dentro de tu definición de nodo estás creando un tipo nuevo llamado Nodo:

 typedef struct nodo
//       ^^^^^^^^^^^ <--- Tipo 'struct nodo';
 {
     int dato;
     struct Nodo *siguiente;
//   ^^^^^^^^^^^^^ <--- Puntero a un TIPO NUEVO que es 'struct Nodo'.
 } Nodo;
// ^^^^ <--- Alias de 'nodo' llamado 'Nodo'.

Cambia tu definición para que el tipo interno struct Nodo coincida con el nombre de tu tipo externo:

typedef struct Nodo
{
    int dato;
    struct Nodo *siguiente;
} Nodo;

Problemas adicionales en tu código.

A parte de lo mencionado en comentarios por SuperG280, tu código tiene un fallo que es especialmente molesto y recurrente en el sitio StackOverflow.

Estás creando un Nodo de datos que se usará como parte de una lista simplemente enlazada y renombras un puntero a ese nodo como lista. Mal hecho, eso es tan disparatado como decir que un escalón es una escalera, sinceramente ¿Te parecen lo mismo?:

Un Nodo es un Nodo no una Lista, así que no deberías llamar lista a un puntero a Nodo. Esta confusión ha sucedido montones de veces y no para de suceder una y otra vez. El problema que provoca no llamar las cosas por su nombre es que el código se hace difícil de entender, de seguir, de depurar y de compartir. Te aconsejo que cambies por completo tu aproximación creando un objeto Lista que contenga Nodos:

typedef struct Nodo
{
    int dato;
    struct Nodo *siguiente;
} Nodo;

typedef struct Lista
{
    Nodo *raiz;
} Lista;
1
  • Pongo opción 1 (1 Ingresar nodo en la lista 1) y solo deja poner un solo número. Commented el 23 ago. 2019 a las 13:56
0

Acabé de ver el código, hay varias fallas. En este prototipo:

void agregar_lista1(list *inicio,list *ultimo);

El primer parámetro de la función recibe la dirección de memoria de un puntero; sin embargo, tu haces esto:

if (inicio == NULL) //Error lógico
{

Lo que estás comparando ahí, es lo que tiene guardado el puntero a puntero y no debería ser de esa manera. Ya que el puntero doble "inicio" tiene la dirección de memoria del puntero que pasaste como referencia y es evidente que esa condición no será igual a NULL y por esa razón, el caso contrario se cumplirá y ahí es donde viene el inconveniente:

else
{
   /* en la linea de abajo es donde me da el error */
   (*ultimo)->siguiente = nuevo;
   *ultimo = nuevo;
   nuevo->siguiente = NULL;
}

Como no existe ningún Nodo inicial, se está tratando de acceder a la dirección 0x00 y desde ahí, el programa se bloquea.

(*ultimo)->siguiente = nuevo;

La solución es cambiar la condición por esta:

if (*inicio == NULL)
{
    nuevo->siguiente = (*inicio);
    *inicio = nuevo;
    *ultimo  = nuevo;
}

Con *inicio, se accede al contenido de la dirección de memoria que pasaste por referencia y de ese modo, el resultado de esa condición será verdadera. El mismo problema tienes en la función "mostrar_lista", debes cambiar la condición a esta:

void mostrar_lista(list *inicio)
{
    pnodo ptr;
    ptr = *inicio;
    if (*inicio != NULL) //aquí faltó el asterisco 
    {
3
  • Ahora cuando apreto 1 para ingresar números a la lista, dice "Ingrese un numero entero positivo: ", per cuando ingreso me vuelve a mostrar el menú. Commented el 25 ago. 2019 a las 22:23
  • Te va mostrar el menú, porqué no estás pausando el programa. Es decir, cada vez que el usuario ingrese un valor, debería pausarlo, de ese modo, no saldrá de una vez el menú.
    – MrDave1999
    Commented el 26 ago. 2019 a las 1:33
  • Pauso después del scanf y queda con el cursor esperando. Commented el 28 ago. 2019 a las 14:08

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.