Objektové programovanie

[Agamemnon]

FileStream fileStream = new FileStream(pathName, FileMode.Open);
StreamReader streamReader = new StreamReader(fileStream);
String command;
Clear();
while ((command = streamReader.ReadLine()) != null)
                {
                   list.Add(command);
                }
               
streamReader.Close();
System.Console.WriteLine("Hra bola nacitana!");


Tak dáko...

skús pozrieť direktívu using a interface IDisposable užitočné veci

btw - filestream nezatváraš?

Kuknem zajtra, teda dneska ale neskorej, teraz by som nepochopil ...a hej už zatváram

EDIT: Dig skúsil som, ako rozdiel tam "laicky" neni žiadny, ale čítam, že je "dobrou vizitkou programátora" , keď to používa pri práci so súbormi, databázami atď...Môžeš to tak laicky vysvetliť, prečo to je vlastne lepšie? veď GC by to uvolnil z pamäte aj tak nie ??

A ešte k tomu, keď to dám do using tak už tu metódu Dispose() nemusím volať hej? sa zavolá sama?

vola sa sama, nemusis volat ty...

ako - nema to ziaden rozdiel oproti tomu, ze to budes volat rucne to je obycajne volanie metody tam... ale nezabudne sa na to zvysuje to prehladnost kodu - aspon pre vacsinu ludi, co poznam... a tak podobne...
a casto je dobre zavriet resource explicitne... a napr. ked mas otvoreny velky subor a pod., tak to moze byt problem, ak si ho nezavries tam, kde to tebe vyhovuje, resp. co najskor a pod - resp. ak mas vseobecne otvoreny nejaky stream alebo connection a pod., tak je dobra prax si ho zavriet po sebe

[tino8]

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

[tino8]

Mam otazku...

Je mozne ist na skusku z OOP bez toho aby sme mali odovzdane zadanie? Lebo datum prvej skusky je hned tu (13.1) a slubovany termin na odovzdavanie este nikde... (akurat tak Novak a Kapa dali datumy ale iba pre svojich studentov...)

kebyze viem tak odovzdam v 13tom...

[Daron]

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?

[tino8]

pozri sa na moodli kedy dal termin...

[Daron]

a tato otazka??

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?

[citro]

a tato otazka??

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?

nebude genci a poriadne kukni do kodu aby si vedel co to povedat, aspon zaklady

[rebro]

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...

Pokial viem tak v jave by to malo byt import Hra.items.* a rovnako to importuje cely balicek itemov

[dEVIANT]

Nechcem byť kurva ale keď to už podľa spraveného kódu neviete vysvetliť ( a na to netreba znalosť OOP ale len angličtiny a trošku google ), tak si trošku vstúpte do svedomia...

[kubiik]

DO NICH! A TVRDOOOOO............ 

[citro]

da sa nejak jednoducho vyrobit prikaz VYMEN z uz existujucich prikazov PUT a GET ?
Chcem vymenit item z batohu do miestnosti, za item v miestnosti (ten v miestnosti je odliseny pomocou Changeable) 

[citro]

prosim pekne, kde som urobil chybu? Stale vracia ze sa neda pouzit, teda mam problem v prvej podmienke ale preco?

Code: [Select]

public GameState use(Game game) {

       
        RoomImpl gamer = (RoomImpl) game.getCurrentRoom();
        Vykres vykres = new Vykres();
        Obraz obraz = new Obraz();



        if (game.getCurrentRoom().getItem(getName())== vykres) {
        game.getBackpack().remove(obraz.getName());
        game.getBackpack().add(vykres);
        game.getCurrentRoom().removeItem(game.getCurrentRoom().getItem(vykres.getName()));

        System.out.println("Dal si mu obraz");
        System.out.println("Dostal si do batohu vykres");
        }
        else {
            System.out.println("Neda sa pouzit!");

        }
        return GameState.PLAYING;
        }
}

[Agamemnon]

da sa nejak jednoducho vyrobit prikaz VYMEN z uz existujucich prikazov PUT a GET ?
Chcem vymenit item z batohu do miestnosti, za item v miestnosti (ten v miestnosti je odliseny pomocou Changeable) 

vymen item1 item2 => put item1 + get item2

[Agamemnon]

prosim pekne, kde som urobil chybu? Stale vracia ze sa neda pouzit, teda mam problem v prvej podmienke ale preco?

Code: [Select]

public GameState use(Game game) {

       
        RoomImpl gamer = (RoomImpl) game.getCurrentRoom();
        Vykres vykres = new Vykres();
        Obraz obraz = new Obraz();



        if (game.getCurrentRoom().getItem(getName())== vykres) {
        game.getBackpack().remove(obraz.getName());
        game.getBackpack().add(vykres);
        game.getCurrentRoom().removeItem(game.getCurrentRoom().getItem(vykres.getName()));

        System.out.println("Dal si mu obraz");
        System.out.println("Dostal si do batohu vykres");
        }
        else {
            System.out.println("Neda sa pouzit!");

        }
        return GameState.PLAYING;
        }
}

to getName co tam mas v tej podmienke ti vrati co? si si isty, ze nazov toho objektu, co chces?
takze asi takto - netusim uplne presne, ako je implementovana ta kniznica a pod.:

Code: [Select]

if (game.getCurrentRoom().getItem(vykres.getName()) != null) { ... }


[citro]

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

[mirek]

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.

[mirek]

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)

[mirek]

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...

[mirek]

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... :-(

[mirek]

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?

[mirek]

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...

[mirek]

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...

[mirek]

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 :-(

[mirek]

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...

[dEVIANT]

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...

Dobrý nápad to s tým rozhraním, ale nezaoberal som sa tým, keďže tam 50 príkazov nie je :-P...keby bolo, tak potom by som to riešil určite nejako ináč, možno aj polom tých príkazov a metódou, ktorá zisti, č itam ten príkaz je. No ale rozhraním to je jednoduchšie to hej.

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

Neviem, či už aj fúkal...ale u dosť ľudí som ho videl, napríklad aj u tých s ktorými nič nemám, akurát na tomto sú ešte aj premenné a metódy také ako mám ja, tak preto viem, že je môj