Java - proměnné z constructoru jsou v jedné metodě prázdné – Java – Fórum – Programujte.com

Zdravím,

dělám něco na rychlo do školy a narazil jsem na jeden problém a už mně nenapadá, jak to opravit.

Zdroják: http://pastebin.com/qJCCS1CT

A problém? 

this.parsedWord, this.gameLevel, this.imagePath, this.levelTitle jsou v metodě getRandomWord() prázdné, ale nikde jinde ne. Vyprintoval jsem si ve všech metodách gameLevel a výsledek:

getRandomWord() - 0
constructor() - 1
createGame() - 1

Kde je chyba? Díky.

#1 1337
Máš chybně napsaný konstruktor. Pokud má chybný parametr, vyrobí nekonzistentní objekt.

Místo metody getRandomWord(), která do třídy Application nepatří, bys měl raději napsat třídu RandomWord. Vrací prázdný string, který je následně použit i v this.parsedWord. Vypadá to, že soubor "./img/1.txt" neexistuje (nevím jak ty, ale já ho nemám).

BTW: Trochu mě zdržela metoda createGame(), která do třídy Application vůbec nepatří. Nešlo mi to kvůli ní zkompilovat a přitom nikde není použita. Bylo by dobré už od začátku programovat objektově. Také bys měl vypakovat ven z třídy skryté závislosti - taková třída se pak nedá rozumně testovat.

#2 Kit

Textové soubory (1.txt, 2.txt, 3.txt) jsem nenahrával, jsou tam jen pod sebou slova, která já vlastně v tom getRandomWord() tahám.

Metoda createGame() je použita až v JFrame třídě.. Je to hodně prasácký, protože jsem to psal hodně na rychlo. Jinak ta metoda vrací prázdný string kvůli tomu, že jsem to zapomněl odmazat. A i kdybych to odmazal, tak to spadne (hodí to vyjímku s indexem u array, že je -1) a soubor se ani neotevře, protože právě gameLevel je prázdný -

File file = new File("./img/" + this.gameLevel + ".txt");

#3 1337
Abych ten program trochu pochopil, tak jsem si tu třídu rozdělil na samostatné třídy Application, Game a RandomWord. Doporučuji ti udělat totéž. Jinak se do toho časem zašmodrcháš ještě víc.

Jako gameLevel jsem si v konstruktoru testu nastavil hodnotu "1", aby mi to alespoň trochu fungovalo. Je mi však záhadou, proč ty 3 parametry předáváš jako pole stringů.

#4 Kit
Máš pravdu. Celý to budu muset překopat.. Udělám to tak, jak píšeš. :) Díky