Probem z losowaniem liczb w funkcji

Question

Witam, mam taki problem. Napisałem program, który losuje liczby bez powtórzeń, przedział liczb i krańce przedziału są określone przez użytkownika, ten program działa i jest wszystko okey. Problem się zaczyna wtedy gdy chce losowanie zawrzeć w funkcji losuj i wywołać ją w main razem z rezorwowaniem i zwalnianiem pamięci. NIestety wywala mi błąd, próbowałem go jakoś rozwiązać, lecz bez skutku. Poniżej wklejam kod:

https://pastebin.com/Kw5scXzx

Comments

W funkcji tylko wylosuj liczbę, po co całą tablicę tworzyć jeszcze raz?? Do funkcji wyślij tylko zakres losowania i zwróc wylosowaną liczbę.

Answer 1

W main:

tab[i] = losuj(int ilosc, poczatek, int koniec);

Co to jest i ? Chcesz przekazac zmienna, a nie deklarowac nowa - bez slowka int.

W funkcji losuj poza forem masz:

return tab[i];

tak samo nie wie skad sie wzielo i.

Answer 2

Do funkcji rand i srand potrzebujesz biblioteki cstdlib.

Answer 3

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

using namespace std;
 
int losuj(int poczatek, int koniec)
{
   int liczba = rand()% (poczatek-koniec)+poczatek;
   return liczba;
}
 
int main()
{
  int ilosc, poczatek, koniec;
  
    cout << "Czesc, podaj ile liczb wyloscowac:"<< endl;
    cin>>ilosc;
	
	int *tab = new int[ilosc];

    cout<< "Podaj poczatek przedzialu: "<<endl;
    cin>>poczatek;
 
    cout<< "Podaj koniec przedzialu: "<<endl;
    cin>>koniec;
    cout<<"\n";
 
	srand(time(NULL));

	for(int i = 0; i < ilosc; i++)
		tab[i]=losuj(poczatek, koniec);
	
	for(int y = 0; y < ilosc; y++)
		cout<<tab[y]<<" ";

	system("pause");
    delete [] tab;
return 0;
}

 

Answer 4

Dziękuje wszystkim za podpowiedzi, program udało mi sie poprawić. Pozdrawiam