close

Se connecter

Se connecter avec OpenID

Comment rater ses revues de code

IntégréTéléchargement
Ratezsesrevuesdecodeen5leçons
Speaker:MichelDomenjoud(Octo)
Format:Quickie
Date:22avril2016
Slides:http://fr.slideshare.net/mdomenjoud/ratez-vos-revue-de-code-en-5-lecons-devoxxfr-2016
Premièrepratiqueàmettreenœuvredansuneéquipe.
Constat:desprojetssesonttrèsbienpassés.Coïncidence,larevuedecodesepratiquait.
D’autreséquipesavaientpeurdechangerleurcodeetavaientpeurdemettreenprod.Pas
derevuedecode?
Dansl’histoiredel’informatique,larevuedecodeestla1ièrepratiquecollaborative(années
60).
Objectif:
ð Détecterdesdéfauts.Etudes:65%desdéfautspeuventêtreévités.1hderevuede
code=4hdedebugéconomisée
ð Bénéficesconséquents
o Partagerlesstandards
o Partagerlesconnaissances
3typesderevuedecode:
ð Revueparunpair
ð Revuecollective
ð Pairprogramming
3histoires
Fail#1:Oncrouletoujourssouslesbugs!
Sitederéservationdebilletsdetrainenligne.
Fonctionnalitécomplexedéveloppéeen2semainesmaisbienpensé.RevuedecodeOK
d’aprèsBenoît.
Passageenprod=>15jours+tardgrosbugquicrash.
Pourquoilarevuen’apaspermisd’identifierlebug?
Elleaétéréaliséetroprapidement.
Bonnespratiques:
• Prendreletemps,àunrythmesoutenable.Pasplusde300lignesdecodepour1h.
Pasplusde1h30
• Demandezunerevuedèsquepossible
• Préparerlarevueviaunelectureàtêtereposée
Fail#2Monéquipeneprogressepas
Coacheuneéquipe.Lesmêmeserreursreviennentrégulièrement.
Raisons:neseparlentpas,corrigeràlaplacedel’auteur
Bonnespratiques:
• Echangezdevivevoixpendantlesrevues
• L’auteurcorrige,enbinômesinécessaire
• Ecrivezvosstandards.Lesfaireévoluer
Fail#3:Onsetapedessus
Lorsd’unerevuedecodecollective,latournuredudébats’estmalpassée.Undéveloppeur
estsortiencolère.
Bonnespratiques:
• Critiquezlecode,pasledéveloppeur
• AppuyezvoussurlesprincipesEgolessProgramming
• Cen’estpaslemomentpouravoirdesdiscussionsàrallonge
• Trouvezdescompromisetenfaireunstandard.
Ces3formatsderevuesontcomplémentaires
Trouvezleoulesformatsquiconviennentetadaptezleaubesoin:
• Pairprogramming:DémarreruneUS,CodeLegacy,Debug,Apprendre
• Revueparunpair:toutletemps
LivreCultureCodegratuitsurlesited’Octo
Auteur
Document
Catégorie
Uncategorized
Affichages
22
Taille du fichier
132 KB
Étiquettes
1/--Pages
signaler