Jak dalej kontynuować?

Question

Napisz program, który wczytuje z klawiatury 2 liczbę naturalne a następnie losuje liczbę, która zawiera się pomiędzy tymi dwiema liczbami.

Dotarłem do połowy tego programu, niestety nie wiem jaka komenda sprawi że wylosuje mi liczbe z przedziału podanego przez nas (przez klawiaturę).

#include "stdafx.h"
#include <iostream>
#include <cstdlib>
#include <ctime>
#include <time.h>

using namespace std;

int main(array<System::String ^> ^args)
{
    int liczba3;
    
    int liczba1;
    cout << "Podaj swoja 1 liczbe: ";
    cin >> liczba1;

    int liczba2;
    cout << "Podaj swoja 2 liczbe: ";
    cin >> liczba2;

    while (liczba3 < liczba2)
{
    
    srand(time(NULL));
    std::cout << "Wylosowanie pierwsze: " << std::rand() << std::endl;
    
    }


    system("PAUSE"); 
    

}

 

Answer 1

Żeby uzyskać losową liczbę z przedziału musisz wykonać kilka obliczeń.

Ale zacznijmy od początku: funkcja rand() zwraca liczbę z przedziału od 0 do RAND_MAX (wartość zależy od implementacji, standard zakłada, że musi to być minimum 32767).
Jeżeli otrzymaną liczbę podzielisz z resztą (modulo) przez X otrzymasz liczbę z przedziału od 0 do X-1.
Jeżeli do tej liczby dodasz liczbę A wówczas przesuniesz przedział. Nowy przedział zawiera liczby od A do X-1+A.
Teraz trochę matematyki:
B = X-1+A
X = B-A+1
Żeby wylosować liczbę z przedziału od A do B musisz wylosować liczbę z przedziału od 0 do X i dodać do niej A.
Czyli musisz wylosować liczbę z przedziału 0 do (B-A+1) i dodać do niej A.

int losuj_z_przedziału(int A, int B) {
    int X = B - A +1;
    return rand() % X + A;
}

 

Comments

error C2601: 'losuj_z_przedziału' : local function definitions are illegal

Wyświetla mi się takie coś :/

---

Umieściłeś funkcję w poprawnym miejscu? Powinna znajdować się poza funkcją main().

---

#include "stdafx.h"
#include <iostream>
#include <cstdlib>
#include <ctime>
#include <time.h>

using namespace std;

int main(array<System::String ^> ^args)
{  
    
    int A;
    int B;
    int X;
    
    cout << "Podaj swoja 1 liczbe: ";
    cin >> A;

    cout << "Podaj swoja 2 liczbe: ";
    cin >> B;

    srand(time(NULL));
    
    int zakres(int A, int B) {
    int X = B - A +1;
    return rand() % X + A;

    cin >> X;
    cout << "Liczba:";
    
    
    system("PAUSE"); 

}

Jestem początkujący i nie za bardzo czasem rozumiem :/

---

Widzę, że stworzyłeś projekt Visual C++ zamiast C++
Jeżeli nie musisz to tego nie używaj

Biblioteki <time.h> i <ctime> to zasadniczo to samo (ctime jest zawarta w przestrzeni nazw std). Wybierz jedną.

#include <iostream>
#include <cstdlib>
#include <ctime>
 
using namespace std;

int losuj_z_przedzialu(int A, int B) {
    int X = B - A +1;
    return rand() % X + A;
}
 
int main(int argc, char **argv)
{
    int A;
    int B;
     
    cout << "Podaj swoja 1 liczbe: ";
    cin >> A;
 
    cout << "Podaj swoja 2 liczbe: ";
    cin >> B;
    
    if (B <= A) {
        cout << "Druga liczba musi byc wieksza od pierwszej";
        return 1;
    }
 
    srand(time(NULL));
     
    int liczba = losuj_z_przedzialu(A,B);
    cout << "Liczba:" << liczba << endl;
     
     
    system("PAUSE"); 
    return 0;
}

 

---

Poczytaj sobie o funkcjach. Bardzo przydatne narzędzie. Twój kod powinien wyglądać tak:

#include "stdafx.h"
#include <iostream>
#include <cstdlib>
#include <ctime>
#include <time.h>
 
using namespace std;
 
int losuj_z_przedzialu(int A, int B) {
    int X = B - A +1;
    return rand() % X + A;
}

int main(array<System::String ^> ^args)
{
    int liczba3;
     
    int liczba1;
    cout << "Podaj swoja 1 liczbe: ";
    cin >> liczba1;
 
    int liczba2;
    cout << "Podaj swoja 2 liczbe: ";
    cin >> liczba2;
 
    while (liczba3 < liczba2)
{
     
    srand(time(NULL));
    std::cout << "Wylosowanie pierwsze: " << losuj_z_przedzialu(liczba1, liczba2) << std::endl;
     
    }
 
 
    system("PAUSE"); 
     
 
}

 

---

@tangarr, o co chodzi z argumentami funkcji main()? Pierwszy raz widzę coś takiego

---

Dziękuje wam, wszystko już działa :)