Concordo fortemente con la risposta di funkymushroom. Se sei un ambiente di squadra, assicurati che gli altri sappiano che stai refattore o riorganizzando il codice, se hai mai intenzione di ottenere buoni incarichi futuri.
Per esperienza personale lo so, sebbene non sia il tuo stile di codifica, se stai mantenendo il codice, che anche altri modificano e mantengono, rimani nello stile del codice esistente. L'aggiunta di commenti e chiarimenti va bene, ma il layout e le convenzioni di base dovrebbero rimanere. I vecchi guru / pistole del progetto si aspettano che il codice sia simile a quello che vedono da anni.
Quando un cliente urla di un bug, la tua direzione andrà alle vecchie pistole per risolvere il problema il più rapidamente possibile. Se queste vecchie pistole, quando sono sotto pressione, ti trovano "ripulito il codice" e quindi ora devono passare il tempo a capire dove hai spostato o rinominato quella variabile che conoscono ha bisogno di essere modificata, il tuo nome nella società verrà cambiato in " fango".
Una volta che la crisi è finita, prima la vecchia pistola ti incolperà lentamente l'aggiornamento critico. Successivamente troverai che manterrai il codice pulito fino a quando sarai in azienda. Alla fine, quando saranno disponibili nuovi progetti interessanti, i tuoi manager chiederanno ai guru su chi dovrebbe lavorare il progetto, e se li hai fregati una volta, non riuscirai mai a raggiungere il nuovo progetto, fino a quando il tuo foraggio non verrà buttato via alla fine per rispettare una scadenza.
Se hai imparato al college il modo "giusto" di codificare e ora sei nella forza lavoro, dimentica quel modo "giusto". Questi non sono incarichi universitari, questi progetti non durano solo un semestre, possono vivere per anni e dovranno essere mantenuti da un gruppo di persone con diversi livelli di competenza e diversi livelli di interesse nell'ultima tendenza CS. Devi essere un giocatore di squadra.
Puoi essere la più grande programmazione hot shot a scuola, ma sul posto di lavoro, il tuo primo lavoro, sei un principiante con zero crediti di strada. Le persone che hanno programmato per anni non danno un grido alla tua scuola o ai tuoi voti, è quanto giochi bene con gli altri e quante interruzioni porti nella loro vita.
Nei miei 20 anni, mi sembra che siano stati licenziati più programmatori di assi, principalmente perché chiedono di fare le cose nel modo "giusto". A meno che tu non porti qualcosa di molto, molto, molto unico nel tuo lavoro, sei sostituibile. Potresti essere stato il migliore della tua classe, ma l'anno prossimo qualcun altro sarà il migliore della sua classe e cercherà un lavoro.
Lo guardo come il tuo lavoro principale, è quello di mantenere il tuo lavoro, fino a quando decidi di cambiare lavoro. Per mantenere il tuo lavoro significa che devi giocare bene nel parco giochi che qualcun altro ha costruito e pagato.
So di sembrare negativo, ma c'è sempre speranza. Man mano che acquisisci esperienza, ottieni successo, acquisirai influenza e sarai in grado di spostare le cose in un modo migliore. Quando si scrive un nuovo codice o su un nuovo progetto, premere per le modifiche desiderate. Se si tratta di un nuovo codice, le vecchie pistole non si aspettano che sia il modo in cui lo hanno lasciato e quando vedono i vantaggi potrebbero imparare e adattare il nuovo modo.
Il vecchio sistema può cambiare, ma richiede tempo. Cambiare qualcosa introduce rischio e rischio di odio per le aziende e devi prenderti tempo e lavoro per rendere la compagnia a proprio agio con il cambiamento.