6

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

// 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) 来获得它，但我想我错过了一些东西。

4

5 回答 5

8

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

如果您的编译器在此行抱怨

p_data = &p_struct->DATA;

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

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

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

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

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

2009 年 10 月 19 日添加： Nate，在您的代码中，您可以使用 index 访问您的数组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;

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

于 2009-10-16T20:30:05.137 回答
3

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

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

于 2009-10-16T19:53:08.353 回答
2

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

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.
于 2009-10-16T19:55:38.347 回答
1

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

p_data = p_struct->DATA;

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

p_data = &p_struct->DATA[0];

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

希望能帮助到你。

于 2009-10-16T19:56:10.137 回答
-1

哎呀！谢谢安德烈

结构 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（或指向 a 的指针struct DEV_STATUS）的指针的地址。

您可能想将该行更改为

    p_data = p_struct->DATA;

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

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

于 2009-10-16T20:01:14.467 回答