【发布时间】:2018-08-16 08:02:52
【问题描述】:
我正在使用 C99 开发一个项目,我正在努力使其符合 MISRA 2012 标准。
在一个文件中,我定义了一个枚举,其中每个值都应被视为一个标志:
/**
* Enumerates the configurable options for performing calibration.
*/
typedef enum
{
CALIBRATION_DEFAULT_OPTIONS=0, /**< Calibrate with default options */
CALIBRATION_RESET_POSITION=1, /**< Ensure window is fully open and motor re-homed */
CALIBRATION_FORCE_RECALIBRATE=2 /**< Force recalibration even if calibration data exists */
} CALIBRATION_OPTIONS_T;
我希望能够声明如下内容:
CALIBRATION_OPTIONS_T options = CALIBRATION_RESET_POSITION | CALIBRATION_FORCE_RECALIBRATE;
我还定义了一个函数,它接受CALIBRATION_OPTIONS_T 参数并根据设置的标志执行不同的逻辑:
// If forced to recalibrate, do so regardless of whether metrics exist in
// EEPROM or not.
if ((options & CALIBRATION_FORCE_RECALIBRATE) != 0U)
{
MOTION_ResetCalibrationData();
calibration = performCalibrationRoutine();
}
// Otherwise try fetching existing metrics from EEPROM. If they exist, return
// these metrics.
else if (tryFetchStoredMetrics(&calibration))
{
if ((options & CALIBRATION_RESET_POSITION) != 0U)
{
calibration.lastPosition = 0;
resetMotorPosition();
storeMetrics(calibration);
}
}
但是,当我使用 PC-lint Plus 对项目进行 lint 时,我得到以下输出,说明此代码违反了 MISRA 2012 规则 10.1:
if ((options & CALIBRATION_FORCE_RECALIBRATE) != 0U)
~~~~~~~ ^
*** LINT: src\c\motionCalibrator.c(645) note 9027: an enum value is not an appropriate left operand to & [MISRA 2012 Rule 10.1, required]
if ((options & CALIBRATION_FORCE_RECALIBRATE) != 0U)
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
*** LINT: src\c\motionCalibrator.c(645) note 9027: an enum value is not an appropriate right operand to & [MISRA 2012 Rule 10.1, required]
if ((options & CALIBRATION_FORCE_RECALIBRATE) != 0U)
^
*** LINT: src\c\motionCalibrator.c(645) warning 641: implicit conversion of enum 'CALIBRATION_OPTIONS_T' to integral type 'unsigned int'
if ((options & CALIBRATION_RESET_POSITION) != 0U)
~~~~~~~ ^
*** LINT: src\c\motionCalibrator.c(655) note 9027: an enum value is not an appropriate left operand to & [MISRA 2012 Rule 10.1, required]
if ((options & CALIBRATION_RESET_POSITION) != 0U)
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~
*** LINT: src\c\motionCalibrator.c(655) note 9027: an enum value is not an appropriate right operand to & [MISRA 2012 Rule 10.1, required]
if ((options & CALIBRATION_RESET_POSITION) != 0U)
^
*** LINT: src\c\motionCalibrator.c(655) warning 641: implicit conversion of enum 'CALIBRATION_OPTIONS_T' to integral type 'unsigned int'
特别是,MISRA 2012 标准建议不要将& 与枚举一起使用,原因有两个:
本质上为枚举类型的操作数不应用于 算术运算,因为枚举对象使用 实现定义的整数类型。涉及枚举的操作 因此,对象可能会产生具有意外类型的结果。注意 来自匿名枚举的枚举常量基本上已签名 输入。
移位和按位运算只能对 本质上是无符号类型。使用它们产生的数值 基本上有符号的类型是实现定义的。
我想知道是否有一种符合 MISRA 的方式,我可以使用类似标志的枚举并测试是否设置了特定标志。
【问题讨论】:
-
使用定义代替标志。枚举不适用于位标志。
-
@Tagc 如果你觉得你真的想在 MISRA 中使用枚举,如果你在对它们进行算术运算之前将它们转换为无符号类型可能会起作用,但我会坚持定义。
-
@Tagc 只是不要写这样错误的代码,你很好;)
-
@Tagc 实际上,您将类型安全与演员一起扔出了窗外。因为它说:忽略它拥有的类型,让它成为我想要的
-
我所说的“类型安全”是指我的编译器(ARM C 编译器 v5.06)确实如果使用枚举并做一些像
options = CALIBRATION_RESET_POSITION | 8这样的傻事会警告我.它仍然可以编译,但发出警告总比没有好。而使用定义,编译器甚至不会发出警告。但是,我动摇了,我现在已经改用定义了。我会尽量确保我不会做像CALIBRATION_RESET_POSITION | 8这样的傻事。