Isso funciona em todos os browsers, mas eu diria que é antiquado, pois é do tempo do chamado "DOM Level 0", quando não havia nenhuma especificação formal de como esse tipo de coisa deveria funcionar. Curioso a professora ensinar assim. Uma maneira mais atual seria:
var estado = document.getElementById('estados').value;
Ou:
var estado = document.querySelector('#estados').value;
QuandoQuanto à definição do event listener inline (no próprio HTML, em <select id="states" name="estados" onChange="mostraCapital()">
), é altamente recomendável evitar. O motivo é a separação de responsabilidades: num código organizado, o HTML deve ser responsável somente pela estrutura do conteúdo, e o JavaScript deve ser responsável pelo comportamento. Nesse trecho você tem estrutura e comportamento misturados no próprio HTML. Para levar para o JS, removarremova o onChange="mostraCapital()"
e use isto no lugar:
var campo = document.getElementById('estados');
campo.addEventListener('change', mostraCapital, false);