Primero necesita tener estándares de codificación y estos son más que simples sintaxis. Cuando las personas comienzan en su empresa, deben aprender las pautas de su empresa tanto como sea posible antes de comenzar a codificar . Si en el proceso de revisión se encuentran todo tipo de violaciones, lo más probable es que:
- no se arregla debido a limitaciones de tiempo
- resultó ser más molesto de lo que valen las pautas
Las pautas deben tener sentido y debe haber herramientas adecuadas para encontrar infracciones y refactorizar lo más fácil posible. Siempre mire el objetivo de las pautas y la revisión del código
El objetivo en mi mente es hacer que el código sea lo más uniforme posible y encontrar problemas de mantenimiento y legibilidad. Un objetivo secundario puede ser poner al día a más personas con una determinada pieza de software.
Las pautas en mi mente podrían, por ejemplo, existir de:
- sintaxis general y pautas de codificación (elija una que ya exista y use herramientas que verifiquen automáticamente)
- Manejo adecuado de excepciones
- Registro adecuado
- Buen uso de los paradigmas para el lenguaje (SOLID para lenguajes OO)
- Dependencias obvias y bien pensadas entre componentes (use herramientas como NDepend)
- Script de compilación de trabajo
- Documentación presente (inicio del desarrollador, manual de instalación)
- bibliotecas internas para usar
- Compañía de Policías
- herramientas de terceros que no están permitidas
- Pruebas unitarias presentes y sin fallas
- cobertura de código del 90%
- ...
Con eso en su lugar, la revisión del código consiste en que el software se verifique con las pautas y:
- discutir violaciones con el programador
- arreglar violaciones innecesarias
- comentar las violaciones necesarias