В программе игнорируется условие

@loadex · Регистрация: 14.05.2017

Author24 — интернет-сервис помощи студентам

Искал в интернете схожие задачи/коды, находил подобные, но с другим синтаксисом.Так как я только начинающий, мне очень затруднительно его понимание. Много раз проверял синтаксис функций и операторов в моем коде, ошибок сам не могу обнаружить.
Ниже прилагаю код:

C++

#include <iostream>
using namespace std;
int main()
{
    int farg = 0;
    int sarg = 0;
    unsigned int divisor = 1;
    int c = 0;
    unsigned int ii = 0;
    cout << "Enter First argument\n";
    cin >> farg;
    cout << "Enter Second argument\n";
    cin >> sarg;
    for (int i = farg; i <= sarg; i++)
    {
        ii = i;
        while (ii>divisor);
        {
            if (ii%divisor == 0)
            {
                c++;
            }
            divisor++;
            cout << "U chisla " << i << "\t" << c << "delitelei\n";
 
        }
    }
    cin.ignore();
    cin.get();
 
}

Исходя из условия в строке 19, переменная c должна увеличиваться только тогда, когда происходит деление без остатка, но она увеличивается каждый круг цикла независимо от условия.

@J5 · 15.05.2017, 11:14

loadex,

C++

1	while (ii>divisor);

Уберите точку с запятой.

@loadex · 15.05.2017, 11:33 **[ТС]**

Убрал точку с запятой, но условие все равно не выполняется.

@sourcerer · 15.05.2017, 11:34

loadex, что ж так сложно-то! Куча лишних переменных зачем-то... Лишний цикл... Можно же проще:

C++

#include <iostream>
 
int main()
{
    std::cout << "Enter an integer: ";
    int number;
    std::cin >> number;
 
    int counter = 0;
    for (int divisor = 1; number >= divisor; ++divisor)
        if (number % divisor == 0)
            ++counter;
 
    std::cout << "The number " << number << " has " << counter << " divisors.\n";
}

@loadex · 15.05.2017, 12:25 **[ТС]**

Виноват, не изложил точно задание. Так заморочился, потому что нужна программа, которая будет выводить количество делителей каждого числа из промежутка чисел, заданного пользователем.

Добавлено через 45 минут
Всем спасибо кто откликнулся, проблема была в том, что переменные divisor и c не обнулялись с каждым началом цикла, поэтому условие выполнялось, но не так, как было запланировано.
Оставлю код здесь, вдруг кто-то будет тупить как я и ему поможет.

C++

#include <iostream>
#include <ctime>
using namespace std;
int main()
{
    int farg = 0;
    int sarg = 0;
    int divisor = 1;
    int c = 0;
    int ii = 0;
    int line = 1;
    cout << "Enter First argument\n";
    cin >> farg;
    cout << "Enter Second argument\n";
    cin >> sarg;
    for (int i = farg; i <= sarg; i++)
    {
        divisor = 1;
        c = 0;
        clock_t start;
        start = clock();
        while (i>=divisor)
        {
            if (i%divisor == 0)
                c++;
            divisor++;
        }
        clock_t stop;
        stop = clock();
        cout << line << ".\t";
        line++;
        if (c > 2)
            cout << "U chisla " << i << "\t" << c << "delitelei\t";
        else
            cout << "Chislo " << i << "\t" << "prostoe\t";
        cout << (stop - start) /(double) CLOCKS_PER_SEC<<"\n";
    }
    cin.ignore();
    cin.get();
}

@sourcerer · 15.05.2017, 12:54

Сообщение от loadex

Виноват, не изложил точно задание. Так заморочился, потому что нужна программа, которая будет выводить количество делителей каждого числа из промежутка чисел, заданного пользователем.

Да всё равно много лишнего у Вас. Вот же:

C++

#include <iostream>
 
int main()
{
    std::cout << "Enter first argument: ";
    int firstArg;
    std::cin >> firstArg;
 
    std::cout << "Enter second argument: ";
    int secondArg;
    std::cin >> secondArg;
 
    for (int i = firstArg, counter = 0; i <= secondArg; ++i)
    {
        for (int divisor = 1; i >= divisor; ++divisor)
        {
            if (i % divisor == 0)
                ++counter;
        }
        std::cout << "The number " << i << " has " << counter << " divisors.\n";
        counter = 0;
    }
}

Добавлено через 3 минуты
Ну или так даже:

C++

#include <iostream>
 
int main()
{
    std::cout << "Enter first argument: ";
    int firstArg;
    std::cin >> firstArg;
 
    std::cout << "Enter second argument: ";
    int secondArg;
    std::cin >> secondArg;
 
    for (int i = firstArg, counter = 0; i <= secondArg; ++i, counter *= 0)
    {
        for (int divisor = 1; i >= divisor; ++divisor)
        {
            if (i % divisor == 0)
                ++counter;
        }
        std::cout << "The number " << i << " has " << counter << " divisors.\n";
    }
}

@loadex · 15.05.2017, 14:27 **[ТС]**

Спасибо за помощь! Еще учиться и учиться, даже основы не очень легко даются

@sourcerer · 15.05.2017, 14:41

Сообщение от loadex

Еще учиться и учиться,

Это нормальное состояние программиста. Прекратил учиться - умер как программист, личность, творец. Превратился в машину.

Сообщение от loadex

даже основы не очень легко даются

Это тоже нормально. Пройдитесь по ссылкам у меня в подписи (я имею ввиду вторую и третью ссылки).

@loadex · 15.05.2017, 19:17 **[ТС]**

Уже прошелся, и взял на вооружение!) Спасибо большое за такой объем информации для новичков, мне кажется это очень круто, и я уверен многие вам очень благодарны за эту работу , как и я.

@loadex 0 / 0 / 0 Регистрация: 14.05.2017 Сообщений: 7
	15.05.2017, 11:33 [ТС]	3
	Убрал точку с запятой, но условие все равно не выполняется. 0

@loadex 0 / 0 / 0 Регистрация: 14.05.2017 Сообщений: 7
	15.05.2017, 14:27 [ТС]	7
	Спасибо за помощь! Еще учиться и учиться, даже основы не очень легко даются 0

@sourcerer Модератор 5284 / 2371 / 342 Регистрация: 20.02.2013 Сообщений: 5,770 Записей в блоге: 20
	15.05.2017, 14:41	8
	Сообщение от loadex Еще учиться и учиться, Это нормальное состояние программиста. Прекратил учиться - умер как программист, личность, творец. Превратился в машину. Сообщение от loadex даже основы не очень легко даются Это тоже нормально. Пройдитесь по ссылкам у меня в подписи (я имею ввиду вторую и третью ссылки). 0

@loadex 0 / 0 / 0 Регистрация: 14.05.2017 Сообщений: 7
	15.05.2017, 19:17 [ТС]	9
	Уже прошелся, и взял на вооружение!) Спасибо большое за такой объем информации для новичков, мне кажется это очень круто, и я уверен многие вам очень благодарны за эту работу , как и я. 0