( SzBlackY | 2014. 07. 12., szo – 21:59 )

Építő jellegű kritika, javaslatok jöhetnek?

1. Jelenleg támogatott, de csúnyán néznek miatta kategóriás, ha az AWT Thread-en kívülről dolgozol UI osztályokkal (néhány Swing Look&Feel speciel ezt egy exception-nel köszöni meg). Az ajánlás a UI kód bootstrap-jére a main()-ből a SwingUtilities.invokeLater vagy SwingUtilities.invokeAndWait hívása.

SwingUtilities.invokeLater(new Runnable() {
    public void run() {
       new WorkingWindow();
    }
});

2. Valszeg az OCD beszél belőlem, de a WorkingWindow ne jelenítse meg magát, bízd a hívó félre (new WorkingDisplay().setVisible(true);)

3. Több helyen is van, hogy int tömbbel kezelsz koordinátákat, miközben van beépített Point osztály - memóriahasználatban kb. ugyanott leszel, viszont később nem kell azonnal agyalnod, hogy melyik koordinátát melyik indexhez rendelted

4. Írod egy helyen az egérkezelést, a mostani kézzel rajzolgatóssal megoldással ezzel szerintem sokat fogsz szívni, amikor elég bátornak érzed magad, megpróbálhatsz elveszni a JTable és kapcsolódó osztályok körében (nem egy egyszerű interfész, az biztos) - addig is hagyd meg egyszerűre a WorkingPanel interfészét (setPosition/setLetter).

BlackY
--
"en is amikor bejovok dolgozni, nem egy pc-t [..] kapcsolok be, hanem a mainframe-et..." (sj)