warnings

[m@o_614]

saludos

tengo el siguiente código de una lista simplemente enlazada, pero el problema es que me aparece un warning que dice:

return makes pointer from integer without a cast

Código (c) [Seleccionar] Expandir

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

typedef struct nodo
{
    int dato;
    struct nodo*sig;
}NODO;

NODO* crear_nodo(int x);
void insertar_cabeza(NODO **cabeza,int x);
NODO* buscar_elemento(NODO *cabeza);

int main()
{
    int i,num,x;
    NODO *cabeza,*ptr;
    cabeza = NULL;
    printf("Cuantos elementos quieres?\n");
    scanf("%d",&num);
    printf("Lista Simplemente Ligada!\n");
    for(i=0;i<num;i++)
    {
        printf("Dame el elemento %d\n",i+1);
        scanf("%d",&x);
        insertar_cabeza(&cabeza,x);
        system("cls");
    }

    for(ptr=cabeza;ptr!=NULL;ptr=ptr->sig)
        printf("\n%d\n",ptr->dato);
    buscar_elemento(cabeza);
    return 0;
}

NODO* crear_nodo(int x)
{
    NODO *p;
    p = (NODO*)malloc(sizeof(NODO));
    p->dato = x;
    p->sig = NULL;
    return p;
}

void insertar_cabeza(NODO **cabeza,int x)
{
    NODO *nuevo;
    nuevo = crear_nodo(x);
    nuevo->sig = *cabeza;
    *cabeza = nuevo;
}

NODO* buscar_elemento(NODO *cabeza)
{
    int elemento,encontro=0;
    NODO *indice;
    printf("Buscar elemento\n\n");
    printf("Cual elemento quieres buscar?\n");
    scanf("%d",&elemento);
    for(indice=cabeza;indice!=NULL;indice=indice->sig)
    {
        if(indice->dato==elemento)
           encontro=1;
        else
           encontro=0;
    }
    if(encontro)
       return indice->dato;
    else
       return 0;
}

de antemano gracias

[naderST]

El warning es porque en tu caso buscar_elemento debe retornar un apuntador a NODO (NODO*) y tu estás retornando un entero.

Código (c) [Seleccionar] Expandir


  if(encontro)
      return indice->dato;
   else
      return 0;


[dato000]

pero el warning no afecta el funcionamiento del programa, deberia retornar NULL entonces??

[0xDani]

pero el warning no afecta el funcionamiento del programa, deberia retornar NULL entonces??

Si.

[m@o_614]

muchas gracias por sus respuestas, ahora ya no me aparece ese warning pero como ya le agregue la función de eliminar_elemento (en esta metí la función de buscar) pero no me hace lo que le pido

Código (c) [Seleccionar] Expandir

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

typedef struct nodo
{
    int dato;
    struct nodo*sig;
}NODO;

NODO* crear_nodo(int x);
void insertar_cabeza(NODO **cabeza,int x);
void eliminar_elemento(NODO **cabeza);

int main()
{
    int i,num,x;
    NODO *cabeza,*ptr;
    cabeza = NULL;
    printf("Cuantos elementos quieres?\n");
    scanf("%d",&num);
    printf("Lista Simplemente Ligada!\n\n");
    for(i=0;i<num;i++)
    {
        printf("Dame el elemento %d\n",i+1);
        scanf("%d",&x);
        insertar_cabeza(&cabeza,x);
        system("cls");
    }

    for(ptr=cabeza;ptr!=NULL;ptr=ptr->sig)
        printf("%d\t",ptr->dato);

    eliminar_elemento(&cabeza);
    for(ptr=cabeza;ptr!=NULL;ptr=ptr->sig)
        printf("%d\t",ptr->dato);
    return 0;
}

NODO* crear_nodo(int x)
{
    NODO *p;
    p = (NODO*)malloc(sizeof(NODO));
    p->dato = x;
    p->sig = NULL;
    return p;
}

void insertar_cabeza(NODO **cabeza,int x)
{
    NODO *nuevo;
    nuevo = crear_nodo(x);
    nuevo->sig = *cabeza;
    *cabeza = nuevo;
}

void eliminar_elemento(NODO **cabeza)
{
   int n;
   NODO *actual,*anterior;
   anterior=NULL;
   printf("Dame el elemento a eliminar\n");
   scanf("%d",&n);
   for(actual=*cabeza;actual!=NULL;actual=actual->sig)
   {
       if(actual->dato==n)
       {
           anterior = actual->sig;
           actual = anterior;
           free(actual);
       }
   }
}


estos punteros por qué son tan difíciles??

[twins]

Hola pues te explico para eliminar un nodo que este en una posicion distinta del inicio o final necesitas usar dos punteros para poder hacer los enlaces en este caso yo solo modifique la funsion eliminar_elemento tu puedes revisar y ver cuales fueron los cambios cordiales saludos 

Código (c) [Seleccionar] Expandir


#include <stdio.h>
#include <stdlib.h>
typedef struct nodo{
    int dato;
    struct nodo *sig;
}NODO;
NODO* crear_nodo(int x);
void insertar_cabeza(NODO **cabeza,int x);
void eliminar_elemento(NODO **cabeza);
int main(void){
    int i,num,x;
    NODO *cabeza,*ptr;
    cabeza = NULL;
    printf("Cuantos elementos quieres?\n");
    scanf("%d",&num);
    printf("Lista Simplemente Ligada!\n\n");
    for(i=0;i<num;i++){
        printf("Dame el elemento %d\n",i+1);
        scanf("%d",&x);
        insertar_cabeza(&cabeza,x);
        system("cls");
    }
    for(ptr=cabeza;ptr!=NULL;ptr=ptr->sig)
        printf("%d\t",ptr->dato);
    eliminar_elemento(&cabeza);
    for(ptr=cabeza;ptr!=NULL;ptr=ptr->sig)
        printf("%d\t",ptr->dato);
    return 0;
}
NODO* crear_nodo(int x){
    NODO *p;
    p = (NODO*)malloc(sizeof(NODO));
    p->dato = x;
    p->sig = NULL;
    return p;
}
void insertar_cabeza(NODO **cabeza,int x){
    NODO *nuevo;
    nuevo = crear_nodo(x);
    nuevo->sig = *cabeza;
    *cabeza = nuevo;
}
void eliminar_elemento(NODO **cabeza){
int n;
NODO *aux1=*cabeza,*aux2=aux1;
    printf("Dame el elemento a eliminar\n");
    scanf("%d",&n);
while(aux2->dato!=n){
aux1=aux2;
aux2=aux2->sig;
}
aux1->sig=aux2->sig;
    free(aux2);
}


[rir3760]

ahora ya no me aparece ese warning pero como ya le agregue la función de eliminar_elemento (en esta metí la función de buscar) pero no me hace lo que le pido

Como ya te comentaron:
A) Si el nodo a eliminar es el primero debes utilizar el parámetro de la función para así afectar la variable "cabeza" de la función main.
B) Para cualquier otro nodo solo hay que actualizar el campo "sig" del nodo anterior.

Para eliminar un nodo de una lista no es necesario utilizar variables auxiliares (en el bucle), en su lugar se trabaja directamente con el parámetro (y sirve para practicar el tema):

Código (c) [Seleccionar] Expandir

void eliminar_elemento(NODO **p)
{
   NODO *aux;
   int dato;
   
   puts("Dame el elemento a eliminar:");
   scanf("%d", &dato);
   
   while (*p != NULL && (*p)->dato != dato)
      p = &(*p)->sig;
   if (*p != NULL){
      aux = *p;
      *p = (*p)->sig;
      free(aux);
   }
}

Un saludo

[durasno]

Hola! muy bueno el aporte rir3760, la verdad me costo entenderlo; trabajar directamente con el puntero a puntero


Saludos

[m@o_614]

Muchas gracias por sus respuestas

Una última pregunta porque cuando le ingreso por ejemplo los números 5,18,67,80 me los imprime al revés 80, 67,18 y 5

no se supone que el puntero *cabeza siempre apunta al inicio de la lista?? ¿por qué la imprime así?

gracias

[twins]

Eso es porque los nodos los estas ingresando al inicio de la lista por ejemplo si ingresas el 1 y despues el 2, en este caso el 2 quedara a la izquierda del 1 es decir 2,1 y tu puntero cabeza apunta al segundo nodo recien ingresado, en este caso el 2, por lo tanto cuando imprimes, imprimes de izquierda a derecha ose 2,1.
si quieres que los nodos sea impresos segun el orden en que fueron ingresados tendras que modificar la funcion insertar_cabeza.