2

Сгенерировать рандомное число от 100000 до 1000000. Есть наработка, но проблема в том, что число постоянно генерируется от 100000 до 170000, а 345956 или 742322 никогда не бывает. Подскажите, что не так я реализовал.

srand(time(NULL));
const int MIN = 100000;
const int MAX = 1000000;
int desiredNumber = rand() % 9 + 1;
int generateNumber = rand() % (MAX - MIN) + MIN;
int generatedNumber = 0;
int numberCoincidences = 0;

generatedNumber = generateNumber;
while (generateNumber)
{
    int temp = generateNumber % 10;
    if (temp == desiredNumber)
    {
        numberCoincidences++;
    }
    generateNumber = (generateNumber - temp) / 10;
}
if (numberCoincidences)
{
    cout << "Numeric " << desiredNumber << " in number " << generatedNumber << " is found " << numberCoincidences << "  times\n";
}
else
{
    cout << "Numeric " << desiredNumber << " in number " << generatedNumber << " does not occur\n";
}
1
  • Намек: задумайтесь о RAND_MAX - максимальном значении, генерируемом rand() Чему оно равно в вашей системе? (что выводит cout << RAND_MAX;?)
    – Harry
    8 дек 2018 в 18:10

2 ответа 2

2

Не стоит использовать rand() - обычно у него "размах" не слишком велик, посмотрите, чему равно RAND_MAX в вашей системе. В результате, если это значение меньше требуемого диапазона - ваш метод просто не будет работать.

Раз уж вы работаете на C++, то генерируйте эти значения с учетом возможностей <random> (см. тут):

using namespace std;

int main()
{
    random_device rd;
    mt19937 gen(rd());
    uniform_int_distribution<> dis(100000, 1000000);

    for (int n = 0; n < 50; ++n)
        cout << dis(gen) << '\n';
}
1

RAND_MAX обычно равно 32767. Для генерации чисел в большем диапазоне можете использовать следующую конструкцию:

    srand(time(NULL));
    const int MIN = 100000;
    const int MAX = 1000000;
    int desiredNumber = rand() % 9 + 1;
    int generateNumber = (rand() % 9)       + 
                         10*(rand() % 9)    + 
                         100*(rand() % 9)   + 
                         1000*(rand() % 9)  + 
                         10000*(rand() % 9) + 
                         100000*(rand() % 9 + 1);
    int generatedNumber = 0;
    int numberCoincidences = 0;

    generatedNumber = generateNumber;
    while (generateNumber)
    {
        int temp = generateNumber % 10;
        if (temp == desiredNumber)
        {
            numberCoincidences++;
        }
        generateNumber = (generateNumber - temp) / 10;
    }
    if (numberCoincidences)
    {
        cout << "Numeric " << desiredNumber << " in number " << generatedNumber << " is found " << numberCoincidences << "  times\n";
    }
    else
    {
        cout << "Numeric " << desiredNumber << " in number " << generatedNumber << " does not occur\n";
    }
4
  • Нули в случайных числах категорически запрещены? :)
    – Harry
    8 дек 2018 в 18:45
  • @Harry Спасибо, отредактировал
    – amiranys
    8 дек 2018 в 18:49
  • "RAND_MAX обычно равно 32767" - это откуда такое "обычно" взялось? 8 дек 2018 в 20:34
  • @amiranys а теперь, видимо, запрещены девятки. Почему не по модулю 10? :) 6 окт 2020 в 20:11

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.