Andy Stow Away 给出的答案修复了我的编译器警告,但没有回答我的第二个问题。eddieantonio 和 Niklas R 对该答案的评论回答了我的第二个问题,但没有修复我的编译器警告。因此,我将它们汇集到一个答案中。
C 不是面向对象的,试图在 C 中模拟面向对象的设计通常会导致风格不佳。复制在结构上调用的方法,以便可以使用指向结构的指针来调用它们,就像我在我的示例中那样。(坦率地说,它违反了 DRY。)结构中的函数指针对于多态性更有用。例如,如果我有一个表示元素线性序列的通用容器的结构向量,则存储一个比较函数成员可能很有用,该成员是一个函数指针,以允许对向量进行排序和搜索。向量的每个实例都可以使用不同的比较函数。但是,对于对结构本身进行操作的函数,最好有一个在结构中不重复的单独函数。
这使得正确的答案变得更加复杂。如何编译我的上述示例是正确的吗?是如何重新格式化我上面的示例以使其具有良好的风格?还是像 C 程序员那样使用函数指针的结构示例是什么?在提出我的问题时,我没有预料到我的问题是错误的。为了完整起见,我将提供每个问题答案的示例。
修复编译器警告
#include <stdio.h>
#include <stdlib.h>
struct my_struct
{
int data;
struct my_struct* (*set_data) (struct my_struct*, int);
};
struct my_struct* my_struct_set_data(struct my_struct* m, int new_data)
{
m->data = new_data;
return m;
}
struct my_struct* my_struct_create()
{
struct my_struct* result = malloc((sizeof(struct my_struct)));
result->data = 0;
result->set_data = my_struct_set_data;
return result;
}
int main(int argc, const char* argv[])
{
struct my_struct* thing = my_struct_create();
thing->set_data(thing, 1);
printf("%d\n", thing->data);
free(thing);
return 0;
}
重新格式化样式
#include <stdio.h>
#include <stdlib.h>
struct my_struct
{
int data;
};
void my_struct_set_data(struct my_struct* m, int new_data)
{
m->data = new_data;
}
struct my_struct* my_struct_create()
{
struct my_struct* result = malloc((sizeof(struct my_struct)));
result->data = 0;
return result;
}
int main(int argc, const char* argv[])
{
struct my_struct* thing = my_struct_create();
my_struct_set_data(thing, 1);
printf("%d\n", thing->data);
free(thing);
return 0;
}
在结构体中演示函数指针的使用
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
struct my_struct
{
void* data;
int (*compare_func)(const void*, const void*);
};
int my_struct_compare_to_data(struct my_struct* m, const void* comparable)
{
return m->compare_func(m->data, comparable);
}
struct my_struct* my_struct_create(void* initial_data,
int (*compare_func)(const void*, const void*))
{
struct my_struct* result = malloc((sizeof(struct my_struct)));
result->data = initial_data;
result->compare_func = compare_func;
return result;
}
int int_compare(const void* a_pointer, const void* b_pointer)
{
return *(int*)a_pointer - *(int*) b_pointer;
}
int string_compare(const void* a_pointer, const void* b_pointer)
{
return strcmp(*(char**)a_pointer, *(char**)b_pointer);
}
int main(int argc, const char* argv[])
{
int int_data = 42;
struct my_struct* int_comparator =
my_struct_create(&int_data, int_compare);
char* string_data = "Hello world";
struct my_struct* string_comparator =
my_struct_create(&string_data, string_compare);
int int_comparable = 42;
if (my_struct_compare_to_data(int_comparator, &int_comparable) == 0)
{
printf("The two ints are equal.\n");
}
char* string_comparable = "Goodbye world";
if (my_struct_compare_to_data(string_comparator,
&string_comparable) > 0)
{
printf("The first string comes after the second.\n");
}
free(int_comparator);
free(string_comparator);
return 0;
}