C++移位运算作业double转int触发C4244警告如何处理

问题原因分析 #

1. C4244警告的触发原因 #

你代码里使用的pow()是<math.h>提供的浮点运算函数，返回值为double类型，当你把double类型的结果直接赋值给int类型变量时，编译器会提示可能存在数据精度丢失的风险，这就是C4244警告的来源。该警告不能仅靠修改变量类型解决，本质是计算方式选择错误。

2. 右移结果的正确性 #

对于非负整数的右移运算，规则是向下取整的除以2的M次方，你测试用例里N=9、M=12时，9/4096=0.002197，取整后结果确实是0，这个输出结果是对的。但你用浮点运算计算的方式存在潜在误差：比如pow(2,10)在部分场景下可能返回1023.9999999999的近似值，转int后会得到1023而非正确的1024，会导致计算错误。

3. 原有实现逻辑的问题 #

你的需求是模拟>>=和<<=位运算符的功能，完全不需要使用浮点运算的pow函数，直接使用C++原生的位运算符即可，既不会有类型转换警告，计算效率和准确率都更高。另外要注意：如果N是负数，有符号整数的右移是算术右移（补符号位），和普通除法向0取整的规则不一样，用除法实现会得到错误结果，只有位运算符才能完全匹配需求。

正确修改方案 #

直接抛弃pow函数，用位运算符实现，修改后的代码如下：

#include <stdio.h>
#include <conio.h>

int main(){
    int N, M;
    int ShiftLeft, ShiftRight;

    printf("Enter integers N: ");
    scanf_s("%d", &N);

    printf("\nEnter integers M: ");
    scanf_s("%d", &M);

    // 校验移位位数合法性，避免未定义行为
    if (M < 0 || M >= sizeof(int)*8) {
        printf("\n非法的移位位数M");
        (void)_getch();
        return 1;
    }

    // 直接用位运算符计算
    ShiftLeft = N << M;
    printf("\nThe value of N <<= M is: %d", ShiftLeft);

    ShiftRight = N >> M;
    printf("\nThe value of N >>= M is: %d", ShiftRight);

    (void)_getch();
    return 0;
}

注：scanf_s和_getch是MSVC编译器特有的函数，如果要兼容其他编译器可以替换为标准的scanf和getchar。

特殊场景适配（作业要求限制不能用位运算符） #

如果要求必须用乘除逻辑实现，可以用整数运算代替浮点运算计算2的M次方，避免使用pow：

// 计算2的M次方的整数值
int pow2 = 1;
for(int i=0; i<M; i++) pow2 *= 2;
ShiftLeft = N * pow2;
ShiftRight = N / pow2;

这种方式也是纯整数运算，不会触发C4244警告。

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