Estoy de acuerdo con los otros colaboradores que han dicho que esto es principalmente un problema de estilo, en lugar de un código malo o anticuado. Y estoy de acuerdo en que puede haber beneficios al discutirlo con su compañero de trabajo.
Por mi parte, puedo ver al menos una ventaja clave de este estilo. Si se va a acceder a la matriz mediante un índice, lo cual es probable, la correspondencia entre el idex y el valor es bastante evidente. En comparación, el estilo que prefiera requiere contar las entradas, desde cero, para encontrar el índice de una entrada.
También evita que el texto salga del borde derecho de la ventana, donde debe desplazarse para poder verlo. Con los IDE modernos, todo el bloque de líneas se puede envolver en una directiva de plegado de código para que pueda colapsarla.
Mantener estas declaraciones en un archivo separado significa que los cambios en ellas se rastrearán por separado en el sistema de control de origen desde los cambios a la lógica. Por lo tanto, puede ver en el registro si se cambió la lógica o se cambiaron las constantes, y por quién y cuándo, sin tener que pasar por la tediosa tarea de comparar versiones en el nivel de origen para determinar qué se cambió.
- Cómo tratar con personas que piensan cosas sobre mí que no son ciertas.
- Mi padre es trabajador y mi madre es campesina y realmente me aman. Sé que confiar en el trabajo para comer no es vergonzoso, pero ¿cómo puedo superar el sentimiento de inferioridad?
- ¿Preferirías caer accidentalmente un martillo en tu dedo del pie o hacerlo accidentalmente en un dedo del pie de un completo extraño?
- Me besé con un chico cuando estaba borracho y ahora me siento raro por eso. ¿Cómo manejo esta situación?
- ¿Por qué formamos una imagen inmediata similar a otra persona nueva que se nos presenta?
En cuanto al tamaño del archivo, aunque el estilo de su compañero de trabajo requiere más líneas, no requiere más bytes. Un separador de línea requiere 2 caracteres en Windows (solo uno en Unix), mientras que una lista de inicializadores requiere 2 caracteres (uno para la coma y otro para el espacio) en ambas plataformas.
Un inconveniente de este estilo es que es más lento de analizar, pero la diferencia es, en el mejor de los casos, marginal, a menos que haya miles de elementos de matriz.
Uno de los colaboradores sugirió que debería tener derecho a cambiar el estilo de codificación una vez que sea responsable del código. Estoy de acuerdo con eso, pero con restricciones.
- Primero, si hay un conjunto de estándares de codificación que debe cumplir, entonces debe seguirlos. Si no estás de acuerdo con ellos, entonces trabaja para cambiarlos.
- Segundo, tenga en cuenta que tener estilos de codificación mixtos en el mismo cuerpo de código es el equivalente cognitivo de pasar de un camino pavimentado a un camino de grava. Es mejor ser consistente. Personalmente, no me importa demasiado si el estilo cambia incluso entre archivos, pero, por favor, ¡¡¡no dentro del mismo archivo !!! Entonces, si va a cambiar el estilo, cámbielo uniformemente dentro de un cuerpo cohesivo de código relacionado.
Además, sea sensible al hecho de que cambiar de estilo crea una incoherencia indeseable, para lo cual el remedio consiste en cambios más generalizados, cuyo resultado es mucho trabajo con quizás poco beneficio real, y con el riesgo adicional de introducir errores.
He estado en ingeniería de software durante cuarenta años y he administrado equipos de hasta 120 personas. Honestamente puedo decir que en todo ese tiempo no creo que me haya encontrado con un ingeniero que pensara que el código de otra persona estaba escrito de forma más limpia de lo que él o ella podría escribirlo. De hecho, como gerente, a menudo era bastante difícil lograr que los ingenieros resistieran la tentación de cambiar el estilo o el código de refactorización simplemente porque no les gustaba la forma en que estaba escrito. Y he tenido que lidiar con las consecuencias de tales cambios de código cuando resultaron en nuevos errores, o peor, la reaparición de errores viejos.