Messages

stale nesu ziadne info ohladom skusky?

su - bude zajtra

kedy a hlavne KDE sa u Mirka dá zajtra odovzdať?

tak jak dnes - od 9:00 v b515. ale dnes to u binasa veru nebolo nic moc...

Musi aj unit test parsera byt podla pravidiel pre PMD?
nemozem sa totiz zbavit chyby Avoid catching NullPointerException; consider removing the cause of the NPE

samozrejme

Ked mi nikto nevie dat odpoved ci su potrebne tie unit testy tak som ich zacal robit...Skoncil som ale pri teste GameImpltest. Chcem sa opytat ako treba testovat tuto triedu? Pretoze ak tam zacnem volat metodu play(); tak to skonci nullom pre line, pretoze tato metoda ocakava vstup a ja mu ho neviem ako dat...

Vedel by mi niekto poradit???

tento unit test nerob - ten netreba. zamer bol, ale nezdaril sa :-(

kod history

Code [Select] Expand


public class History {
    private static History history;
    private ArrayList<String> list;

    public static History getInstance(){
        if( history == null ){
            history = new History();
        }

        return history;
    }

    private History(){
        this.list = new ArrayList<String>();
    }

    public void add( String line ){
        this.list.add(line);
    }

    public void clear(){
        this.list.clear();
    }

    public void save( String path ) throws IOException{
        LibraryMethods.save(this.list, path);
    }

    public void load( String path ) throws FileNotFoundException, IOException{
        this.list = LibraryMethods.load(path);
    }

    public int size(){
        return this.list.size();
    }

    public String get( int index ){
        return this.list.get(index);
    }

}


ak treba nieco zmenit v LOAD tak tu je kod neviem kde by to mohlo byt

Code [Select] Expand

public class Load implements Command{
    private String params;

    public GameState execute(Game game) {
        if( this.params == null || this.params.isEmpty() ){
            this.params = "history";
        }

        History history = History.getInstance();
        try {
            history.load(params.toLowerCase());
        } catch (FileNotFoundException ex) {
            System.out.println(ex.getMessage());
            return GameState.PLAYING;
        } catch (IOException ex) {
            System.out.println(ex.getMessage());
            return GameState.PLAYING;
        }

        return GameState.LOADEDPOSITION;
    }

    public void setParams(String params) {
        this.params = params;
    }

    public String getParams() {
        return this.params;
    }

    public String getName() {
        return "NAHRAJ";
    }

    public String getDescription() {
        return "Prikaz nahra ulozeny stav hry z disku";
    }

}


nuz - poslal si sice vsetko mozne, ale to podstatne veru nie. preskumame pri odovzdavani, ze kde je chyba inac - veeeelmi pomaha projekt krokovat pri vykonavani... (debug)

ako rozsirenie hry staci ked mam v hre osoby, da sa s nimi pokecat, pri dobrej odpovedi daju predmet, alebo ak im dam predmet tak mi daju iny? ta vymena ale nema samostatny prikaz je realizovana cez "pouzi"

ano - staci, ak na pokecanie sa pouziva specialny prikaz, predmety (itemy), s ktorymi sa da pokecat, su nejako specialne oznacene (rozhranim? premennou? metodou?) a nie je to len o tom, ze po zadani prikazu pouzi sa na obrazovke objavi spominany dialog a pouzije sa vec - toto nie je rozsirenie.

A nevie teraz ujo Binas napisat, ze kedy bude mozne odovzdat zadanie, ak teraz v pondelok nemozem? Bude nejake dalsie hromadne odovzdavanie, alebo potom uz iba osobne so svojim cviciacim? Je mi jasne, ze pridem o jeden termin na skusku, ale co uz...

pravedpodobne v utorok

takyto mail mi poslal moj oblubeny cviciaci:

dobry vecer

verim, ze vsetci uz mate svoje hry hotove a vasi spolubyvajuci/rodinni prislusnici/priatelia/priatelky ich hratelnost uz dokladne otestovali. takze je na case, aby ich dokladne otestovali aj vasi oblubeni cviciaci. spolocne sa preto pri tejto prilezitosti stretneme v pondelok v case od 9:00 (predbezne) v miestnosti B515. takze nezabudnite si este raz prezriet, ci mate pred odovzdanim splenene vsetky nalezitosti a pridte sa pochvalit tym, co ste sa pocas semestra naucili. radi sa na to pozrieme spolu s vami

za tim objektovych cviciacich sa na vas uz teraz tesi

miroslav binas

juhuuuuuuuu, fungujeeeee  no nechapem preco ma to nenapadlo...dakujem 

tusim preto, lebo si to asi chcel povodne porovnavat s nazvom predmetu a nie s referenciou na predmet. nuz - myslienka bola sice spravna, ale...

kedy dal novak termin?? a aky je pri dodatocnom odovzdavani na studentov, ktory si zadanie zadovazili inak, ako vlastnorucne nakodili?? daljme tomu kupili??
Bude ako genci, ze bude mat tazko vpici, alebo bude robit problemy?

taki studenti si to zopakuju na rok. ako ja :-(

ludia... nejde mi to PMD vypisuje mi:

'java' is not recognized as an internal or external command, operable program or batch file

Vie niekto vo co de? 

A máš aj JAVU nainštalovanú? nie len netové fazole

Instaloval som JDK with NetBeans, v ProgramFiles mam adresar Java v nej jdk1.6.0_17 a jre6 takze myslim ze ano

2tino8: ano zadavam to spravne, dokonca aj ked som uz v pmd/bin a zadam iba pmd tak mi vypise to iste (malo by vypisat ak pmd pouzivat)

skus jdk-4.2.5

to mu asi nepomoze. nema nastavenu cestu ku jave, aby ju mohol volne spustit z prikazoveho riadku. joj deti...

netusim o com tocite...
nemate na mysli import TextGame.LibraryMethods;  ?

a pouzitie je tak jednoduche:  LibraryMethods.save(commands, path);
pricom treba vychytat exception

My točíme o C#  ...tam nie je import ale using...a dáva sa tam celý TextGame, alebo celý Hra.items, čo je dosť výhoda, lebo tam nemáš 10 importov jednotlivých predmetov, ale len import priečinku...

v jave sa da zapisat napriklad aj toto:
import commands.*;

a hadam netreba rozpitvavat, ze aky je toho zapisu vyznam...

public void load(String path) throws FileNotFoundException, IOException {
   BufferedReader in = new BufferedReader(new FileReader(path));
   list.clear();
   String line;
   while ((line = in.readLine()) != null) {
   list.add(line);
   }

in.close();
}

PLS vie mi to niekto prerobit do C#? prosiiiiim 

a preco? sak metoda Load() je uz implementovana. na co ju chces robit znova?

no mozno aj hej,koho sa spytam nikto nechce pomost ci uz ked som chcela s pmd pomost,nakoniec som to s kamosom nejak upchala do netbeansu a preslo mi to v pohode,ale nejako ludia su neochotni pomost,ale pomoc by chceli vsetci :/ a tebe dakujem za ochotu kiezby to rano slo....a sak binas si to nespusti aj sam?>

nebude - on je na to moc lenivy. aj vlani som musel robit vsetko pred nim sam... :-(

ano v pricinku lib mam tu kniznicu....jar a v pricinku bin,kde mam priecinok ten lib mam tu moju hru blabla.jar ktoru spustam
a tsale mi vyhadzuje to java.lang.noclassdeffound........

Ta potom neviem...iď spať, ráno vstaneš a pojde to...

Taaak toto je riadne kruty Parser. dEVIANT, vsak by si ho uz teraz takto, po tych par hrach nerobil

Furt ho tak robím...

public Command getCommand(String ret){
         Command navrat = null;
                for(Command command : list)
        {
           
            if(ret.startsWith(command.getName()))
            {
                History historyObjekt = History.getInstance();
                historyObjekt.addP(ret);
                if(ret.startsWith("PRESKUMAJ")
                        || ret.startsWith("POUZI")
                        || ret.startsWith("POLOZ")
                        || ret.startsWith("VEZMI")
                        || ret.startsWith("ULOZ")
                        || ret.startsWith("NAHRAJ"))

                {
                    parameterPrikazu = ret.substring(command.getName().length()).trim();
                    command.setParams(parameterPrikazu);

                }
               
                navrat = command;
                }
          }

        return navrat;
    }

šak pekný je...či vieš o lepšom spôsobe?...

fiha - a ked tam budes mat tych prikazov s parametrom 50? nebolo by lepsie dat prikazom s parametrom nejakym sposobom znacku, ze ho maju? premenna v abstraktnej triede? prazdne rozhranie? ked si sa uz vydal cestou, ze len istej sade prikazov budes podhadzovat dalej parameter...

public Command getCommand(String ret) {                                     
    Command navrat = null;
    for (int i=0; i<list.size(); i++) {

        boolean pravdaOK=ret.startsWith(list.get(i).getName());           
        if (pravdaOK) {                                                     
            History objekt = History.getInstance();
            objekt.addP (ret);
            if(ret.startsWith("PRESKUMAJ")
            || ret.startsWith("POUZI")
            || ret.startsWith("POLOZ")
            || ret.startsWith("VEZMI")
            || ret.startsWith("ULOZ")
            || ret.startsWith("NAHRAJ")
            || ret.startsWith("ROZPRAVAJ"))
            {
                int premennac = list.get(i).getName().length();                         
                param = ret.substring(premennac).trim();                             
                list.get(i).setParams(param);
            }
                navrat = list.get(i);
        }
    }
    return navrat;
}

Toto je moj PARSER........potreboval by som poradiť ako môžem zmeniť to s "ret.startsWith......" lebo mi pritom cez PMD vyhadzuje "Cyclomatic Complexity" skúšal som to už všelijako meniť ale neviem si s tým poradiť   

Tak po prvé...to je môj Parser Ktovie ako sa dostal k tebe, síce dosť hier som robil :D a oprav to na ...

...dobre vediet, odkial vietor fuka... (alebo len bude fukat? :-P)

EDIT: Otázka na mireka ešte raz, ako to bude s pomerom príklady/ teória na skúške?a typy otázok?

tak pol na pol.

v c# sa to da dat do switchu... v jave netusim... c# povoluje vo switchi testovat aj stringy (ai.)...

oks... ak to chapem spravne, tak chces ku miestnosti vypisat jej help?
predpokladam, ze mas objekt Room alebo aky, kde je aj nazov a aj help? nedaju sa roomy ulozit do dictionary a potom pristupovat ku nim pomocou kluca v dict (ktorym bude napr. nazov)? alebo urobit keyedcollection, co je vlastne skoro to iste?

No help chcem vypisat len k nejakym miestnostam, ktore som si zistoval cez if a nasledne som k nim vypisal hlasku (pomoc) (a v niektorych if-och som mal aj ci je napriklad predmet uz v miestnosti/batohu a ak ano tak k tomu sa uz vypisala ina hlaska(pomoc)).

Momentalne to skusam urobit cez "List", do ktoreho ukladam booleany z vyhodnoteni o miestnostiach...Potom budem tento zoznam prechadzat forom a ked najde true, tak sa vypise pomoc...Moze by aj takto? Alebo to robim uplne zle?

na jeden problem existuje niekolko rieseni. da sa napriklad aj tak, ze ked vytvaras miestnosti, tak mozes opat pretazit konstruktor o jeden parameter naviac, ktorym bude dotycny help. mozes pridat jednu metodu do triedy RoomImpl napr. getHelp() a v pripade potreby ju volat. bud ti vrati hlasku zadanu konstruktoru alebo null, ked help nie je.

ale tu ma napada nieco ine - co ked danu ulohu v miestnosti splnis? nemala by hlaska zmiznut? lebo zatial riesis iba to, ako ju dostat.

hmm, ja som napísal taký strašný kód ako je tá knižnica? omg

to ani nehovor nahlas, lebo ja som to po tebe nekontroloval. a to sme uz jeden problem odstranovali :-P

taky mam zdrojak tej mapy.....len to rozhranie mi niejak nejde...

mam taky dojem, ze kedze ti nefacha to rozhranie Storeable, tak sa snazis implementovat nieco, co si videl u niekoho ineho. inac si to vysvetlit proste neviem. jednoducha rada je, ze zmaz tu informaciu, v ktorej sa hovori, ze trieda implementuje rozhranie Storeable. a cuduj sa svete - ono to pojde... (ak moje slepe oko videlo spravne...)

nevie niekto ako sa robi rozhranie Storeable.....lebo to chcem použit na zobrazenie mapy a niejak sa mi to nedari......

hmm... zeby

Code [Select] Expand


interface Storeable{
}


?

a este mi PMD hlasi jednu chybu, ale neviem co s nou.. chyba: Avoid really long methods.  vypisuje mi to pri metode createRooms kde mam zadefinovane tie miestnosti, vychody a predmety v miestnostiach. neviem ako mam skratit tu metodu. dost blba hlaska 

no - pokial sa jedna o porusenie pravidla ExcessiveMethodLength z pravidiel Code Size, tak tam je predvoleny limit nastaveny na 100. kolko tam mas teda riadkov ty?

tak mam tam 110 riadkov

hmm... skus poslat tu metodu binasovi. nech ti povie, co s tym. ze ci nahodou nevie poradit, ci sa tam neda nieco skratit. zvykne aj odpovedat (obcas).

a este mi PMD hlasi jednu chybu, ale neviem co s nou.. chyba: Avoid really long methods.  vypisuje mi to pri metode createRooms kde mam zadefinovane tie miestnosti, vychody a predmety v miestnostiach. neviem ako mam skratit tu metodu. dost blba hlaska 

no - pokial sa jedna o porusenie pravidla ExcessiveMethodLength z pravidiel Code Size, tak tam je predvoleny limit nastaveny na 100. kolko tam mas teda riadkov ty?

staci ak budem mat v hre rozhrania Moveable a Useable ? ci treba este daco ?

Treba aj jedno vlastné: {zdroj moodle }
#  implementovať aspoň jedno ďalšie rozšírenie hry - b u ď  z uvedených alebo vymyslieť svoje vlastné
# v hre navrhnúť a použiť aspoň jedno vlastné rozhranie

Podľa mna stačí moveable, useable...

samozrejme ze nestaci. moveable a usable su predpisane rozhrania a nie vlastne - vami vymyslene...

ako sorry, len som chcel poradit, nic v zlom, je to celkom jednoduche a pochopitelne riesenie.

ano - to tvoje riesenie je najelegantnejsie. v podstate v tej hre nenastane situacia, ze budes potrebovat zmazat viac predmetov - ci uz z batohu alebo z miestnosti. teda - nastane vtedy, ked budes pouzivat dva predmety (alebo viac), ktore po pouziti zmiznu. ale je to len o tom, ze zavolas metodu remove() dvakrat - raz pre jeden predmet a druhykrat pre druhy predmet. takze presne ako to riesis - v momente, ked prides na to, ze si dotycny predmet nasiel, netreba uz dalej zoznam iterovat a zbytocne zabavat procesor s niecim, co uz netreba riesit, ale staci ho uz len ukoncit (v tomto pripade vratit jeho referenciu)...