Esa es mi historia: uno de mis colegas lo utiliza para revisar todo el código, alojado en el sistema de revisión. No estoy hablando de una revisión adecuada de los cambios en las partes a las que pertenece. Él mira el código archivo a archivo, línea a línea. Cada archivo nuevo y cada modificado. ¡Tengo ganas de ser espiado!
Supongo que si el código ya estaba alojado para controlar el sistema, al menos debería confiar en que sea viable. Mi pregunta es, ¿tal vez soy demasiado paranoico y la práctica de revisar el código de los demás es buena?
PD: Somos un equipo de solo tres desarrolladores, y me temo que si hay más de nosotros, el colega simplemente no tendrá tiempo para revisar todo el código que escribiremos.
fuente
Su colega suena como un desarrollador diligente, debe seguir su ejemplo.
fuente
Miro nuestro sistema de control de versiones de manera similar. Nuestra base de código es demasiado grande para ver todas las líneas, pero trato de obtener una sensación de alto nivel para la mayoría de los cambios. También miro los registros que tienen más probabilidades de tener efectos secundarios y los reviso línea por línea. Por el tiempo mínimo que paso haciendo esto, la recompensa es enorme. (También tenga en cuenta: no soy el único desarrollador en nuestro equipo con este hábito).
Este tipo de revisión tiende a detectar errores o invocar debates semanalmente. Eso ahorra tiempo al hacer QA. Las discusiones abarcan desde las mejores prácticas hasta el diseño de algoritmos y más. La clave en este frente es que todos lo ven como constructivo.
Personalmente, también me da una mejor comprensión de lo que está sucediendo en otras partes de la base de código que no toco regularmente. Cuando otros necesitan ayuda, puedo saltar más rápido. Además, cuando aparecen nuevas ideas, puedo aprovecharlas antes.
fuente
¿Sientes que te espían (!)? Pero desde la perspectiva de su colega, diría que está haciendo lo correcto para el desarrollo de su carrera. Lea el código de otros y descubra cómo diseñan e implementan la lógica, ¡esto le hará ganar mucho!
En mi humilde opinión, si alguien señala algo mal en su código, debe aceptarlo y aprender de ellos sobre cómo escribir un buen código
fuente
Durante 6-7 meses estuve haciendo lo mismo. No para espiar, sino para controlar la calidad. Cada línea del código para una aplicación desarrollada activamente, comprometida con el repositorio central, 2 idiomas principales, algunos otros idiomas, enormes archivos MAKE para 4 plataformas.
Es muy mala práctica . Algún día descubrí que no puedo capturar todo debido a la robustez. El otro argumento en contra de esto es la subjetividad: todos pueden equivocarse.
Es mejor cuando los desarrolladores revisan los códigos de los demás y hay alguien con experiencia para tomar decisiones finales y definir direcciones.
fuente
Las revisiones de código dentro de un equipo (usando ojo de pez , crisol u otras herramientas) son extremadamente importantes y útiles. Lo único mejor es la programación directa de pares para asegurarse de que el código que ingresa al sistema la primera vez esté bien pensado y haya pasado por el cerebro de más de una persona.
fuente
Esto sucedió en mi equipo una vez. Lamentablemente resultó en un juego de culpa. La gente continuamente esperaba que otros revisaran el código y siempre intentaban encontrar algo incorrecto y jugaban al juego de la culpa todo el tiempo.
Espero que tengas una audiencia más madura.
fuente
Esta es una práctica bastante estándar en la industria. Las compañías en las que he trabajado tienen pautas de revisión de código muy estrictas. Incluso uno no le permitiría comprometerse a menos que se haya revisado el código.
No te ofendas ni te sientas vigilado. Piense en ello como una red de seguridad y una experiencia de aprendizaje.
fuente
En un trabajo anterior, el desarrollador senior observaba y revisaba todos los registros y con frecuencia recibía excelentes comentarios que me ayudaban a ser un mejor desarrollador.
En mi trabajo actual, miro muchos de los registros y hace tres días encontré un error y notifiqué al desarrollador.
Esta práctica atrapará errores y mejorará a todo su equipo, si lo acepta.
fuente