Fatidico suspisious pointer conversion.

[joanan46]

Buenas gente, ya entiendo mejor la listas, un gran paso ejje.

Pero hay una cosa que no logro resolver, los malditos warnings, y cunado son de suspisious pointer no me gustan nada.

tengo declarada la tupla nodo, con el prototipo de la funcion añadir.(Lo siento esta en Catalan ejej):

Código C:

struct LEnters{
    int info;
    struct Enters *seg;
};
 
 
int afegirInici(struct LEnters **pinici, int num);
 

Aqui el c:

Código C:

int afegirInici(struct LEnters **pinici, int num){
 
    struct LEnters *aux;
 
    aux=malloc(sizeof(struct LEnters));
 
    if(aux==NULL) return 1;
 
    aux->info=num;
    aux->seg=*pinici;
    *pinici=aux;
 
   return 0;
}
 

El suspisious pinter me lo lanza siempre que accedo aux->seg;, osea siempre que accedo al seg, que es un puntero a struct.

Sera que no accedo bien a ese puntero?

Gracias.

[joanan46]

esta todo bien?

[starfix]

que raro, a mi no dio ninguna advertencia, solamente no compilaba, para que compile cambie las siguientes lineas:

en el primero codigo:
linea 3:

Código C:

struct LEnters *seg;//el tipo estaba mal, le agrege la L
 

en el segundo codigo:
linea 5:

Código C:

//la funcion malloc devuelve un puntero a void, asi que tienes que agregar antes
//(LEnters*) para la compatibilidad de tipo
LEnters *aux=(LEnters*)malloc(sizeof(LEnters));
 

[joanan46]

Lo primero es verdad, que pajaro!!! Eso es por tanto tocar jejej. A mi incluso me compila, como vamos con el borland c++...

lo segundo no hace falta asegurarte que te devuelva un puntero a LEnters. No entiendo porque lo haces.

Salduos y muchas gracias.

[cesar_casla]

La funcion malloc te devuelve un puntero a void por lo que deberias castear ese retorno al tipo (LEnters *), si bien compila perfectamente e incluso no tira warning es porque internamente el compilador lo "interpreta" como si fuese correcto. ojo no te acostumbres a dejarlo sin castear es de buena practica hacerlo ademas cuando trabajes con el tipo de dato void * vas a ver que si o si necesitas castear
No soy un experto en C pero mas o menos asi me explicaron el tema de malloc

Saludos

[starfix]

por lo que estuve leyendo, el warning te lo da al trabajar con un puntero de puntero, o sea, cuando usas el dobre asterisco **Puntero......

ahora bien, como a mi no me tira el warning te voy a dar las posibles soluciones que deduje:

1

- que la funcion afegirInici reciba como parametros un puntero a LEnters, el protoripo quedaria de la siguiente manera:

int afegirInici(struct LEnters *pinici, int num);

- luego en la linea 10 del segundo codigo borrar el asterisco:

aux->seg=pinici;

2

- cambiar la linea del warning por la siguientes:

- aux->seg=(LEnter*)pinici;

esta ultima tendrias que probarla..... no sabia de que trataba el doble asterisco, asi que alguien mejor entendido de este tema te podria ayudar.

tambien mira esta pagina donde te explica porque te da el warning y las posibles soluciones.

http://c-pointer.blogspot.com/2010/0...s-pointer.html

[gonzo13]

El uso del puntero a puntero es correcto. Con eso está añadiendo el nodo en la lista original, y no en una copia.

Viendo el código original, diría que tu lista sólo contiene un nodo y que cuando intentes añadir otro nodo, dejarás el anterior perdido por la memoria del ordenador.

Un saludo,
gonzo

[joanan46]

Iniciado por starfix

por lo que estuve leyendo, el warning te lo da al trabajar con un puntero de puntero, o sea, cuando usas el dobre asterisco **Puntero......

ahora bien, como a mi no me tira el warning te voy a dar las posibles soluciones que deduje:

1

- que la funcion afegirInici reciba como parametros un puntero a LEnters, el protoripo quedaria de la siguiente manera:

int afegirInici(struct LEnters *pinici, int num);

- luego en la linea 10 del segundo codigo borrar el asterisco:

aux->seg=pinici;

2

- cambiar la linea del warning por la siguientes:

- aux->seg=(LEnter*)pinici;

esta ultima tendrias que probarla..... no sabia de que trataba el doble asterisco, asi que alguien mejor entendido de este tema te podria ayudar.

tambien mira esta pagina donde te explica porque te da el warning y las posibles soluciones.

http://c-pointer.blogspot.com/2010/0...s-pointer.html

Estoy trabajando con un puntero a struct LEnters. Sino me pasaria como dixe gonzo13.

[joanan46]

Código C:

struct LEnters{
    int info;
    struct LEnters *seg;
};
 
 
int afegirInici(struct LEnters **pinici, int num);
 
 
int afegirInici(struct LEnters **pinici, int num){
 
    struct LEnters *aux;
   //printf("Primer pinici apunta a %p\n", pinici);
    aux=malloc(sizeof(struct LEnters));
    if(aux==NULL) return 1;
    aux->info=num;
    aux->seg=*pinici;
    *pinici=aux;
 
    //printf("Despres: %p\n", pinici);
    //printf("%d  -  %p \n", aux->info, pinici);
 
   return 0;
}
 

Arreglao, el warning era que al nodo, al atributo seg no lo tenia bien declarado al nombre de struct LEnters.

[gonzo13]

La lista sigue teniendo sólo un nodo y perdiendo el resto de nodos en la memoria. Haz una comprobación "manual" (un folio y un lápiz) de tu código y verás qué ocurre con el nodo que ya tenías en la lista.

Podríamos decirte: "este es el fallo", pero intenta darte cuenta por tí mismo. Es la mejor forma de que no vuelva a ocurrirte.

Un saludo,
gonzo.