RSS El Diario ¿Qué Demonios?
Seguir
Línea Representativa: Un parámetro más, colega
Matt necesitaba añadir un nuevo campo a un formulario, pero la tarea se complicó debido al método utilizado para guardar los cambios en la base de datos. El método, saveQualif, tenía 36 parámetros de tipo cadena, lo que dificultaba su uso. El método creaba un objeto de acceso a datos con el mismo número de parámetros y actualizaba los campos en la base de datos si no eran nulos. Los parámetros incluían información sensible como pdfPassword, que se almacenaba en texto sin formato. El método se llamaba dos veces, una con todos los parámetros rellenados y otra con la mayoría de los parámetros establecidos como nulos. Matt tuvo la tentación de refactorizar el método para mejorarlo, pero le preocupaba romper algo y decidió añadir otro parámetro en su lugar. Este enfoque ha sido adoptado por desarrolladores anteriores, que también han evitado refactorizar el método. El resultado es una base de código compleja y difícil de mantener. La refactorización a menudo se pospone hasta más tarde, dejándola para que los futuros desarrolladores se ocupen de ella. Este enfoque puede provocar problemas en el futuro, como errores y vulnerabilidades de seguridad.