mas a pergunta era mais em relação à boas práticas mesmo, por usar uma palavra reservada pra nomear a variável
Oi Juliana,
obrigado pela observação!
Você tem toda razão, é sempre bom evitar o uso de palavras reservadas para nome de variáveis. Nesse caso, em particular, como a função max não está sendo utilizada dentro da função em questão, não faria diferença, mas foi um descuido nosso.
Se você desejar pode alterar o nome desse parâmetro, sem prejuizo com relação a nota. Só não pode é eliminar o parâmetro, ou seja, sua função deve continuar recebendo 3 parâmetros, senha, chute e o número máximo de dígitos, nessa ordem.
Outra coisa importante com relação ao parâmetro max é que a especificação dessa função é mais genérica (poderosa!) que o exigido pelo enunciado do EP, ou seja, deve funcionar para max digitos e não apenas para 4 dígitos, o que permitira usar essa função com jogos de 10 dígitos por exemplo.