我已经开始解决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.
答案 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语句更改为:
while ((i*i)<number && isPrime == true) {
答案 3 :(得分:0)
您需要查看整体类型尺寸。您的原始数字约为6000亿,不适合32位整数。签名溢出所发生的情况在技术上是未定义的,但是大多数现代系统都会为您进行模运算。
在某些系统上,long
为32位,64位(适用于此处)。假设您的平台上有一些C99,long long
至少为64位。使用它。
还要注意中间结果。您在某些处理中使用了int
i
和divider
来自LargestFactor
,您甚至将int
传递给IsItPrime
})。对计算中涉及的所有变量使用long long
,否则有时会截断部分值。
而且,正如Sylvain Defresne建议的那样,将LL
放在数字文字的末尾。至少这样清楚。
如果你的数字有很大的素因子,那么你的算法会遇到问题,因为它似乎需要永远,但那将是一个单独的问题。
答案 4 :(得分:0)
相反只是长...
您还可以使用long long
和double long
。
这样你就能得到积极的回答。
答案 5 :(得分:0)
看起来我们在同一条船上......我刚刚发布了一个相关的问题Time Running out during execution
我已经花了很长时间作为数据类型,我的程序工作,除了它在三个结果后停止输出,因为它适用于ProjectEuler中的问题3中的另一个数字,即13195。
等待它解决!!