Skip to content

PP MEX - Felipe Victoria#366

Closed
victoriapl wants to merge 2 commits into
ironhack-labs:masterfrom
victoriapl:master
Closed

PP MEX - Felipe Victoria#366
victoriapl wants to merge 2 commits into
ironhack-labs:masterfrom
victoriapl:master

Conversation

@victoriapl
Copy link
Copy Markdown

Terminamos el Pair Programming y todos los bonus, esperamos los stickers!

Alt Text

for (i = 0; i < hacker1.length; i++ ){
capitalizedName += " "+ hacker1[i].toUpperCase();
}
console.log(capitalizedName);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Muy bien, pero si notan está sobrando un espacio en el início de la cadena. Pueden removerlo usando los métodos .slice(1) o .trim().

}
console.log(capitalizedName);

for (a = hacker2.length; a >= 0; a--){
Copy link
Copy Markdown

@ta-web-mex ta-web-mex Mar 26, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Una pequeña corrección: si notan, hay un espacio en el início del reverseName. Eso pasa porque el índice de los caracteres en la cadena empieza en 0, pero el conteo del length empieza en 1. Para empezar en la última letra de la cadena, el valor inicial de i deberia de ser hacker2.length-1

}
}

checkPalindrome();
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excelente. Una sugerencia: si convierten storeString y palindromeString a minúsculas antes de checarlas, podrían identificar tambien palíndromos en nombres ("Ana", por ejemplo).

@ta-web-mex
Copy link
Copy Markdown

ta-web-mex commented Mar 26, 2019

Excelente trabajo y felicidades por haber terminado los bonus! :octocat:
Hice un par de comentarios con algunas pequeñas sugerencias.

@ta-web-mex ta-web-mex closed this Mar 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants