Entiendo que cada persona tiene su propio estilo de programación y que debería poder leer los estilos de otras personas y aceptarlos tal como son. Sin embargo, ¿se consideraría a uno como un programador descuidado si el estilo de codificación de uno fuera inconsistente en el estándar en el que estaban trabajando?
Algunos ejemplos de inconsistencias podrían ser:
- A veces nombra variables privadas con _ y otras veces no
- A veces, con sangrías variables dentro de los bloques de código
- No se alinean los apoyos hacia arriba, es decir, en la misma columna si se usa comenzar con el nuevo estilo de línea
- El espaciado no siempre es consistente alrededor de los operadores, es decir, p = > p + 1, p + = 1 frente a otras veces p = > p + 1 o p = > p + 1 etc
¿Esto es algo que, como programador, debería ocuparme de abordar? ¿O es algo tan insignificante que al final del día no debería preocuparme por lo que ve el usuario final y si el código funciona en lugar de cómo se ve mientras se trabaja?
¿Es una programación descuidada o simplemente sobre la selección obsesiva de liendres?
EDIT : Después de algunos excelentes comentarios, me di cuenta de que podría haber omitido alguna información en mi pregunta. Esta pregunta surgió después de revisar el registro de código de otros colegas y notar algunas de estas cosas y luego darme cuenta de que he visto este tipo de incoherencias en los registros anteriores. Luego me puse a pensar en mi código y en si hago las mismas cosas y noté que normalmente no lo hago, etc. No estoy sugiriendo que su técnica sea mala o buena en esta pregunta o si su forma de hacer las cosas es correcta o incorrecta. .
EDIT : para responder a algunas consultas y obtener más comentarios positivos. La instancia específica en la que se realizó esta revisión fue usar Visual Studio 2010 y la programación en c #, por lo que no creo que el editor cause ningún problema. De hecho, sólo debería ayudar, espero. Lo siento si dejé esa información y afecta las respuestas actuales. Intenté ser un poco más genérico para comprender si esto se consideraría descuidado, etc. Y para agregar un ejemplo aún más específico de una pieza de código que vi durante la lectura del registro:
foreach(var block in Blocks)
{
// .. some other code in here
foreach(var movement in movements)
{
movement.Moved.Zero();
} // the un-formatted brace
}
Una cosa tan pequeña que sé, pero muchas cosas pequeñas se suman (???), y tuve que echar una doble mirada al código en el momento para ver dónde estaba todo alineado, supongo. Tenga en cuenta que este código fue formateado adecuadamente antes de este check-in.
EDIT : después de leer algunas respuestas geniales y pensamientos variados, el resumen que he tomado de esto fue.
- No es necesariamente un signo de un programador descuidado. Sin embargo, como programadores tenemos el deber (para nosotros y otros programadores) de hacer que el código sea lo más legible posible para ayudar en el desarrollo continuo. Sin embargo, puede sugerir deficiencias, que es algo que solo es posible revisar caso por caso (persona por persona).
- Hay muchas razones por las que esto puede ocurrir. Deben tomarse en contexto y trabajar con la persona / personas involucradas si es razonable. ¡Tenemos el deber de intentar y ayudar a todos los programadores a convertirse en mejores programadores!
- En los viejos tiempos, cuando el desarrollo se realizaba con un buen bloc de notas antiguo (u otra herramienta de edición de texto simple), esto ocurría con mucha más frecuencia. Sin embargo, ahora contamos con la asistencia de los IDE modernos, por lo que, aunque no deberíamos necesariamente convertirnos en OTT al respecto, probablemente debería abordarse en cierta medida.
- Nosotros, como programadores, variamos en nuestros estándares, estilos y enfoques de soluciones. Sin embargo, parece que, en general, todos tomamos PRIDE en nuestro trabajo y, como rasgo, es algo que puede diferenciar a los programadores. Hacer lo mejor de nuestras capacidades, tanto interno (código) como externo (resultado del usuario final) nos ayuda a darnos esa gran palmadita en la espalda que quizás no busquemos pero que llena nuestro corazón de orgullo.
- Y finalmente, para citar a CrazyEddie de su publicación a continuación. No te preocupes por las cosas pequeñas