Code Reviews – The secret of MobiMedia’s quality

Code Review – Ovvero, parliamo della qualità made by MobiMedia.

Una buona qualità del software è uno dei fattori di successo di un software, che noi, in qualità di sviluppatori, possiamo influenzare in modo significativo. Se noi sviluppatori svolgiamo un buon lavoro, il cliente è soddisfatto e potenzialmente il software ha maggior successo.

MobiMedia ha sempre effettuato il collaudo del software in modalità manuale. Nel corso degli anni, abbiamo lavorato sempre di più con collaudi automatizzati (inizialmente unit testing), e successivamente con collaudi UI completamente automatizzati, che simulavano il funzionamento del software per garantire che, modificando il codice, tutto continuasse a funzionare senza variazioni.

Una questione d’onore – Perché noi di MobiMedia siamo così precisi.
Questo articolo dovrebbe trattare un’altra misura che negli ultimi anni ci ha portato un grandissimo valore sia come sviluppatori di software sia come azienda. Le Code Review sono un processo di lavoro adattato in cui uno sviluppatore non pubblica direttamente le modifiche al proprio codice, bensì prima le fa controllare da un altro sviluppatore (il revisore del codice o “Code Reviewer”). Il revisore del codice può commentare il lavoro, formulare suggerimenti per il miglioramento e, alla fine, approvare o rifiutare il lavoro. In caso di rifiuto, lo sviluppatore originale ha la possibilità di migliorare nuovamente il suo codice e di sottoporlo nuovamente alla Code Review. Questo ciclo può essere ripetuto più volte, fino a quando il revisore del codice non muove più nessuna obiezione e, in pratica, entrambe le parti sono soddisfatte della modifica.

La Code Review funge da controllo  – Il lavoro di squadra è fondamentale.
Quando non era ancora previsto il processo di Code Review, non solo il tasso di errore era più alto, ma anche la qualità del codice sorgente era peggiore. C’erano molte più soluzioni isolate perché gli sviluppatori spesso creavano soluzioni solo interne ai propri progetti, nonostante queste avrebbero potuto essere applicate allo stesso modo anche ad altri progetti.

Oggi molti errori vengono individuati prima ancora che qualcuno avvii il software. Semplicemente perché il revisore del codice controlla il lavoro di uno sviluppatore. È interessante notare che le Code Review sono state ampiamente accettate dal team e dall’azienda fin dal principio. Oggi fanno parte del processo di lavoro tanto quanto lo sviluppo e il collaudo del software

Attraverso il processo di Code Review, possiamo anche affidare più rapidamente agli sviluppatori nuovi o inesperti compiti più complessi, poiché c’è sempre uno sviluppatore esperto che controlla il lavoro. Inoltre, le Code Review risolvono automaticamente anche il problema delle sostituzioni per malattia o per ferie, perché abbiamo sempre almeno due sviluppatori che hanno più o meno familiarità con le modifiche. Nei team di sviluppo più grandi, gli sviluppatori possono controllare il loro codice tra di loro, il che ha un effetto positivo sulla comunicazione del team e spesso aiuta a risolvere i problemi.

Per mappare il processo di Code Review utilizziamo un software chiamato “Phabricator”. Con questo software possiamo anche determinare più revisori del codice per una modifica. In tal caso, diversi sviluppatori devono dare il loro OK prima di accettare una modifica. Questo procedimento è particolarmente efficiente per gli interventi più profondi a livello di codice sorgente, poiché la responsabilità è distribuita su più persone e il rischio di errori è ridotto.

L’unico svantaggio delle Code Review è un leggero aumento del dispendio di tempo e costi, perché deve sempre esserci almeno un secondo sviluppatore per controllare le modifiche. Ciò può comportare ritardi di uno o due giorni, anche quando il revisore del codice lavora a pieno regime.

Alla fine, però, le Code Review hanno cambiato soprattutto un aspetto in MobiMedia: lo spirito di squadra nello sviluppo è più forte che mai. Comunichiamo di più l’uno con l’altro, ci critichiamo a vicenda e risolviamo i problemi insieme. Questo si traduce in un valore aggiunto, per noi come azienda sotto forma di un migliore ambiente di lavoro, e per i nostri clienti sotto forma di una migliore qualità del software.

Autore: C. Zeller, Responsabile Sviluppo MobiMedia AG