[feelfree]

#java #gamedev #libgdx #programowanie
Mirki, posiadam dwie funkcje które loopują ten sam ArrayList. Pierwsza funkcja jest podpięta pod Timer(około 25 razy na sekunde). W momencie gdy druga funkcja usuwa jeden obiekt z ArrayList'a, program wywala błąd:java.lang.IndexOutOfBoundsException: Index: 0, Size: 0
at java.util.ArrayList.rangeCheck(ArrayList.java:653)
at java.util.ArrayList.get(ArrayList.java:429)
Kod funkcji: http://pastebin.com/fMPQx1Nm

[s.....a]

@feelfree: to niezbyt dobry pomysł żeby w tym samym czasie dwa wątki jeździły po liście

[s.....a]

@feelfree: i ją modyfikowały

[feelfree]

@slucham-psa-jak-gra: Loop jest na kopi tej listy, modyfikuje po index'ach.

[feelfree]

for(OtherPlayer _otherpl : **new ArrayList(other_players)**)

[sorhu]

@feelfree:
I nie pisz if(d--a == false) czy if(d--a == true).
Pisz if(!d--a) lub if(d--a).

[sorhu]

@feelfree: A potem wewnątrz pętli odwołujesz się do oryginału, a nie do kopii.

[P.....3]

@feelfree: To nie ma znaczenia, że na kopiach bo i tak modyfikujesz tą główną, jednym zgetujesz, drugim usuniesz to co zgetowałeś wcześniej i cuda wyjdą.

[feelfree]

@Porana123: @sorhu: @slucham-psa-jak-gra:
Ograniczyłem kod do jednej pętli w tym samym czasie, niestety program nadal wywala ten błąd ( ͡° ʖ̯ ͡°)
http://pastebin.com/jLRUsmxK

[s.....a]

@feelfree: a możesz mi powiedzieć jaki to ma sens?
for(OtherPlayer other : new ArrayList(otherplayers)){

[s.....a]

@feelfree: a chyba wiem, a spróbuj może przed pętlą ją skopiować czy coś

[feelfree]

@slucham-psa-jak-gra: Właśnie ta linia jest odpowiedzialna za kopiowanie listy

[s.....a]

@feelfree: i pamiętaj, że przy kopiowaniu listy przez tworzenie nowej ze starą w konstruktorze to tworzy ci się nowa lista, która posiada te same obiekty co stara

[fegwegw]

@feelfree: To jest świetny przypadek, w którym warto zaznajomić się z działaniem debuggera.

[fegwegw]

@feelfree: Dodatkowo parę hintów:

Staraj się pisać tak, aby zachować pewnie naturalne 'flow'. Masz 2 główne 'kanał' przepływu wykonania:
a) trzeba usunąć 'packet'
b) nie

[kozunio12]

@feelfree: for(OtherPlayer otherpl : new ArrayList(otherplayers))
Przekazujesz tutaj tylko referencję do listy, a nie kopię obiektów. Musisz zrobić głęboką kopię listy, bo inaczej będziesz miał problem z referencjami do pól, które też są jakimiś obiektami.

Konwencja kodowania w Javie - zmienne + settery/gettery.

[P.....3]

@feelfree: Ogarnąłeś to? Nie wiem czym jest ten x, ale w tym kodzie tylko w 1 miejscu masz get, zrób na samym początku indexpl = -1 nie 0, i potem jak robisz te gety to zrób if(indexpl != -1) tylko wtedy rób gety, bo iniciujesz indexpl na 0 i liczysz, że ten for go zmieni a for może go nie zmienić a x z poprzedniego wywołania może być true i