function palindrome(str) { const forward = str.replace(/[^a-zA-Z ]/g, "").toUpperCase() const reversed = str.replace(/[^a-zA-Z ]/g, "").toUpperCase() for (let i = 0; i < forward.length; i++) { for (let k = reversed.length - 1; k >= 0; k--) { if (forward[i] === reversed[k]) { return true } else { return false } } } } console.log(palindrome("almostomla"));¿Por qué esto no funciona? ¿Mi ciclo simplemente crea una nueva "s"?
No necesita bucles anidados, eso comparará cada carácter con cualquier otro carácter. Solo desea comparar el primer carácter con el último carácter, el segundo carácter con el segundo hasta el último carácter, y así sucesivamente. Por lo tanto, solo debería haber un solo bucle que incremente i y disminuya k en el paso de bloqueo.
No debe devolver true cuando encuentre una coincidencia, porque podría haber caracteres posteriores que no coincidan. Devuelve false cuando encuentres una discrepancia y devuelve true si superas el ciclo sin volver.
No necesita variables forward e reversed , ya que son lo mismo. Simplemente convierta la cadena de entrada a mayúsculas una vez y utilícela para ambas.
No necesita iterar a través de toda la cadena, puede detenerse cuando llegue al medio.
function palindrome(str) { const upper = str.replace(/[^a-zA-Z ]/g, "").toUpperCase() for (let i = 0, k = upper.length - 1; i < upper.length/2; i++, k--) { if (upper[i] !== upper[k]) { return false } } return true; } console.log(palindrome("almostomla")); console.log(palindrome("almotomla")); console.log(palindrome("almottomla"));Es posible que desee esto:
function palindrome(str) { const forward = str.replace(/[^a-zA-Z ]/g, "").toUpperCase() var n = forward.length for (let i = 0; i < n; i++) { if (forward[i] !== forward[ni-1]) { return false; } } return true; }