• Welcome to TUKE FÓRUM - Fórum pre študentov Technickej Univerzity v Košiciach.
 

OOP

Started by NDK, 22.09.2011, 21:02:52

« predchdzajce - alie »

pietro316

Quote from: MartinK on  16.12.2011, 21:09:29
Nie este som nezapisal  :ropebanana: ... ale uz sa to blizi! bp :emot-dance:

Bude sa dat buduci tyzden este odovzdat zadanie? Lebo nietkori cviciaci este budu preberat zadania aj buduci tyzden.

xxx3

Quote from: MartinK on  16.12.2011, 21:09:29
Nie este som nezapisal  :ropebanana: ... ale uz sa to blizi! bp :emot-dance:

bolo by na case, chcel by som sa prihlasit totiz na nejaky normalny termin na skusku  :ropebanana:

MartinK

Quote from: pietro316 on  16.12.2011, 21:55:58
Quote from: MartinK on  16.12.2011, 21:09:29
Nie este som nezapisal  :ropebanana: ... ale uz sa to blizi! bp :emot-dance:

Bude sa dat buduci tyzden este odovzdat zadanie? Lebo nietkori cviciaci este budu preberat zadania aj buduci tyzden.

Odporucal by som sledovat Moodle, mam take tusenie, ze na dalsi tyzden sa vypise uz jeden termin na hromadne odovzdavanie zadani.

milaninho

so far je termin na odovzdavanie zadania 21. 12. 2011, streda od 8:00 v PC1

xxx3

Quote from: MartinK on  16.12.2011, 21:09:29
Nie este som nezapisal  :ropebanana: ... ale uz sa to blizi! bp :emot-dance:
kedy to tam uz bude ? :D

pietro316

Quote from: milaninho on  17.12.2011, 03:56:04
so far je termin na odovzdavanie zadania 21. 12. 2011, streda od 8:00 v PC1

Ten termin je len pre ludi od Tomaseka? Alebo aj pre ludi od kapa?

zaky

zdar, mam pruser, mam vela prikazov ktore mozu nieco robit s predmetmi, to znamena ze ak zistujem co je na vstupe mam dost dlhy if (param.startsWith("oslov")||param.startsWith("sprav si")||param.startsWith("hod")) atd.. a teraz mi pmd stale vypisuje chybu ze
Znížte počet: 'if', 'while', 'for', 'case' príkazov v rámci 'method processCommand' (zložitosť je 11 z 10).
co s tym ? neviem ci toto mu vadi ze mam vela moznych prikazov ale skusil som zmazat 2 prikazy a uz mi tu chybu nevypisovalo, jak to mam obist? nemozem 2 prikazy vynechat, potrebujem ich :/
skuste mi helfnut

92% teenagerů neposlouchá svojí maminku. Patříš-li mezi zbývajících 8%, dej si tohle do podpisu

Agamemnon

Quote from: zaky on  19.12.2011, 08:26:19
zdar, mam pruser, mam vela prikazov ktore mozu nieco robit s predmetmi, to znamena ze ak zistujem co je na vstupe mam dost dlhy if (param.startsWith("oslov")||param.startsWith("sprav si")||param.startsWith("hod")) atd.. a teraz mi pmd stale vypisuje chybu ze
Znížte počet: 'if', 'while', 'for', 'case' príkazov v rámci 'method processCommand' (zložitosť je 11 z 10).
co s tym ? neviem ci toto mu vadi ze mam vela moznych prikazov ale skusil som zmazat 2 prikazy a uz mi tu chybu nevypisovalo, jak to mam obist? nemozem 2 prikazy vynechat, potrebujem ich :/
skuste mi helfnut

vadi mu, ze mas vela if-ov... ti to tam aj pise...
urob mensi pocet ifov... pripadne pastni kod a mozno ti poradim, ze ako
All code is crap.

42

razer08

Urobil som junit test takto:
    @Test
    public void testGetCommand() {
        List<String> listCommands = Arrays.asList("SEVER", "juh",
        "   VYCHOD", "zapad   ", "rozhliadni SA", "inVENtar", "KONIEC",
        "VEZMI", "POLOZ", "POUZI", "PRESKUMAJ", "PRIKAZY", "NACITAJ",
        "ULOZ", "VERZIA", "RESTART" );
        ParserImpl instance = new ParserImpl();
        Command expResult = null;
        for(String key: listCommands){
            assertNotNull(instance.getCommand(key));
        }
    }

ale vzdy mi hadze chybu:
No test passed, 1 test failed
adventure.ParserImplTest FALED
testGetCommand Failed:junit.framework.AssertionFailedError

kto by mi vedel pomoct?

Brand

Ze test pada, neznamena nutne, ze je zly, skor to znamena, ze splnil ucel a nesiel chybu :).

Daj si na riadok

assertNotNull(instance.getCommand(key));

breakpoint a skus to trosku podebugovat, na zaciatok len prejdi ten cyklus, aby si videl, pri ktorom prikaze Ti to pada, mozno nemas osetreny niektory pripad (case sensitive? trim?)

razer08

Quote from: Brand on  19.12.2011, 19:50:56
Ze test pada, neznamena nutne, ze je zly, skor to znamena, ze splnil ucel a nesiel chybu :).

Daj si na riadok

assertNotNull(instance.getCommand(key));

breakpoint a skus to trosku podebugovat, na zaciatok len prejdi ten cyklus, aby si videl, pri ktorom prikaze Ti to pada, mozno nemas osetreny niektory pripad (case sensitive? trim?)

diky...mal som iny problem...toUpperCase som robil vTextUI v loop-e a v getCommand-e som nemal toUpperCase a kvoli tomu neakceptoval velke pismena...akoze hra uz akceptoval len v getCommand-e mi neurobil toUpperCase :) uz len backpackimpltest mi chyba

esperian

#636
ahojte, tak trocha sa bavkam v jave a chcel by som sa spytat mozno niekto skusal nieco podobne.. nasiel som kniznicu java.util.Calendar; no a teraz by som chcel spravit nieco take, ze získam si koľko je práve sekúnd int sec=cal.get(Calendar.SECOND); , no a teraz by som chcel aby sa mi ten cas prekresloval na výstupe .. ze 8 potom by tam bolo 9 ..10.. atd.. len neviem ci to je aj realne s mojimi znalostami, ale tak aspon skusam xDDD
---
dik  ;)

Agamemnon

Quote from: esperian on  20.12.2011, 02:47:01
ahojte, tak trocha sa bavkam v jave a chcel by som sa spytat mozno niekto skusal nieco podobne.. nasiel som kniznicu java.util.Calendar; no a teraz by som chcel spravit nieco take, ze získam si koľko je práve sekúnd int sec=cal.get(Calendar.SECOND); , no a teraz by som chcel aby sa mi ten cas prekresloval na výstupe .. ze 8 potom by tam bolo 9 ..10.. atd.. len neviem ci to je aj realne s mojimi znalostami, ale tak aspon skusam xDDD


http://docs.oracle.com/javase/1.3/docs/api/java/util/Timer.html
All code is crap.

42

Eriik

QuoteMetóda obsahuje viac ako 99 príkazov. Skúste redukovať jej veľkosť vytvorením pomocných metód.
Je to vážne nutné ? nechce sa mi teraz prerábať celú metódu createRooms() ...

ppeess

Quote from: esperian on  20.12.2011, 02:47:01
ahojte, tak trocha sa bavkam v jave a chcel by som sa spytat mozno niekto skusal nieco podobne.. nasiel som kniznicu java.util.Calendar; no a teraz by som chcel spravit nieco take, ze získam si koľko je práve sekúnd int sec=cal.get(Calendar.SECOND); , no a teraz by som chcel aby sa mi ten cas prekresloval na výstupe .. ze 8 potom by tam bolo 9 ..10.. atd.. len neviem ci to je aj realne s mojimi znalostami, ale tak aspon skusam xDDD
---
dik  ;)
Preco to chces robit takto?????? :)

Agamemnon

Quote from: Eriik on  20.12.2011, 03:29:33
QuoteMetóda obsahuje viac ako 99 príkazov. Skúste redukovať jej veľkosť vytvorením pomocných metód.
Je to vážne nutné ? nechce sa mi teraz prerábať celú metódu createRooms() ...

však len zober skupinu príkazov, ktorá robí nejakú zmysluplnú vec a exportni ich do zvláštnej metódy... ja nvm... createRoomsInFirstBuilding...
All code is crap.

42

xxx3

nam Kapa vravel, ze v createrooms ked bude PMDcko hlasit chybu o pocte riadkov, ze je prilis rozsiahla, tak ze tato chyba nebude vadit ...

Eriik

Quote from: Agamemnon on  20.12.2011, 04:13:15
Quote from: Eriik on  20.12.2011, 03:29:33
QuoteMetóda obsahuje viac ako 99 príkazov. Skúste redukovať jej veľkosť vytvorením pomocných metód.
Je to vážne nutné ? nechce sa mi teraz prerábať celú metódu createRooms() ...

však len zober skupinu príkazov, ktorá robí nejakú zmysluplnú vec a exportni ich do zvláštnej metódy... ja nvm... createRoomsInFirstBuilding...
čo sa dalo tak som takto spravil .. zvyšok by si žiadal rozsiahlejšie zmeny a to sa mi už vôbec nechce robiť..

Quote from: xxx3 on  20.12.2011, 05:01:31
nam Kapa vravel, ze v createrooms ked bude PMDcko hlasit chybu o pocte riadkov, ze je prilis rozsiahla, tak ze tato chyba nebude vadit ...
super, tak dík :)

NightWalker

neviete niekto poradit so save() ??? vytvorim si subor,ale z history mi don nezapise nic

Agamemnon

Quote from: NightWalker on  20.12.2011, 22:42:03
neviete niekto poradit so save() ??? vytvorim si subor,ale z history mi don nezapise nic

tak niečo robíš zle...












-------

pastni kód, napíš čo robíš, etc... si myslíš, že viem čítať myšlienky alebo kód bez toho, aby som ho videl?
All code is crap.

42

NightWalker

v tride save som si vytvoril
    public void execute(Game game, String path) {       
        try {
           History.getInstance().save(path);
        } catch (IOException ex) {
            //Logger.getLogger(Save.class.getName()).log(Level.SEVERE, null, ex);
            System.out.print(ex);
        }       
          game.setGameState(GameState.PLAYING);
    }

malo by to sluzit na ulozenie

a v triede history ulozenie array listu do suboru

public void save(String path) throws IOException{
        String subor = path.trim();
        if (subor.length() == 0) { subor = "save.txt"; }

         File aFile = new File(subor);
    aFile.createNewFile();
    try {
    LibraryMethods.save((ArrayList<String>) this.historyZoznam, subor);
    } catch (Exception ex) {
        System.out.println(ex.getMessage());
    }
    System.out.println("Game Saved !!");

    }

Agamemnon

odhadom... nemáš ten súbor vytvárať... pretože tak, ako ho ty vytvoríš, tak súbor ostane otvorený, a potom tá metóda save z LibraryMethods, keď sa ho pokúsi vytvoriť znovu, tak nemá šancu... nie je to síce v javadoc špecifikované, ale tak nejak to z toho, čo tam je napísané, vyplýva...
pastni akú ti to píše exception... keď už tam máš ten výpis chyby urobený

btw... handling resources... každý súbor, ktorý otvoríš, máš za sebou aj zavrieť...
All code is crap.

42

excellent

Čaute, vy čo ste odovzdávali zadanie v stredu, máte už zápočet na MAIS-e?
F5 can be so refreshing :thumbs-up:

darkoooooo

Quote from: excellent on  23.12.2011, 23:51:33
Čaute, vy čo ste odovzdávali zadanie v stredu, máte už zápočet na MAIS-e?

nie ...

zaky

anooo.. s tomaskom mam

92% teenagerů neposlouchá svojí maminku. Patříš-li mezi zbývajících 8%, dej si tohle do podpisu