Komentáře k článku

Antipatterny, smradlavý kód a Peterův princip v IT

Články - - Různé -

U předchozího článku se už v titulku objevilo slovo „antipattern“. Jak samotný název napovídá, jde o určitý protipól návrhových vzorů, „patternů“, ovšem tentokrát v opačném smyslu, tedy „jak to nedělat“. Pojďme si taková „antidoporučení“ projít – a nezapomeňme se přepnout do módu „nadsázka povolena“.

Zpět na článek

93 komentářů k článku Antipatterny, smradlavý kód a Peterův princip v IT:

  1. Blaazen

    Antipatternofobie

    Pěkný článek, člověk se v něm občas najde – to (ne)klidně přiznám. Jen si musím dát pozor na antipatternofobii – stav, kdy se člověk tak bojí, že jeho kód bude odpovídat nějakému antipatternu, že kvůli tomu píše pomalu a pořád se kontroluje. To by byl vlastně nový antipattern.
    Pro jistotu si ale dám článek do záložek. :-)

    Odpovědět
    1. Ondřej Mirtes

      Re: Antipatternofobie

      Zkušenný/dobrý programátor voňavý kód píše automaticky, ne? :) A pokud je otestovaný, není problém ho refaktorovat.

      Odpovědět
  2. kraken

    Poltergeist

    Jak se bránit „Poltergeistu“, pokud v době jeho vytváření předpokládám, že časem funkčnost přidám (a tedy vytvořím takový objekt)?

    Odpovědět
    1. jehovista

      Re: Poltergeist

      Jestli casem znamena za hodinu, tak to asi takovy problem nebude. Jestli casem znamena mozna v pristi verzi SW, tak je asi lepsi obejit se bez nej.

      Odpovědět
    3. X

      Re: Poltergeist

      Joshua Bloch: „When in doubt, leave it out.“ – prostě situace „někdy se tady ta složitost bude hodit“ řešit tím, že tam tu složitost přidám až teprve to skutečně bude potřeba. Ne preventivně dříve!

      Odpovědět
  3. belzebub

    opisovani wikipedie

    Popisovana temata povazuji za extremne dulezita, takze na jednu stranu jsem rad ze se o nich pise – na stranu druhou mi pride celkem ubohy opisovat wikipedii a vydavat to za clanek..

    Odpovědět
    1. biq

      Re: opisovani wikipedie

      preco, mne sa to na wiki hladat nechce, tu to mam pekne pokope precitane za par minut. Diki autorovi

      Odpovědět
      1. belzebub

        Re: opisovani wikipedie

        „hledat na wiki“?!?! to opravdu myslite vazne? zadat do vyhledavaciho okenka „wiki antipatterns“ je otazkou mene nez 5 sekund..

        ..nehlede na to ze pokud se o tom chci neco dozvedet, stejne musim rozkliknout odkazy z clanku – a ty me privedou hadejte kam ?
        ..no ale vzhledem k tomu, ze se vam to asi nebude chtit hledat, to prozradim: na popisy antipatternu na wikipedii :)

        Odpovědět
            1. kronta

              Re: opisovani wikipedie

              Souhlasím. Autor se zcela jasně netají vazbou na wiki a každá osvěta dobrá.

              Musíme o těch problémech mluvit, jinak se nám těmato hnusama bude kód plíživě zanášet.

              Myslím, že nestačí propagovat, jak by se to mělo dělat. Strašně důležité je pořád si opakovat, na co si dát pozor.

              Odpovědět
          2. koudy

            Re: opisovani wikipedie

            Zrovna vcera, protoze tu(nebo abclinuxu?) byl nejaky clanek o programovani a presne tuhle stranku na wiki jsem vcera cetl, ale je pravda ze jsem nevedel, ze neco takoveho je popsane. Uz nekolikrat jsem se s antipatterny setkal a ani jsem to nevedel. :)
            Clovek se snazi nedelat tyto veci a je dobre vedet, ze „tudy ne pratele!“ :)

            Odpovědět
        2. ???

          Re: opisovani wikipedie

          Tak si ale uvědom, kolik článků je na internetu skutečně původních, tzn. takových že autor publikuje výhradně své myšlenky a občas se na něco odkáže. Tenhle článek je shrnutí toho co je na wikipedii v Češtině. A kdyby článek nebyl, většina lidí by o antipatternech nic nezjišťovala, někteří by ani dosud nevěděli co to je. Myslím že takovéto články mají smysl, je to jako slevo-mánie. Napadlo by tě normálně zajít na Křižíkovu fontánu do Prahy, mě spíš ne, kdybych k tomu neměl jiný podnět, ale když to máš na slevo-serveru, jako nabídku a ještě k tomu se slevou, tak se někdo chytne.

          Odpovědět
          1. uf

            Re: opisovani wikipedie

            Je dobře, že se o tom píše. Já jsem třeba o některých nevěděl, že jsou pojmenované.

            Ale fakt nevím, proč píše o mýc programech …

            Odpovědět
      2. Radek

        Re: opisovani wikipedie

        Akorát že na Wikipedii je před spoustou těch článků několik varování (např. neutralita článku je zpochybněna, neuvádí žádné zdroje, apod.), viz http://en.wikipedia.org/wiki/BaseBean
        Prostě čtenář je varován, že se tam můžou tvrdit dost pochybné věci. Naproti tomu Zdroják nám to předkládá jako fakt. Ano viděl jsem v perexu „nadsázka povolena“. Ale vidíte to sami – jakmile někdo něco napíše, byť na místo, kam může psát každý, spousta lidí tomu začne slepě věřit :-)

        Odpovědět
        1. wellczech

          Re: opisovani wikipedie

          A proto máme pod články diskuze, abychom se o tématu dále pobavili. A ještě lepší by bylo po diskuzi vyvodit závěry, shrnout je do nového článku nebo diplomové práce a upravit wiki včetně odkazů :)

          Odpovědět
      3. pht

        Re: opisovani wikipedie

        Ono je to celkem jedno odkud je to opsane. Z takovehleho clanecku ani z clanku na wiki z vas mistr nebude. Na to je potreba precist toho trochu vic a hlavne to zkouset v praxi. Doporucuju napr. knihu „refactoring to patterns“, z toho clovek ziska docela dobry nahled.

        Odpovědět
        1. tdvorak

          Re: opisovani wikipedie

          Nebude z nikoho mistr, ale uvedomi si ze nejaky problemy jsou. A dost mozna si hned vybavi nejakej svuj kod. A to by mohlo stacit k tomu, aby to cloveka motivovalo dal to resit a opravit. Jiste ze ucelena publikace to vysvetli lip, ale tenhle clanek si clovek prolitne o poledni pauze a neco mu to da.

          Odpovědět
    1. cpt. Rumpill

      Re: je to opisovanie

      Uvést zdroje? Jako napsat třeba „Tento text čerpá z popisů antipatternů na Wikipedii, a vychází proto pod licencí CC-BY-SA“? A zkusil ses podívat na konec textu?

      Odpovědět
    1. Fandik Hacker

      Re: Loop-switch sequence

      Myslim, ze v tomto pripade se uplne nejedna o antipattern loop-switch sequence.
      Jde tu spise o velmi prehlednou a udrzovatelnou konstrukci parsovaci funkce.

      Autor se urcite poucil u knihovni funkce printf(...) a jejich derivatu, kterou programatorsky svet zna cca od roku 1970 a jeji implementaci muzete v jazyce C nalezt napriklad v <stdio.h>.

      Odpovědět
      1. JScore

        Re: Loop-switch sequence

        no přiznám se tak jsem kód do hloubky nezkoumal. ale opravdu může mít daná konstrukce opodstatnění. Napadá mě například počítání znaků, čísel a speciálních znaků v řetězci. Vlastně algoritmus pro frekvenční analýzu.

        Odpovědět
        1. mmad

          Re: Loop-switch sequence

          Její jednoduchá verze:
          <?php
          $retezec_ptakovin = „neco neco blbost necojineho ptakovina“;
          $res = array();
          for ($i=0; $i<256; $i++) {
          switch ($i) {
          case 1:
          $d = „neco“
          break;
          case 7:
          $d = „necojineho“
          break;
          case 44:
          $d = „blbosti“
          break;
          /* … */
          default:
          $d = „“;
          }
          if (!$res[$d]) $res[$d] = 0;
          $res[$d] += substr_count($re­tezec_ptakovin, $d);
          }
          print_r($res);
          ?>
          Verze bez switche…
          <?php
          $retezec_ptakovin = „neco neco blbost necojineho ptakovina“;
          $hled = array(„neco“,“ne­cojineho“,“blbos­t“);
          $res = array();
          foreach ($hled as $i => $v) {
          if (!$res[$v]) $res[$v] = 0;
          $res[$v] += substr_count($re­tezec_ptakovin, $v);
          }
          print_r($res);
          ?>
          A jednotlive znaky – kdy ma for vyznam…
          <?php
          $retezec_ptakovin = „neco neco blbost necojineho ptakovina“;
          $res = array();
          for ($i=0; $i<256; $i++) {
          $c = char($i);
          if (!$res[$c]) $res[$c] = 0;
          $res[$c] += substr_count($re­tezec_ptakovin, $c);
          }
          print_r($res);
          ?>

          Odpovědět
      2. phr

        Re: Loop-switch sequence

        Ano, přibližně tohle jsem chtěl napsat. Taková konstrukce se někdy veeelmi hodí, protože dokáže dobře zpřehlednit co ten kód vlastně dělá.

        Odpovědět
    2. Jirka

      Re: Loop-switch sequence

      Mezi antipatternem uvedeným na Wikipedii a Vámi uvedeném javascriptovém kódu je obrovský rozdíl. V prvním případě autor ví, že první parsovaná hodnota je „key“, druhá „value“ a pak následují parametry. V druhém případě je pořadí volání parsovacích funkcí libovolné.

      Odpovědět
  6. maryo

    Re: Antipatterny, smradlavý kód a Peterův princip v IT

    Komu se clanek nelibi, nepochybne jeho kod spada do jednoho z bodu a jen si to nechce priznat :)…

    Odpovědět
    1. Vebloud

      Re: Antipatterny, smradlavý kód a Peterův princip v IT

      Mě se článek extrémě líbí, protože právě dělám na projekty, který se tím hemží. Takže jsem nasadil PHP Mass detector, Copy Paste detector a snažíme se vylepšit co se dá. Jenže…

      Odpovědět
      1. Branik

        Re: Antipatterny, smradlavý kód a Peterův princip v IT

        Copy Paste detector :) to je dobrej napad to je nejakej tool ? da se to pouzit na c++ ?

        Odpovědět
        1. Michal Augustýn

          Re: Antipatterny, smradlavý kód a Peterův princip v IT

          V TeamCity je třeba vestavěnej Duplicates Finder (pro Javu a .NET).

          Odpovědět
  7. Inkvizitor

    Code Complete jsem nečetl,

    ale nabízená definice „nekompetentního vývojáře“ mi přijde pojatá dost nešťastně. Samozřejmě, kód by měl být vždy psaný tak, aby byl co nejpochopitelnější, ale primární problém přece není v tom, že někdo použije „málo známou vlastnost jazyka“. V prvé řadě jde o to, aby v kódu bylo co nejméně závislostí na okolí, jednoznačné identifikátory, rozumné komentáře (hlavně u popisu funkcí/metod). Používání „obskurních konstrukcí“ by samozřejmě nemělo být samoúčelné, ale často vede k daleko kratšímu nebo rychlejšímu kódu. A ať se na mě nikdo nezlobí, programátor, který nerozumí svému vlastnímu programovacímu jazyku nebo mu dělá problém konstrukci najít někde v dokumentaci nebo na Internetu, by se měl nad sebou minimálně zamyslet.

    Odpovědět
    1. Martin MalýAutor příspěvku

      Re: Code Complete jsem nečetl,

      Tam je, že tyhlety konstrukce „upřednostňuje před čitelností kódu“. Namlátit dvoukilový zdroják plný triků a hacků a říct: „Kdo se v tom nevyzná, je lama, UTFG nebo jdi na ekonomku!“ je znakem určité programátorské nedostatečnosti. Pokud použití obskurní a málo známé konstrukce má svůj význam, tak kompetentní programátor k ní přidá pořádný komentář, kde vysvětlí proč je tam tohle, jaké to má důsledky atd., nebo aspoň jak se to jmenuje. Nelze spoléhat na to, že další programátor na první pohled rozpozná v kódu nějakou konstrukci, dokáže si ji pojmenovat a najít (kdyby si ji dokázal pojmenovat, tak ji asi nemusí hledat, že?) A nemusí to být špatný programátor – třeba zná jen jiné hacky a ne zrovna ten váš.

      Odpovědět
      2. Inkvizitor

        Díky za odpověď

        Obecně naprostý souhlas (až na to, že hacky a „obskurní konstrukce“ bych rozhodně nesrovnával), ale chtělo by to asi pár konkrétních případů. Asi hodně záleží na jazyku; osobně mě nenapadá třeba v Pythonu (který mě z větší části živí) žádnou konstrukci (když vynechám staré třídy) konstrukce, kterou by si programátor v našem týmu mohl dovolit ignorovat a čekat, že ho v tom nechám. Na druhou stranu jsem velice přísný, co se týče přehlednosti a dokumentace kódu. Chceš použít nějakou cool feature? OK, ale ať už používáš cokoliv, chci, aby kódu pokud možno porozuměl i ten, kdo nezná kontext a zdroják vidí poprvé. Ono to většinou opravdu jde.

        Odpovědět
      3. Aichi

        Re: Code Complete jsem nečetl,

        Mohu se zeptat v jake kapitole popisuje nekompetentniho vyvojare? Rad bych si to precetl :)

        Odpovědět
  8. Martin Soušek

    bohužel programátorem se člověk rodí - to se nenaučíte

    Tohle je sice všechno krásné, ale být dobrý programátor se nedá naučit. Buď to máte v krvi, anebo nemáte.

    Řadu těch antipatternů lidé používají především proto, že to lépe neumí a i kdybychom jim v tom tisíckrát hlavu vymáchali, tak to líp nikdy nesvedou.

    Odpovědět
    1. Branik

      Re: bohužel programátorem se člověk rodí - to se nenaučíte

      s tim nemohu souhlasit programovani se da naucit . Anti paterny jsou vysledkem lenosti ! Typicky magicke konstanty.

      Odpovědět
  10. Ondřej Mirtes

    Další antipatterny

    Ke článku bych rád doplnil další antipatterny.

    Nejzákeřnější jsou ty, které jsou prezentovány jako návrhové vzory – tedy osvědčená „nejlepší“ řešení, které chytře řeší určený problém.

    Na školách se vesele vyučuje např. Singleton, který je ovšem příšernou ukázkou antipatternu. Jedna instance objektu v aplikaci se dá zařídit i jinak, než že udělám privátní konstruktor a globální statickou metodu getInstance(). Stačí se domluvit, že daný objekt nebudu nikde instanciovat, ale zavolám na něm new jen jednou a budu si ho všude předávat. Takové řešení je testovatelné a nezblázním se, když takové objekty najednou potřebuju dva.

    Dalším takovým je Service Locator, kterým se zabýval díl zdejšího seriálu o Dependency Injection.

    Dalšími „code smelly“ jsou globální proměnné, statické atributy tříd, „statické třídy“ – tedy třída sloužící jen jako jmenný prostor pro funkce, které nemají žádnou návaznost na instanci objektu.

    Nedostatečná dekompozice objektů – pokud budu mít systém s jemnější granularitou, jeho části se pak snadněji znovupoužívají. Také by se neměla využívat dědičnost tříd tam, kam nepatří.

    Aleš Roubíček nedávno prohlásil, že mutable objekty jsou také anti-pattern, byl bych rád, kdyby tuto myšlenku rozvedl :)

    Z manažerských anti-patternů se mi líbí Brooksův zákon, který říká, že přidáním dalších programátorů do zpožděného projektu narůstá jeho zpoždění :) A deadliny jsou vlastně taky anti-pattern :)

    Odpovědět
    1. Opravdový odborník :-)

      Re: Další antipatterny

      1) Singleton: ten se vyučuje v první řadě proto, aby studenti na jednoduchém příkladu pochopili, co to návrhový vzor je. Na lepších školách zároveň řeknou, že to má i svoje úskalí a není moc dobré ho používat. Ale jako ilustrační školní příklad na pochopení smyslu návrhových vzorů je dobrý.

      2) Service Locator: záleží na kontextu, s čím to srovnáváte — ve srovnání s zpraseným kódem, kde jsou stejné věci na spoustě míst, je SL spása a krok dopředu. Samozřejmě to lze dělat lépe. A opět, jako ilustrační příklad dobré.

      3) „statické třídy“ jsou v pořádku, jsou to normální knihovny funkcí — a ano, třída v takovém případě plní pouzi jmenného prostoru, ale pokud chceme používat funkce (a ne jen metody) v jazyce, který funkce formálně nemá, tak je knihovní třída jasná volba.

      Odpovědět
    2. blizz

      Re: Další antipatterny

      „Dalšími „code smelly“ jsou globální proměnné, statické atributy tříd, „statické třídy“ – tedy třída sloužící jen jako jmenný prostor pro funkce, které nemají žádnou návaznost na instanci objektu.“

      To neni žiadny code smell, statická trieda = modul.

      citujem: „Zapáchající kód vznikne i přílišným lpěním na dogmatech – například použitím složitého návrhového vzoru tam, kde by stačilo jednoduché a přímočaré řešení.“

      Odpovědět
      1. asdasd

        Re: Další antipatterny

        +1 Přesně tahle věta se mi hned vybavila. Bohužel to je velice rozšířená úchylka a s postiženými se většinou nedá diskutovat.

        Odpovědět
        1. Aleš Roubíček

          Re: Další antipatterny

          Ovšem modul musí mít všechny znaky modulu a jeho fukce musejí být opravdovými funkcemi. tj. žádný sdílený stav, žádné vedlejší efekty. Čož bývá velmi zřídka k vidění, a proto to všeobecné lpění na tom, že jde o zlo.

          Odpovědět
            1. Aleš Roubíček

              Re: Další antipatterny

              Zkušenosti, za ta léta jsem viděl soustu code base a viděl spoustu takových statických API se sdíleným stavem a side effecty, že si musím nalít dalšího panáka, když si na to vzpomenu…

              Odpovědět
          2. asdasd

            Re: Další antipatterny

            S první větou souhlasím. Ovšem to, že někdo něco neumí použít z toho ještě nedělá špatný postup, ne?

            Odpovědět
            1. Aleš Roubíček

              Re: Další antipatterny

              Pokud není blbu vzdorný a umožňuje snadno dělat prasárničky, pak to z něj špatný postup dělá.

              Odpovědět
              1. asdasd

                Re: Další antipatterny

                Aha. A co říkáte třeba na jazyky, ve kterých jsou přístupné všechny objekty skrz na skrz? Ty by se asi měly přinejmenším zakázat, protože o nějaké „blbuvzdornosti“ nemůže být ani řeč, přitom v nich vznikají skvěle navržené programy – není někde chyba? Chorobná preventivní opatrnost by klidně mohl být další antipattern.

                Odpovědět
          3. Michal Augustýn

            Re: Další antipatterny

            V C# třeba považuju prakticky za jediné moduly s funkcemi statické třídy s extenzníma metodama.

            Odpovědět
          4. František Kučera

            Re: Další antipatterny

            A přesně k tomu jsou ty návrhové vzory — knihovní třída má mít soukromý konstruktor a nikde se pak její instance nevytváří, sdílený stav ani vedlejší efekty nemá a jediné co obsahuje jsou statické metody (ve skutečnosti ony opravdové funkce) a třída je pouze modulem, jmenným prostorem.

            Odpovědět
    3. ondra.novacisko.cz

      Re: Další antipatterny

      Moc zrovna nesouhlasím s tím, co tvrdíte o singletonech. Možná jde jen o použití. Singleton samozřejmě není speciální třída, ale speciální použití a pokud jde o singleton sedící za rozhraním, na kterém je metoda getInstance(), tak samozřejmě u mě má tato metoda ještě sestřičku, setInstance(), kterou mohu dosadit vlastní implementaci. Singletony mají rozhodně své opodstatnění.

      To samé se Service Locátory, třeba například v mé knihovně mám vlastní implementace funkcí, které na základě jména otevřou někde stream, což může být soubor, ale i TCP spojení. Je tam rozhraní se singletonem, za ním sedí service locátor, který na základě názvu vydá patřičný objekt, který následně je použit k uspokojení požadavku. Přijde mi to jako nejčiščí řešení, protože žádná část programu následně nemusí řešit, koho se mají doptat na konkrétní implementaci služby, a všechny části jsou schopné pracovat i se službami, které v době návrhu neexistovali. Maximální flexibilita.

      Odpovědět
      2. Michal Augustýn

        Re: Další antipatterny

        Co řeší ta další metoda setInstance()? A jak se to bude chovat ve vícevláknovém prostředí (třeba webový server) ?
        Singleton je prostě zlo – jediné použití, kde bych ho toleroval, je tam, kde není technicky možné injectovat závislosti (a v tom případě je oním Singletonem Service Locator).

        Odpovědět
        1. ondra.novacisko.cz

          Re: Další antipatterny

          Co to je za zbytecnou otazku? To je snad jasne ne? Tenhle typ singletonu je vetsinou, nebo spis vzdycky urcen pro zajisteni sluzeb s vnejsim prostredi, ktere je vzdy jedno. Zpravidla vetsina aplikaci bezi napriklad v ramci jednoho operacniho systemu soucasne, ktery ma zpravidla jeden filesystem (i kdyz jich ma vic, ale aplikaci poskytuje jedno rozhrani) atd. Podle vaseho dotazu hadam, ze jste tak uplne nepochopil, k cemu to je.

          A proc chci mit moznost to globalne zmenit? Treba proto, ze chci rozsirit moznosti vnejsiho prostredi, nebo emulovat funkce vnejsiho prostredi, ktere to prostredi genericky nenabizi

          Odpovědět
          1. Michal Augustýn

            Re: Další antipatterny

            To není zbytečná otázka. Takže znovu – co řeší ta metoda setInstance() ? Co dělá je jasné, ale já se ptám, co z architektonického hlediska řeší. IMHO jen zavádí další smell a případnou race condition, kdyby se nedejbože nějaká třída rozhodla tu metodu zavolat.

            Třída by měla jasně deklarovat své závislosti. V C++, C#, Javě apod. by se tedy měly všechny závislosti předávat přes parametry konstruktoru (příp. přes property setter injection). Použití singletonu vede k tomu, že třída používá něco, co v konstruktoru nedeklaruje.

            Pokud je například to zmiňované API operačního systému pro práci s file systemem poskytováno jako singleton, tak s tím nic nenaděláme, a v takovém případě je potřeba se této neprůhledné závislosti zbavit na vhodné úrovni.
            Např. jestli by měla třída FileLogger používat API pro komunikaci s file systemem napřímo nebo jestli by měla používat nějakou non-singleton abstrakci, to už je do jisté míry otázka vkusu. Já osobně bych třeba použil to API napřímo, protože třída vyjadřuje do jisté míry tuto závislost svým jménem.

            To jsou vynucené singletony, se kterými nemůžeme nic moc dělat. Ale v žádném případě bych nezaváděl vlastné singleton – je to prostě špatně.
            Třídy pak jasně nedeklarují své závislosti a třída, která singleton používá, může být netestovatelná (např. když chceme v jednom kontextu spustit více testů paralelně a každá třída potřebuje singleton jinak namockovaný).

            Odpovědět
            1. ondra.novacisko.cz

              Re: Další antipatterny

              Proto tvrdim, ze to moc nechapete. V jazycich zalozenych na tridach a instancich mate ke kazde instanci adekvatni automaticky vytvoreny singleton, jak vy tvrdite asi vynuceny. Ano, je to trida, s tridnimi funkcemi a promennymi, s jednou a neprepsatelnou metodou new.

              Pokud to nahradim tovarnimi objekty, vzdycky chte nechte narazime na problem s tim, ze potrebujete aby vam nekdo vyrobil instanci tovarny, nebo instanci tovarny tovaren. Nakonec skoncite u singletonu. Ze ne? No minimalne na operator new, coz je vlastne singleton.

              A co delat setInstance? No to co u new nikdy neudelate, mate moznost zmenit superglobalni tovarnu tovaren, treba tim, ze dodate vlastni instanci, ktera implementuje nejaka vylepseni. Ja neobhajuju objekty navrzene jako singletony. At jsou navrzene tak aby si nekdo tu superglobalni tovarnu mohl predavat v promenne a tim se vyhne tem vasim race condition… a zmene instance tovarny za behu. Ona setInstance by se taky mela volat tak jednou za cely beh, na zacatku treba jako konfigurace knihovny, nebo pri vyznamnych udalostech… prirovnal bych to treba udalosti zmeny runlevelu v linuxu

              Odpovědět
              1. Michal Augustýn

                Re: Další antipatterny

                > Ano, je to trida, s tridnimi funkcemi a promennymi, s jednou a neprepsatelnou metodou new.

                Jak už psal jiný Ondřej, statické položky tříd a statické třídy jsou taky smell. První jmenované jsou totiž prakticky globální proměnné a druhé vytváří singleton. Proto by se neměly používat.
                A metoda new se volá jen v kompozičním rootu – jak říkal Miško Hevery na jedné ze svých přednášek, tak jedním ze znaků špatně testovatelného kódu bývá přítomnost volání new někde jinde.

                > Pokud to nahradim tovarnimi objekty, vzdycky chte nechte narazime na problem s tim, ze potrebujete aby vam nekdo vyrobil instanci tovarny, nebo instanci tovarny tovaren.

                Ano – tomu se říká kompoziční root aneb konfigurace aplikace.

                > A co delat setInstance? No to co u new nikdy neudelate, mate moznost zmenit superglobalni tovarnu tovaren, treba tim, ze dodate vlastni instanci, ktera implementuje nejaka vylepseni.

                Pokud chci změnit superglobalni tovarnu tovaren, udělám to na úrovni konfigurace aplikace. Pokud by to měl dělat někdo níže odspodu, popíralo by to IoC.

                > Ona setInstance by se taky mela volat tak jednou za cely beh, na zacatku treba jako konfigurace knihovny, nebo pri vyznamnych udalostech…

                Než nějaké „by se mela volat“ raději navrhnu aplikaci tak, aby bylo nesnadné dostat ji do nějakého špatného stavu.

                Odpovědět
                1. ondra.novacisko.cz

                  Re: Další antipatterny

                  Nojo, jenze cokoliv se nachazi v rootu je vlastne singleton. Takze je ekvivalentni globalni deklaraci. Jedina vyhoda mit jeden root a v nem singletony je v tom, ze si teoreticky mohu vytvorit vice rootu.

                  Pak je otazkou, zda zbytecne netvorime kotvy nebo poltergaisty. Ono to, „co kdyby nahodou“ pak vede k tomu ze na jednom miste obchazime pasti, ktere jsme si jinde vytvorili

                  Odpovědět
    4. Aleš Roubíček

      Re: Další antipatterny

      Užití singletonu je vlastně speciální případ service locatoru. Navíc je to lifestyle. Pokud se volání getInstance() vyskytuje více než právě jednou jde o smell. Ve spojení s IoC je implementace vzoru singleton ok.

      Odpovědět
      1. xxx

        Re: Další antipatterny

        Naopak, singleton je absolutne nekompatibilni s IoC. V tom pripade jste totiz omezovan na jednu konkretni implementaci service locatoru (za predpokladu, ze nemate neco jako setInstance(), coz ale pak neni pravy singleton). Je to pak prakticky to same, jako by jste service locator nepouzili vubec a instance si delali „na miste“.

        Odpovědět
        1. Aleš Roubíček

          Re: Další antipatterny

          No chce to asi ještě trochu představivosti praxe. Ono vám totiž vůbec nic nebrání v tom, aby singleton implementoval rozhraní, volání getInstance() bylo provedeno právě jednou v composition rootu aplikace a všude byla tato instance injektována (klidně pomocí poor man’s DI). V takovém případě máme čisté IoC, s troufám si tvrdit, že nejde o smradlavé užití Singletonu.

          Pro lepší představu připojuji ukázku:

          class FooSingleton : IFoo {
    static IFoo instance = new FooSingleton();
    private FooSingleton() {}
    public static IFoo Instance { get { return instance; } }
}

class Bar {
    public Bar(IFoo foo) {}
}

class Program {
  static void Main() {
    var bar = new Bar(FooSingleton.Instance);
  }
}
          Odpovědět
    5. Branik

      Re: Další antipatterny

      Singletony jsou krasnej priklad paternu, kterej se velice rychle muze zvrhnout v antipattern . Nekolik singletonu propojenejch jako jojo poruznu rozhazenejma volanima a callbackama a clovek se pak jen divi co to vzniklo za spiders nest .

      Odpovědět
    6. phr

      Re: Další antipatterny

      Globální proměnné jsou softwarovou implementací Archimédova principu „Dejte mi pevný bod a pohnu Zemí“. A jako takové mohou mít smysl, ale samozřejmě nesmí se to s nimi přehánět nebo je používat zbytečně.

      Odpovědět
    7. Tonda

      Re: Další antipatterny

      Singleton se asi z nějakého důvodu jmenuje singleton a svoji fukci plní dokonale. Používá se tam kde potřebuješ mít jednu instanci a ne dvě nebo tři. To že v průběhu implementace zjistíš, že najednou potřebuješ instance dvě není problémem singletonu, ale tvůj, protože jsi tu aplikaci blbě navrhnul. Chtěl bych vidět jak bys v týmu dvaceti vývojářů všem vysvětloval co si kam mají předávat protože to není singleton, ale instance má být jenom jedna :-D

      Odpovědět
      2. jos

        Re: Další antipatterny

        class foo
{
  function __construct()
  {
    if(self::$counter) {
      throw new LogicException(
        'použij existující instanci pičo'
      );
    }
    ++self::$counter;
  }
  private static $counter = 0;
}
        Odpovědět
      3. Nox

        Re: Další antipatterny

        Je ovšem rozdíl mezi „používá se jen jedna“ a „je absolutně nemyslitelné, aby se v aplikaci objevila druhá instance“. Druhá verze je singleton (a nic jiného singleton není) a osobně si nedovedu představit případ, kdy by bylo nutné něco takového vynucovat
        (a navíc se stejně často odhalí že instancí je třeba víc http://phpfashion.com/singleton-sofie-s)

        Větu o týmu neberu, pokud má tým takové komunikační a programátorské problémy, tak nevím jak by mohl vůbec fungovat

        Odpovědět
  11. vonRibbentrop

    heje

    Ono se ale těžko programuje, když se člověk nemůže spoléhnout na to, že funkce vrátí správnou hodnotu.

    Odpovědět
    2. Sten

      Re: heje

      Tipuji to na chybu v komunikaci mezi Wikipedií, autorem článku a publikem :-). Funkce nemusí vrátit správnou hodnotu, protože může vrátit chybový stav a to i postranním kanálem (třeba readdir_r), což při neodchycení často znamená katastrofu.

      Odpovědět
  12. utx

    Příklad: GTK+

    Už párkrát jsem si říkal, že GTK+ nezveřejňuje věci, které programátoři potřebují, a musí složitě reimplementovat. To jsem ještě nevěděl, že se tomu říká Obrácení abstrakce.
    https://bugzilla.gnome.org/show_bug.cgi?id=554229#c4
    http://developer.pidgin.im/ticket/14245

    Při ladění jsem už několikrát narazil na Poltergeist. U jednoho kuriózního nebyla kvůli chybě ve vyšší vrstvě nikdy volána velká část kódu ve vrstvě nižší, aniž si toho kdokoliv všiml (viz Skrývání chyb). Až jednoho dne jeden člověk našel v nikdy nevolaném kódu bezpečnostní chybu…

    A sám se přiznávám k tomu, že jsem párkrát použil Programování metodou pokus-omyl dokonce v kombinaci s metodou Magická čísla.

    Odpovědět
    1. ondra.novacisko.cz

      Re: Příklad: GTK+

      Obrácená abstrakce – od jisté doby nepoužívám private:, pouze protected:, která chrání atributy z venku, ale umožňuje přístup zevnitř, takže funkce, které zůstávají neveřejné lze případně potřeby volat poděděním objektu. Nevýhodou je, že za správné použití zodpovídá potomek a s tím souvisí i problémy související třeba s nečekaným updatem předka například v další verzi. Nicméně, pořád to má tu výhodu, že při náhlé změně se musí opravit jedno místo v programu (potomka), než kdyby různé části programu přistupovali k atributům deklarovaným public.

      Private je moc restriktivní, a kolikrát proklínám programátory, že do toho zabalí deklarace, které by se mohly venku hodit.

      Odpovědět
      1. Ondřej Mirtes

        Re: Příklad: GTK+

        Třída by měla být na dědění uzpůsobená a měla by k tomu poskytovat prostředky – např. protected metody, které umožňují překrytí. protected atributy umožňují zasahování do implementace i na místech, kde to není vhodné.

        Odpovědět
            1. ondra.novacisko.cz

              Re: Příklad: GTK+

              Bohuzel, kristalove koule jsou prakticky nesehnatelne. Takze vymyslenim mist, kde se bude trida v budoucnu rozsirovat je podle me zbytecna ztrata casu a vytvari akorat zbytecne kotvy

              Odpovědět
      2. uf

        Re: Příklad: GTK+

        Tendence by měla být spíš opačná: Schovat toho co nejvíc a v případě nutnosti lze restrikce změkčit.

        Já se nepovažuji za dobrého programátora, ale nesnáším protected atributy. Je to zdroj zla. Dokonce ani ve třídě, která je navržená jako (nejlépe abstraktní) bázová pro dědění. Dnešní IDE vám sice řeknou, odkud proměnná pochází, ale plnit proměnnou, kterou nemám deklarovanou, je magie. O to víc mě mrzí, že je to běžné v JDK.

        Odpovědět
          1. uf

            Re: Příklad: GTK+

            Deklarovaná je, a to záměrně protected v předkovi. Ale (mluvím tedy o Javě) porušuje zapoudření.

            Odpovědět
  13. Aminux

    EMACS

    „EMACS je skvělý operační systém, kterému chybí jen použitelný textový editor“
    Ta to by chtělo implementovat Vim do EMACS třeba jako plugin.

    Odpovědět
  14. verajo

    Hezky clanek!

    Diky za tento clanek. Ja myslim, ze jestli se v nem nekdo aspon v necem castecne nenasel, tak nikdy neprogramoval, protoze vsechny tyhle „antipatterns“ jsou dusledkem bezne praxe.

    Odpovědět
  15. rooobertek

    Super článok

    vyzerá to, že niekto písal o firme, v ktorem pracujem :) Tu by ste našli asi všetky antipatterny popísané v článku.

    Odpovědět
  16. carlo.kokoth

    Původně jsem chtěl komentovat ...

    … ale komentář se tak rozrostl, ze jsem si (konečně) založil blog, udělal z toho první zápisek a časem budu přidávat věci které jsem už chtěl dávno napsat.

    Nenechte se odradit nazvem blogu / permalinkem (nesnáším politickou korektnost, svět je plný věcí hodných kritiky a servítky si brát nehodlám), samotný zápisek je až překvapivě uhlazený :
    http://svetjeplnyblbcu.blog.root.cz/2011/06/27/obskurni_featury_my_ass/

    Odpovědět
    1. 100% Lenin

      Re: Původně jsem chtěl komentovat ...

      Ideální kodér píše správný kód ve stylu
      try …
      try …
      try …
      catch nic

      Na dotaz proč to tak napsal mi odpověděl – správná aplikace nesmí strašit uživatele chybami

      a to je vše, drzí přátelé. :D

      Odpovědět
      1. 100% Lenin

        Re: Původně jsem chtěl komentovat ...

        :D (už na to nevidím). Myslel jsem“A to je vše, drazí přátelé.“

        Odpovědět
  17. veselej

    Naprosto sedí

    Takhle jsem se dlouho nenasmál :-) Největší joke je, jak to naprosto sedí na reálné situace projekty. Člověk trochu zavzpomíná na léta minulá a na některé oblíbené projekty. Super článek!

    Odpovědět

Napsat komentář

Přihlásit se

Tato diskuse je již příliš stará, pravděpodobně již vám nikdo neodpoví. Pokud se chcete na něco zeptat, použijte diskusní server Devel.cz

Zdroj: https://www.zdrojak.cz/?p=3509