Richiesto re-post da StackOverflow:
Sto lavorando in un piccolo tempo di sviluppo con tempi di sviluppo molto limitati. Sviluppiamo uno strumento importante per il risultato del nostro lavoro, ma non utilizzato quotidianamente. Sono l'unica persona nella squadra che ha un background come programmatore.
Il mio problema è che sto spingendo per le revisioni del codice prima di ricollegarmi al trunk per oltre un anno. Tutti erano d'accordo su questo, ma è ancora solo il mio codice che è stato rivisto. Di ritorno da una lunga vacanza torno a un baule con commenti in codice poiché "questa è una brutta soluzione - rimuovila al più presto" e "soluzione rapida". La novità è che un ragazzo è stato nominato responsabile dello strumento. (Un ruolo che prima mi è stato offerto, ma ho rifiutato a causa di un motivo non correlato al lavoro.) E pensa che questo sia un modo ok di lavorare: dato che abbiamo un tempo di sviluppo così limitato, dovremmo tagliare angoli del genere.
La mia preoccupazione è che gli altri sviluppatori scrivano brutti codici: spesso rompono l'incapsulamento, scrivono enormi classi, aggiungono classi interne in posti strani, hanno pochi o nessun test unitari e così via. Alla fine sarà impossibile sviluppare ulteriormente lo strumento.
Devo insistere sul fatto che eseguiamo revisioni del codice prima di ricollegarle al trunk o sono solo una puttana di qualità del codice?