Come convincere un collega che la duplicazione del codice è errata?

Question

Un mio collega stava implementando una nuova funzionalità in un progetto su cui lavoriamo insieme e lo ha fatto prendendo un file contenente l'implementazione di una funzionalità simile dello stesso progetto, creando una copia di esso rinominando tutte le dichiarazioni globali e leggermente modificare l'implementazione. Quindi abbiamo finito con due file di grandi dimensioni quasi identici a parte la ridenominazione.

Ho cercato di spiegare che rende il nostro progetto più difficile da mantenere, ma non vuole cambiare nulla dicendo che è più facile per lui programmare in questo modo e che non c'è motivo di correggere il codice se esso "non è rotto".

Come posso convincerlo che tale duplicazione del codice è una cosa negativa?

È correlato a this questions, ma sono più interessato alle risposte rivolte a una persona tecnica (un altro programmatore), ad esempio un riferimento a una fonte autorevole come un libro sarebbe ottimo. Ho già provato argomenti semplici e non ci sono riuscito.

Answer 1

Chiedigli cosa farà quando troverà un errore nel suo codice. Quanti posti avrà ora per sistemarlo?

È inoltre possibile visualizzare le risposte a this question (Perché "copia e incolla" del codice è pericoloso?).

Answer 2

Dagli una copia di Refactoring.

Answer 3

Appello al tuo capo per motivi tecnici. Se il capo è d'accordo con i metodi del tuo collega/e/o non gli fa aggiustare, allora non c'è molto che tu possa fare se il ricorso alla ragione non funziona.

Answer 4

Perché quando trovi un bug, devi cambiarlo in due punti. Perché quando vuoi aggiungere una nuova funzione, devi aggiungerla in due punti.

Answer 5

Migliora il tuo versione del codice così tanto che lui è frustrato con la gelosia, poi dire - se si fosse appena collegato al mio codice ...

Answer 6

dirgli che non si sa mai dove un'applicazione può andare in un business ... Una semplice applicazione di test a volte può essere modificata continuamente, e alla fine finisce per essere usata molto ... L'ho vista spesso nelle piccole aziende. E poi, invece di ricominciare da capo e perdere tempo su cose che avrebbero potuto essere risolte prima, potresti semplicemente farlo ora, breve e dolce, mentre può essere ...

Answer 7

Quando desidera il caffè, fallo prendere un sorso alla volta dalla caffettiera invece di una tazza intera. Ciò è particolarmente efficace se aggiunge panna e zucchero che dovranno essere sbucciati in minuscole porzioni. Questo dovrebbe illustrare come i compiti ripetitivi siano molto macchinosi e faticosi (come correggere 20 pezzi di codice anziché uno).

Quindi, invia un link a questo post in modo che possa vedere tutte le altre persone che hanno le spalle.

Answer 8

Ci sono due opzioni qui:

1. Lui è una persona razionale che semplicemente non ha troppa esperienza. In questo caso, puoi razionalizzare il tuo argomento, magari mostrandogli un esempio più chiaro di duplicazione del codice di qualcun altro nel tuo codice. Puoi anche trovare un bug nella copia originale (o anche meglio, alcuni bug), e dirgli che ora il suo codice è rotto e che deve ripararlo.

2. È un asino testardo: Allora non dovresti sprecare energia su di lui. Vai dal suo capo e lascia che sia il capo a occuparsene. Alcune persone sono proprio così.

Mentre la prima opzione è ovviamente molto meglio, a volte non si ha scelta. E se tu sarai quello che alla fine avrà bisogno di mantenere il suo codice alle 3 del mattino perché un cliente importante inizia a gridare dall'altra parte della terra - allora è sicuramente il tuo problema, e il tuo capo dovrebbe gestirlo.

Infine, se il tuo capo pensa che ti sbagli, probabilmente sei nel posto sbagliato.

Answer 9

Non si tratta di far riparare il tuo amico proprio ora. Si tratta di far crescere la tua squadra.

Fargli capire che è ingiusto per il team & il progetto. Se ancora non è d'accordo, prendilo una tazza di caffè e chiedigli di sedersi sorseggiandolo, mentre potresti prendere la sua tastiera e sistemare il codice di fronte a lui.

Potrebbe vergognarsi e non farlo la prossima volta (grande vincita). Ho usato questo 4 volte e ha sempre funzionato!

Buona fortuna.

Answer 10

Probabilmente suppone che non si sia rotto, e non lo sarà. Inoltre, il perfetto è il nemico del bene. Non penso che sia ignaro dei pericoli del copia/incolla, ha solo una diversa valutazione del potenziale di errore di te.

Forse potresti romperlo per lui, per mostrare quanto sia facile. Se non puoi, forse ha ragione.

Answer 11

Ci sono molti validi motivi per non duplicare il codice, ma basta chiedere ... il tuo team desidera mantenere linee di codice di 100K (con duplicazioni di codice) o 50K di codice? Potrebbe sembrare che la duplicazione del codice a questo punto sia minima, motivo per cui il tuo collaboratore non vede l'importanza del concetto ASCIUTTO, ma immagina se duplica sempre più codice per i prossimi 5 anni. Chi manterrà quel codice? La tua squadra? Cosa succede se un giorno lascia il lavoro? La tua squadra vuole mantenere questa merda? :) In caso contrario, hai già creato un caso molto convincente per non duplicare il codice, per non parlare di "more duplications" = "più incline ad avere più bug in futuro".

Answer 12

Se è superiore (o supervisore) a te, chiedi ulteriori spiegazioni - è possibile saperne di più sul contesto ... forse non vale la pena di rifattorizzare il codice (forse è un piccolo progetto).

Se è uguale a te, puoi segnalare al tuo superiore, proponendo questa soluzione (che è una migliore).

Se siete superiore a lui, solo "chiedere" a lui di fare la tua strada ...

Answer 13

vostro collega sta ottimizzando la sua efficacia a breve termine, sacrificando l'efficacia lungo termine dell'organizzazione (ad esempio, il resto dei suoi collaboratori pure come lui). Eventuali modifiche richieste nel primo file sono probabilmente necessarie nella seconda, ma nessuno ricorderà ... e che causerà 2 cicli di ricerca e correzione, anziché uno.

È possibile eseguire un rilevatore di cloni sul codice e mostrare semplicemente i risultati al proprio manager.

Vedere Wikipedia on duplicate code per un elenco.

È possibile visualizzare campioni di rilevamento clone per varie lingue utilizzando il nostro rilevatore CloneDR. È stato progettato per individuare un grande blocco di codice con costanti rinomanze e può mostrare esattamente cosa è successo.

Answer 14

In primo luogo, riconoscere che ha ragione: copia-incolla è infatti più veloce ora.

Quindi, diciamo che il problema è il costo a lungo termine e che il costo aumenterà perché con la duplicazione, il sistema non è così ordinato come potrebbe essere. Sta introducendo disordine, confusione e più disordine hai, più è difficile lavorare con un sistema. Applicare qualche sforzo ora per organizzarlo meglio (di solito) ripaga a lungo termine. È come tenere organizzata la tua scrivania o stanza.

Questa è l'idea di Ivar Jakobson di software entropy