Генерация массива случайных чисел код выдаёт неправильный результат

@Slenon · Регистрация: 25.07.2016

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

Вот код :

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
#include <stdio.h>
#include <iostream>
#include <cstring>
#include <cstdlib>
#include <cmath>
using namespace std;
int main(){
int n, q, j, x, r, z, w , s, i = 0;
int y[] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0};
printf("Vvedite n = ");
scanf("%i", &n);
while(r <= n) {
j = 0; r = 0;      
srand(time(NULL)); 
s = floor(rand()/rand());
if((s > 0) && (s <= n)) {
 for(x = 0; x < 10; x++) {if(s != y[x]) {++j;}}}  
  if(j == 10) {y[i] = s; ++i;}
 for(z = 0; z < 10; z++) {if(y[z] != 0) {++r;}} 
   }
 for(w = 0; w < r; w++) {printf("%i", y[w]);}
 getchar();
}

По идеи он должен создавать массив чисел от 1 до n (не > 10), без нулей, без повторов, без исключений,
вобщем смотри ветку CLIPS...
Пока меня интересует, происходит ли сравнивание r с n ...

ValeryS · 13.12.2017, 23:52

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

srand(time(NULL));
s = floor(rand()/rand());

зачем каждый раз настраивать генератор?

Добавлено через 2 минуты

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

while(r <= n)

r здесь чему равно?

@Herji · 14.12.2017, 00:10

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

rand()/rand()

иногда можно и на ноль делить)

@Slenon · 18.12.2017, 18:46 **[ТС]**

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

r здесь чему равно?

ValeryS, r должно быть равно числу элементов (не нулей), получаемых в результате
работы проги ( при n = 5, y[], например, = 3, 2, 4, 5, 1, 0, 0, 0, 0, 0) ...
Вот исправленный кусок кода:

C
1
2
3
4
5
6
7
8
9
10
11
int y[] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0};
//-------------------------------------------------
srand(time(NULL));
printf("Vvedite n = ");
scanf("%i", &n);
while(r < n) {
j = 0; r = 0;
s1 = rand(); s2 = rand();
if((s1 > 0) && (s2 > 0)) {s = floor(s1/s2);}
//---------------------------------------------------
if(s <= n) {for(x = 0; x < 10; x++) {if(s != y[x]) {++j;}}}

ValeryS · 18.12.2017, 18:56

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

r должно быть равно числу элементов

и где это в коде?

@Slenon · 19.12.2017, 02:12 **[ТС]**

C
1
for(z = 0; z < 10; z++) {if(y[z] != 0) {++r;}}

Должна происходить проверка всех (10) элементов массива
на неравенство 0, если y[z] != 0, происходит приращение r ...

ValeryS · 19.12.2017, 07:57

еще раз, где в коде r присваивается значение?

@Antikl · 19.12.2017, 08:05

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

еще раз, где в коде r присваивается значение?

у него 0 постоянно присваивать к r в цыкле

C++
1
r = 0;

ValeryS · 19.12.2017, 08:07

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

у него 0 постоянно присваивать к r в цыкле

а строчку выше видел?

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

C
1
while(r < n)

вот я про неё и спрашиваю

zer0mail · 19.12.2017, 08:52

Как программа написана, так и работает. По поводу [не]правильности - все претензии к автору кода.

@Slenon · 19.12.2017, 21:17 **[ТС]**

Заработало!

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
#include <stdio.h>
#include <iostream>
#include <cstring>
#include <cstdlib>
#include <cmath>
using namespace std;
int main(){
int n, q, j, x, z, w , s, s1, s2;
int i = 0; int r = 0;
int y[] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0};
srand(time(NULL));
printf("Vvedite n = ");
scanf("%i", &n);
while(r < n) {
j = 0; r = 0;
s1 = rand(); s2 = rand();
if((s1 > 0) && (s2 > 0)) {s = floor(s1/s2);}
if(s <= n) {for(x = 0; x < 10; x++) {if(s != y[x]) {++j;}}}  
if(j == 10) {y[i] = s; ++i;}
 for(z = 0; z < n; z++) {if(y[z] != 0) {++r;}} 
   }
 for(w = 0; w < n; w++) {printf("%i", y[w]);}
 getchar();
}

Спасибо за настойчивый ? ...

Добавлено через 32 минуты
Изменим вывод:

C
1
2
printf("%s", " S = ");
 for(w = 0; w < n; w++) {printf(" %i ", y[w]);}

ValeryS · 19.12.2017, 21:32

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

Заработало!

ну и хорошо

а теперь другое решение
забивает массив из 9 элементов числами от 1 до 9

C++
1
2
3
4
5
6
7
8
9
10
11
int arr[9];
int boolArr[10]={0};
for(int i=0;i<9;i++)
{
 int RandTmp=0;
 do{
    RandTmp=rand()%9+1
   }while(boolArr[RandTmp]!=0);
 boolArr[RandTmp]=1;
 arr[i]=RandTmp;
}

@TheCalligrapher · 19.12.2017, 21:35

Нет, не заработало.

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

C++
1
2
s1 = rand(); s2 = rand();
if((s1 > 0) && (s2 > 0)) {s = floor(s1/s2);}

Ну а чему будет равно s если и s1, и s2 окажутся равны 0?

@Slenon · 19.12.2017, 22:13 **[ТС]**

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

Ну а чему будет равно s если и s1, и s2 окажутся равны 0?

C
1
if((s1 > 0) && (s2 > 0)) {s = floor(s1/s2);} else {s = 0;}

и ещё дополнение, что бы окно не схлопывалось:

C
1
2
getchar();
getchar();

@Slenon · 11.02.2018, 03:21 **[ТС]**

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
#include <stdio.h>
#include <iostream>
#include <cstring>
#include <cstdlib>
#include <cmath>
using namespace std;
int main(){
int n, q, j, x, z, w , s, s1, s2, r, i;
srand(time(NULL));
while(2) {
printf("Vvedite n = ");
scanf("%i", &n);
if(n == 0) {break;}
printf("\n");
i = 0; r = 0;
int y[850]={0};
while(r < n) {
j = 0; r = 0;
s1 = rand(); s2 = rand();
if((s1 > 0) && (s2 > 0)) {s = floor(s1/s2);} else {s = 0;}
if(s <= n) {for(x = 0; x < 850; x++) {if(s != y[x]) {++j;}}}  
if(j == 850) {y[i] = s; ++i;}
 for(z = 0; z < n; z++) {if(y[z] != 0) {++r;}} 
      }
printf("%s", " S =");
for(w = 0; w < n; w++) {printf(" %i", y[w]);}
printf("\n\n");
   }
}

@zss · 11.02.2018, 11:51

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

floor(s1/s2)

s1 и s2 целые числа в диапазоне [0, RAND_MAX]
Результат деления - целое число в диапазоне [1, RAND_MAX]
причем равномерность распределения при таком делении сильно искажена,
маленькие значения будут встречаться гораздо чаще, чем большие
(плотность распределения будет иметь вид 1/x).

И зачем это Вам, когда в помощи по rand приведен пример, как это делать правильно

C
1
2
3
4
5
6
7
8
9
10
11
12
13
void RangedRandDemo( int range_min, int range_max, int n )
{
   // Generate random numbers in the half-closed interval
   // [range_min, range_max). In other words,
   // range_min <= random number < range_max
   int i;
   for ( i = 0; i < n; i++ )
   {
      int u = (double)rand() / (RAND_MAX + 1) * (range_max - range_min)
            + range_min;
      printf( "  %6d\n", u);
   }
}

Привел Ваш код в божеский вид:

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
#include <stdio.h>
#include <time.h>
//#include <iostream>
//#include <cstring>
#include <stdlib.h>
//#include <cmath>
//using namespace std;
int RangedRand( int range_min, int range_max)
{
   // Generate random numbers in the half-closed interval
   // [range_min, range_max). In other words,
   // range_min <= random number < range_max
      int u = int((double)rand() / (RAND_MAX + 1) * (range_max - range_min))
            + range_min;
      return u;
}
int main(){
    int n, j, is, i;
    const int MAX_SIZE=850;
    int y[MAX_SIZE];
    srand((unsigned)time(NULL));
    while(1) 
    {
        printf("Enter n: ");
        scanf("%i", &n);
        if(n <= 0 || n > MAX_SIZE)
            break;
        puts("");
        i = 0;
        for(j = 0; j < n; j++)
            y[j]=0;
        while(i < n) 
        {
            is = RangedRand(1,n+1);
            for(j = 0; j < n; j++) 
            {
                if(is == y[j])
                    break;
            }
            if(j == n) 
                y[i++] = is;
        }
        printf(" S =");
        for(j = 0; j < n; j++) 
            printf(" %i", y[j]);
        puts("\n");
    }
    return 0;
}

Но на С++ это можно сделать гораздо красивее

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
#include <iostream>
#include <algorithm>
using namespace std;
int main()
{
    const int MAX_SIZE=850;
    int y[MAX_SIZE];
    while(true) 
    {
        cout<<"Enter n: ";
        int n;
        cin>>n;
        if(n <= 0 || n > MAX_SIZE)
            break;
        cout<<endl;
        for(int j = 0; j < n; j++)
            y[j]=j+1;
        random_shuffle(y,y+n);
        cout<<" S =";
        for(int j = 0; j < n; j++) 
            cout<<y[j]<<' ';
        cout<<endl;
    }
    return 0;
}

@Slenon · 03.11.2018, 04:26 **[ТС]**

Просьба объяснить что в коде неправильно ...

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
#include <stdio.h>
#include <iostream>
#include <cstring>
#include <cstdlib>
#include <cmath>
void RangedRandDemo(int n ) {
   int i;
   for ( i = 0; i < n; i++ ) {
   int u = (double)rand() / (RAND_MAX + 1) * (n - 1) + 1;
   printf( " %d", u);
   }
}
using namespace std;
int main(){
int n;
while(1) {
printf(" Vvedite n = ");
scanf("%i", &n);
printf("%s", "         S =");
RangedRandDemo(n);
printf("\n");
   }
}

Добавлено через 16 минут
Мне надо, чтобы прога выдавала примерно следующее:
Vvedite n = 7
S = 3 5 6 2 1 7 4
Vvedite n = 7
S = 4 7 2 6 1 5 3
...........................

@alexu_007 · 03.11.2018, 08:14

Самому подумать не пробовал?

@Slenon · 03.11.2018, 18:04 **[ТС]**

Вот что я напридумывал (всё равно ...

)

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
#include <stdio.h>
#include <iostream>
#include <cstring>
#include <cstdlib>
#include <list>
#include <cmath>
using namespace std;
 
void spisok(int n) {
    list<int> Spisok(0);
     while(Spisok.size() <= n) {
         int J = 0, T = round((double)rand() * n);
          if(T != 0) {
           list<int>::iterator p = Spisok.begin();
            while(p != Spisok.end()) {
                if(*p != T) {++J;}
         }
      }
                 if(J == Spisok.size()) {printf(" %d", T); Spisok.push_back(T);}
   }
}
int main() {
int n;
srand(time(NULL));
while(1) {
printf(" Vvedite n = ");
scanf("%i", &n);
printf("%s", "    Spisok =");
printf("\n");
   }
}

@TheCalligrapher · 03.11.2018, 18:12

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

Вот что я напридумывал (всё равно ... )

Тема генерации неповторяющихся случайных чисел уже миллион раз обсуждалась.

Непонятно только, зачем вы писали эту функцию spisok, если она у вас в программе ни разу не вызывается.

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

ValeryS Модератор 8978 / 6744 / 921 Регистрация: 14.02.2011 Сообщений: 23,854
	19.12.2017, 07:57
	еще раз, где в коде r присваивается значение? 0

zer0mail 2688 / 2260 / 244 Регистрация: 03.07.2012 Сообщений: 8,231 Записей в блоге: 1
	19.12.2017, 08:52
	Как программа написана, так и работает. По поводу [не]правильности - все претензии к автору кода. 0

@alexu_007 736 / 700 / 110 Регистрация: 29.05.2015 Сообщений: 4,267
	03.11.2018, 08:14
	Самому подумать не пробовал? 0