如何解决C语言编译警告“warning: comparison between pointer and integer”

警告根源 #

你遇到的“comparison between pointer and integer”警告，核心原因是运算符优先级错误：!的优先级高于==，所以代码里的!q==(char*)NULL会被编译器解析为(!q) == (char*)NULL。!q会把指针q转换成布尔值（非空指针转1，空指针转0），这是整数类型，再和指针类型的NULL比较，就触发了类型不匹配的警告。

正确的判断写法 #

直接用以下两种方式判断realloc的返回值即可，完全没必要绕!和强制转换：

• 标准写法：if (q != NULL)
• 简洁写法：if (q)（C语言中空指针等价于布尔假，非空指针等价于布尔真）

如果一定要保留!的逻辑，必须加括号改变优先级：if (!(q == NULL))，但这种写法冗余，不推荐。

关于直接赋值p = realloc(p, i)的隐患 #

你替换成直接赋值后程序能运行，但这是不安全的：当realloc失败返回NULL时，原来的p指向的内存地址会被覆盖，导致内存泄漏——你再也无法通过指针释放这块内存。用临时指针q先接收返回值，判断成功后再赋值给p，才能在realloc失败时保留原指针，后续可以正常释放内存。

修正后的完整代码 #

#include <stdio.h>
#include <stdlib.h>

int main(void) {
    char* p = malloc(sizeof(char));
    // 先判断malloc是否成功
    if (!p) {
        printf("malloc error\n");
        return 1;
    }
    printf("%p\n", p);

    for (int i = 2; i < 100; i++) {
        char* q = realloc(p, i);
        printf("%p\n", q);
        // 正确的非空判断
        if (q != NULL) {
            p = q;
            printf("%p %dB\n", p, i);
        } else {
            printf("realloc error for %dB\n", i);
            // 失败时可选择退出或继续，注意后续要释放原p
            break;
        }
    }
    free(p);
    return 0;
}

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