Ç命名建议

我正在写一个简单的解析器读取配置文件。该接口的config.h只有三个他们在简略如下主要功能，Ç命名建议

config_init(); 
config_dinit(); 
config_parse(); 
config_read_value();

我的问题是这些功能将发射不同类型的错误，对于一个示例，

config_init() emit , FILE_NOT_FOUND,FILE_EOF_ERROR,FILE_OPEN_ERROR, ... 
config_dinit() emit , NOT_INIT_ERROR , 
config_parse() emit , PARSE_ERROR, OVERFLOW_ERROR, INVALID_CHARACTER_FOUND_ERROR,... 
config_read_value() emit, SECTION_NOT_FOUND,KEYWORD_NOT_FOUND,OVERFLOW_ERROR,NOT_INITIALIZED_ERROR,INVALID_STATE_ERROR,... etc. 

Then I create enums for each function, for by using these names , 
enum Config_ParseError{...} , enum Config_InitError{...} ,enum Config_ReadValueError{..} 
etc.

一些枚举值相互重叠并点击“编译器错误”过。像溢出错误，

我打开您的建议，

，我已经做了谷歌的快速调研，发现最流行的IRC客户端源代码已经定义了这样的枚举，

enum { 
    CMDERR_OPTION_UNKNOWN = -3, /* unknown -option */ 
    CMDERR_OPTION_AMBIGUOUS = -2, /* ambiguous -option */ 
    CMDERR_OPTION_ARG_MISSING = -1, /* argument missing for -option */ 

    CMDERR_UNKNOWN, /* unknown command */ 
    CMDERR_AMBIGUOUS, /* ambiguous command */ 

     CMDERR_ERRNO, /* get the error from errno */ 
    CMDERR_NOT_ENOUGH_PARAMS, /* not enough parameters given */ 
    CMDERR_NOT_CONNECTED, /* not connected to server */ 
    CMDERR_NOT_JOINED, /* not joined to any channels in this window */ 
    CMDERR_CHAN_NOT_FOUND, /* channel not found */ 
    CMDERR_CHAN_NOT_SYNCED, /* channel not fully synchronized yet */ 
    CMDERR_ILLEGAL_PROTO, /* requires different chat protocol than the active server */ 
    CMDERR_NOT_GOOD_IDEA, /* not good idea to do, -yes overrides this */ 
    CMDERR_INVALID_TIME, /* invalid time specification */ 
    CMDERR_INVALID_CHARSET, /* invalid charset specification */ 
    CMDERR_EVAL_MAX_RECURSE, /* eval hit recursion limit */ 
    CMDERR_PROGRAM_NOT_FOUND /* program not found */ 
};

它定义的enum没有任何名称，那是一个很好的风格吗？那么为什么那个原因是什么呢？

严重需要一些更好的命名决定。请不要伤害我我只是开始阅读“写美丽的C代码”一书。

在此先感谢。 Sandun。

来源

2011-06-09 sandun dhammika

我通常是很整个库的一组错误返回的风扇。这种方式在消费者中，他们不必担心“是X的-1输入错误或无法连接到Y”。

我也E_前缀的粉丝，但算不上什么都行：

enum _config_error 
{ 
    E_SUCCESS = 0, 
    E_INVALID_INPUT = -1, 
    E_FILE_NOT_FOUND = -2, /* consider some way of returning the OS error too */ 
    ... 
}; 

/* type to provide in your API */ 
typedef _config_error error_t; 

/* use this to provide a perror style method to help consumers out */ 
struct _errordesc { 
    int code; 
    char *message; 
} errordesc[] = { 
    { E_SUCCESS, "No error" }, 
    { E_INVALID_INPUT, "Invalid input" }, 
    { E_FILE_NOT_FOUND, "File not found" }, 
    ... 
};

来源

2011-06-09 04:05:30 user7116

哇好建议标题先生， – 2011-06-09 13:30:11

'类型定义枚举_config_error error_t;'为我工作。 – Danijel 2017-04-07 07:06:11

我认为这是很好的风格。 CMDERR_前缀将相关的错误代码组合在一起（假设它们与某种“命令调用/执行”有关）

由于您的所有示例似乎都与您的配置功能相关，因此我只需要一个使用CONFIG_前缀的枚举定义（或简称为CFG_）。

enum Config_Errors { 
    CONFIG_FILE_NOT_FOUND, 
    CONFIG_FILE_EOF_ERROR, 
    CONFIG_FILE_OPEN_ERROR, 
    //etc. 
};

背后的公共前缀的原因是使用枚举类型，当你想清楚该类型的成员都属于同一组。

来源

2011-06-09 02:16:30

在IRC客户端源代码中的CMDERR_前缀是一个很好的风格，但定义枚举没有任何名称是不是一个好的风格。不是好事，因为你不能说这是一个枚举类型，只有整数类型如下图所示：

CMDERR function1(); 

int function1(); // actually returning CMDERR unnamed enum

，并使用像下面的枚举类型不能定义变量：

CMDERR errResult; 

int errResult; // actually errResult is CMDERR unnamed enum

来源

2011-06-09 03:20:54 dragon135

回答

相关问题