c - C语言结构中 "dereferencing ` void *' pointer"警告的解决方案？

Question

我试图创建一个伪超结构来打印结构数组。我的基本
结构如下。

/* Type 10 Count */
typedef struct _T10CNT
{
    int _cnt[20];
} T10CNT;

...

/* Type 20 Count */
typedef struct _T20CNT
{
    long _cnt[20];
} T20CNT;
...

我创建了以下结构来打印上述结构的数组。在编译下面的代码片段时，我遇到了取消引用 void 指针错误。

typedef struct _CMNCNT
{
    long  _cnt[3];
} CMNCNT;

static int printCommonStatistics(void *cmncntin, int cmncnt_nelem, int cmncnt_elmsize)
{
    int ii;
    for(ii=0; ii<cmncnt_nelem; ii++)
    {
        CMNCNT *cmncnt = (CMNCNT *)&cmncntin[ii*cmncnt_elmsize];
        fprintf(stout,"STATISTICS_INP: %d\n",cmncnt->_cnt[0]);
        fprintf(stout,"STATISTICS_OUT: %d\n",cmncnt->_cnt[1]); 
        fprintf(stout,"STATISTICS_ERR: %d\n",cmncnt->_cnt[2]);
    }
    return SUCCESS;
}

T10CNT struct_array[10];
...
printCommonStatistics(struct_array, NELEM(struct_array), sizeof(struct_array[0]);
...

我的意图是有一个通用的功能来打印所有的数组。请告诉我正确的使用方法。

提前感谢帮助。

编辑:参数名称从 cmncnt 改为 cmncnt。抱歉，这是拼写错误。

谢谢，
马修·李居

Answer 1

我认为您的设计将会失败，但我也不相信我看到的其他答案完全可以解决更深层的原因。

您似乎正在尝试使用 C 来处理泛型类型，这总是很麻烦。如果你小心点，你可以做到，但这并不容易，在这种情况下，我怀疑这是否值得。

更深层次的原因 : 让我们假设我们解决了单纯的句法(或仅是句法)问题。您的代码显示 T10CNT 包含 20 int T20CNT 包含 20 long .在现代 64 位机器上 - 除了在 Win64 下 - sizeof(long) != sizeof(int) .因此，您的打印函数中的代码应该区分取消引用 int数组和 long数组。在 C++ 中，有一条规则是您不应该尝试多态地处理数组，这就是原因。 CMNCNT 类型包含 3 long值(value)观；尽管数组的基本类型与 T20CNT 匹配，但在数量上与 T10CNT 和 T20CNT 结构不同。

风格推荐:我强烈建议避免在名称中使用前导下划线。通常，以下划线开头的名称保留给实现使用，并用作宏。宏不尊重范围；如果实现定义了一个宏 _cnt 它会破坏你的代码。保留哪些名称存在细微差别。我不打算讨论这些细微差别。认为“保留以下划线开头的名称”要简单得多，它会让你摆脱麻烦。

风格建议:您的打印功能无条件返回成功。这是不明智的；您的函数不应返回任何内容，以便调用者不必测试成功或失败(因为它永远不会失败)。仔细观察函数返回状态的编码人员将始终测试返回状态，并具有错误处理代码。该代码永远不会被执行，所以它已经死了，但是任何人(或编译器)都很难确定这一点。

Surface Fix:暂时，我们可以假设您可以处理 int和 long作为同义词；但是，您必须改掉认为它们是同义词的习惯。 void *参数是说“这个函数需要一个不确定类型的指针”的正确方式。但是，在函数内部，您需要从 void * 转换在进行索引之前将其转换为特定类型。

typedef struct _CMNCNT
{
    long    count[3];
} CMNCNT;

static void printCommonStatistics(const void *data, size_t nelem, size_t elemsize)
{
    int i;
    for (i = 0; i < nelem; i++)
    {
        const CMNCNT *cmncnt = (const CMNCNT *)((const char *)data + (i * elemsize));
        fprintf(stdout,"STATISTICS_INP: %ld\n", cmncnt->count[0]);
        fprintf(stdout,"STATISTICS_OUT: %ld\n", cmncnt->count[1]); 
        fprintf(stdout,"STATISTICS_ERR: %ld\n", cmncnt->count[2]);
    }
}

(我也喜欢名为 stout 的文件流的想法。建议:在真实源代码上使用 cut'n'paste ——这样更安全!我通常使用“ sed 's/^/ /' file.c”来准备 cut'n 的代码'粘贴到 SO 答案中。)

那条类型转换线是做什么的？我很高兴你问...

第一个操作是转换 const void *变成 const char * ;这允许您对地址执行字节大小的操作。在标准 C 之前的日子里，char *用于代替 void *作为通用寻址机制。

下一个操作添加正确的字节数以到达 i 的开头。大小为 elemsize 的对象数组的第一个元素.

然后第二个 Actor 告诉编译器“相信我 - 我知道我在做什么”和“将此地址视为 CMNCNT 结构的地址”。

从那里开始，代码就足够简单了。请注意，由于 CMNCNT 结构包含 long值，我使用了 %ld说实话给 fprintf() .

由于您不打算修改此函数中的数据，因此使用 const 是一个不错的主意。像我一样的预选赛。

请注意，如果您要忠实于 sizeof(long) != sizeof(int) ，那么您需要两个单独的代码块(我建议使用单独的函数)来处理“ int 的数组” ' 和 ' long 的数组' 结构类型。