Queria saber se a forma que fiz é eficiente.

<meta charset="UTF-8">
<script>

    var loginCadastrado = "alura";
    var senhaCadastrada = "alura321";

    var loginInformado = prompt("Informe seu login");
    var senhaInformada = prompt("Informe sua senha");
    var tentativas = 3;

    if(loginCadastrado == loginInformado && senhaCadastrada == senhaInformada ) {
        alert("Bem-vindo ao sistema " + loginInformado);
        } else {

            alert("Senha incorreta. Você tem 3 tentativas antes do sistema travar.");

            while( tentativas >= 1){

                tentativas--;
                loginInformado = prompt("Informe seu login");
                senhaInformada = prompt("Informe sua senha");

                alert("Senha ou login incorretos!");
            } alert("Login rejeitado, encerrando sistema!");

    }
</script>

Basicamente o título. Queria saber se essa forma é eficiente ou se a feita pelo exercício é feita de forma mais otimizada. Ou tanto faz numa aplicação desse tamanho?