0

Tengo un problema en C. Tengo un puntero tipo const void llamado p1, el cual a la hora de acceder a su valor me da errores. Se supone que es porque tengo que hacerlo tipo struct, pero no se hacer el cast para poder acceder a la estructura. Alguna solución o ejemplo de como hacer correctamente el cast de *void a struct? Los errores que me salen son del tipo: warning: dereferencing 'void *' pointer in: if(*p1 == *p2)

Gracias por adelantado.

/**
 * @brief Compares two points.
 * 
 * @param p1,p2 Points to compare.
 *
 * @return Returns True or False. 
 * In case of error, returns FALSE. 
 */
Bool point_equal (const void *p1, const void *p2){

    if(*p1 == *p2){
        return TRUE;
    }
    else{
        return FALSE;
    }
}
5
  • 1
    (struct tu_estructura *) p1
    – abulafia
    Commented el 14 feb. 2022 a las 13:13
  • ¿Podrías aportar la parte del código donde te da error? "Me da errores" no es una buena explicación, ¿puedes aportar qué tipo de errores te dan?
    – Londo
    Commented el 14 feb. 2022 a las 13:13
  • Ya lo he subido. Gracias por molestarte en comentar :)
    – Nico221003
    Commented el 14 feb. 2022 a las 20:32
  • Quita el operador de puntero * del if y ya podrás comparar valores de direccionamiento entre las variables p1 y p2. Además, las variables booleanas deben ir en minúsculas, y tú tienes 3 referencias mal puestas. Éstas son: TRUE y FALSE (deben ir en minúscula), y el retorno de la función debe ser bool, la primera letra también en minúsculas.
    – Londo
    Commented el 14 feb. 2022 a las 20:43
  • muchas gracias !!
    – Nico221003
    Commented el 14 feb. 2022 a las 20:50

1 respuesta 1

0

Como bien dijeron, el casteo se debe hacer mediante (struct nombre_struct *)p1. Ahora, la comparacion (struct nombre_struct *)p1 == (struct nombre_struct *)p2 posiblemente no esté haciendo lo que vos esperás.

Lo que estas haciendo al comparar un struct contra otro es ver si la direccion de memoria de cada estrucutra coincide. Estimo que lo que queres hacer en este caso es corroborar si algun campo o todos coinciden.

Algo por el estilo de:

Bool point_equal (const void *p1, const void *p2){

    if((struct tu_struct *)p1->a == (struct tu_struct *)p2->a){
        if((struct tu_struct *)p1->b == (struct tu_struct *)p2->b){
            if((struct tu_struct *)p1->c == (struct tu_struct *)p2->c){
                return TRUE;
            }
        }
    }
 
    return FALSE;
}

Y de yapa: No es necesario que uses else en el ejemplo que diste, ya que si se entra al if vas a retornar. Bien podrias haber hecho (y es mas optimo):

Bool point_equal (const void *p1, const void *p2){

    if(*p1 == *p2){
        return TRUE;
    }

    return FALSE;
}

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.