Lavorando su basi di codice esistenti, di solito mi imbatto in procedure che contengono Uso abusivo di istruzioni IF e Switch . Le procedure consistono in codice travolgente , che a mio avviso richiede un riscrittura errata. La situazione peggiora quando identifico che alcuni di questi sono ricorsivi .
Ma questa è sempre una questione di dibattito poiché il codice funziona bene e nessuno vuole risvegliare il drago. Ma tutti lo accettano è un codice molto costoso da gestire. Mi chiedo se ci sono raccomandazioni per determinare se un particolare metodo è un colpevole e ha bisogno di una rivisitazione / riscrittura, in modo che possa essere scomposto o polimorfizzato in modo efficace. Ci sono metriche (come numero di righe nella procedura) che possono essere utilizzate per identificare tale segmento di codice. La lista di controllo o il consiglio per convincere tutti, sarà fantastico!