铸造结构* void*

Question

我在C中具有以下功能：

Bool message(void* msg, unsigned timeout);

以及我想传递给该功能的结构：

msgSPI_t msgSPI; //msgSPI_t is of struct type

像这样：

message(&msgSPI, (unsigned)0)

现在，这是按预期工作的，但是由于我正在使用Sonarqube进行代码质量检查，因此我得到了传递的错误＆amp; msgspi：

“去除这种危险的铸件。” （根据Misra C：2004，11.2和11.3）。

由于功能消息是第三方代码，因此我不能（或不应该）更改它。这里有什么好的解决方案，可以满足声纳的投诉？ （在＆amp; msgspi的前面添加（void*）清除该错误，但我不确定这是否是正确的方法。）

Answer 1

我认为这是一个长度预备的块。在具有继承的语言中，您将使用类似的内容：

class BaseMessage {
   uint32_t size;
};

class MyMessage extends BaseMessage {
   // ...
};

bool message( BaseMessage *msg, unsigned timeout );

MyMessage my_message;
my_message.size = sizeof( MyMessage );
...

message( &my_message, timeout );

C没有继承。相反，您倾向于看到，

bool message( void *msg_body, size_t msg_body_size, unsigned timeout );

但这需要消息才能构建消息，可能涉及内存分配。这种模式可能很浪费。它仍然使用void *。

尝试使用工会或What的实施继承会凌乱，它们不会真正增加任何价值。您只需添加代码，因此即将成为一个有效的铸件。

因此，这是一个假阳性。这些是可以从衬里中获得的，因此您应该有一个处理这些处理的过程。

我希望一个过程可以标记某些预期的警告，但是可以使用以下内容进行沉默：

// Silence SonarQube false positive.
#define message( msg, timeout ) message( (void*)(msg), (timeout) )

message( &msgSPI, 0U )

我强烈建议您不要在每次使用Message> Message的呼叫站点上使用铸件。指针铸件是危险的，因为违抗类型系统。以上确实有演员表，这就是为什么它不是我首选的方法。但这只是一个位置，因此更容易管理。