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

• #11068 Peter Kiss őstag Sk8erPeter #11061

  Új Válasz 2012-09-08 15:53:35 #11068

  Ú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 Sk8erPeter #11061 üzenetére

  Inkább örültél volna, hogy még megoldásnak neveztem a tiéd. Soakot leszólod azért, mert sántít az OO látásmódja, de a tiéd sem jobb. Azt mondod:

  public function addNewTab(tabId, tabLabel, htmlContentOfTab)

  Első körben syntax error, második körben pontosan tudod, hogy egy Tab-ot akarsz hozzáadni, akkor adj hozzá egy Tab-ot, ne pedig egy metódusban hozz létre raw adatból újat, és azt add hozzá. Single responsibility mond valamit? Nem hiszem.

  $tabs = array(
  'fragment-1' => array(
  'label' => 'One',
  'content' => '<p>tartalom egy.............</p>'
  )
  );

  Ezt meg hol tanultad? Tömb mire való? Azonos elemek (néha a végletekig leeegyszerűsítve (pl. valami alap osztály)) tárolására, nem arra, hogy újabb olyan tömböket rakjunk bele, amelyek inkább szótárak, így az azokban lévő KeyValuePair-ok csak úgy lógnak a levegőben.

  private $tabs = array();

  Miért private? Mi van, ha később hozzá kell férni egy leszármazott osztályban? Vagy ilyet nem lehet, annyira egységbe zársz?

  Soak kódját nem magasztaltam, az ő megoldása PHP-san ronda és finom, egyedül kiemeltem benne egy jó elgondolást, de az is igaz, hogy az övé több és jobb, mint a te általad berakott pár gondolat, ráadásul te csak leszaroztad, ahelyett, hogy megpróbáltad volna megfogni benne a jót, esetleg a jó útra terelni. Sőt, azt mondtad, hogy a public field-ek miatt akármit berakhat oda bárki, miért, a tiédbe talán nem?

  Ráadásul az egyszerű megoldást emeled ki, szerintem a te általad folytatott sokkal bonyolultabb azzal, hogy gyakorlatilag egy helyen intézel mindent.

  És még egyszer ne add be senkinek, hogy nekem könnyebb volt a rövid beszélgetések után összehozni azt, amit.

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