Me gustaría saber cuáles son las mejores prácticas de las personas para eliminar el código no utilizado. Personalmente, soy partidario de eliminar (no solo comentar) cualquier cosa que no esté siendo utilizada actualmente. Pero no estoy seguro de qué tan lejos llegar.Mejores prácticas para eliminar el código no utilizado
tomar esto como un ejemplo (aunque estoy interesado en el debate). En mi proyecto, tengo una docena de UserControls. Para una característica que luego se enlazó, implementé un par de métodos y propiedades en uno de los UserControls. El código adicional no es específico de la función, pero es necesario para admitirlo. Posiblemente sea útil más adelante.
- ¿Debo eliminar el código porque no lo estamos usando en este momento, y cuanto menos código hay, más fácil es leerlo? El problema con esto es, ¿cómo saben los futuros desarrolladores que este trabajo ya se ha realizado?
- O debería mantenerlo allí, por lo que otro desarrollador puede encontrar fácilmente si necesitan usarlo más tarde (que no van a pensar que pasar por control de fuente para ver si alguien había hecho esto y borrado)?
- ¿O hay otra opción?
Lo mismo se aplica a UserControls actualmente no utilizados. ¿Debería eliminarlos o conservarlos?
Edit: Sobra decir (o pensé que sí) que estamos usando el control de fuente.
Sugiere agregar un # 3: el código no utilizado no es probable que se pruebe bien y teóricamente podría introducir errores y/o agujeros de seguridad. – JohnFx
Eso cae bajo 'mantenimiento', por lo que es más o menos lo mismo que razón # 1. –
+1. También encuentro que los bloques de código comentados parecen llamar tu atención mientras escaneas, no quieres que te distraiga el código. –