Hirdetés

Új hozzászólás Aktív témák

  • WonderCSabo
    félisten

    Sziasztok!

    Írtam egy nagyon alap Black Jack-et JAVA-ban. Biztos, hogy profi szemmel nézve gányolás lehet néhány helyen a kód. Valaki nem nézne bele és mondaná el, hogy mi az ami nagyon nem jó és, hogy min kellene változtatni, csak hogy tanuljak belőle. :K

    Letöltések:

    Forrás / JAR

    Két java fileból áll. Egy BlackJack.java (JFrame), itt semmit nem csinálok szinte és egy Asztal.java-ból:

    Forrás (pastebin)

    Megjegyzés: Van benne egy ékezetes függvény, az azóta javítottam. Gondolom nem célszerű használni.

    Néhány kép:




    Átfutottam a kódot. fordfairlane megjegyzése teljesen helytálló, azt kell mondjam. Gyakorlatilag a teljes programért az Asztal osztály felel. Nem túl szép, hogy a GUI, az eseménykezelést és az üzleti logikát mind egyetlen osztályba szuszakolod bele. Először is szét kéne választani a GUI-t és a logikát. Erre lehet használni mindneféle architektúrát, pl. MVC-t de az se kell ide, elég legyen annyi, hogy a különböző Swing-es leszármozott osztályaid nem tartalmaznak logikát, továbbá a logikáért felelős osztályok nem is tudnak a GUI-ról. Továbbá az egész kódot lehetne dekomponálni, modulárisabbá tenni. Egy ilyen kártyajátéknál ez egész könnyü, hiszen ez iskolapélda is, pl. ilyen szereplőkről lehetne beszélni: játékos, pakli, kártya, menedzser (lépéseket vezérli), stb.

Új hozzászólás Aktív témák