EvgeniyGlazirin / oop

0 stars 0 forks source link

Замечания по генератору простых чисел #6

Closed alexey-malov closed 6 years ago

alexey-malov commented 6 years ago
void ExcludeMultipliesOf(int i, std::vector<bool> &isPrime)
{
    int step = 1;
    if (step == 1) // <-- Условие всегда равно true
    {
        for (int j = i * i; j < isPrime.size(); j += i)
        {
            isPrime[j] = false;
            step += 1; // <-- изменение step уже ни на что не влияет
        }
    }
    else // <-- этот код никогда не выполнится
    {
        for (int j = i * i; j < isPrime.size(); j += 2 * i)
        {
            isPrime[j] = false;
        }
    }
}
alexey-malov commented 6 years ago
void PrintPrimeNumbers(std::ostream& output, const std::set<int>& primeNumbers)
{
    std::copy(primeNumbers.begin(), primeNumbers.end(), std::ostream_iterator<int>(output, " "));
    output << std::endl;
}