Administrace komentářů

Pokud jsem ten zdroják pochopil, děje se toto. Při startu programu se vytvoří dvě instance třídy Tank, které mají veřejné proměnně int x a int y, které představují počáteční polohu tanku na hrací ploše. Pro vykreslení polohy tanku používáš ITank a příslušné 4 třídy, které jej implementují, pro každou orientaci tanku jednu. Když překreslovací vlákno aktualizuje polohu tanku, volá nad TankD/L/N/P metodu setPozice() z interfacu IPosuvny. Problém je, že pokud ten zdroják čtu dobře, není tam nic, co by aktualizovalo souřadnice x,y tanku jako takového, upravuje se jen místo, kde se tank nakreslí. Detekce kolizí ale kontroluje pozici tanku oproti x,y z třídy Tank, která se po celou dobu běhu programu nemění.

Obávám se, že žádný quick fix mě nenapadá, máš to totiž celé trochu divně navržené. Nevím, zda to po vás takhle ve škole chtěli, ale mít dva totálně nezávislé objekty pro tank a pro cosi, co zařídí jeho vykreslení mi přijde dost nesmyslné.

Pár hnidopišských poznámek na závěr ;)

• Pokud chceš, aby ti kód někdo revidoval, přidej tam nějaké komentáře. Docela mi trvalo, než jsem pochopil, jak ta věc vlastně pracuje. Zejména ta detekce kolizí byla schovaná doopravdy důmyslně:)
• Zkus dodržovat nějaký coding style. Vím, že když něco urputně odmítá fungovat, je otrava zarovnávat závorky a odsazení, ale ten kód se fakt nečte moc dobře.
• Z metod typu

  public Koule pal() {
     Koule strela;
     strela = new Koule(...);
     return strela;
  }
  

  mi vypadávají vlasy, tohle totiž funguje úplně stejně dobře

  public Koule pal() {
     return new Koule(...);
  }
  

• Je doopravdy nutné mít čtyři třídy jen proto, abys vykreslil tank ve čtyřech možných orientacích? Jednak duplikuješ dost kódu, což zvyšuje riziko nějakého bugu (který budeš navíc opravovat 4x) a druhak je do dost nepřehledné (taky by to mohlo mít nehezký dopad na výkon).
  

Hezké Velikonoce, vyřešit problém sám je vždycky lepší, než si od někoho vyslechnout řešení, takže s chutí do toho:)