0 / 0 / 0
Регистрация: 19.12.2014
Сообщений: 12
1

Заполнить массив случайными числами в диапазоне

23.12.2014, 22:38. Показов 25199. Ответов 16
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Помогите, пожалуйста, найти ошибку

Нужно заполнить двумерный массив случайными простыми числами, которые могут повторяться.
Почему-то большая часть выпадающих цифр - нули! Хотя число рандомное я задала (вроде бы) от двух до 50. Функция проверки на простоту вообще игнорируется! В чем проблема?

C++ (Qt)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
#include <iostream>
#include <cstdlib>
#include <ctime> 
#include <locale>
 
using namespace std;
int Prime(int b);
int main()
{
    setlocale(LC_ALL, "Russian");
    const int  n = 4; const int m = 4;
    int Sluch[m][n] = {}; int i; int j;
    ///заполним-ка массив
    srand(time(NULL)); //srand()  получает в виде параметра текущее системное время
    for ( i = 0; i < n; i++)
    for ( j = 0; j < m; j++)
        {
            int ch = 0; 
                ch = 2 + rand() % 50; // запись случайного числа, которое вернет rand() (от 2 до 50)
            if (Prime(ch) == 1) Sluch[i][j] = ch; //если число простое, записываем его в элемент массива
        }
 
    ///////////вывод
    for ( i = 0; i < n; i++)
    {
        for ( j = 0; j < m; j++)
        {
            cout << Sluch[i][j] << " ";
        }
        cout << endl;
    }
            
} 
 
 
int Prime(int b)// откуда тут вылезают нули???
{
    if (b == 0 || b == 1) return 0;// не простое
    else
    {
        float end = sqrt((float)b); //если натуральное число n>1 не делится ни на одно 
                                    //простое число, не превосходящее sqrt{n}, то оно простое
    for (int im = 2; im <= end; ++im)
    {
        if (b%im == 0) return 0; // не простое
        else return 1;// простое
    }
    }
}
Миниатюры
Заполнить массив случайными числами в диапазоне  
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
23.12.2014, 22:38
Ответы с готовыми решениями:

Заполнить массив случайными числами в заданном диапазоне
Сгенерировать числа от -5 до 4

Как заполнить одномерный массив случайными существенными числами в диапазоне?
ПОМОГИТЕ!!!

Заполнить одномерный динамический массив случайными числами в заданном диапазоне
как заполнить одномерный динамический массив случайными числами от -100 до +100 вот вариант от 1...

Заполнить массив случайными числами в диапазоне (6, 32), вывести в строку, упорядочить вставкой
Плиз очень сильно надо не знаю

16
Заблокирован
23.12.2014, 22:41 2
Annnie, от 2 до 50 будет так:
C++
1
ch = 2 + rand() % 49;
0
0 / 0 / 0
Регистрация: 19.12.2014
Сообщений: 12
23.12.2014, 22:46  [ТС] 3
да я уже поняла мне это не особо важно, я вот не пойму, почему функция проверки не срабатывает
0
Заблокирован
23.12.2014, 22:51 4
Annnie, я бы сделал по-другому. Заполните массив простыми числами. И получайте случайный индекс, который будет указывать на элемент массива случайных числе. Так вы избежите проверок на каждой итерации и точно будете гарантировать, что запишется именно простое число.

Добавлено через 2 минуты
Annnie, вот код, который заполняет массив всеми простыми числами до заданного числа:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
#include <iostream>
 
const int PRIMES = 2 * 100;
int primes[ PRIMES ];
int primeSize = 0;
 
void primeGenerate()
{
    for( int i = 0; i < PRIMES; ++i )
        primes[ i ] = 1;
 
    primes[ 0 ] = primes[ 1 ] = 0;
 
    for( int i = 2; i * i < PRIMES; ++i )
        if ( primes[ i ] )
        {
            for( int j = i * i; j < PRIMES; j += i )
                primes[ j ] = 0;
        }
    for( int i = 0; i < PRIMES; ++i )
        if ( primes[ i ] )
            primes[ primeSize++ ] = i;
}
 
int main()
{
    primeGenerate();
    for( int i = 0; i < primeSize; ++i )
        std::cout << primes[ i ] << " ";
    std::cout << std::endl;
 
    return 0;
}
P.S. Писал быстро - получилось не красиво.
0
31 / 31 / 6
Регистрация: 23.10.2014
Сообщений: 107
23.12.2014, 22:53 5
Всё у вас работает. В массиве, изначально, нули. Не каждое число от 2 до 50 простое, так ведь? Поэтому не простые числа не записываются и в массиве остаётся ноль.
0
0 / 0 / 0
Регистрация: 19.12.2014
Сообщений: 12
24.12.2014, 22:02  [ТС] 6
подправила кое-что, проверка на простоту работает, но как обойти нули - так и не пойму :/

Цитата Сообщение от NotNot Посмотреть сообщение
Всё у вас работает. В массиве, изначально, нули. Не каждое число от 2 до 50 простое, так ведь? Поэтому не простые числа не записываются и в массиве остаётся ноль.
а как сделать так, чтобы они не оставались?
Цитата Сообщение от student_201 Посмотреть сообщение
Заполните массив простыми числами. И получайте случайный индекс, который будет указывать на элемент массива случайных числе. Так вы избежите проверок на каждой итерации и точно будете гарантировать, что запишется именно простое число.
но индексы ведь тоже придется проверять на простоту или я не поняла идею?


C++ (Qt)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
#include <iostream>
#include <cstdlib>
#include <ctime> 
#include <locale>
 
using namespace std;
int Prime(int b);
int main()
{
    setlocale(LC_ALL, "Russian");
    const int  n = 4; const int m = 4;
    int Sluch[m][n] = {}; int i; int j;
    ///заполним-ка массив
    srand(time(NULL)); //srand()  получает в виде параметра текущее системное время
    for ( i = 0; i < n; i++)
    for ( j = 0; j < m; j++)
        {
            int ch; 
                ch = 2 + rand() % 19; // запись случайного числа, которое вернет rand() (от 2 до 20)
            if (Prime(ch) == 1) Sluch[i][j] = ch; //если число простое, записываем его в элемент массива
        }
 
    ///////////вывод
    for ( i = 0; i < n; i++)
    {
        for ( j = 0; j < m; j++)
        {
            cout << Sluch[i][j] << " ";
        }
        cout << endl;
    }
    
            
} 
 
 
int Prime(int b)
{
    if (b == 0 || b == 1) return 0;// не простое
    else
    {
        float end = sqrt((float)b); //если натуральное число n>1 не делится ни на одно 
                                    //простое число, не превосходящее sqrt{n}, то оно простое
        for (int im = 2; im <= end; ++im)
        {
            if (b%im == 0) return 0;// не простое
        }
        return 1;// простое
    }
    
}
Добавлено через 10 минут
если кому еще надо, я кажись сделала

C++ (Qt)
1
2
3
4
5
6
7
8
9
10
11
srand(time(NULL)); //srand()  получает в виде параметра текущее системное время
    for ( i = 0; i < n; i++)
    for ( j = 0; j < m; j++)
        {
            do
                ch = 2 + rand() % 19; // запись случайного числа, которое вернет rand() (от 2 до 20)
            while (Prime(ch) == 0);
                if (Prime(ch) == 1) Sluch[i][j] = ch;
             //если число простое, записываем его в элемент массива
            
        }
0
31 / 31 / 6
Регистрация: 23.10.2014
Сообщений: 107
24.12.2014, 22:04 7
Цитата Сообщение от Annnie Посмотреть сообщение
но индексы ведь тоже придется проверять на простоту или я не поняла идею?
Таки не поняли.
Если у вас будет одномерный массив содержащий простые числа, то это место
C++
1
if (Prime(ch) == 1) Sluch[i][j] = ch;
можно будет заменить на
C++
1
Sluch[i][j] = primes[rand() % N]
где N - размер массива простых чисел.
1
Заблокирован
24.12.2014, 22:06 8
Annnie, индексы не придется проверять на простоту. Нужно задать диапазон генерируемых значений 1..n, где n - максимальный индекс массива с простыми числами. Но раз оптимальность решения не нужно, то...

Добавлено через 1 минуту
Annnie, NotNot, дело написал )
0
31 / 31 / 6
Регистрация: 23.10.2014
Сообщений: 107
24.12.2014, 22:07 9
student_201, от 0 до n, индексация ведь с нуля.
0
Заблокирован
24.12.2014, 22:08 10
NotNot, я общий смысл описал. Тогда, конечно, будет от 0..n-1.
0
31 / 31 / 6
Регистрация: 23.10.2014
Сообщений: 107
24.12.2014, 22:13 11
student_201, вы написали
Цитата Сообщение от student_201 Посмотреть сообщение
где n - максимальный индекс
а если n - это индекс то с 0.
0
Заблокирован
24.12.2014, 22:15 12
NotNot, не понимаю, к чему такие придирки. Написал же человеческие диапазоны: от 1 до n. Если нумеруем с 0, то до n-1.
Цитата Сообщение от NotNot Посмотреть сообщение
а если n - это индекс то с 0
с 0, только не до n, а до n - 1, т.к. |0, .., n-1| = n элементов
0
31 / 31 / 6
Регистрация: 23.10.2014
Сообщений: 107
24.12.2014, 22:35 13
student_201, просто вдумайся в слова.
Цитата Сообщение от student_201 Посмотреть сообщение
Нужно задать диапазон генерируемых значений 1..n, где n - максимальный индекс массива с простыми числами.
Этими словами ты, вполне конкретно, говоришь, что значения будут: rand() % (N-1) + 1, где N - размер, (N-1) - индекс последнего элемента - он же твой n. Что означает, что первый элемент (по индексу 0) - мы не берём.

Я понимаю что ты имел в виду с первого по энный, т.е натуральный счет, но сказал ты именно про индексы.
0
0 / 0 / 0
Регистрация: 19.12.2014
Сообщений: 12
24.12.2014, 23:06  [ТС] 14
Цитата Сообщение от NotNot Посмотреть сообщение
Если у вас будет одномерный массив содержащий простые числа, то это место
Код C++
1
if (Prime(ch) == 1) Sluch[i][j] = ch;
можно будет заменить на
Код C++
1
Sluch[i][j] = primes[rand() % N]
где N - размер массива простых чисел.
аааа, то есть выбираем любое из массива простых чисел и пишем его в таблицу?
0
31 / 31 / 6
Регистрация: 23.10.2014
Сообщений: 107
24.12.2014, 23:07 15
Annnie, да.
0
Заблокирован
24.12.2014, 23:12 16
Annnie, я же так и написал в самом начале:
Цитата Сообщение от student_201 Посмотреть сообщение
Annnie, я бы сделал по-другому. Заполните массив простыми числами. И получайте случайный индекс, который будет указывать на элемент массива случайных числе. Так вы избежите проверок на каждой итерации и точно будете гарантировать, что запишется именно простое число.
0
0 / 0 / 0
Регистрация: 19.12.2014
Сообщений: 12
24.12.2014, 23:17  [ТС] 17
Цитата Сообщение от student_201 Посмотреть сообщение
Annnie, я же так и написал в самом начале
ну извиняйте, мне понадобилось время
спасибо за помощь! буду знать про такой метод
0
24.12.2014, 23:17
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
24.12.2014, 23:17
Помогаю со студенческими работами здесь

Функции для обработки матриц: объявить и заполнить двумерный динамический массив случайными числами в заданном диапазоне
Как переделать этот код так , чтоб он соответствовал этому заданию: /*Объявить и заполнить...

Сформировать одномерный массив случайными числами в диапазоне [-1; 1]
3. Сформировать одномерный массив случайными числами в диапазоне и вывести его на экран. ...

Заполнить массив случайными числами
Задайте с помощью генератора случайных чисел массив в диапазоне от 1 до 100. Целое число

Заполнить массив случайными числами
Помогите, пожалуйста, как заполнить массив случайными числами в диапазоне от -20 до 20.


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
17
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2023, CyberForum.ru