Diskusia – Program pre správu kontaktovať v Jave
SpäťUpozorňujeme, že diskusie pod našimi online kurzami sú nemoderované a primárne slúžia na získavanie spätnej väzby pre budúce vylepšenie kurzov. Pre študentov našich rekvalifikačných kurzov ponúkame možnosť priameho kontaktu s lektormi a študijným referentom pre osobné konzultácie a podporu v rámci ich štúdia. Toto je exkluzívna služba, ktorá zaisťuje kvalitnú a cielenú pomoc v prípade akýchkoľvek otázok alebo projektov.
David Hartinger:12.8.2014 21:43
Komentář se smazal protože obsahoval vulgarismy, ne proto, že obsahoval
kritiku 
Ahoj,
dobrý začátek, ale kód má mouchy. Začneme samotným středem dění -
třídou Osoba, tam sice není vyložená chyba, ale funguje (hodně ti to
ušetří čas) následující kód.
private String meno, priezvisko, skype, email, mobil;nemusíš tedy to pokaždé vypisovat. Další v pořadí budou OsobyList.
máš tam několik importů, které nepoužiješ. Netbeans ti je podtrhne žlutě a pod žárovičkou nabídne možnost Remove unsed import, nebo Remove all unused imports, která smaže všechny. (viz. obr) Konstruktor máš prázdný, je tam zbytečný, každá třída v Javě má výchozí konstruktor, který je v tomto případě úplně stejný jako ten tvůj prázdný, proto je to tam zbytečné.
metoda uloz a nacitaj všechno vypisuje, pro uchovávání objektů je lepší použít XmlSerializaci a XmlDeserializaci. Serializací a deserializací by se ti výrazně zkrátili metody uloz a nacitaj. Zde na ITnetowrk bohužel zatím není článek o XML serializaci a deserializaci. využít můžeš třeba <a href="http://xstream.codehaus.org/tutorial.html">tohoto tutorialu</a>
Na formuláři PridatJFrame máš všechny textové políčka špatně pojmenované, neměl by se menoJLabel jmenovat menoJTextField, když je to instance JTextField? Tlačítko nemáš pojmenované, v kódu nemusí být na první pohled jasné co jButton1 dělá, lepší by byl třeba pridatJButton. Zvýrazněné to máš na obrázku 2.
Na hlavním formuláři nemáš žádný prvek pojmenovaný, v kódu vůbec nepoznáš co dělá jButton1, jButton2 a jButton3, u jScrollPane1 je to celkem jedno, protože ten prvek je v aplikaci 1×.
Formuláři DetailJFrame předáváš metodou vypis, lepší je to předat v konstruktoru, zbavíš se té metody, která může být třeba v tom objektu soukromá (private) a žádná jiná třída o ni nemusí vědět, protože o ni vědět nemají. O vypisovaní na formulář se má starat dotyčný formulář, ne nějaké jiné okno nebo nějaká jiná třída.
Formulář jsi předpokládám vytvářel v designéru čemuž odpovídá i kód.
Zatiaľ nikto nevložil komentár - buď prvý!