C语言sscanf类型转换警告求解：消除指针转整数无强制转换警告

首先，咱们得先搞明白为啥会出现那个sscanf的警告——你完全用错sscanf了！ sscanf的第一个参数要求是const char*（字符串指针），用来从字符串里解析出数据，但你却把int类型的变量强制转成int传进去，这本质上是把整数当成了内存地址，既不符合语法，也完全没必要，因为你手里已经是int类型的变量了，根本不需要再解析一遍。

接下来咱们一步步修复代码，顺便把其他隐藏的逻辑bug也补上：

核心问题1：多余的sscanf调用 #

看你代码里的两处sscanf：

1. 在main函数里：

   sscanf((int)numbers[j],"%d",&x);
   

   numbers[j]本身就是int类型，直接赋值给x就行：x = numbers[j];，完全不需要sscanf。

2. 在checkPowerOfTwo函数里：

   sscanf((int)n,"%d",&x);
   

   传入的n已经是int了，直接x = n;就好，这行sscanf纯粹是画蛇添足。

核心问题2：字符串转数字的错误方式 #

你原来的代码里把token（char*类型的字符串指针）直接转成int：

numbers[i]=(int)token;

这是把字符串的内存地址存进了数组，而不是解析字符串里的数字！正确的做法是用atoi()（或者更安全的strtol()）把字符串转成整数：

numbers[i] = atoi(token);

核心问题3：数组填充的循环逻辑错误 #

原来的while(token!=NULL)里面套了for循环，会导致i直接从0跑到size，不管token是不是已经用完了，正确的做法是用一个计数器，在token不为空且没超过数组大小的时候循环：

int i = 0;
token = strtok(str," ");
while(token != NULL && i < size) {
    numbers[i++] = atoi(token);
    token = strtok(NULL," ");
}

额外优化：内存分配与输入处理 #

你给str分配内存的计算有点复杂，其实可以直接分配一个足够大的缓冲区（比如1024字节），或者更灵活的方式——不过如果要严格计算的话，每个数字最多占11位（包括负号和数字），加上空格，分配size*12字节就足够了。

修复后的完整代码 #

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

int checkPowerOfTwo(int x);

int main() {
    int exp, size, *numbers, sum = 0;
    char term, *str = NULL, *token;

    printf("Enter size of input:");
    if (scanf("%d%c", &size, &term) != 2 || term != '\n') {
        printf("Invalid Size\n");
        return 0;
    }
    if (size <= 0) {
        printf("Invalid size\n");
        return 0;
    }

    // 分配数组内存
    numbers = (int*)malloc(size * sizeof(int));
    // 分配字符串缓冲区：每个数字最多11位+空格，预留足够空间
    str = (char*)malloc(size * 12);
    if (numbers == NULL || str == NULL) {
        printf("Out of memory\n");
        return 0;
    }

    printf("Enter numbers:");
    // 读取整行输入
    fgets(str, size * 12, stdin);
    // 去掉fgets读取的换行符（如果有的话）
    str[strcspn(str, "\n")] = '\0';

    // 分割字符串并填充数组
    int i = 0;
    token = strtok(str, " ");
    while (token != NULL && i < size) {
        numbers[i++] = atoi(token);
        token = strtok(NULL, " ");
    }

    // 检查每个数字是否为2的幂
    for (int j = 0; j < size; j++) {
        exp = checkPowerOfTwo(numbers[j]);
        if (exp >= 0) {
            int x = numbers[j];
            printf("The number %d is a power of 2: %d=2^%d\n", x, x, exp);
            sum += exp;
        }
    }
    printf("Total exponent sum is %d\n", sum);

    // 释放内存
    free(numbers);
    free(str);
    return 0;
}

int checkPowerOfTwo(int n) {
    int x = n;
    int exponent = 0;

    // 处理0和负数的情况
    if (x <= 0) {
        return -1;
    }
    // 1是2^0
    if (x == 1) {
        return 0;
    }

    while (x != 1) {
        if (x % 2 != 0) {
            return -1;
        }
        x /= 2;
        exponent++;
    }
    return exponent;
}

关键修改总结 #

• 移除了所有不必要的sscanf调用，彻底消除警告
• 用atoi()正确将分割后的字符串转成整数存入数组
• 修复了数组填充的循环逻辑，避免越界或未填充完全的问题
• 优化了字符串缓冲区的内存分配，同时处理了fgets读取的换行符
• 完善了checkPowerOfTwo函数对0和负数的判断逻辑

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