遍历c中的结构

Question

我正在编写一个与此代码类似的单元测试，并且我正在尝试在设置值时测试我的值，以便我知道发生了什么。当我运行以下代码时，我不明白为什么 ptr 值没有设置为 1。相反，当我运行它时，它会输出 10、64、0、0。

任何解释或建议将不胜感激！

#include <stdio.h>
#include <stdbool.h>

typedef struct
{
    bool bOne;
    bool bTwo;
    bool bThree;
    bool bFour;
} items;

int main()
{
    items item;
    item.bOne = 0;
    bool *ptr = &(item.bOne);

    for(int i = 0; i < sizeof(items)/sizeof(bool); i++)
    {
        *ptr = 1;
        *ptr++;
        printf("ptr value = %d\n", *ptr);
    }
    return 0;
}

Answer 1

在*ptr++中，++的优先级高于*，所以这个post递增指针，读取并丢弃原来指向的值。现在指针已经递增，您正在读取printf 中的未初始化内存。如果您的意图是增加所指向的值，请尝试：

(*ptr)++;

或者

ptr[0]++;

编辑

嗯，由于您的循环绑定基于适合结构大小的bools 的数量，因此您的意图可能是增加指针。在这种情况下，您不需要同时取消引用它，并且您不应该期望在printf 中获得任何有意义的东西。此外，正如所指出的，由于结构不是数组，您将徘徊在未定义的行为领域，因为编译器可能会决定添加填充：

来自 C99 §6.7.2.1：

12 结构或联合对象的每个非位域成员都以适合其类型的实现定义方式对齐。

13 在结构对象中，非位域成员和位域所在的单元的地址按声明顺序递增。一个指向结构对象的指针，经过适当的转换，指向它的初始成员（或者如果该成员是位域，则指向它所在的单元），反之亦然。结构对象中可能有未命名的填充，但不是在其开头。

Answer 2

你有两个主要问题。

首先，您有一个逻辑错误，因为您在打印出它指向的值之前提前了指针；而不是打印您刚刚设置的值，而是打印下一个未初始化成员的值。您需要颠倒提前和打印语句的顺序，如下所示：

printf("ptr value = %d\n", *ptr);
ptr++; // note no * operator

或者只是将操作组合在一个表达式中：

printf("ptr value = %d\n", *ptr++);

这假设您可以使用 %d 打印 bool 值 - 我不确定那个临时（可能必须明确地将参数转换为 (int)）。

但是你有一个更大的问题——不能保证你可以用这样的指针遍历 struct 类型的成员。即使所有成员都是同一类型，也不能保证成员之间不会有填充（这取决于bool 类型的大小和您平台的对齐要求）。

您的代码可能按预期工作，或者它可能导致您的struct 实例中的数据损坏，或任何数量的其他问题。更安全的做法是使用指向这些成员的单独指针数组，因为它是保证您可以使用指针遍历数组：

bool *members[] = { &item.bOne, &item.bTwo, &item.bThree, &item.bFour, NULL };

bool *ptr = members[0];
while ( ptr )
{
  *ptr++ = 1;
}

如果你仍然想要输出，你会写

*ptr = 1;
printf( "ptr value = %d\n", *ptr++ );

您可以发挥创意，在打印语句中结合赋值和指针前进：

printf( "ptr value = %d\n", (*ptr++ = 1) );

但这可能会让你受到打击。