OP - zadanie

[Shwollo]

Vzhladom k tomu ze niekto vymazal temu s op tak zakladam novu.

Rata sa ako vlastne rozsirenie taka vec ze napr. Pokial nepouzijem kluc tak sa mi neukaze dalsi vychod z miestnosti...?

[tino8]

ne

[tino8]

class diagram tvojho projektu... podrobnejsie sa to neda napisat

[smelyzajo]

tino8 mam otazku akym stylom si to robil v GameImpl si nastavil vychody napr. setExits(null,null,null,null)
potom v triede napr.kluc v metode use si to poriesil cez if a v setExits v tej use metode si to nastavil uz aj z vychodmi )??

[Brand]

Ak to chces urobit takto, ze znovu zavolas na objekte miestnosti setExists, tak potrebujes vediet ziskat minimalne referencie tychto miestnosti z objektu hry, pretoze volanie setExits ma zmysel iba tam, kde "poznas" miestnosti.

Riesenim by mohlo byt deklarovanie miestnosti na urovni GameImpl ako clenskych premennych, ale to by nevyzeralo dobre a istotne by PMD protestovalo a takisto cviciaci (ak by si tam deklaroval vsetky miestnosti...).

Celkom jednoduche by bolo vytvorit kolekciu vsetkych miestnosti ako clensku premennu triedy GameImpl. ArrayList<Room>, v hre je pouzitych niekolko ArrayListov, takze by si s tym nemal mat problem. V metode createRooms jednoducho kazdu miestnost vlozis do tohto zoznamu, na zoznam spravis nejaky getter a tym padom mas pristup ku referenciam vsetkych miestnosti z hocikadial z kodu, ktory ma pristup ku referencii hry. Resp. priamo nejaka metoda public Room getRoomByName(String name) {}, ktora by vratila uz konkretnu miestnost na zaklade mena. Takto dokazes ziskat kazdu miestnost a tym padom dokazes pouzit volanie setExits aj mimo createRooms.

[Safyia]

hm... ja som to riesila tak, ze som si meistnost, ktoru mal predmet otvarat dala do parametra konstruktoru... Ak som potrebovala vediet, v ktorej meistnosti sa s predmetom nachadzam, tak som dala dva parametre.
Napr Key(Room room). Pri vytvarani noveho kluca tam potom das len nazov, kedze ten zoznam miestnosti mas tiez v GameImpl, kde vytvaras aj predmety. A potom len v metode use zavolas napr setExits(blabla.getNorth(), blabla.getSouth(), room, blabla.getWest()) . And it is!

[Brand]

Celkom napadite

[Shwollo]

čo mám robiť s týmto?    
Pretypujte deklaráciu (namiesto typu 'ArrayList' použite rozhranie, ktoré tento typ implementuje).

mám tam toto:

Code: [Select]

private ArrayList<Item> backpackItems = new ArrayList<Item>();

[revelc]

private List<Item> backpackItems = new ArrayList<Item>();

[Shwollo]

Metóda obsahuje viac ako 99 príkazov. Skúste redukovať jej veľkosť vytvorením pomocných metód.

toto môže zostať?

[revelc]

moze ak mas tolko miestnosti 

[ondrej0071]

musi byt alebo moze byt na zaciatku to menu?

[freshmakerik]

v tom spominanom vymazanom topicu bol aj navod na pmd, ak ho niekto ma, tak ho tu prosim hodte, diky 

[Shwollo]

v netbeanse - Files - adventure - buils.xml - pmd - klikneš pravým a dáš run

[ondrej0071]

PMD mi zahlasilo chybu: odstrante nepouzivanu lokalnu  premennu path  ...v prikaze SAVE v metode execute()

 public GameState execute(Game game) {
        String path;
        path = this.getParams();
        try {
            History.getInstance().save();
        } catch (IOException ex) {
            Logger.getLogger(History.class.getName()).log(Level.SEVERE, null, ex);
        }
        return GameState.PLAYING;
    }

ibaze tato premenna path je vyuzita,cize nechapem preco mi hadze chybu 

[ondrej0071]

inak..dakujeme tomu kto zmazal tu predchadzajucu diskusiu OP..nemohol radsej FJAP 

[chaser]

PMD mi zahlasilo chybu: odstrante nepouzivanu lokalnu  premennu path  ...v prikaze SAVE v metode execute()

 public GameState execute(Game game) {
        String path;
        path = this.getParams();
        try {
            History.getInstance().save();
        } catch (IOException ex) {
            Logger.getLogger(History.class.getName()).log(Level.SEVERE, null, ex);
        }
        return GameState.PLAYING;
    }

ibaze tato premenna path je vyuzita,cize nechapem preco mi hadze chybu  

nevyuzivas ju nikde
iba si si ju zadefinoval a konec
ci daco nevidim ?

[Shwollo]

mám to dobre?

[Makaveli]

bude velmi vadit ak nemam triedy GameImpl a RoomImpl a mam to vsetko v main???

[Buici]

mozte napisat ako vytvorim tie diagramy???

[Agamemnon]

bude velmi vadit ak nemam triedy GameImpl a RoomImpl a mam to vsetko v main???

áno, lebo to nie je objektové... etc...

[dEVIANT]

mám to dobre?

Vyzerá to ok. Akurát pre Commandy by si mal mať abstraktnú triedu, ale zas v podmienkach to nieje, keď sa nemýlim.

[ondrej0071]

musi byt na zaciatku to uvodne menu??? 

[smelyzajo]

ako to poriesit ked mi pmd hadze chybu typu:a method should have only one exit point,and that should  be the last statement in the method.treba to rozdelit potom na viac metod ak mam viac returnov ak dobre chapem tak metoda moze mat iba jeden return???

[revelc]

musi byt na zaciatku to uvodne menu??? 

musi