Version corta
Si el trabajo consiste en mantener una aplicación, las habilidades que necesita probar durante las entrevistas son:
La capacidad de comprender la gran base de código con su documentación, pruebas unitarias , etc.
La capacidad de refactorizar el código y traer cambios sin romperlo todo.
Pedirle a la gente que lea el código no te ayudará a evaluar esas habilidades.
Versión larga
¿Te pidieron que escribieras el código? En caso afirmativo, como señaló Sign en su respuesta , esto es suficiente. Si generalizamos un poco, una persona que escriba un código fuente claro y fácil de entender podría leer el código fuente escrito por otras personas.
Si no se le pidió que escribiera el código, entonces, probablemente fue entrevistado por una persona del departamento de recursos humanos. Tales entrevistas no pueden ser demasiado técnicas, y en su mayoría no valen nada, ya que no valoran sus habilidades y su capacidad para trabajar bien, sino la cantidad de años que pasó en la universidad y otras cosas que no tienen nada que ver con el trabajo.
Hay algunas razones más para no pedir leer el código para un trabajo de mantenimiento:
1. Es difícil hacerlo de manera confiable
Concretamente, ¿qué harías si fueras un entrevistador? Haz que tus candidatos lean un código. ¿Qué código? ¿En que idioma? ¿Qué tan bien o mal escrito? ¿Con o sin comentarios? ¿Con o sin documentación?
Más importante aún, ¿qué dice sobre el candidato? ¿Qué tan bien se correlaciona con la propia base de código?
Supongamos que tiene que mantener una aplicación VB.NET heredada. Usted sabe que el código fuente es en su mayoría feo y no probado, y algunos comentarios son obsoletos o engañosos. Durante los últimos tres meses, tuvo un desarrollador muy hábil trabajando en la solución; refactorizó y probó la unidad de las partes más críticas de la aplicación, agregó comentarios donde era necesario hacer comentarios y, lo más importante, escribió documentación detallada sobre la arquitectura general, las partes críticas y las trampas.
Ahora está contratando a un desarrollador para mantener esta base de código. Durante una entrevista, ¿le daría un código heredado (feo no probado) o el código que fue refactorizado por el desarrollador anterior?
¿Darías la documentación? Para leer la documentación, el candidato deberá pasar al menos unas horas. Esto hace que sea imposible hacerlo durante una entrevista.
2. Leer un fragmento de código corto no es lo mismo que leer un código de un proyecto familiar
Recuerde, el trabajo es mantener un proyecto. Es difícil mantener una base de código grande los primeros días o semanas cuando no está familiarizado con el proyecto. Es mucho más fácil hacerlo después de unos meses cuando ha escrito toda la documentación y tiene una visión clara de la base de código general.
Lo más importante para probar es si la persona será eficiente esos meses . No le importa si la persona no podrá entender nada durante los primeros dos días.
Al pedirle a una persona que lea un pequeño fragmento de código desde cero, no está probando cómo esta persona podría lidiar con una base de código familiar y documentada de miles de LOC .
3. Mantener el código fuente no es solo leerlo
Cuando mantiene una base de código, la está modificando . Un desarrollador que solo lee código no aporta nada útil a su empresa.
Las habilidades útiles son la capacidad de refactorizar el código , agregar pruebas unitarias , predecir el impacto de un cambio , etc. No prueba esas habilidades pidiéndole a una persona que lea el código durante la entrevista.