È accettabile avere un codice inutile?

Question

Vedo che alcuni programmatori aggiungono un codice che, dopo tutto, non fa nulla di utile. Per esempio (C#):

[Serializable] 
class Foo { 
    // ... 
    string SerializeMe() { 
     return new XmlSerializer(typeof(this)).Serialize(this).ToString(); // serialize to xml (syntax wrong, not important here) 
    } 
} 

La classe è contrassegnato come Serializable, ma l'unico modo è serializzato è mediante XmlSerialization, che non richiede l'attributo classe affatto.

Personalmente odio questo tipo di codice inutile, ma lo vedo molto spesso e sono curioso di sapere cosa ne pensano gli altri. Dopotutto, non è un difetto così grave? È una pratica comune nel settore? O è semplicemente brutto e dovrebbe essere rimosso, non importa quale?

Answer 1

È necessario uno sforzo minimo per leggere il codice aggiuntivo (inutile, come aggiunto)?

Se sì (e penso che sia così), allora non dovrebbe essere nel codice. Codice come questo è contaminato da frammenti extra che non sono utili, sono lì "nel caso in cui".
Il "dopo il refactoring" di queste cose può essere doloroso, dopo 6 o 12 mesi, chi ricorderà se è stato effettivamente usato?

Answer 2

Pensa che il significato di inutile è che non può e non verrà utilizzato. Penso anche che una porta aperta sia molto aperta.

Answer 3

In un progetto commerciale, direi di no, soprattutto se qualcuno può smontarlo per la lettura e poi ha un momento WTF.

In un progetto di casa - certo perché no, mantengo spesso alcuni frammenti per un uso successivo.

Answer 4

Beh, non posso commentare questo particolare pezzo di codice senza conoscere il tuo background ma personalmente seguo rigorosamente una regola per non avere nulla in codice a meno che non ne abbia davvero bisogno. O almeno sapere che potrebbe essere utile un giorno per qualcosa che ho già in mente.

Answer 5

mia regola empirica,

Se proprio non utilizzato sbarazzarsi di esso.

Tutti i commenti in eccesso, interessanti sono solo rumore e aiutano il tuo codice a diventare illeggibile. Se lasciato lì, incoraggiano più codice in eccesso nella tua base di codice. Quindi rimuovilo.

Answer 6

Controllo origine buona significa che il codice o il codice inutile che non è più necessario devono essere rimossi. Può sempre essere recuperato dal controllo sorgente in un secondo momento.

Answer 7

Provo a seguire il principio YAGNI, quindi anche questo mi infastidisce.

Answer 8

Questo è un codice errato.

Il codice errato è una pratica comune.

Ciò detto a volte non vale la pena di cambiare materiale che non è "rotto".

Answer 9

Sembrerebbe che la perfezione non sia raggiunta quando non si può aggiungere altro, ma quando non può più essere rimosso.— Antoine de Saint-Exupéry

Answer 10

Davvero non piace.

Perché trascorro quindi del tempo prezioso cercando di capire perché è lì. Supponendo che se è lì, è lì per un motivo, e se non vedo il motivo, allora c'è una possibilità che mi manca qualcosa di importante che dovrei capire prima di iniziare a scherzare con il codice. Mi irrita quando mi rendo conto che ho appena perso un'ora cercando di capire perché, o cosa, qualche frammento di codice sta facendo, che è stato lasciato lì da qualche sviluppatore troppo pigro per tornare indietro e rimuoverlo.

Answer 11

Se c'è un piano attiva per utilizzare questa funzione aggiuntiva, quindi tenerlo in.

Se si sa che non potrà mai essere utilizzato, o se quel piano ha smesso di essere rilevanti 6 mesi fa, quindi estrarla. Prima commenterei, quindi rimuoverlo completamente più tardi.

Sono nel bel mezzo del salvataggio di un sacco di variabili che erano pubbliche ma che in realtà devono essere solo protette o private, semplicemente perché non hanno senso da una prospettiva API per essere esposte - che e niente usa loro.

Answer 12

Il codice one-liner inutile può e deve essere rimosso.

Il codice inutile che ha richiesto tempo per scrivere può e deve essere rimosso ... ma i responsabili tendono a sentirsi male. Forse la soluzione migliore è avere un progetto ProbablyUselessButWhoKnows in cui tutti possono verificare in un codice inutile che potrebbe essere riutilizzato in due o tre anni. Se non altro, in questo modo le persone non devono sentirsi nervose per averlo cancellato.

(Sì, in teoria è possibile ottenere fuori di controllo del codice sorgente, ma vecchio codice eliminato nel controllo del codice sorgente non è esattamente altamente rilevabile.)