Основные факторы поколения C ++ - PullRequest
2 голосов
/ 17 октября 2011

Вот что у меня есть:

//project Eular Problem 4: Prime Factors
#include<iostream>
#include<cmath>
typedef unsigned long long int uint_64;
using namespace std;

void storeFactors(uint_64 factors[], uint_64 num)
{
    for(uint_64 i=0;i*i<num;i++
       factors[i]=1;       //assign 1 to all the values

    for(uint_64 j=2; j*j<num;j++){
       if(num%j!=0)
          factors[j]=0;       //assign 0 to non-factors
    } 
}  

//Sieve of Eratosthenes to generate primes
void gen_primes(uint_64 arr[],uint_64 firstElement, uint_64 lastElement, uint_64 size)
{
    for(uint_64 i=0;i<size;i++)   //assigning 1 to all the values
       arr[i]=1;

    for(uint_64 i=2;i*i<=lastElement;i++){   //loop until the square-root of n
       if(arr[i])
          for(uint_64 j=i;j*i<=lastElement;j++)      //eliminate multiples by assigning them 0
            arr[j*i]=0;
       if(firstElement==1)
          arr[firstElement]=0;
    }
}

void arrayComp(uint_64 factors[],uint_64 primeArray[], uint_64 size)
{
    for(uint_64 i=2; i<=size; i++){
        if(factors[i] && primeArray[i]){
                cout<<i<<endl;
        }
    }
}

void processFactors(uint_64 num)
{
    uint_64 size = sqrt(num);
    uint_64 *factors = new uint_64[size];
    uint_64 *primeArray = new uint_64[size];

    storeFactors(factors, num);
    gen_primes(primeArray, 2, num, size);
    arrayComp(factors, primeArray,size);

    delete [] factors;
    delete [] primeArray;
}

int main()
{
    uint_64 number;
    cout<<"Enter a number: "<<endl;
    cin>>number;
    cout<<"The prime factors of "<<number<<" are: "<<endl;
    processFactors(number);

    return 0;
}

Я пытался использовать решетчатый подход для расчета факторов. Всем не факторам присваивается 0. ArrayComp отображает числа, если они оба являются факторами ввода, а также простыми числами.

Проблема, с которой я столкнулся, заключается в том, что вывод не завершен, и программы сталкиваются с ошибкой сегментации. Например, коэффициенты для 10 равны 5 и 2, но он показывает тот же ответ для 100.

РЕДАКТИРОВАТЬ: Еще одна вещь, в которой я не слишком уверен, это размер массива. Эта программа показывает 3 как главный фактор 21, а не 7, но если я увеличу размер на 1, она показывает 3 и 5 (неверно)

Ответы [ 2 ]

7 голосов
/ 17 октября 2011

Простые множители 100 такие же, как и простые множители 10, отличается только кратность.

Похоже, что ваше представление не имеет никакого способа хранить повторяющиеся факторы.

1 голос
/ 18 октября 2011
uint_64 size = sqrt(num);
uint_64 *factors = new uint_64[size];
uint_64 *primeArray = new uint_64[size];

неправильный резервный размер.

исх.)

число = 10

размер = 3

коэффициент 10 = 2,5

    if(factors[i] && primeArray[i]){
            cout<<i<<endl;
    }

факторов [5]?тогда размер массива равен 3 (0,1,2)

...