[szczepi]

#programowanie #java #javafx
Robię projekt w javie. Samolot lata po pewnej trasie. Gdy dodaje drugi samolot to się zawiesza(nowy wątek). Współrzędne cały czas oblicza i wypisuje dobrze. Problem jest z rysowaniem. Wszystko oparte na wątkach.
Najczęściej wyrzuca błędy:


at java.util.ArrayList.elementData(ArrayList.java:418)
at java.util.ArrayList.get(ArrayList.java:431)
at javafx.scene.Parent.updateCachedBounds(Parent.java:1583)
at javafx.scene.Parent.recomputeBounds(Parent.java:1527)
at javafx.scene.Parent.impl_computeGeomBounds(Parent.java:1380)

[p.....n]

@szczepi: a gdzie kod?

[szczepi]

@pstjepan:
Main: http://paste.ofcode.org/ca7S83zU2wKZ4HALuxp4SU
Klasa pojazd: http://paste.ofcode.org/NwRcBW3KTFTNKPFHNTVFrK

[p.....n]

@szczepi: a co z klasą Budynek?

[kapelusz]

Ale ArrayListy nie używamy do wielowątkowości.

[szczepi]

@pstjepan: http://paste.ofcode.org/tzm88rBcJcpHjHQANhRyDw

[szczepi]

@kapelusz: dlaczego?

[p.....n]

@szczepi: nie jest 'thread safe'

[p.....n]

@szczepi: możesz najwyżej takie coś użyć:

List list = Collections.synchronizedList(new ArrayList());

[p.....n]

@szczepi: nie wiem po co tak robisz:

listaLotnisk.add(new Budynek(10,10,"pierwszeLotnisko"));
 listaLotnisk.get(0).rysuj(pane);

nie lepiej:

Budynek b = new Budynek(10,10,"pierwszeLotnisko"));
b.rysuj(pane);
listaLotnisk.add(b);

?

[szczepi]

@pstjepan: Uważasz że ta lista tworzy problem?
Z budynkiem to robi jakąś różnicę?

[p.....n]

@szczepi: możliwe. Musisz sprawdzić. Przede wszystkim uporządkuj kod, bo jest mało czytelny. Przede wszystkim nie powinieneś zakładać, że dany element będzie miał określony indeks np. 0 po dodaniu do listy - dlatego używałbym konstrukcji takiej co podałem wcześniej.

[p.....n]

@szczepi: i jeszcze jedna rzecz. Całe rysowanie musisz wykonywać w głównym wątku.
https://docs.oracle.com/javafx/2/threads/jfxpub-threads.htm