le pointeur de fonction et la fonction est incompatible à cause des arguments

Question

J'ai lu le livre Comprendre et utiliser les pointeurs C et essayer de compiler le code ci-dessous. Mais après la compilation, j'ai reçu l'avertissement: affectation à partir du type de pointeur incompatible.

J'ai vérifié le code et trouver le pointeur de fonction fptrSet et la fonction ShapeSetX est incompatible parce que le premier argument de fptrSet est void * et la fonction ShapeSetX est Shape *.

Comment puis-je résoudre ce problème? Merci!

typedef void (*fptrSet)(void*, int); 
typedef int (*fptrGet)(void*); 
typedef void (*fptrDisplay)();  

typedef struct _vfunc 
{ 
    fptrSet  setX; 
    fptrGet  getX; 
    fptrSet  setY; 
    fptrGet  getY; 
    fptrDisplay display;  
} vFunc;  

typedef struct _shape 
{ 
    vFunc function; 
    int x; 
    int y; 
} Shape; 

void displayShape(){ 
    printf("Shape\n"); 
} 
void ShapeSetX(Shape *shape, int x){ 
    shape->x = x; 
} 
void ShapeSetY(Shape *shape, int y){ 
    shape->y = y; 
} 
int ShapeGetX(Shape *shape){ 
    return shape->x; 
} 
int ShapeGetY(Shape *shape){ 
    return shape->y; 
}  

Shape *newShape() 
{ 
    Shape *shape = (Shape *)malloc(sizeof(Shape)); 
    shape->x = 10; 
    shape->y = 10; 
    shape->function.setX = ShapeSetX; 
    shape->function.getX = ShapeGetX; 
    shape->function.setY = ShapeSetY; 
    shape->function.getY = ShapeGetY; 
    shape->function.display = displayShape; 
    return shape; 
} 

Answer 1

Vous devez respecter la définition du pointeur: pointeur besoin que le premier paramètre est un pointeur d'annuler, de sorte que votre mise en œuvre de la fonction doit avoir le premier paramètre comme vide:

void ShapeSetX(void *void_shape, int x){ 
    Shape *shape = (Shape*) void_shape; 
    shape->x = x; 
} 
void ShapeSetY(void *void_shape, int y){ 
    Shape *shape = (Shape*) void_shape; 
    shape->y = y; 
} 
int ShapeGetX(void *void_shape){ 
    Shape *shape = (Shape*) void_shape; 
    return shape->x; 
} 
int ShapeGetY(void *void_shape){ 
    Shape *shape = (Shape*) void_shape; 
    return shape->y; 
} 

Answer 2

J'allais dire « Pourquoi ne remplacez-vous pas void par Shape puis? ", jusqu'à ce que je me rende compte que Shape n'a pas encore été défini - vous ne pouvez pas non plus échanger les deux définitions, car Shape a besoin de vFunc qui a besoin de typedef s.

Alors, faites ceci:

typedef struct _shape Shape; // Define _shape and Shape later 

typedef void (*fptrSet)(Shape*, int); 
typedef int (*fptrGet)(Shape*); 
typedef void (*fptrDisplay)();  

Si votre compilateur ne pas comme ça, vous devrez peut-être changer pour:

typedef struct _shape; // Define _shape later 

typedef void (*fptrSet)(struct _shape*, int); 
typedef int (*fptrGet)(struct _shape*); 
typedef void (*fptrDisplay)();