Форум программистов, компьютерный форум, киберфорум
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
 
Рейтинг: Рейтинг темы: голосов - 10, средняя оценка - 4.70
Annnie
0 / 0 / 0
Регистрация: 19.12.2014
Сообщений: 12
#1

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

23.12.2014, 22:38. Просмотров 3873. Ответов 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
Миниатюры
Заполнить массив случайными числами в диапазоне  
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
23.12.2014, 22:38
Я подобрал для вас темы с готовыми решениями и ответами на вопрос Заполнить массив случайными числами в диапазоне (C++):

Заполнить одномерный динамический массив случайными числами в заданном диапазоне - C++
как заполнить одномерный динамический массив случайными числами от -100 до +100 вот вариант от 1 до 100: srand(time(NULL)); ...

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

Заполнить массив случайными значениями, лежащими в диапазоне от 50 до 100 - C++
Массив предназначен для хранения значений весов двадцати человек. С помощью датчика случайных чисел заполните массив целыми значениями,...

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

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

Заполнить массив случайными числами от -50 до 50 - C++
Нужно заполнить массив числами от -50 до 50 .Написал такой код #include &quot;stdafx.h&quot; #include&lt;iostream&gt; int...

16
student_201
Заблокирован
23.12.2014, 22:41 #2
Annnie, от 2 до 50 будет так:
C++
1
ch = 2 + rand() % 49;
0
Annnie
0 / 0 / 0
Регистрация: 19.12.2014
Сообщений: 12
23.12.2014, 22:46  [ТС] #3
да я уже поняла мне это не особо важно, я вот не пойму, почему функция проверки не срабатывает
0
student_201
Заблокирован
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
NotNot
30 / 30 / 5
Регистрация: 23.10.2014
Сообщений: 107
23.12.2014, 22:53 #5
Всё у вас работает. В массиве, изначально, нули. Не каждое число от 2 до 50 простое, так ведь? Поэтому не простые числа не записываются и в массиве остаётся ноль.
0
Annnie
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
NotNot
30 / 30 / 5
Регистрация: 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
student_201
Заблокирован
24.12.2014, 22:06 #8
Annnie, индексы не придется проверять на простоту. Нужно задать диапазон генерируемых значений 1..n, где n - максимальный индекс массива с простыми числами. Но раз оптимальность решения не нужно, то...

Добавлено через 1 минуту
Annnie, NotNot, дело написал )
0
NotNot
30 / 30 / 5
Регистрация: 23.10.2014
Сообщений: 107
24.12.2014, 22:07 #9
student_201, от 0 до n, индексация ведь с нуля.
0
student_201
Заблокирован
24.12.2014, 22:08 #10
NotNot, я общий смысл описал. Тогда, конечно, будет от 0..n-1.
0
NotNot
30 / 30 / 5
Регистрация: 23.10.2014
Сообщений: 107
24.12.2014, 22:13 #11
student_201, вы написали
Цитата Сообщение от student_201 Посмотреть сообщение
где n - максимальный индекс
а если n - это индекс то с 0.
0
student_201
Заблокирован
24.12.2014, 22:15 #12
NotNot, не понимаю, к чему такие придирки. Написал же человеческие диапазоны: от 1 до n. Если нумеруем с 0, то до n-1.
Цитата Сообщение от NotNot Посмотреть сообщение
а если n - это индекс то с 0
с 0, только не до n, а до n - 1, т.к. |0, .., n-1| = n элементов
0
NotNot
30 / 30 / 5
Регистрация: 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
Annnie
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
NotNot
30 / 30 / 5
Регистрация: 23.10.2014
Сообщений: 107
24.12.2014, 23:07 #15
Annnie, да.
0
24.12.2014, 23:07
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
24.12.2014, 23:07
Привет! Вот еще темы с ответами:

Заполнить двумерный массив случайными числами - C++
Заполнить двумерный массив с++ случайными числами

Заполнить массив случайными не повторяющимися числами - C++
Здравствуйте! Подскажите пожалуйста, случайными числами заполнить как бы не проблема, но как сделать, что бы они не повторялись - вот это...

Заполнить массив неодинаковыми случайными числами - C++
нужно дополнить ф-кцию рандома так что бы заполнить массив не одинаковыми числами. Как прописать возвращение на внутренний цыкл, чтобы...

Заполнить массив размером 10 случайными числами от 0 до 10 - C++
#include &lt;vector&gt; #include &lt;algorithm&gt; #include &lt;iostream&gt; using namespace std; int main() { int n = 10; vector&lt;int&gt;...


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

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

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2018, vBulletin Solutions, Inc.
Рейтинг@Mail.ru