Losowanie liczb w c++

Question

"Spytaj użytkownika o przedział liczb do losowania. Ale uwaga użytkownik może podać pierwszą liczbę większą od drugiej. Zastosuj instrukcje warunkowe. Wykorzystaj zamianę wartości zmiennych i zmienną pomocniczą".

Napisałam program, który kompletnie nie działa tak jak powinien i nie wiem gdzie popełniłam błąd.

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

using namespace std;

int main()
{

	int liczba1, liczba2, liczba3, losowanie;

	cout<<"Podaj liczbe: ";
	cin>>liczba1;
	
	cout<<"Podaj liczbe: ";
	cin>>liczba2;
	
	srand(time(NULL));
	 
	 if(liczba1>liczba2)
	 {
	 losowanie=liczba1+rand()%(liczba2-liczba1+1);
	 cout<<"Wylosowano liczbe : "<<losowanie;
	}
	
	else if(liczba2>liczba1)
	{
		liczba1=liczba2;
		liczba2=liczba3;
		losowanie=liczba2+rand()%(liczba3-liczba2+1);
	 	cout<<"Wylosowano liczbe : "<<losowanie;	
	}
	
	else
	{
		cout<<"Liczby sa rowne";
	}
		
system("pause");	
return 0;
}

 

Answer 1

int main()
{
    std::cout << "Podaj poczatek: ";
    int min = 0;
        std::cin >> min;
    std::cout << "Podaj koniec. ";
    int max = 0;
        std::cin >> max;
    if (min > max) {
        int temp = max;
        max = min;
        min = temp;
    }
    std::random_device randomGenerator;
    std::mt19937 rng(randomGenerator ());
    std::uniform_int_distribution <std::mt19937::result_type> dist (min, max);

    for (int i = 0; i < 10; i ++) {
        std::cout << dist(randomGenerator) << "\n";
    }

    return 0;
}

Troche bardziej zmodernizowane

Comments

std::uniform_int_distribution <std::mt19937::result_type> dist (min, max);

Przekombinowałeś. AFAIK argumentem szablonu jest typ wyjściowy, który nie ma związku z wyjściem generatora. Możesz tutaj zwykłego inta wstawić.

---

Oh okay, pierwszy raz to widzialem tak i jakos tak juz zostalo, ale dzieki za info

std::random_device randomGenerator;
    std::uniform_int_distribution <int> dist (min, max);