Kdy je refaktorizace už hodně?

milos

Kdy je refaktorizace už hodně?
« kdy: 01. 07. 2018, 12:04:57 »
Nazdar kluci a holky,
Chci slyset vas nazor, kdy je refaktorizace uz hodne? Proc se ptam? Pracuji ve firme uz nekolik let, kde nedavno ku nam nastoupil kolega a strasne ma potrebu neco refaktorovat. I celkem dost starej kod. My si to jako uvedomujem, ze nejaka ta refaktorizace by se hodila, jenomze nekdy nemame na to prostor. Podle me nedelame zly kod, taky si delame codereviews. A ten novy udelal asi pred mesicem takovy refaktor, tak to byla zmena cca. 60 fajlu. A nase nova funkcionalita mela cca 10 souboru, takze kdyz jsme s kolegama delali review, tak misto pulhodky, jsme zabili hodku a pul.
« Poslední změna: 08. 07. 2018, 18:55:24 od Petr Krčmář »


anonym

Re:Kdy je refaktorizace uz hodne?
« Odpověď #1 kdy: 01. 07. 2018, 12:09:54 »
Tak pokud na to neni prostor, a ten protor na to neni proto, ze vas kod je na hovno (investice do budouci uspory casu), tak v ten moment je refaktorizace asi uz hodne, ne?

Kit

Re:Kdy je refaktorizace uz hodne?
« Odpověď #2 kdy: 01. 07. 2018, 12:29:13 »
Pokud změnil za měsíc jen 60 souborů, tak je to docela OK. Někdo jich i za den udělá víc a to už pak problém. Podstatné je, že ses zmínil o kvantitě, ale nikoli o kvalitě. Právě kvalita rozhoduje o tom, zda to refaktorování má smysl nebo ne.

gll

  • ****
  • 429
    • Zobrazit profil
    • E-mail
Re:Kdy je refaktorizace uz hodne?
« Odpověď #3 kdy: 01. 07. 2018, 12:33:29 »
když se nezeptáš na konkrétní příklady refaktorizace, tak to IMHO na technický web nepatří.

Re:Kdy je refaktorizace uz hodne?
« Odpověď #4 kdy: 01. 07. 2018, 13:51:40 »
když se nezeptáš na konkrétní příklady refaktorizace, tak to IMHO na technický web nepatří.

Technické disciplíny neznají kouzlo generalizace?


Franta <xkucf03/>

Re:Kdy je refaktorizace uz hodne?
« Odpověď #5 kdy: 01. 07. 2018, 14:13:18 »
Nazdar kluci a holky,
Chci slyset vas nazor, kdy je refaktorizace uz hodne? Proc se ptam? Pracuji ve firme uz nekolik let, kde nedavno ku nam nastoupil kolega a strasne ma potrebu neco refaktorovat. I celkem dost starej kod. My si to jako uvedomujem, ze nejaka ta refaktorizace by se hodila, jenomze nekdy nemame na to prostor. Podle me nedelame zly kod, taky si delame codereviews. A ten novy udelal asi pred mesicem takovy refaktor, tak to byla zmena cca. 60 fajlu. A nase nova funkcionalita mela cca 10 souboru, takze kdyz jsme s kolegama delali review, tak misto pulhodky, jsme zabili hodku a pul.

V první řadě bych se vyvaroval nějakých hurá akcí. Ono jedna věc je opravit nějakou drobnost ve starém kódu, když už daný soubor edituji kvůli jinému úkolu, a jiná věc je rozsáhlejší refaktoring – ten už je potřeba dělat nějak řízeně a po domluvě s ostatními a s vedením projektu. Jak sám píšeš, je potřeba udělat revizi (což žere další čas). Tím to ale nekončí, dále je potřeba výsledek otestovat a ověřit, že nedošlo ke změně chování navenek. K tomu potřebuješ dobré automatické testy a/nebo dobrou dokumentaci a ruční testy. To stojí zase další čas a peníze.

Dokud to neprojde revizí a testy, tak refaktoring není hotový. A polovičatý refaktoring víc škodí než přináší. Pokud tedy nemáš kapacity/rozpočet i na ty revize a testy, tak je lepší nechat starý kód tak, jak je.

Za rozumný přístup považuji vyčlenit si nějaký paušální podíl času (% pracovní doby), který se věnuje údržbě kódu potažmo produktu – tzn. refaktoring, dopisování dokumentace, doplňování testů atd.

gll

  • ****
  • 429
    • Zobrazit profil
    • E-mail
Re:Kdy je refaktorizace uz hodne?
« Odpověď #6 kdy: 01. 07. 2018, 14:13:31 »
když se nezeptáš na konkrétní příklady refaktorizace, tak to IMHO na technický web nepatří.

Technické disciplíny neznají kouzlo generalizace?

diskuze směřuje k nadávání do lopat a hodnocení kódu, který nikdo neviděl. Anonym už začal.

L.

Re:Kdy je refaktorizace uz hodne?
« Odpověď #7 kdy: 01. 07. 2018, 14:26:39 »
A ten novy udelal asi pred mesicem takovy refaktor, tak to byla zmena cca. 60 fajlu.

Samotný počet souborů nic neříká. Stačí opravit gramatickou chybu v názvu hodně používané knihovní funkce a je to. Co přesně předělal?

pepa

Re:Kdy je refaktorizace uz hodne?
« Odpověď #8 kdy: 01. 07. 2018, 14:46:25 »
Code review  je skvělá věc, ale jak tu psali nademnou, bez testů to je stejně nejde.

anonym

Re:Kdy je refaktorizace uz hodne?
« Odpověď #9 kdy: 01. 07. 2018, 15:00:55 »
Podle mého názoru, v červenci bude hodně horko a bude pršet jen občas.

uuuuuuuu

Re:Kdy je refaktorizace uz hodne?
« Odpověď #10 kdy: 01. 07. 2018, 19:29:14 »
Ja myslim, ze to muze byt dobry krok dat novemu seniornimu kolegovi
Za ukol refaktoring. Koukne se na to novyma ocima, neco zjisti o produktu a muzete pri code review dohodnout tohle ano, tohle uz ne.

Ferda

Re:Kdy je refaktorizace uz hodne?
« Odpověď #11 kdy: 04. 07. 2018, 09:53:43 »
Co funguje, to se neopravuje.  :)

Trollopata

Re:Kdy je refaktorizace uz hodne?
« Odpověď #12 kdy: 04. 07. 2018, 11:58:58 »
Jo, to moc dobře znám... tolik write-only sračkódu ve starém projektu, že jakákoliv i malá změna je utrpení a kvůli začarovanému kruhu s tím nikdo nemá čas něco dělat, takže se to už jen zhoršuje. Je to dokonce tak strašné, že to vidí i nováček, který zatím ještě má snahu a odhodlání to aspoň trochu napravit navzdory ošoupaným lopatám, které už ve firmě mají vlastní inventární číslo a těžce nesou, když jim kdokoliv sáhne na ty mnohaleté výplody, jež považují za vrchol geniality.

Já už jsem na projektu několik let, spoustu těch ošoupaných lopat se už povedlo odsud vyštípat nebo vyměnit a refactoring mě pořád ještě neomrzel, takže za chvíli ta aplikace snad už bude trochu k světu ;-)

Inkvizitor

Re:Kdy je refaktorizace uz hodne?
« Odpověď #13 kdy: 04. 07. 2018, 12:22:32 »
Co funguje, to se neopravuje.  :)

Definuj "funguje".

Re:Kdy je refaktorizace uz hodne?
« Odpověď #14 kdy: 04. 07. 2018, 12:29:48 »
Co funguje, to se neopravuje.  :)

Co se nedá lehce upravit moc nefunguje.
Co nemá testy nefunguje.