Losowanie bez powtorzen

Question

Witam, program ma wylosować 5 liczb bez powtórzeń, liczby wylosowane mają być z przedziału od 1 do 10. Zdaje sobie sprawę, że podobne posty są już na forum, jednak zależy mi na zrozumieniu gdzie popełniam błąd.

Pozdrawiam

 

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

using namespace std;

bool czyjest(int liczby[], int kandydat) {
	for (int i = 0; i < 5; i++) {
		if (liczby[i] == kandydat) return true;
		else return false;
	}
}

int main()
{
	srand(time(NULL));
	int liczby[5];
	int kandydat;
	
	liczby[0] = rand() % 10 + 1;
	
	for (int i = 1; i < 5; i++) {
		kandydat = rand() % 10 + 1;
		for (; czyjest(liczby, kandydat) != false;) {
			kandydat = rand() % 10 + 1;
			
		}
		
	}

	for (int i = 0; i < 5; i++) {
		cout << liczby[i] << endl;
	}

}

 

Answer 1

W pierwszej pętli liczba nie jest przypisywana do finałowej tablicy

for (int i = 1; i < 5; i++) {
        kandydat = rand() % 10 + 1;
        for (; czyjest(liczby, kandydat) != false;) {
            kandydat = rand() % 10 + 1;
             
        }
         
    }

Także polecał bym spojrzenie na unordered_set

Comments

Dodałem po tej pętli, ale dalej nie działa

for (int i = 1; i < 5; i++) {
		kandydat = rand() % 10 + 1;
		for (; czyjest(liczby, kandydat) != false;) {
			kandydat = rand() % 10 + 1;
			
		}
		liczby[i] = kandydat;
		
	}

 

---

Funkcja czyjest powinna wyglądać tak

bool czyjest(int liczby[], int kandydat) {
    for (int i = 0; i < 5; i++) {
        if (liczby[i] == kandydat) return true;
    }
	return false;
}
 

 

---

Rzeczywiście, dziękuje za pomoc!