[Dúvida] Corigam para mim se está certo no meu ta normal

Está correto, a única coisa que mudaria seria passar a variável mensagemDeErro no seu alerta já que o texto é o mesmo. É bom evitar se repetir.

Outra coisa, nos seus if's você está testando se lua == voce, o correto seria checar se nome == voce, já que o nome é a variável que comporta a string "lua".

O código ficaria assim:

let nome = "lua"

let idade = 25;

let numeroDeVendas = 50;

let saldoDisponivel = 1000;

let mensagenDeErro = "Erro! preencha todos os campos";

alert(mensagenDeErro);

let voce = prompt("Ola Qual e o Seu nome");

let longa = prompt("Qual e a sua idade");

if (nome == voce) { 
    console.log("Nome coreto"); 
}

if (idade = longa) { 
    console.log("Idade ideal") 
}

if (idade >=18){
    alert("pode tirar a carteira");
}

valeu