Versão curta
Se o trabalho consiste em manter um aplicativo, as habilidades que você precisa testar durante as entrevistas são:
A capacidade de entender a grande base de código com sua documentação, testes de unidade , etc.
A capacidade de refatorar o código e trazer alterações sem quebrar tudo.
Pedir às pessoas que leiam o código não ajudará você a avaliar essas habilidades.
Versão longa
Você foi solicitado a escrever código? Se sim, como Sign observou em sua resposta , isso é suficiente. Se generalizarmos um pouco, uma pessoa que escrever código fonte claro e fácil de entender seria capaz de ler o código fonte escrito por outras pessoas.
Se você não foi solicitado a escrever código, então provavelmente foi entrevistado por uma pessoa do departamento de recursos humanos. Essas entrevistas não podem ser muito técnicas e são inúteis, pois não agregam suas habilidades e sua capacidade de funcionar bem, mas sim o número de anos que você passou na faculdade e outras coisas que não têm nada a ver com o trabalho.
Existem mais alguns motivos para não solicitar a leitura do código de um trabalho de manutenção:
1. É difícil fazer de maneira confiável
Concretamente, o que você faria se fosse um entrevistador? Faça seus candidatos lerem algum código. Qual código? Em que língua? Quão bem ou mal escrito? Com ou sem comentários? Com ou sem documentação?
Mais importante, o que isso diz sobre o candidato? Quão bem ele se correlaciona com a própria base de código?
Digamos que você tenha um aplicativo VB.NET herdado para manter. Você sabe que o código fonte é geralmente feio e não testado, e alguns comentários estão desatualizados ou enganosos. Nos últimos três meses, você teve um desenvolvedor muito habilidoso trabalhando na solução; ele refatorou e testou por unidade as partes mais críticas do aplicativo, adicionou comentários onde havia necessidade de comentários e, o mais importante, escreveu documentação detalhada sobre a arquitetura geral, as partes críticas e as armadilhas.
Agora você está contratando um desenvolvedor para manter essa base de código. Durante uma entrevista, você daria um código legado (feio não testado) ou o código que foi refatorado pelo desenvolvedor anterior?
Você daria a documentação? Para ler a documentação, o candidato precisará passar pelo menos algumas horas. Isso torna impossível fazer durante uma entrevista.
2. Ler código curto não é o mesmo que ler código de um projeto familiar
Lembre-se, o trabalho é manter um projeto. É difícil manter uma grande base de código nos primeiros dias ou semanas em que você não está familiarizado com o projeto. É muito mais fácil fazê-lo após alguns meses, quando você tiver escrito toda a documentação e ter uma visão clara da base de código geral.
A coisa mais importante a testar é se a pessoa será eficiente nesses meses . Você não se importa se a pessoa não será capaz de entender nada nos dois primeiros dias.
Ao pedir a uma pessoa que leia um pequeno pedaço de código do zero, você não está testando como essa pessoa seria capaz de lidar com um código familiar e documentado de milhares de LOC .
3. Manter o código fonte não é apenas lê-lo
Quando você está mantendo uma base de código, está modificando -a. Um desenvolvedor que apenas lê código não traz nada de útil para sua empresa.
As habilidades úteis são a capacidade de refatorar o código , adicionar testes de unidade , prever o impacto de uma mudança etc. Você não testa essas habilidades pedindo a uma pessoa que leia o código durante a entrevista.