[ROZWIĄZANE] Losowanie bez powtórzeń

Szukam sposobu na utworzenie funkcji, która losowałaby bez powtórzeń, czyli:

 

Pierwsze losowanie wśród liczb od 1 do 10 - wylosowało 5

Drugie losowanie wśród liczb od 1 do 10 (nie bierze pod uwagę 5) - wylosowało 2

Trzecie losowanie wśród liczb od 1 do 10 (nie bierze pod uwagę 5 i 2) - wylosowało 7

itd.

 

Oczywiście ze zmianą mapy wszystko się resetuje.

 

Z góry dzięki za pomoc

Zrób tablice dynamiczną , wrzuć do niej liczby , potem losuj index ( od 0 do rozmiar -1 ), pobieraj liczbe z indexu , usuwaj pole z tablicy pod danym indexem

Zrób tablicę, która ma elementów tyle co maksymalna liczba do wylosowania i tam zapisuj prawdę na pozycji x, gdy wylosuje się x, a potem tylko będziesz losować dotąd, aż nie wylosuje się jakaś wartość, która ma fałsz w tamtej tablicy. Jest to sposób mniej optymalny, ale łatwiejszy do zrobienia. Drugi sposób, to także zrobić tablicę, ale jeśli wylosujemy x, to w tablicy tam gdzie jest indeks x, wrzucamy ostatnią wartość z tej tablicy i zmniejszam ilość do wylosowania. Trochę ciężko to szybko opisać, ale może coś Ci to rozjaśni, być może ktoś Ci tu rzuci też jakimś kodem.

Może ktoś sypnąć kodem? Bo nie bardzo rozumiem jak to wykonać

new liczby[] = {

1,2,3,4,5,6,7,8,9,0

};

new i, j, r, s = sizeof(liczby), Array:ah = ArrayCreate(1, 10);



for(i=0; i<s; ++i)

ArrayPushCell(ah, liczby[i]);



for(i=0; i<(j=ArraySize(ah)); ++i) {

log_amx("%i", ArrayGetCell(ah, (r = random(j))));

ArrayDeleteItem(ah, r);

}

Nie testowane.

new model_zh[10];
new modele[] = {
	"zh_corpse",
	"zh_crispy2",
	"zh_classic",
	"zombie_big_origin",
	"zombie_girl",
	"zombie_leech",
	"zombie_nemesis",
	"zombie_poison",
	"zombie_resident",
	"zombie_speed"
};

public LosujModele()
{
	new i, j, r, s = sizeof(modele), Array:ah = ArrayCreate(1, 10);

	for(i=0; i<s; ++i)
	ArrayPushString(ah, modele[i]);

	for(i=0; i<(j=ArraySize(ah)); ++i)
	{
		r = random(j);
		ArrayGetString(ah, r, model_zh[r], 31);
		ArrayDeleteItem(ah, r);
	}
}

L 10/27/2013 - 12:33:18: Info (map "de_dust2") (file "addons/amxmodx/logs/error_20131027.log")
L 10/27/2013 - 12:33:18: [AMXX] Displaying debug trace (plugin "zombiehell_v1.6.amxx")
L 10/27/2013 - 12:33:18: [AMXX] Run time error 4: index out of bounds 
L 10/27/2013 - 12:33:18: [AMXX]    [0] zombiehell_v1.6.sma::LosujModele (line 628)
L 10/27/2013 - 12:33:18: [AMXX]    [1] zombiehell_v1.6.sma::plugin_init (line 223)

W funkcji ArrayCreate (620) jako pierwszy parametr (maksymalna wielkość danych, w Twoim wypadku długość stringa) podajesz 1, podczas, gdy stringi z tablicy są dłuższe, niż 1 i to powoduje błąd.

 

Pomijając już kwestię inicjacji tablicy dynamicznej o za małym rozmiarze, zastanów się, jaki warunek zostanie wykonany przy szóstej iteracji pętli z linijki 625. i dlaczego nie zostanie on spełniony.

Zmieniłem wielkość na 20 i pobieram wielkość tablicy i dodaję jeden (inaczej nie wylosowałoby 10. pozycji) przy każdym wywołaniu pętli. Wydaje mi się, że o to chodziło.

public LosujModele()
{
	new i, r, s = sizeof(modele), Array:ah = ArrayCreate(20, 10);

	for(i=0; i<s; ++i)
	ArrayPushString(ah, modele[i]);

	for(i=0; i<ArraySize(ah); ++i)
	{
		r = random(ArraySize(ah)+1);
		ArrayGetString(ah, r, model_zh[random(10)], 20);
		ArrayDeleteItem(ah, r);
	}
}

Nadal występuje ten sam błąd co wcześniej, w związku z czym modele się nie zmieniają, tylko są domyślne.

1. Usuń +1 (pierwsze id=0).
2. random(11) zamień na i, inaczej wstawisz losowy model w losowe miejsce.
3. while(ArraySize(ah)) wystarczy (625).

Teraz kod wygląda w ten sposób:

public LosujModele()
{
	new i, r, s = sizeof(modele), Array:ah = ArrayCreate(20, 10);

	for(i=0; i<s; ++i)
	ArrayPushString(ah, modele[i]);

	while(ArraySize(ah))
	{
		r = random(ArraySize(ah));
		ArrayGetString(ah, r, model_zh[r], 20);
		ArrayDeleteItem(ah, r);
	}
}

Błąd już nie występuje, ale modele nadal są ustawione na domyślne, zamiast na Zombie.

new model_zh[10][21];

new modele[][] = {

"zh_corpse",

"zh_crispy2",

"zh_classic",

"zombie_big_origin",

"zombie_girl",

"zombie_leech",

"zombie_nemesis",

"zombie_poison",

"zombie_resident",

"zombie_speed"

};

public LosujModele()

{

new i, j, r, s = sizeof(modele), Array:ah = ArrayCreate(20, 10);



for(i=0; i<s; ++i)

ArrayPushString(ah, modele[i]);



while((j = ArraySize(ah)))

{

r = random(j);

ArrayGetString(ah, r, model_zh[r], 20);

ArrayDeleteItem(ah, r);

}

}

I takie uwagi:
1. Pobieraj tylko raz wielkość tablicy dynamicznej.
2. Źle tworzyłeś na początku zmienne.

OK. Trochę pozmieniałem, żeby było mi wygodniej, lecz model nadal się nie zmienia.

new Array:ah;
new modele[] = {
	"zh_corpse",
	"zh_crispy2",
	"zh_classic",
	"zombie_big_origin",
	"zombie_girl",
	"zombie_leech",
	"zombie_nemesis",
	"zombie_poison",
	"zombie_resident",
	"zombie_speed"
};

public plugin_init() 
{
	(...)
	
	ah = ArrayCreate(21, 10)
	WczytajModele()
}

public WczytajModele()
{
	new s = sizeof(modele);
	
	for(new i=0;i<s; ++i)
	ArrayPushString(ah, modele[i]);
}

public UstawModel() // Jest wykonywana co rundę
{
	new model[21], j, r;
	
	if(!(j = ArraySize(ah)))
	WczytajModele();	

	r = random(j);
	ArrayGetString(ah, r, model, 20);
	ArrayDeleteItem(ah, r);
	
	for(new i=1;i<33; i++)
	{
		if(is_user_connected(i) && get_user_team(i) == 1)
		cs_set_player_model(i, model);
	}
}

I dalej swoje...

new modele[][] = {

Tak masz tworzyć tablicę.

Dobra, moja wina. Zmianę w 1. tablicy zauważyłem, ale w drugiej już nie

Wysłane z mojego HTC Desire S przy użyciu Tapatalk 4

Wszystko działa. Do zamknięcia.

Wiadomość wygenerowana automatycznie


Ten temat został zamknięty przez moderatora.

Powód: Pomoc udzielona

Jeśli się z tym nie zgadzasz, raportuj ten post, a moderator lub administrator rozpatrzy go ponownie.


Z pozdrowieniami,
Zespół AMXX.PL