Veo varios problemas en tu código:
function contarCaracteres(valor = prompt("Escriba algo")){
if (valor == 2){
Esto no es un error, pero en general no es una buena práctica: la función prompt
devuelve texto, no un número. Tu código funciona hasta aquí porque == usa coerción de tipos y transforma los elementos comparados para que tengan el mismo tipo. Es más correcto transformar el texto leído a number
y usar el triple igual ===
para comparar de manera estricta: if (+valor === 2) {
.
while (val == 2) {
Eso sí es un fallo: la variable val no existe, con lo que esta comparación es igual a comparar undefined == 2
, lo que siempre dará false
. Esto hace que las siguientes dos líneas no se ejecuten nunca y por tanto val
nunca tendrá asignado un valor. Podrías reutilizar valor
aquí.
let val = prompt("Vuelva a escribir");
val();
}
} else{
document.write("El texto ${valor} tiene ${valor.length} caracteres");
Esta línea tiene otro fallo: estás intentando usar un string template pero al usar comillas dobles ("
) en lugar de las comillas inversas, los placeholders no serán interpretados.
}
}
contarCaracteres();
Y ahora en conjunto: no hay problemas por usar un while dentro de un if, pero si estás comprobando en ambos lo mismo, el if es innecesario. Has de familiarizarte con el control del flujo del programa (se consigue con la experiencia, no te preocupes) y saber cuándo aplicar condiciones y/o bucles:
function contarCaracteres(valor = prompt("Escriba algo")){
while (+valor !== 2) {
document.write(`El texto ${valor} tiene ${valor.length} caracteres\n`);
valor = prompt("Vuelva a escribir");
}
}
contarCaracteres();