AntonioGanci

Il blog di Antonio Ganci
posts - 201, comments - 420, trackbacks - 31

Il Copia e Incolla è male

Il Copia e Incolla è male, Il Copia e Incolla è male, Il Copia e Incolla è male... non mi stancherò mai di ripeterlo. 

Nel codice di produzione la scorsa settimana è finito un bug a causa di uno stupido errore di copia e incolla di codice.

Il codice era simile al seguente:

for(int i1 = 0; i1 < array1.Length; i1++)
  array1[i1] = // un valore qualunque

A questo punto interviene la pigrizia che prende il sopravvento sul buon senso e faccio un copia delle righe precendenti e incolla:

for(int i2 = 0; i2 < array2.Length; i2++)
  array1[i2] = // un valore qualunque

Poi sostituisco il numero 1 con il 2, ovviamente dimenticandomene uno, quello evidenziato in rosso.

Ecco uno dei risultati nel farsi sconti quando si sviluppa. Apparentemente risparmi 30 secondi, per poi perdere mezz'ora per trovare il bug e un'ora per fare una release che risolve il problema.

Non infierite, sono umano.

Print | posted on giovedì 29 giugno 2006 21:14 |

Feedback

Gravatar

# re: Il Copia e Incolla è male

Se avessi prima scritto un test che controllasse il codice prima di scriverlo, avresti perso solo due secondi, il tempo di lanciarlo e vederlo fallire :)
29/06/2006 22:05 | Francesco Carucci
Gravatar

# re: Il Copia e Incolla è male

Il codice originale non è quello riportato, ho esemplificato al massimo in questo post.
In linea di massima per il nome dell'indice cerco di evitare nomi poco significativi come i, i1, index, ecc. preferisco nomi tipo rowIndex, customerIndex, ecc.
Inoltre per rendere più facilmente testatibili le funzioni cerco di mantenere più bassa possibile la complessità.
29/06/2006 23:22 | Antonio Ganci
Gravatar

# re: Il Copia e Incolla è male

Cosa vedo ... un orrido for!!! Nella mia azienda sono banditi gli if e i for :)
PierG
30/06/2006 14:25 | PierG
Gravatar

# re: Il Copia e Incolla è male

Per Pier:
Per forza perchè nella tua azienda fate tutto in prolog ;-)
30/06/2006 17:52 | Antonio Ganci
Gravatar

# re: Il Copia e Incolla è male

Il capo progetto vuole che usi il Copia e Incolla per fare prima. Ma invece si fanno più errori di prima :-P
01/07/2006 16:41 | Sem Tamburella
Comments have been closed on this topic.

Powered by:
Powered By Subtext Powered By ASP.NET