La función convertHTML("Dolce & Gabbana & Dolce");
debe convertir los &,<,>,"", y ' que encuentre en el argumento a los valores de html correspondientes: &ersand...etc
function convertHTML(str) {
var ampersand = "&"
var left = "<"
var right = ">"
var doubleq = """
var singleq = "'";
if (str.includes("&")){
var newStr = str.replace(/&/g, ampersand);
}
else if (str.includes("<")){
var newStr = str.replace(/</g, left);
}
else if (str.includes(">")){
var newStr = str.replace(/>/g, right);
}
else if (str.includes('"')){
var newStr = str.replace(/"/g, doubleq);
}
else if (str.includes("'")){
var newStr = str.replace(/'/g, singleq);
}
else {return str}
return newStr;
}
He hecho la función pero hay una situación en la que no funciona y es si en el argumento se dan dos entidades html diferentes: convertHTML("<>")
Entonces sólo convierte la primera que se encuentra: // Resultado: "<>"
Imagino que hay otra manera de hacerlo para que checkee las dos diferentes pero me preguntaba si podía arreglar esto sin tener que encarar el problema de forma diferente, es decir, poner arriba una función checkear()
que compruebe si todavía existen caracteres sin convertir, y si todavía existen pues que los If statemente de abajo se vuelvan a ejecutar.
function convertHTML(str) {
var ampersand = "&"
var left = "<"
var right = ">"
var doubleq = """
var singleq = "'";
function checkear{
if ((str.includes("&") || str.includes("<") || str.includes(">") || str.includes('"') || str.includes("'")){
primerCheck() // vuelve a ejecutar los If statement de abajo
}
else {return newStr}
}
function primerCheck(){
if (str.includes("&")){
var newStr = str.replace(/&/g, ampersand);
}
else if (str.includes("<")){
var newStr = str.replace(/</g, left);
}
else if (str.includes(">")){
var newStr = str.replace(/>/g, right);
}
else if (str.includes('"')){
var newStr = str.replace(/"/g, doubleq);
}
else if (str.includes("'")){
var newStr = str.replace(/'/g, singleq);
}
else {return str}
}
checkear()
return newStr;
}
Pero me estoy perdiendo en los scopes y bueno, no se si esto se puede hacer más sencillo y es posible, o estoy perdiendo el tiempo y debería empezar todo de nuevo.
if
de manera incorrecta. De hecho te devolverá la primera que evalúe cuando se ejecute. Creo que deberías usar un.split
, luego recorrer elArray
resultante y reemplazar, y por último usar un.join
. En fin que opciones tienes varias. Saludosif
seguido unelse if
entonces tú código entra a un soloif
no a los dos, aunque la condición que esté evaluando elelse if
sea verdadera, si ya entró en elif
oelse if
anterior, no entra en ningún otro. La forma fácil de resolver lo que tenías sin explicar muchas cosas más de código es cambiar loselse if
porif
nada másif
con elelse
seguramente el siguiente error que te seguía era porque estabas intentando redefinir una variable ya definida, es decir por usarvar
para que eso no te sucediera tienes que al principio de tu función definirvar newStr = ''
y luego dentro de cadaif
quitar elvar
y solo dejarnewStr = str.replace...
Otro error que posiblemente tendrías es lo mismo que comenté en la respuesta que te dieron, mira el comentario y lo identificarás