Diskuze – Program pre správu kontaktov v Javě
ZpětUpozorňujeme, že diskuze pod našimi online kurzy jsou nemoderované a primárně slouží k získávání zpětné vazby pro budoucí vylepšení kurzů. Pro studenty našich rekvalifikačních kurzů nabízíme možnost přímého kontaktu s lektory a studijním referentem pro osobní konzultace a podporu v rámci jejich studia. Toto je exkluzivní služba, která zajišťuje kvalitní a cílenou pomoc v případě jakýchkoli dotazů nebo projektů.
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.
Zobrazeno 5 zpráv z 5.