Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@victoriapl
Copy link

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);

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

@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();

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

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.