Форум программистов, компьютерный форум, киберфорум
Наши страницы
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 5.00/20: Рейтинг темы: голосов - 20, средняя оценка - 5.00
Annnie
0 / 0 / 0
Регистрация: 19.12.2014
Сообщений: 12
#1

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

23.12.2014, 22:38. Просмотров 4084. Ответов 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
Ответы с готовыми решениями:

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

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

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

Заполнить массив случайными числами от -50 до 50
Нужно заполнить массив числами от -50 до 50 .Написал такой код #include...

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

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 / 6
Регистрация: 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 / 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
student_201
Заблокирован
24.12.2014, 22:06 #8
Annnie, индексы не придется проверять на простоту. Нужно задать диапазон генерируемых значений 1..n, где n - максимальный индекс массива с простыми числами. Но раз оптимальность решения не нужно, то...

Добавлено через 1 минуту
Annnie, NotNot, дело написал )
0
NotNot
30 / 30 / 6
Регистрация: 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 / 6
Регистрация: 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 / 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
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 / 6
Регистрация: 23.10.2014
Сообщений: 107
24.12.2014, 23:07 #15
Annnie, да.
0
student_201
Заблокирован
24.12.2014, 23:12 #16
Annnie, я же так и написал в самом начале:
Цитата Сообщение от student_201 Посмотреть сообщение
Annnie, я бы сделал по-другому. Заполните массив простыми числами. И получайте случайный индекс, который будет указывать на элемент массива случайных числе. Так вы избежите проверок на каждой итерации и точно будете гарантировать, что запишется именно простое число.
0
Annnie
0 / 0 / 0
Регистрация: 19.12.2014
Сообщений: 12
24.12.2014, 23:17  [ТС] #17
Цитата Сообщение от student_201 Посмотреть сообщение
Annnie, я же так и написал в самом начале
ну извиняйте, мне понадобилось время
спасибо за помощь! буду знать про такой метод
0
24.12.2014, 23:17
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
24.12.2014, 23:17

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

Заполнить случайными числами 5 значный массив
От программы требуется заполнить случайными числами 5 значный массив, так,...

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


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

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

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