Rivedi e annota il codice
Correggi il codice
In sostanza, NON diventare questo ragazzo fastidioso che tutti sappiamo che cambia solo gli spazi bianchi o gli stili di parentesi basati su una preferenza personale ma senza riguardo per la coerenza con il resto della base di codici, e che arriva addirittura a spaccare le build mentre aggiusta le violazioni minori del codice introducendo bug semplici ma profondamente nascosti.
Comunicare
-
Contatta gli autori .
-
FARE discutere le tue modifiche con gli altri (non necessariamente gli autori).
-
Richiedi la revisione per i tuoi cambiamenti dagli altri; Questo:
- mostra umiltà,
- invita gli altri a saltare sul carro del carro,
- mostra che si tratta di un processo fluido.
-
FARE regole e convenzioni con i tuoi colleghi per evitare che questo problema si espanda e paralizzi il codebase nel tempo.
- FARE rapidamente la questione.
Chiedere
Chiedi. Piuttosto che dire "Sto aggiustando questo perché X", chiedi direttamente "perché lo hai fatto in questo modo, e come pensi che possiamo migliorarlo?" Se escono con qualcosa di meglio, allora inizia con quello e lavora insieme su qualcosa di ancora meglio. Se non lo fanno, allora fai un suggerimento. Se escono con qualcosa di meglio, o non capiscono perché è sbagliato, allora spiegano e pongono di nuovo la seconda domanda.
Fallo uno sforzo di squadra
È più probabile che la tua squadra salti sul carro del vincitore se effettivamente partecipi a questo. Se si tratta di un progetto di grandi dimensioni con una base di codice un po 'datata, è probabile che a loro non piaccia lo stato di decomposizione del codice, ma forse non si è mai preso la briga di fare qualcosa al riguardo.
Se hai il potere (o il supporto di quelli che lo fanno), può essere una buona idea organizzare piccoli sprint dedicati a migliorare la qualità del codice. L'impostazione di analizzatori di codice per rilevare gli odori del codice e monitorarne la diminuzione costante durante il refactoring è anche abbastanza motivante per una squadra.
Ovviamente, potrebbe essere un po 'più difficile per i team remoti, poiché è un po' più difficile trasmettere entusiasmo su queste cose senza lavorare veramente insieme.
Quasi tutto si riduce a quello.
Semantica
Inoltre, non confondere "politically correct" con "diplomatic" :
- Il primo riguarda la possibilità di non offendere la gente dicendo qualcosa che molto probabilmente sarà negativo.
- Quest'ultimo tratta di affrontare i problemi senza offendere le persone perché potrebbe esserci un contesto emotivo.
Sono cose molto diverse.
Domande correlate