C语言：主函数调用自定义宏返回异常错误求助

嘿，我来帮你搞定这个宏的问题！这种位操作相关的宏很容易踩运算符优先级或者参数展开的坑，咱们一步步拆解解决。

首先，先明确核心需求：统计数组中满足「存在至少一对左右对称位均为1」的数字数量（比如你举的10110111，二进制第7位和第0位都是1，符合条件）。如果你的需求是「所有对称位要么都为1要么都为0」，后面我也会给出调整方案。

常见的宏错误原因 #

先排查你可能踩的坑：

• 参数未加括号：比如宏里写num >> 31 - bit，会因为减法优先级高于移位，导致计算顺序出错，必须给运算表达式加括号。
• 宏未用语句表达式包裹：如果宏直接写循环逻辑，没有用({ ... })（GNU C扩展），在主函数里直接赋值会报错，因为宏展开后不是一个合法的表达式。
• 循环范围错误：比如检查32位数字时，只需要遍历前16位（后半位是前半位的对称位），遍历全部位会重复检查。
• 逻辑判断错误：比如误把「存在一对」写成「所有对都必须是1」，或者位取值时没和1做按位与，导致拿到的不是单个位的值。

正确的宏实现（针对「存在至少一对对称位均为1」） #

这里给你一个能直接用的宏，兼容32位无符号整数，同时避免了上述坑：

#define COUNT_SYMMETRIC_ONES(arr, size) \
({ \
    int count = 0; \
    const int half_bits = 16; /* 32位数字的前半部分位数 */ \
    for (int i = 0; i < (size); ++i) { \
        unsigned int num = (arr)[i]; \
        int is_valid = 0; /* 默认不符合，找到符合的位对就设为1 */ \
        for (int b = 0; b < half_bits; ++b) { \
            int left_pos = 31 - b; /* 对称的高位位置 */ \
            unsigned int left_bit = (num >> left_pos) & 1; \
            unsigned int right_bit = (num >> b) & 1; \
            if (left_bit == 1 && right_bit == 1) { \
                is_valid = 1; \
                break; /* 找到一对就停止检查，提升效率 */ \
            } \
        } \
        if (is_valid) { \
            count++; \
        } \
    } \
    count; \
})

主函数调用示例 #

把宏放在头文件或者主函数上方，调用方式如下：

#include <stdio.h>

/* 上面的宏定义 */

int main() {
    // 测试数组：0b10110111（符合）、0b110011（符合）、0b1010101（符合）、0b00000000（不符合）
    unsigned int nums[] = {0b10110111, 0b110011, 0b1010101, 0};
    int arr_size = sizeof(nums) / sizeof(nums[0]);
    
    int result = COUNT_SYMMETRIC_ONES(nums, arr_size);
    printf("符合条件的数字数量：%d\n", result); // 输出应该是3
    
    return 0;
}

如果你的需求是「所有对称位要么都为1要么都为0」 #

只需要修改宏内部的判断逻辑：

#define COUNT_SYMMETRIC_ALL_VALID(arr, size) \
({ \
    int count = 0; \
    const int half_bits = 16; \
    for (int i = 0; i < (size); ++i) { \
        unsigned int num = (arr)[i]; \
        int is_valid = 1; /* 默认符合，找到不符合的位对就设为0 */ \
        for (int b = 0; b < half_bits; ++b) { \
            int left_pos = 31 - b; \
            unsigned int left_bit = (num >> left_pos) & 1; \
            unsigned int right_bit = (num >> b) & 1; \
            // 只要有一对位一个是1一个是0，就不符合
            if ((left_bit ^ right_bit) == 1) { \
                is_valid = 0; \
                break; \
            } \
        } \
        if (is_valid) { \
            count++; \
        } \
    } \
    count; \
})

关键注意事项 #

1. 如果你需要支持8位、16位数字，只需要修改half_bits和left_pos的计算（比如8位数字的half_bits是4，left_pos是7 - b）。
2. 宏里的所有参数和运算表达式都加了括号，避免运算符优先级导致的错误。
3. 使用GNU C的语句表达式({ ... })让宏可以返回值，这在大多数编译器（GCC、Clang）上都支持。

内容的提问来源于stack exchange，提问作者Danny Nitman