Si luego de leer el post anterior te has preguntado ¿Cómo evito escribir comentarios? te recomiendo que sigas leyendo.

1. ¿Qué debo hacer para dejar de escribir comentarios sin valor y qué solo generan ruido en el código?

Respuesta: Debes escribir nombres claros y fáciles de entender.

Nota: Se sugiere que las clases tengan como nombre un sustantivo y los métodos un verbo.

1.1 Clases:

Evil Code:

//Clase que genera reportes PDF
public class Reporte{}

//Clase que contiene funciones matemáticas
public class Calc{}

//Clase que contiene la lógica para caducar Polizas
public class ServCadPol{}

Clean Code

public class GeneradorReportePdf{}
 
public class Calculadora{}
 
public class ServicioCaducarPoliza{}

Nota: Cualquier clase que solo se llame: Util, Helper o Manager, quiere decir que no tiene una responsabilidad clara y hace muchas cosas a las vez.

1.2 Métodos:

Evil Code:

//Obtiene los codigos generados por fecha
public List<string> Obtener(Datetime fecha)
 
//Calcula los descuentos que se tiene que aplicar por fecha
public void Procesar(Datetime fecha)
 
//Caduca una poliza
public void Opera84784(RequestOpera84784 request)

Clean Code:

public List<string> ObtenerCodigosGeneradosPorFecha(Datetime fecha)
 
public void CaclcularDescuentosPorFecha(Datetime fecha)
 
public void CaducarPoliza(Poliza poliza)

1.3 Variables:

Evil Code:

//Nombre completo del usuario
string n;
 
//Fecha de regsitro
Datetime d;

//IGV
double i = 1.8;

Clean Code:

string nombreCompletoUsuario;
Datetime fechaRegistro;
double IGV;

2. ¿Qué debo hacer para dejar de usar regiones que encierren distintas funcionalidades dentro de clases o métodos?

Respuesta: Debes aplicar el Principio de Responsabilidad Única.

Nota: Cuando se usan regiones para ocultar muchas responsabilidades ocasionan que la cohesión de una clase disminuya. Recuerden alta cohesión y bajo acoplamiento.

2.1 Clases

Evil Code:

public class Util
{
	#region Encriptacion
	//Codigo
	#endregion

	#region Notificacion
	//Codigo
	#endregion

	#region Formato
	//Codigo
	#endregion

	#region Log
	//Codigo
	#endregion
}

Clean Code:

Por cada responsabilidad identificada se debe crear una clase para que haga exactamente una cosa y que la haga bien:

public class ServicioEncriptacion{}
public class ServicioNotificacion{}
public class ServicioLog{}

2.2 Métodos:

Evil Code:

//Caduca una poliza y notifica al administrador
public void Actualizar(Poliza poliza)
{ 
	//Llamada al servicio web del cliente para caducar una poliza
	ServicioWebPolizaExterno servicioPoliza = new ServicioWebPolizaExterno();
    RequestOpera105874 requestCaducarPoliza;
	//Código...
	servicioPoliza.Opera105874(requestCaducarPoliza);
	//Llamada a web service
	 
	//Envió de correo al responsable de la poliza
	ServicioNotificacion servicioNotificacion = new ServicioNotificacion();
    Mensaje mensajeResponsable;
	//Código...
	servicioNotificacion.EnviarNotificacion(mensajeResponsable);
	//Envió de correo

	//Log Traza 
	ServicioTraza servicioTraza = new ServicioTraza();
    Traza trazaPolizaCaducada;
	//Código...
	servicioTraza.Registrar(trazaPolizaCaducada);
	//Log Traza
}

Clean Code:

public void CaducarPoliza(Poliza poliza)
{
	CaducarPolizaServicioExterno(poliza);
	EnviarNotificacionResponsable(poliza);
	RegistroLogPoliza(poliza);
}

private void CaducarPolizaServicioExterno(Poliza poliza)
{
	ServicioWebPolizaExterno servicioPoliza = new ServicioWebPolizaExterno();
    RequestOpera105874 requestCaducarPoliza;
	//Código...
	servicioPoliza.Opera105874(requestCaducarPoliza);
}

private void EnviarNotificacionResponsable(Poliza poliza)
{
	ServicioNotificacion servicioNotificacion =  new ServicioNotificacion();
    Mensaje mensajeResponsable;
	//Código...
	servicioNotificacion.EnviarNotificacion(mensajeResponsable);
}

private void RegistroLogPoliza(Poliza poliza)
{
	ServicioTraza servicioTraza =  new ServicioTraza();
    Traza trazaPolizaCaducada;
	//Código...
	servicioTraza.Registrar(trazaPolizaCaducada);
}

Nota: Si los métodos resultantes se utilizan también en otras clases lo recomendable es crear una clase por cada responsabilidad para evitar repetir ese código, recuerden el principio DRY.

3. ¿Qué hago si el código es difícil de entender?

Respuesta: Debes dividir el código en distintos métodos.

Evil Code:

//Si el usuario existe, no esta bloqueado, no expiro su contraseña
//y esta habilitado para cambiar su contraseña
if(
(usuario != null && !string.IsNullOrEmpty(usuario.DistinguishedName) ) &&
!(((int)usuario.Propiedad["msDS-User-Account-Control-Computed"]) 
& 16/*flag AccountLockedOut*/ ) &&
!(((int)usuario.Propiedad["msDS-User-Account-Control-Computed"]) 
& 64/*flag PasswordCannotChange*/ ) &&
(((int)usuario.Propiedad["msDS-User-Account-Control-Computed"]) 
& 8388608/*flag PasswordExpired*/ )
)
{
	//Codigo
}
else
{
	//Codigo
}

Clean Code:

var usuarioPuedeCambiarContrasena = 
		UsuarioExiste(usuario) && 
		!UsuarioBloqueado(usuario) && 
		!UsuarioContrasenaExpiro(usuario) && 
		UsuarioPuedeCambiarContrasena(usuario);

if(usuarioPuedeCambiarContrasena )
{
	//Codigo
}
else
{
	//Codigo
}

const string AtributoControlCuentaAD ="msDS-User-Account-Control-Computed";

bool UsuarioExiste(Usuario usuario)
{
	return usuario != null && 
		!string.IsNullOrEmpty(usuario.DistinguishedName);
}

bool UsuarioPuedeCambiarContrasena(Usuario usuario)
{
	int flagNoPuedeCambiarContrasena = 64;
	return !(((int)usuario.Propiedad[AtributoControlCuentaAdam]) 
		& flagNoPuedeCambiarContrasena );
}

bool UsuarioBloqueado(Usuario usuario)
{
	int flagCuentaBloqueada = 16;
	return (((int)usuario.Propiedad[AtributoControlCuentaAdam]) 
		& flagCuentaBloqueada );
}

bool UsuarioContrasenaExpiro(Usuario usuario)
{
	int flagContrasenaExpiro = 8388608;
	return (((int)usuario.Propiedad[AtributoControlCuentaAdam]) 
		& flagContrasenaExpiro );
}

4. ¿Dónde puedo guardar los cambios hechos en el código y sobre todo que sean durables?

Respuesta: Debes usar un administrador de código fuente.

Evil Code:

//BUG: No se valido que la variable tenga un valor 10/12/2012
if(objeto != null) {}
 
//Agregado por laldazabal
if(objeto != null) {}

//Cambio 10/10/2001 - Se renombro el metodo
//Cambio 10/12/2012 - Se agrego un parametro nuevo
//Cambio 10/06/2014 - Se reemplazo el metodo, no funcionaba
servicioCorreo.EnviarNotificacionEnCadena();

Clean Code:

En estos caso se debe usar una herramienta de control de código fuente como: TFS, GitHub, Subversion, Mercurial, Visual SourceSafe, etc.

En mi caso uso el TFS para controlar las versiones, cada vez que se sube un nuevo archivo queda registrado con el usuario que hizo el check-in.

También nos ayuda a ver en el tiempo todos los cambios que han ocurrido, con esta funcionalidad ya no es necesario dejar código comentado, ya que podemos recuperar una versión anterior en cualquier momento.

Conclusión

Dejar de escribir comentarios en el código fuente no es difícil, solo debemos invertir un poco más de tiempo en definir nombres que sean claros y legibles. Aplicando principios de diseño, como el de responsabilidad única, nos permite volver al código fácil de leer y mantener. Finalmente, para controlar las versiones del código fuente debemos apoyarnos en herramientas como: TFS, Github, Subversion, etc.

Si te gusto este post entonces por favor ayúdame a difundirlo y logremos que el conocimiento se expanda, para lograr esto dale me gusta, compártelo en tus redes sociales a tus amigos o suscríbete a mi canal RSS, Gracias :).

Referencias:

• Use Team Foundation Version Control
• Github Guides – Hello World
• 10 Best Practices to Follow while writing Code Comments
• Express names in code: Bad vs Clean
• Single Responsability Example

Metal Tip:

Este artículo lo escribí escuchando la canción Desperate Cry de la banda de Sepultura de Brazil, les comparto el enlace.

Happy coding and Stay Heavy lml