12 komentářů k článku Co nás naučilo Code Review:

  1. omelkes

    překlep
    … práci do zvládnutelných jednotka. -> do zvládnutelných jednotek.

    Jinak díky za zajímavý článek.

  2. EdaCZ

    Chybky
    Ahoj.
    našel jsem pár chybek.

    Pokud nechápete všechny důsledky každého řádku pak vaše review nebude
    velmi cenné.

    Chybí čárka před slovem pak.

    API třetích stran byli používány správně

    Chyba v i.

    Jemné šťouchnutí od reviewera ve správnou chvíli je někdy nezbytné,
    aby časem nezchátrali.

    Nezchátral*y*, pokud jsou tedy myšleny ty dokumentace.

    …když se konečně rozhodnete se reviews vypořádat.

    S reviews, nikoliv se.

    …že se vám situace vymykne z rukou.

    Vymkne.

    …vždy je dobré uvažovat o code reviews jako součásti vašeho
    každodenního pracovního dne a ne jako o rozptýlení.

    Před „a ne“ bych dal čárku. To „a“ lze případně úplně vypustit.

    …přistupte k další inkrementální změně a tak dále až dokud není celý
    refaktoring dokončen.

    Čárka před „až“.

    Snad jsem už nic gramatického nepřehlédl. Dále by se tam našlo dost míst, která by šla přeformulovat stylisticky lépe, ale to už jsou detaily.

    Bohužel už patří ke koloritu Zdrojáku, že články, ač informačně a věcně velmi přínosné, nejsou jazykově zpracované příliš pečlivě. Mám pocit, že podobný jazykový příspěvek píšu ke každému druhému článku.

    Pokud byste měli zájem, můžu na nové články kouknout z pohledu korektora ještě předtím, než je uveřejníte, ať se vyhneme takovému množství chyb, které bylo například v tomto článku.

    PS: Nic ve zlém. Jsme lidi… :-)

    PS2: Editor příspěvků neinterpretuje dobře odřádkování. Za slovem „Ahoj“ mám odřádkování, ve výsledném příspěvku tomu tak ale není.

    Eda

    1. EdaCZ

      Re: Chybky
      Tak ta nepřesná interpretace odřádkování je jen v náhledu příspěvku, po odeslání už je to ok.

  3. uetoyo

    @EdaCZ
    „API třetích stran byli používány správně“
    Proč je tam špatně i? To API nemusím skloňovat… pokud si chci nějak pomoci tak to aplikační rozhraní.

  4. Tomáš Tintěra

    Čeho konkrétně pomocí CR dosahujete?
    Zajímala by mne konkrétní data, čeho je běžné nebo čeho lze pomocí code review dosáhnout. Snížení chybovosti, zlepšení zastupitelnosti mezi členy týmu, zlepšení čitelnosti kódu a vnitřní komunikace.

    V článku jsou chyby, viditelné na první přečtení. Je vidět, že by mu pomohlo review. Alespoň standardní jazyková korektura. Každopádně: díky za článek.

    Podle mé zkušenosti na review často stačí i méně času. (Měli jsme jednoduchou aplikaci. DB a web, pět vývojářů, projekt běžel jednotky měsíců).

Napsat komentář

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=16318