Author Topic: OOP  (Read 114496 times)

pietro316

  • Newbie
  • *
  • Posts: 3
    • View Profile
Re: OOP
« Reply #625 on: 16.12.2011, 20:55:58 »
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

  • Hero Member
  • *****
  • Posts: 1010
    • View Profile
Re: OOP
« Reply #626 on: 16.12.2011, 21:48:45 »
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

  • Newbie
  • *
  • Posts: 6
    • View Profile
Re: OOP
« Reply #627 on: 17.12.2011, 00:59:22 »
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

  • Jr. Member
  • **
  • Posts: 99
    • View Profile
Re: OOP
« Reply #628 on: 17.12.2011, 02:56:04 »
so far je termin na odovzdavanie zadania 21. 12. 2011, streda od 8:00 v PC1

xxx3

  • Hero Member
  • *****
  • Posts: 1010
    • View Profile
Re: OOP
« Reply #629 on: 17.12.2011, 04:41:26 »
Nie este som nezapisal  :ropebanana: ... ale uz sa to blizi! bp :emot-dance:
kedy to tam uz bude ? :D

pietro316

  • Newbie
  • *
  • Posts: 3
    • View Profile
Re: OOP
« Reply #630 on: 19.12.2011, 06:01:31 »
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

  • Jr. Member
  • **
  • Posts: 95
    • View Profile
Re: OOP
« Reply #631 on: 19.12.2011, 07: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

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

Agamemnon

  • Hero Member
  • *****
  • Posts: 2942
  • Per aspera ad Astra
    • View Profile
Re: OOP
« Reply #632 on: 19.12.2011, 16:03:20 »
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

  • Newbie
  • *
  • Posts: 37
    • View Profile
Re: OOP
« Reply #633 on: 19.12.2011, 18:30:57 »
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

  • Sr. Member
  • ****
  • Posts: 317
    • View Profile
Re: OOP
« Reply #634 on: 19.12.2011, 18: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?)

razer08

  • Newbie
  • *
  • Posts: 37
    • View Profile
Re: OOP
« Reply #635 on: 19.12.2011, 19:27:51 »
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

  • Hero Member
  • *****
  • Posts: 873
    • View Profile
Re: OOP
« Reply #636 on: 20.12.2011, 01: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  ;)
« Last Edit: 20.12.2011, 01:58:13 by esperian »

Agamemnon

  • Hero Member
  • *****
  • Posts: 2942
  • Per aspera ad Astra
    • View Profile
Re: OOP
« Reply #637 on: 20.12.2011, 01:48:46 »
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

  • Newbie
  • *
  • Posts: 33
    • View Profile
Re: OOP
« Reply #638 on: 20.12.2011, 02:29:33 »
Quote
Metó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

  • Guest
Re: OOP
« Reply #639 on: 20.12.2011, 02:35:11 »
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

  • Hero Member
  • *****
  • Posts: 2942
  • Per aspera ad Astra
    • View Profile
Re: OOP
« Reply #640 on: 20.12.2011, 03:13:15 »
Quote
Metó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

  • Hero Member
  • *****
  • Posts: 1010
    • View Profile
Re: OOP
« Reply #641 on: 20.12.2011, 04: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 ...

Eriik

  • Newbie
  • *
  • Posts: 33
    • View Profile
Re: OOP
« Reply #642 on: 20.12.2011, 18:14:36 »
Quote
Metó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ť..

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

  • Newbie
  • *
  • Posts: 8
  • http://www.arim.sk
    • View Profile
Re: OOP
« Reply #643 on: 20.12.2011, 21:42:03 »
neviete niekto poradit so save() ??? vytvorim si subor,ale z history mi don nezapise nic

Agamemnon

  • Hero Member
  • *****
  • Posts: 2942
  • Per aspera ad Astra
    • View Profile
Re: OOP
« Reply #644 on: 20.12.2011, 23:00:54 »
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

  • Newbie
  • *
  • Posts: 8
  • http://www.arim.sk
    • View Profile
Re: OOP
« Reply #645 on: 20.12.2011, 23:59:34 »
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

  • Hero Member
  • *****
  • Posts: 2942
  • Per aspera ad Astra
    • View Profile
Re: OOP
« Reply #646 on: 21.12.2011, 01:01:25 »
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

  • Full Member
  • ***
  • Posts: 231
    • View Profile
Re: OOP
« Reply #647 on: 23.12.2011, 22:51:33 »
Č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

  • Guest
Re: OOP
« Reply #648 on: 24.12.2011, 20:49:56 »
Čaute, vy čo ste odovzdávali zadanie v stredu, máte už zápočet na MAIS-e?

nie ...

zaky

  • Jr. Member
  • **
  • Posts: 95
    • View Profile
Re: OOP
« Reply #649 on: 25.12.2011, 18:43:41 »
anooo.. s tomaskom mam

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