[mirkobogdan]

Witam cieplutko,
Chciałem wylosować liczby z pewnego przedziału do dwóch tablic, tak aby te liczby nie miały duplikatów w obu tablicach.
W metodzie SpawnPos1 wszystko wykonuje się ok, ale w SpawnPos2 liczby wylosowane do drugiej tablicy duplikują się ze sobą oraz z tymi z pierwszej tablicy.
Co zrobiłem nie tak w tym warunku?

#programowanie #csharp

[mirkobogdan]

@budyn: Kolega mi tak podpowiedział, sam wczesniej nie korzystałem z Array.IndexOf bo się dopiero uczę ( ͡º ͜ʖ͡º)

Ale w pierwszej medodzie działa.

[budyn]

@mirkobogdan: Daj || zamisat &&.
Szukasz kolejnej liczby jeśli jest duplikat w jednej tablicy ALBO drugiej

[Goglez]

Szukasz kolejnej liczby jeśli jest duplikat w jednej tablicy ALBO drugiej

@budyn: Na pewno?

tak aby te liczby nie miały duplikatów w obu tablicach.

[mirkobogdan]

@budyn: Własnie z || mi się to losuje w nieskończoność :(

[Goglez]

@mirkobogdan: Polecam wklejać kod na Pastebin i wrzucić tu link, jest to wygodniejsze, bo kod można u siebie łatwo skopiować i wkleić i sprawdzić co się dokładnie tam dzieje.

[mirkobogdan]

Ok, to tak wrzucę. Hmm, tylko to kod z metodami z Unity, musiałbym przerobić ( ͡° ͜ʖ ͡°)

[Goglez]

w SpawnPos2 liczby wylosowane do drugiej tablicy duplikują się ze sobą oraz z tymi z pierwszej tablicy.

@mirkobogdan: A przedział i elementy w tablicach dają możliwość znalezienia takiego elementu, jakiego oczekujesz? Może warunek jest dobry, tylko niemożliwy do spełnienia? Na przykład w tablicy masz elementy 1,2,3,4,5, z przedziału [2,4] nie wylosujesz unikalnego elementu.

[mirkobogdan]

@Goglez: Przedział jest od 0 do 18, każda z tablic ma dostać 9 elementów

[Goglez]

@mirkobogdan: A tablice są puste na wejściu? Pamiętaj, że tablice to typ referencyjny, w metodzie przekazujesz odwołanie do istniejącej tablicy, nie tworzysz nowej. Ja niedawno o tym zapomniałem i straciłem sporo czasu na szukaniu przyczyny tego, że elementy się nie zgadzały, okazało się że właśnie nie uwzględniłem tego że tablica to typ referencyjny, a dwie metody traktowały ją jak typ wartości i wzajemnie sobie szkodziły.

Hmm, tylko to kod z metodami

[mirkobogdan]

@Goglez
Zapełniłem teraz tablice początkowymi wartościami, z poza zakresu który dostaną w metodach i efekt się nie zmienił.

Próbowałem poprzeklejać te metody do pustego projektu c#, ale dostałem tyle błędów, że nie mam głowy wszystkich naprawiać, szczególnie że siedzę teraz w pracy i co innego jest moim zajęciem a nie programowanie ( ͡º ͜ʖ͡º)

Spróbuję w domu to inaczej losować.

[zibizz1]

@mirkobogdan:
1. nie wstawiasz elementów do drugiej tablicy?, albo jako druga tablice podajesz tez pierwsza
2. Jeśli to mały przedział to najpierw zrób Liste elementów a potem wyciągaj z niej losowo, będzie szybciej w przypadku losowania 18 z 18 bez powtórzeń i znacząco szybciej przy losowaniu np 100 z 100 bez powtórzeń