Hirdetés

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

  • pmonitor
    aktív tag

    Nem használok 2-nél mélyebben if-eket, mert annál több pont az ellenkezőjét éri el.

    A verziód azért nem jó, mert:
    - ha a 'this' megváltozik, akkor a unreg-es párját több helyen is át kell írni
    - ha új resource jelenik meg, mint modjuk a 'these', akkor lehet copy-pastelni és +1 mélységet kap ez a csoda if szerkezet
    - antipattern és jobb helyeken az ilyeneket be sem lehet szállítani, mert a checkerek megfogják

    #6262 pmonitor: de hisz nem is emlegette...

    Mondjuk sztem. ez az if mélység még nem veszélyes(értelmezés szempontjából).
    De itt is működik új változó bevezetésével az, hogy ha még tovább kellene mélyíteni, akkor a változó(mint kapcsoló) "beállításával" lehet követő if-ben rendezni a további feltételeket.
    Nekem is azért nagyon oda kellett figyelnem, hogy ezt a goto-val tele kódot megértsem egyáltalán, hogy mi miért is van. És még így sem biztos, hogy sikerült. Ezért írtam, hogy nem biztos, hogy ekvivalens. Úgyhogy nekem meg a goto-kkal teletűzdelt kód antipattern.

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