为何return语句与print语句输出结果不同？C++代码问题咨询

嘿，我一眼就揪出问题的根源了——返回值类型不匹配导致的整数溢出！

问题原因 #

你在largestproduct()函数里定义的greatestProduct是unsigned long类型，但函数本身的返回类型却是int。当程序计算出的最大乘积超过int类型的取值上限（通常是2^31-1，也就是2147483647）时，将unsigned long转换为int会发生溢出截断，导致返回给main函数的值是错误的；而函数内部直接用cout输出greatestProduct时，是按照unsigned long的完整取值范围打印，所以结果正确。

举个直观的例子：如果greatestProduct的值是3000000000（超过int的最大值），转换成int后会变成一个负数或者乱码值，但直接输出unsigned long就能显示正确的3000000000。

修复方案 #

只需要把函数的返回类型和声明都改成unsigned long，让返回值的类型和变量类型保持一致即可：

#include <iostream>
#include <string>
using namespace std;
// 修改函数声明的返回类型
unsigned long largestproduct();
int main() {
    cout << largestproduct() << endl;
}
// 修改函数定义的返回类型
unsigned long largestproduct() {
    const int adjacentDigits = 13;
    int n = 0;
    unsigned long greatestProduct = 0;
    signed char num[] = "7316717653133062491922511967442657474235534919493496983520312774506326239578318016984801869478851843858615607891129494954595017379583319528532088055111254069874715852386305071569329096329522744304355766896648950445244523161731856403098711121722383113622298934233803081353362766142828064444866452387493035890729629049156044077239071381051585930796086670172427121883998797908792274921901699720888093776657273330010533678812202354218097512545405947522435258490771167055601360483958644670632441572215539753697817977846174064955149290862569321978468622482839722413756570560574902614079729686524145351004748216637048440319989000889524345065854122758866688116427171479924442928230863465674813919123162824586178664583591245665294765456828489128831426076900422421902267105562632111110937054421750694165896040807198403850962455444362981230987879927244284909188845801561660979191338754992005240636899125607176060588611646710940507754100225698315520005593572972571636269561882670428252483600823257530420752963450";
    while(n <= 1000-adjacentDigits) {
        unsigned long product = 1;
        for(int i = n; i < n+adjacentDigits;i++) {
            product *= (num[i] - '0');
        }
        if(greatestProduct < product) {
            greatestProduct = product;
        }
        n++;
    }
    cout << greatestProduct << endl;
    return greatestProduct;
}

额外说明 #

13个9相乘的结果是9^13 = 2541865828329，这个值在32位unsigned long的取值范围内（32位unsigned long最大为4294967295），所以用unsigned long完全足够容纳这个结果，不需要担心溢出问题。

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