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

• #11192 Peter Kiss őstag Soak #11186

  Új Válasz 2012-09-14 23:06:03 #11192

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  Peter Kiss

  őstag

  válasz Soak #11186 üzenetére

  Nekem bűzlik ez a példa, mert ránézve hard code-olt az egész.

  Maga a configuration injection is elég nehézkes, de itt főleg, gyakorlatilag tesztelhetetlen configfájl nélkül. Biztosan nem tenném bele a Settings::getInstance(); hívást, hagyni kell a getFactorynak (vagy másnak) egy olyan pontot, ahol kódból bele tudom tolni az aktuális lehetséges válaszokat. (Ez a Settings cucc azért nincs itt, mert a factory-k mellett nincs nagy jelentősége, minden látható, ami kell belőle.) A másik, ami problémás lehet, az a switch-case; ha valaki ilyet lát, akkor gondolkodjon el azon, hogy ezt biztosan lehet máshogyan is.

  Például, ha én csinálnám, akkor valószínűleg egy Activator vagy valamilyen más osztályt írnék, ami a konfigurációban szereplő adatok alapján tudna létrehozni objektumokat, nem pedig így hard code-olva switch-case-zel, ezt nagyon nehéz módosítani.

  Factory se static lenne szerintem, hanem egy DI container-től el tudnám kérni az összes elemet, ami kellene.

  DB_Abstraction_AbstractFactory-nak private constructor-t rak, de ennek ellenére pár sorral lejjebb példányosítja, aha. A require hívásokról ne is beszéljünk...

• #11188 Brown ügynök senior tag Soak #11186

  Új Válasz 2012-09-14 15:53:02 #11188

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  Brown ügynök

  senior tag

  válasz Soak #11186 üzenetére

  A Settings osztály getInstance() metódusa egy statikus fv. amit anélkül meghívhatunk, hogy előtte példányosítanánk az osztályt.

  Itt ezt úgy sikerült meghívni, hogy vagy egy fájlban van a két osztály vagy include-tuk az osztályt tartalmazó fájlt, esetleg használtuk a namespace importálás szolgáltatását egy kis __autoload()-dal megtámogatva.

  Szerk: design patternre én ezt találtam: [link]. Érthető példákat hoz.

• #11187 Speeedfire félisten Soak #11186

  Új Válasz 2012-09-14 15:47:03 #11187

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  Speeedfire

  félisten

  válasz Soak #11186 üzenetére

  Úgy, hogy odarakták.
  Gondolom a Settings static osztályban van pár dolog, ami nekik kell és azért hívták meg. Itt ebben nem sok jelentősége van szerintem. De majd megköveznek, ha nincs igazam.

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