Losowanie i zliczanie wyników

Question

Witam, próbuję stworzyć program który wylosuje milion liczb losowych z zakresu od 0 do 100, a następnie zliczy ich powtórzenia i wyświetli liczbę najczęściej wylosowaną oraz ilość jej powtórzeń, napisałem taki kod: 

 

#include <iostream>
#include <time.h>
#include <cstdlib>

using namespace std;

int main() {
int liczba;
int tab[101] = { 0 };
cout << "Wylosowanie 1000000 liczb calkowitych ze zbioru od 0 do 100" << endl;

srand(time(NULL));
for (int i = 0; i <= 1000000; i++) {
    liczba = rand() % 100 + 1;
    tab[liczba]++;
}
int ilosc;
int max = tab[0];
int pozycja = 0;
for (int i = 0; i < ilosc; i++)
{
   if (tab[i] > max)
   {
       pozycja = i;
       max = tab[i];
    }
}
std::cout<<pozycja+1<<"  "<<tab[pozycja+1];
system("pause");
}

ale niestety coś w nim jest nie tak i nie mogę zrozumieć co.

Z góry dziękuję za wszelką pomoc i porady.

Answer 1

Zmienna ilość nie jest zainicjowana. Poprawiłem też kilka rzeczy, bo jak rozumiem chcesz losować 1'000'000 liczb z zakresu od 0 - 100 włącznie z 0 i 100:

#include <iostream>
#include <ctime>
#include <cstdlib>

int main() {
	
	const int ilosc = 101;
	int tab[ilosc] = { 0 };
	std::cout << "Wylosowanie 1000000 liczb calkowitych ze zbioru od 0 do 100" << std::endl;

	srand(time(NULL));
	for (int i = 0; i < 1000000; i++) // tutaj <, bo losujesz 1'000'000, a nie 1'000'001
	{
		int liczba = rand() % ilosc;
		tab[liczba]++;
	}
	
	int max = tab[0];
	int pozycja = 0;
	for (int i = 0; i < ilosc; i++)
	{
		if (tab[i] > max)
		{
			pozycja = i;
			max = tab[i];
		}
	}

	std::cout << pozycja << "  " << tab[pozycja] << "\n";
	system("pause");
}

PS. Warto wiedzieć, że C++ dostarcza wiele gotowych funkcji, które łatwo rozwiązują takie zadania:

#include <iostream>
#include <algorithm>
#include <cstdlib>
#include <ctime>

int main() 
{
	srand(time(NULL));

	const size_t size = 101;
	int array[size]{};
	std::cout << "Wylosowanie 1000000 liczb calkowitych ze zbioru od 0 do 100\n";

	for (size_t i = 0; i < 1'000'000; i++) 
	{
		int number = rand() % size;
		array[number]++;
	}
	
	auto max = std::max_element(array, array + size);
	std::cout << std::distance(array, max) << "  " << *max << "\n";
}

Comments

Niestety, żaden z kodów które napisałeś nie działa u mnie, w pierwszym po prostu konsola jest pusta, a w drugim wyrzuca błąd w linii "auto max = std::max_element(array, array + size);"

---

Nie może być pusta w pierwszym przypadku...

---

Może masz wolny komputer i musisz dać mniejszą liczbę zamiast 10000000 np 1000

---

Nie, nie mam wolnego komputera. Poza tym nawet dla szkolnego złomka to nie powinien być problem.

---

Zauważyłem że program działa, ale wyświetla tylko na chwile wynik, po czym okno konsoli robi się malutkie, a po powiekszeniu jest puste. Jak to naprawić?

---

Okej, udało mi się to naprawić, zastąpiłem system("pause") za pomocą getchar() i return 0.

Dziękuję za pomoc.

---

Dziwne. Z jakiego środowiska korzystasz? CodeBlock/Dev/Visual?

EDIT: Okej ;)