Problem z zadaniem "Liczby Pierwsze" - Polski SPOJ

Question

Witam, mam problem z zadaniem "Liczby pierwsze" z serwisu Polski SPOJ. Chciałem zrobić aby program sprawdzał także liczby ujemne, więc zrobiłem funkcję zwracającą liczbę bezwzględna, lecz dla każdej ujemnej liczby zwraca iż liczba należy do liczb pierwszych...

#include <iostream>

int number;
bool isPrime;

int wartoscBezwzgledna(int number);

int main()
{
    for (int i = 0; i < 100000; i++){
        std::cin>>number;
        wartoscBezwzgledna(number);
        isPrime = true;
        for (int i = 2; i < number; i++)
        {
            if (number%i == 0)
            {
                isPrime = false;
            }
        }
        if (isPrime == true)
        {
            std::cout<<"NIE"<<std::endl;
        }else
        {
            std::cout<<"TAK"<<std::endl;
        }
    }
    return 0;
}

int wartoscBezwzgledna(int number)
{
    if (number >= 0)
    {
        return number;
    }else
    {
        return -number;
    }
}

Z góry dziękuję za wszelką pomoc :)

Comments

12. linijka nic nie robi.

---

Jesteś pewien?

---

Liczy co prawda wartość bezwzględną, ale nic z tym nie robi.

---

Chodzi Tobie o to, że funkcja nie otrzymuje oryginalnej zmiennej?

---

Funkcja liczy wszystko dobrze i zwraca poprawny wynik, tylko ten wynik przepada, bo nic się z nim nie dzieje. W odpowiedzi TOM_CPP masz przykład jak to powinno być (pomijając, że on użył funkcji bibliotecznej a ty własnej).

Answer 1

Nie lepiej dodać:

number = abs(number);

Pomyśl także o przyśpieszeniu swojego algorytmu - nie musisz sprawdzać w pętli całego zakresu od 2 do number.