O problema é que o código nem entra no loop já que o valor inicial não atende a condição colocada. Você tem que executar a primeira vez incondicionalmente usando do ... while
.
console.log(gen(1, 100, 10, 20));
function gen(min, max, excludeMin, excludeMax){
var value;
do {
value = Math.random() * (max - min) + min;
} while(value > excludeMin && value < excludeMax)
return value;
}
Coloquei no GitHub para referência futura.
Eu pensei em setar um valor inicial que atendesse a condição, mas esse caso é complicado, nem sempre será possível.
Ainda há um problema se os parâmetros forem mandados de forma inconsistente. Então seria melhor fazer uma verificação:
//seria bom verificar se recebeu null antes de usar o valor
console.log(gen(1, 100, 10, 20));
function gen(min, max, excludeMin, excludeMax){
if (min > max || excludeMin > excludeMax) {
return null;
}
var value;
do {
value = Math.random() * (max - min) + min;
} while(value > excludeMin && value < excludeMax)
return value;
}
Coloquei no GitHub para referência futura.
Não precisa ser exatamente esta lógica, você dá o tratamento que desejar, mas se o valor que deveria ser menor for maior que o que deveria ser maior, o loop sai imediatamente após o primeiro número. Pode até ser o que você deseja mas fica o alerta.