Ia só comentar mas ficou grande.

Essencialmente não, afinal ele faz muito pouca coisa.

A não ser que você queira aplicar uma série de coisas que se costuma recomendar em termos de arquitetura, mas seria um absurdo fazer isto em um exemplo tão simples.

Claro que tem pequenos detalhes que podem ser mudados, mas provavelmente mais por gosto do que por necessidade.

- Eu padronizaria os espaçamento entre operadores
- eventualmente daria nomes melhores para as variáveis
- cuidaria para **garantir** o fechamento do arquivo mesmo se der erro, que sequer está sendo feito em condições normais
- Não atribuiria valores para a mesma variável sem necessidade (`op = 0`)
- declararia as variáveis mais próximas de onde são usadas
- talvez usaria um `op > 5` no lugar de `op >= 6`
- mudaria o `op==0` para `op <= 0` e mudaria a mensagem para `"Os valores devem ser entre 1 e 5."`

tem outras perfumarias que podem ser feitas, mas

Se lembrar de mais alguma coisa, eu edito.