c - 如何命名一个好的/有意义的类型？

Question

设备管理器.h

typedef enum
{
    DNM = 0x2A,

}TYPE_e;

typedef struct DEVICE_s* p_DEVICE;
typedef p_DEVICE(*FUNC)(char* name, TYPE_e type, uint32_t ip, uint16_t method, uint16_t groupRule);   

p_DEVICE DeviceManager_New(void);
p_DEVICE DeviceManager_Ctor(char* name, TYPE_e type, uint32_t ip, uint16_t method, uint16_t groupRule);
p_DEVICE DeviceManager_Dtor(p_DEVICE element);

设备管理器.c

struct DEVICE_s
{
    uint32_t IP;
    TYPE_e Type;
    uint16_t Method;
    uint16_t GroupRule;
    char Name[40];
    FUNC fp_Ctor, fp_Dtor;    //this line needs modification
}DeviceSet[32],DeviceTemp;

p_DEVICE DeviceManager_InitObject(p_DEVICE* self)
{
    (*self) = DeviceManager_New();
    (*self)->IP = 0;
    (*self)->Type = 0;
    (*self)->Method = 0;
    (*self)->GroupRule = 0;
    memset((*self)->Name, 0, NAME_SIZE);
    (*self)->fp_Ctor = DeviceManager_Ctor;
    (*self)->fp_Dtor = DeviceManager_Dtor;    // warning: assign to wrong type
    return (*self);
}
p_DEVICE DeviceManager_New(void)
{
    return &DeviceTemp;
}
p_DEVICE DeviceManager_Ctor(char* name, TYPE_e type, uint32_t ip, uint16_t method, uint16_t groupRule)
{
    memcpy(DeviceTemp.Name, name, sizeof(name));
    DeviceTemp.Type = type;
    DeviceTemp.IP = ip;
    DeviceTemp.Method = method;
    DeviceTemp.GroupRule = groupRule;
    return &DeviceTemp;
}
p_DEVICE DeviceManager_Dtor(p_DEVICE element)
{
    element->IP = 0;
    element->Type = 0;
    element->Method = 0;
    element->GroupRule = 0;
    memset(element->Name, 0, NAME_SIZE);
    return element;
}

这是我第一次实现封装概念，遇到了一些问题。

在头文件中，我使用 typedef 将类型“FUNC”定义为函数指针。

我觉得“FUNC”这个名字不够明确，因为这种命名时尚会导致:

struct DEVICE_s
{
    uint32_t IP;
    TYPE_e Type;
    uint16_t Method;
    uint16_t GroupRule;
    char Name[40];
    FUNC1 fp_Ctor;    //not clear
    FUNC2 fp_Dtor;    //not clear
}DeviceSet[32],DeviceTemp;

fp_Ctor 和 fp_Dtor 都是相同的类型(函数指针)，只是参数的个数不同。

我总是在命名类型上挣扎。能否提供一些关于命名类型的建议？

Answer 1

这一切都有点主观，但我会从放弃将指针隐藏在 typedef 后面的风格开始，并在其上涂上某种“匈牙利符号”。很多 C 程序员都会同意这一点。

所以第一个建议是去

typedef struct DEVICE_s DEVICE_s;

然后根据 DEVICE_s* 定义不透明接口(interface)。它不仅更易于阅读，而且可以过滤掉混淆，例如调用者试图将 p_DEVICE* 传递给用户定义的函数等，因为他们没有意识到他们已经有了一个指针。 (Win32 API 严重受此问题影响。)

那么你的构造函数就变成了

DEVICE_s* DeviceManager_Ctor ( ...

并且所有成员函数都将采用 DEVICE_s* 参数而不是 p_DEVICE 值。调用者必须声明指针而不是对象，让他们清楚他们有一个指向不完整类型的指针，他们不能/不应该玩弄任何东西。

接下来，您也可以删除隐藏在函数指针中的指针。这不是什么大问题，但最好保持一致:

typedef DEVICE_s* DeviceManager_Ctor_t ( ...

您的函数指针定义将变为:

DeviceManager_Ctor_t* Ctor;

您可以删除“fp”命名，因为该类型已经很明显是一个函数指针。

---

作为旁注，我建议避免使用 obj.member 表示法模仿 C++ 成员函数。因为在 C 中，缺少 this 指针，您最终会得到 obj.member(&obj, ...)，这有点多余。

宁愿接受 C 就是它的样子，并将成员函数调用为 DeviceManager_Ctor(obj); 其中 obj 声明为 DEVICE_s* obj;。可读 OO 代码的关键是对属于“类”的所有函数使用一致的源代码前缀，就像您已经做的那样:DeviceManager_。