¿Es normal recibir muchos comentarios sobre la revisión de código?

Dices que estás en un nuevo trabajo, pero ¿es el primero después de la escuela o tienes mucha experiencia en desarrollo de software y simplemente cambiaste de trabajo? Probablemente hay una gran diferencia entre los dos, pero déjame intentar y darte algunas respuestas plausibles.

A su primera pregunta, la respuesta es sí. Si acaba de salir de la escuela, es probable que no haya estado expuesto al tipo de entorno en el que la codificación es un deporte de equipo y que no lo realicen programadores aislados con poca o ninguna supervisión. En una organización estructurada, existe la expectativa de mantener la coherencia en el estilo de codificación (para que otros puedan ver fácilmente lo que ha hecho), arquitectura (ya que hay modismos preferidos para usar y el diseño de arquitectura de alto nivel probablemente se completó antes de ser asignado al código ) y errores (con suerte eso se explica por sí mismo). Lo mismo ocurriría si realiza la transición desde otro trabajo que no tenía esa estructura.

Para su segunda pregunta, “cuánto tiempo se espera esto”, la respuesta simple (y algo sarcástica) es “mientras siga cometiendo los mismos errores en código / arquitectura / errores”.

Permítanme ofrecer respetuosamente algunos consejos que abordan algo ‘detrás’ de su pregunta. Cuando usa el término “problemas” para referirse a la obtención de comentarios sobre su desempeño, y cuando pregunta “por cuánto tiempo se espera esto”, lo está haciendo sonar como si se tratara de usted.

Las revisiones de códigos no son una forma de ritual o castigo para el chico nuevo, como pintar la cara de un chico dormido con crema de afeitar en una casa de fraternidad. Una empresa lo contrató para hacer un trabajo y esperan que se haga profesionalmente; está recibiendo comentarios valiosos y al mismo tiempo se asegura de producir un producto de mucha mayor calidad que si lo hiciera solo y sin supervisión. Por lo menos, debe sentirse afortunado de trabajar en una empresa que toma su producto tan en serio que siguen procedimientos que le enseñarán muchísimo sobre cosas que quizás no sepa.

Cada vez que participé en el desarrollo de SW que involucra revisiones de código, terminé aprendiendo mucho y siempre el producto generado tenía menos errores, era fácil de modificar / extender para otra persona y era menos costoso de desarrollar.

Último comentario, y lo digo con respeto. Hay una actitud muy contraproducente que aparece en su pregunta y necesita revisar esta actitud. Estas no son cosas “hechas a ti”; no se trata de ti, se trata de ser un profesional. Cuanto antes te des cuenta de esto, más pronto te convertirás en uno.

Una buena revisión del código casi siempre genera comentarios. El objetivo de una buena revisión de código no es una prueba de aprobación / falla, es traer otro par de ojos, otro cerebro a la solución codificada, y ver si hay algo que pueda mejorarse. Es una programación de par retrasado cuando se hace correctamente.

Además, es 100% normal que los nuevos empleados reciban muchos comentarios sobre el estilo. Aún no te has adaptado a cómo se hacen las cosas allí. Comience a aprender de esos y esos comentarios disminuirán.

En mi último puesto estuve allí durante siete años, y trabajé con otros dos desarrolladores que habían estado allí durante más de diez años (¡uno casi 17! Años) y casi todas las revisiones de código entre nosotros tenían al menos un comentario. A veces, mucho, mucho más 🙂

¡No se preocupe por la cantidad de comentarios, úselos para aprender! No importa lo bueno que seas, nunca escribirás un código perfecto. ¡Disfruta de un segundo par de ojos y perspectiva para escribir el mejor código que puedas!

Sí, habrá un montón de cosas que haces mal, como nuevo empleado.

  • Tienes que trabajar duro para aprender la arquitectura del proyecto en el que estás trabajando.
  • No conoce el estilo y las convenciones a las que se adhiere un equipo de software, y tiene que aprender a adaptarse. Es posible que pueda ayudarse un poco y solicitar que se instale un “estilo de código” en su IDE para que se aplique automáticamente a su código, por lo que coloca las llaves en los lugares correctos y maneja automáticamente otras preferencias de formato.
  • Eres un novato e incluso probar tu código aún puede dejar pasar algunos errores lógicos que conducen a problemas como un rendimiento subóptimo o casos extremos no manejados. Ciertamente no debería tener errores de tiempo de compilación, y los errores de tiempo de ejecución que puede descubrir probando su código más a fondo.

Buenas noticias sin embargo. Aparentemente tiene la ventaja de estar en un equipo de software excelente y atento, y debe aprovecharlo al máximo, absorbiendo todas las sugerencias mientras pueda. Si está aprendiendo de manera proactiva, los comentarios de estilo / convención deberían desaparecer de inmediato, los errores deberían desaparecer en cuestión de meses, y los comentarios de arquitectura probablemente no se reduzcan durante años, si es que lo hacen. La arquitectura del software es difícil.

Esto es muy normal. De hecho, si no recibiera muchos comentarios, le aconsejaría que buscara activamente un mentor talentoso para que se lo diera. Los comentarios de estilo desaparecerán rápidamente a medida que se ajuste al estilo del equipo (y las herramientas automáticas pueden ayudar). Los errores se reducirán a unos pocos aquí y allá porque, bueno, todos cometemos errores y para eso están las revisiones de código. Las discusiones sobre arquitectura y diseño son las más valiosas y formarán parte de la mayoría de los códigos que haya escrito como desarrollador de software. Eventualmente, sin embargo, se convertirá más en una discusión de ida y vuelta sobre los méritos de los diferentes enfoques, en lugar de decirte directamente cómo debes abordar el problema (como junior).

Para calificar mi respuesta a esta pregunta, he sido desarrollador durante 23 años y administré desarrolladores durante 18 años.

Obtener una gran cantidad de comentarios en una revisión de código no es normal, pero mucho depende de las personalidades de los revisores.

En su caso, no me preocuparía cuántos comentarios hay, sino el espíritu con el que se hacen. Supongo que porque eres nuevo en esto, la mayoría de los comentarios son útiles y simplemente pretenden ser útiles para ti. Podría haber un elemento de algunas personas que quieran “probar” al nuevo tipo. Pero la mayoría de los ingenieros de software no son así.

Tomaría todos los comentarios al pie de la letra, que las personas realmente están tratando de ayudarlo. Si hay algunos comentarios que no entiendes, pediría aclaraciones a las personas en privado.

Con respecto a los comentarios de estilo, generalmente sugeriría que si decide incorporar esos cambios depende de usted. En su caso, sin embargo, usted es bastante nuevo y la mayoría de los nuevos ingenieros tienen un estilo de codificación realmente terrible. Un buen estilo de codificación generalmente permite que las personas entiendan lo que ha hecho en el futuro con mayor claridad.

En resumen, asumo las buenas intenciones de tus revisores y acepta que, como nuevo ingeniero, tienes mucho que aprender que no puedes aprender de la escuela.