Lavoro su una nuova impresa in una grande azienda di software aziendale (più di 3000 programmatori). Nel mio gruppo, abbiamo un sacco di progetti e le persone di solito lavorano su diversi progetti nel corso di un anno.
Ho appena iniziato a lavorare su un progetto che è stato precedentemente gestito da un mio amico (consulente che è stato con noi per 3+ anni) per aggiungere alcune funzionalità. Ho inserito il codice e la qualità era davvero scarsa. Sia che si trattasse del frontend dell'interfaccia utente o del backend dei servizi, il codice semplicemente non era rientrato, c'erano centinaia di righe commentate senza una ragione apparente, la documentazione era praticamente inesistente, gli standard di codifica non erano applicati in modo coerente (ad es.camelCase
e under_scored_variables
), i nomi delle variabili erano inintelligibili, le scelte dei tipi di dati erano sbagliate, ecc. ecc.
Sono una persona non conflittuale, quindi non voglio attaccare il mio collega, ma non voglio nemmeno andare dal mio capo e lamentarmi della sua prestazione. Quali sono i tipi di cose che potrei dire per dire gentilmente che il codice è scarsamente strutturato?
Modifica
Voglio chiarire che, mentre capisco, c'è un elemento di "Il codice di tutti gli altri fa schifo" a tutti i programmatori, quando vedo qualcosa di simile (i nomi vengono scelti di proposito e alcuni dettagli lasciati / modificati in questo esempio):
public void doCalculate(Object argument) {
if (argument instanceof String) {
String argument2 = (String) argument;
if (argument2 == "DataBase") {
// do something
} else {
long argument3;
try {
argument3 = Long.parseLong(argument2);
} catch (Exception e) {
argument3 = -1;
}
// do something completely unrelated
}
}
}
Penso che sia obiettivamente corretto dire che questa è not una buona idea. Inoltre, non ho a che fare con un principiante qui (sto solo codificando da 3 anni). Ha forse 20 anni di esperienza su di me. Il consiglio che avete dato finora è grandioso; voglio solo assicurarci che non stiamo parlando di una "linea sottile" qui.