欧拉问题3 - 我需要帮助这一小块C

时间:2011-04-18 14:18:57

标签: c

我已经开始解决http://projecteuler.net/的问题,但我似乎无法解决问题#3。我认为对你们大多数人来说都很容易。

#include <stdio.h>
#include <stdbool.h>  //This is to bring in the define of true
#include <math.h>  //This is to bring in the define of sqrt()

int LargestFactor (long number);
bool IsItPrime (int number);

int main (int argc, const char * argv[]) {
    long number;
    number = 600851475143;
    printf("The largest prime factor of %ld is %d.", number, LargestFactor(number));
}


int LargestFactor (long number) {
    int divider,i=1;
    bool foundIt=false;
    while (foundIt == false) {
        i++;
        if (number % i == 0 && IsItPrime(number/i)) {
            divider = number / i;
            foundIt=true;
        }
    }
    return divider;
}


bool IsItPrime (int number) {
    int i=1;
    bool isPrime=true;
    while (i<sqrt(number) && isPrime == true) {
        i++;
        if (number % i == 0) {
            isPrime=false;
        }
    }
    return isPrime;
}

我也得到了这个结果:

The largest prime factor of 600851475143 is -127237759.

6 个答案:

答案 0 :(得分:2)

问题在于长类型可以容纳的最大值,当您尝试存储的数字大于您可以容纳的最大类型时,可能会因为2的恭维系统。

请参阅此页:http://en.wikipedia.org/wiki/Limits.h

这将显示数据类型将保留的保证最小值。

因此,您应该以这种方式声明数字:long long int number = 600851475143LL

因此它足够大以容纳数字。

答案 1 :(得分:1)

当整数用32位表示时,数字600851475143大于INT_MAX(在大多数常见平台上就是这种情况)。您应该更改所有变量并将值从int返回到uint64_t(来自stdint.h)。此外,您应该将您的立即值更改为600851475143LL,以便编译器不会将其转换为int

答案 2 :(得分:1)

像其他人说的那样,你需要使用64位整数。 您还应该在%lld中使用printf。 理想情况下,通过将while语句更改为:

来避免使用sqrt和libm
while ((i*i)<number && isPrime == true) {

答案 3 :(得分:0)

您需要查看整体类型尺寸。您的原始数字约为6000亿,不适合32位整数。签名溢出所发生的情况在技术上是未定义的,但是大多数现代系统都会为您进行模运算。

在某些系统上,long为32位,64位(适用于此处)。假设您的平台上有一些C99,long long至少为64位。使用它。

还要注意中间结果。您在某些处理中使用了int idivider来自LargestFactor,您甚至将int传递给IsItPrime })。对计算中涉及的所有变量使用long long,否则有时会截断部分值。

而且,正如Sylvain Defresne建议的那样,将LL放在数字文字的末尾。至少这样清楚。

如果你的数字有很大的素因子,那么你的算法会遇到问题,因为它似乎需要永远,但那将是一个单独的问题。

答案 4 :(得分:0)

相反只是长... 您还可以使用long longdouble long。 这样你就能得到积极的回答。

答案 5 :(得分:0)

看起来我们在同一条船上......我刚刚发布了一个相关的问题Time Running out during execution

我已经花了很长时间作为数据类型,我的程序工作,除了它在三个结果后停止输出,因为它适用于ProjectEuler中的问题3中的另一个数字,即13195。

等待它解决!!