¿Cuáles son los comentarios de revisión de código más frecuentes?

El comentario de revisión de código más común que solía recibir y es uno de los olores de código más vistos es “Envidia de características”. Deberíamos mantener la lógica de la clase que pertenece. No tener la lógica en la clase apropiada crea mucha duplicación en el código y se vuelve difícil de probar.

Tomemos un ejemplo: (uno muy simple en Java)

Podrá apreciar esto más para las clases que tienen una lógica compleja pero la idea es la misma, tener clases pequeñas y mantener la lógica que necesita.

Solicitud de clase pública
{
String requestId;
}

Supongamos que hay una clase Request que tiene un requestId y ahora digamos que hay n clases que usan esta clase Request. En todos los lugares donde se usa, no sorprende que vea la línea a continuación.

if (requestA.requestId.Equals (requestB.requestId)
{
// Hacer algo
}

En cambio, si la clase Request tuviera un método llamado “es” , se simplificarían todos los usos de lo anterior.

Solicitud de clase pública
{
String requestId;

bool público es (Solicitar otra solicitud)
{
devuelve this.requestId.Equals (otherRequest.requestId);
}
}

Y en n lugares donde tenías el bloque if ahora se verá como

// Esto también es más legible

if (requestA.is (requestB)
{
// Hacer algo
}

¿Qué hace esto y por qué lo hiciste de esta manera?

Poner nombre de variable relevante y comentarios. 😉

Gracias por A2A

Soy un probador, no un desarrollador. Lo siento, no puedo ayudarte aquí.