#include <cstdio>
#include <ctime>
int populate_primes(int array[])
{
const int max = 1000000;
char numbers[max+1];
int count=1;
array[0]=2;
for(int i=max;i>0;i-=2)numbers[i]=0;
for(int i=max-1;i>0;i-=2)numbers[i]=1;
int i;
for(i=3;i*i<=max;i+=2){
if(numbers[i]){
for(int j=i*i;j<max+1;j+=i)numbers[j]=0; array[count++]=i;
}
}
int limit = max/2;
for(;i<limit;i++) if(numbers[i])array[count++]=i;
return count;
}
int factorize(int number,int array[])
{
int i=0,factor=1;
while(number>0){
if(number%array[i]==0){
factor++;
while(number%array[i]==0)number/=array[i];
}
i++;
}
printf("%d\n",factor);
return factor;
}
int main()
{
int primes[42000];
const int max = 1000000;
int factors[max+1];
clock_t start = clock();
int size = populate_primes(primes);
factorize(1000,primes);
printf("Execution time:\t%lf\n",(double)(clock()-start)/CLOCKS_PER_SEC);
return 0;
}
我想找到不。使用简单算法的因素。 populate primes部分运行正常,但factorize部分不执行并给出浮点异常错误。
请查看代码并告诉我的错误。
答案 0 :(得分:1)
在factorize方法中,您访问数组[0],因为i的初始值为0。 这个数组是由populate_primes填充的primes数组。但是填充素不会写入素数[0],因为count的初始值是1。
因此,第一个元素未初始化,您可能会得到一个div 0错误。
答案 1 :(得分:0)
尝试使用以下功能查看一些问题:
#define MAX_PRIMES 42000
int factorize(int number,int array[])
{
int i=0,factor=1;
for (i=0; number>0 && i< MAX_PRIMES; i++){
if (array[i] == 0 || array[i] == 1) {
printf("Error: array[%d] = %d\n", i, array[i]);
} else {
if(number%array[i]==0){
factor++;
while(number%array[i]==0 && number>0) {
printf("%d %d\n", number, array[i]);
number/=array[i];
}
}
}
}
printf("%d\n",factor);
return factor;
}
答案 2 :(得分:0)
您需要将填充的大小传递给factorize。
factorize(int number,int array [],int size);
问题是你的数组[]没有完全加载,它只加载到size变量。所以你可能想检查一下。
factorize中的逻辑也是错误的。您需要检查(数字&gt; 1)而不是(数字&gt; 0)。