在 C 中：如何设置指向数组结构成员的指针？

Question

我应该如何编写代码来举例说明恰好是结构成员的数组的特定数组索引？以下代码给我带来了问题。

// main.c

void clean_buffers(void); // prototype

struct DEV_STATUS {
    unsigned char ADDR;
    unsigned char DEV_HAS_DATA;
    unsigned char ETH_HAS_DATA;
    unsigned char DATA[20];
};

struct DEV_STATUS g_cmdQueue[60] = {0};

void main(void) {

    clean_buffers();

    while (1) {
        ;// MCU tasks        
    }
}

void clean_buffers(void) {
    unsigned char theCount = 0;
    byte queIdx;
    for (queIdx = 0; queIdx < 59; queIdx++) {
        struct DEV_STATUS *p_struct;
        unsigned char *p_data;
        p_struct = &g_cmdQueue[queIdx];
        p_data = &p_struct->DATA;
        p_struct->ADDR = 0;
        p_struct->DEV_HAS_DATA = 0;
        p_struct->ETH_HAS_DATA = 0;
        theCount = 0;
        while(*(p_data+theCount) != 0) {
            *(p_data+(theCount++)) = 0;
        }
    }    
} // EOF main.c

我在以下行收到编译器错误“struct/union member expected”：

p_data = &p_struct->DATA;

如果要访问例如结构成员 DATA[3] 的特定值，我应该如何编写指针？我很困惑，我认为 p_data = &p_struct->DATA;已定义，我应该可以使用 *(pdata+3) 来获取它，但我想我错过了一些东西。

Answer 1

您确定要编译与您在此处发布的代码相同的代码吗？

如果你的编译器在这一行抱怨

p_data = &p_struct->DATA;

如果出现“预期结构/联合成员”消息，您的编译器可能已损坏。

请注意，&p_struct->DATA 在 C 中是一个完全有效的表达式。这个表达式本身绝对没有问题。

这里的问题是这不是你需要的。 &p_struct->DATA 返回指向整个数组 'DATA' 的指针，即 unsigned char (*)[20] 类型的指针。您正在尝试将此值分配给unsigned char * 类型的指针。这在 C 中是非法的，因为类型完全不同，但传统上 C 编译器仅以“类型不匹配”警告响应它并执行隐式转换（顺便说一句，这意味着您的原始代码，尽管“脏”，应该仍然按预期工作）。

即使某些编译器决定将此不匹配标记为错误（这很好），它仍然不应该抱怨任何“预期的结构/联合成员”类型的问题。这里不存在这样的问题。

附：正如其他人已经说过的，您真正需要的是p_data = &p_struct->DATA[0]，但这仍然不能解释您的编译器的奇怪行为。难道'DATA'是在'clean_buffers'定义之前某处定义的宏？

于 2009 年 10 月 19 日添加： Nate，在您的代码中，您使用索引 theCount 访问您的数组。由于您无论如何都在使用索引访问，因此甚至没有理由创建您尝试创建的指针。该代码无需任何额外指针即可完美运行，只需直接访问 DATA 字段

theCount = 0;
while (p_struct->DATA[theCount] != 0) {
  p_struct->DATA[theCount++] = 0;

（我可能会在这里使用for 循环）。

如果你真的坚持创建这个指针并且仍然使用索引访问，代码应该如下所示（其他人已经不止一次建议过）

p_data = p_struct->DATA; /* or &p_struct->DATA[0] */
...
theCount = 0;
while (p_data[theCount] != 0) {
  p_data[theCount++] = 0;

此外，您可以选择更“异国情调”的变体:)

unsigned char (*p_data)[20]; /* <- note: declared differently */
...
p_data = &p_struct->DATA; /* <- note: your original version */
...
theCount = 0;
while ((*p_data)[theCount] != 0) {
  (*p_data)[theCount++] = 0;

但是，返回到 unsigned char *p_data 版本，由于您创建了该指针，因此使用“滑动指针”技术而不是使用索引访问可能更有意义

unsigned char *p_data;
...
p_data = p_struct->DATA; /* or &p_struct->DATA[0] */
...
while (*p_data != 0) {
  *p_data++ = 0;

与往常一样，这完全取决于个人喜好。当然，在您摆脱宏的干扰之前，这些都不会起作用。

Answer 2

去掉 & in p_data = &p_struct->DATA;

p_struct 已经是一个指针。然后，使用 p_data[] 访问您的数组。

Answer 3

你应该写的是两件事之一：

p_data = p_struct->DATA; // DATA is the address of the first element.

或

p_data = &p_struct->DATA[0]; // taking the address of the first element.

Answer 4

只需删除开头的 &，如下所示：

p_data = p_struct->DATA;

这是数组的特殊语法（记住它们总是作为引用传递），它相当于：

p_data = &p_struct->DATA[0];

是的，现在你可以使用 *(pdata+3)

希望对你有帮助。

Answer 5

哎呀！谢谢AndreyT

结构 DEV_STATUS *p_struct; 无符号字符 *p_data; p_struct = &g_cmdQueue[queIdx]; p_data = &p_struct->DATA;

p_struct 是指向struct DEV_STATUS 的指针。
&p_struct 是指向struct DEV_STATUS 的指针的地址（或指向struct DEV_STATUS 的指针的指针）。

您可能想将该行更改为

    p_data = p_struct->DATA;

哦...您的 clean_buffers() 函数不会“清理”元素 g_cmdQueue[59]。

而且，因为它是一个全局对象，所以数组 g_cmdQueue 甚至在 main() 的第一条语句执行之前就被初始化为全零。