Devo avvisare i miei colleghi quando trovo un bug nel loro codice? [chiuso]

7

Se mi viene assegnato un bug, a volte controllo il controllo della versione per vedere quando è stato introdotto. Devo avvisare lo sviluppatore che ha introdotto il bug, anche se l'ho già risolto? Il vantaggio è che potrebbe aiutare a imparare ma lo svantaggio è che potrebbe sembrare una critica

    
posta JoelFan 21.01.2016 - 02:27
fonte

5 risposte

18

Se fa parte del tuo processo, allora si. Se non è specificamente parte del tuo processo, allora no.

Se non è specificato, la cosa migliore è chiedere agli sviluppatori se desiderano che li informi se correggi bug nel loro codice e rispetti i loro desideri individuali.

Se non vuoi farlo o è troppo complicato, puoi far loro sapere che hai trovato un bug. Se pensi che si sentiranno criticati, puoi dire "Ho risolto questa cosa su cui avevi lavorato prima, e ora sto facendo appello alla tua conoscenza precedente per assicurarmi di aver preso in considerazione tutte le sottigliezze nel farlo". In altre parole, non stai dicendo che il loro codice è stato rovinato, ma che rispetti che la loro soluzione profondamente pensata potrebbe essere più complessa della tua comprensione superficiale, e vuoi la loro considerazione esperta per il lavoro di riparazione che hai intrapreso. Questo ha il vantaggio di assicurarsi che siano a conoscenza del difetto del codice (e delle successive opportunità di apprendimento) senza essere offeso.

    
risposta data 21.01.2016 - 02:52
fonte
6

Penso che, se hai trovato un bug in qualche area dello sviluppatore, la cosa migliore da fare sarebbe chiedere allo sviluppatore di revisionare il codice delle modifiche.

In questo modo non stai ferendo l'ego di qualcuno, stai solo facendo il tuo lavoro (risolvendo un bug che ti è stato assegnato e assicurandoti di non rompere di più a causa della "correzione").

Crea una conversazione in questi termini.

  • Mi è stato assegnato il compito di lavorare su Problema X (non menzionare che si tratta di un bug).
  • Ho analizzato il codice e vedo che la soluzione migliore è quella di modificare il codice nell'area Y, che possiedi.
  • Puoi, per favore, rivedere la mia soluzione che non ho infranto qualcosa che non so.

Per esperienza, le persone reagiscono molto meglio se chiedi la loro opinione in modo proattivo, piuttosto che scoprendo le tue modifiche nel loro codice.

    
risposta data 21.01.2016 - 14:56
fonte
2

Per lo meno, dovresti avvisare che hai scoperto un potenziale bug con il loro codice. Questa potrebbe essere considerata una critica (e tu devi essere certamente costruttivo quando critichi); ma c'è anche il potenziale che ciò che hai trovato è in realtà non un bug .

Notificando la parte responsabile, non stai solo assegnando la colpa. Stai anche dando all'autore l'opportunità di difendere le sue azioni. Può darsi che il "bug" fosse una funzione prevista (o scoperta) su cui i tuoi utenti fanno affidamento. È anche molto probabile che, anche nel caso di un bug reale scoperto, la tua soluzione trascuri casi d'angolo con cui l'autore del codice è più familiare.

Notificando l'autore di un bug, dai anche la possibilità di correggere eventuali altri errori simili che potrebbero aver commesso. Se hanno commesso un errore in un posto, è probabile che abbiano commesso un errore simile in altri.

    
risposta data 21.01.2016 - 16:27
fonte
-1

Assicurati solo che siano critiche costruttive e digli come hai risolto il bug, quindi non crea nuovamente il bug.

    
risposta data 21.01.2016 - 02:44
fonte
-3

Sei il manutentore del codice? Se è così, correggi il bug, documenta la correzione e non andare puntando le dita su chi è "da incolpare" per questo in primo luogo.
Qualcun altro è il manutentore? Cosa stavi facendo scavando nel loro codice in primo luogo? Informali che hai trovato un potenziale problema e dove si trova. Forse dai loro consigli su come risolverlo se ne hai qualcuno. Non andare a puntare le dita e dare la colpa al problema.

Non hai avuto problemi a scavare in quel codice? Vergogna, stavi sprecando il tuo tempo facendo cose che non sono di tua responsabilità e potrebbero persino violare le politiche sul posto di lavoro relative all'accesso a documenti che non sono necessari per svolgere il tuo lavoro.

Come hai scritto la tua domanda, sembra che tu voglia solo gongolare alle persone che "hanno fatto l'insetto". Che spesso non è nemmeno il caso. Le modifiche al codice, qualcosa che è corretto a un certo punto potrebbero non essere corrette in seguito dopo che il codice da qualche altra parte cambia, causando problemi con l'input di quel codice originale che lo induce a comportarsi male.

    
risposta data 21.01.2016 - 15:56
fonte

Leggi altre domande sui tag