«целочисленная константа слишком велика для типа long» при поиске наибольшего простого множителя - PullRequest
4 голосов
/ 25 января 2011

Я работаю над решением проекта Эйлера 3:

Description: The prime factors of 13195 are 5, 7, 13 and 29.
             What is the largest prime factor of the number 600851475143 ?

Это мой код для генерации ответа.Однако мне нужен целочисленный тип для хранения 600851475143.Когда я компилирую это в GCC на Mac, я получаю:

integer constant is too large for ‘long’ type". 

Я ожидаю, что long long может легко удерживать этот номер.Я также попытался сделать это без знака.Почему мой код не содержит этого небольшого числа и что я могу сделать, чтобы он работал?

#include <iostream>
#include <vector>

using namespace std;

static bool IsPrimeFactor(int numToTest,int testNum,vector<int> factors)
{
    if(numToTest%testNum==0) // is it a factor?
    {
        // see if it is a prime factor
        for(unsigned int i=0; i < factors.size(); i++)
        {
            if(testNum%factors[i]==0)  // see if this factor
            {                          //is divisble by one we have already

                return false;
            }
        }

        return true;
    }
    return false;
}

int main() {
    unsigned long long numToTest=600851475143;
    unsigned int testNum=2;  // 0 and 1 are assumed
    vector<int> factors;   

    while(testNum<numToTest)   // don't go higher than the max num
    {
        if(IsPrimeFactor(numToTest,testNum,factors)) // is it a factor?
        {
            factors.push_back(testNum); // got through prime factors 
        }                                   // and none divided it

        testNum++;
    }

    for(unsigned int i=0; i < factors.size(); i++)
    {
        cout << "factor " <<factors[i] << endl;
    }

    cout<<"Highest factor: " << factors[factors.size()-1]<<endl;

    return 0;
}

Ответы [ 2 ]

6 голосов
/ 25 января 2011

Проверьте этот вопрос. Вы должны указать свой литерал так:

600851475143LL
1 голос
/ 25 января 2011

Как сказал @ Space_C0wb0y, необходимо указать суффикс для литерала.

Кроме того, у вас возникнет проблема с функцией IsPrimeFactor - параметры являются целыми числами, но, как вы уже обнаружили, int или даже long недостаточно велики для хранения числа проходить многократно ...

...