[wedptmad0418] Edna Domingos #134
Conversation
ta-web-mad
left a comment
ta-web-mad
left a comment
There was a problem hiding this comment.
Dale un repasillo a lo que te he puesto Edna, por lo demás, muy buen trabajo! 👍
|
|
||
|
|
||
| hacker1=hacker1.toUpperCase() + ""; | ||
|
|
There was a problem hiding this comment.
Por qué concatenas comillas al final?, las comillas sin espacio es igual a nada. No son necesarias
|
|
||
| space = "", | ||
| navegator = hacker1.split(space); | ||
|
|
There was a problem hiding this comment.
Está bien igualar variables para acortar lo que tienes que escribir, pero en el caso de tu variable 'space', que además has olvidado declarar el tipo de variable (var), escribes más poniendo 'space' que las comillas.
Podías haberlo hecho igual con:
var navigator = hacker1.split("")
| o += s[i]; | ||
| return o; | ||
| } | ||
|
|
There was a problem hiding this comment.
Bien hecho, pero intenta usar nombres de variables que definan bien el sentido que tienen, s podría ser mejor string y la o podría ser mejor newString. Una vez que sabes cómo invertir la cadena manualmente, échale un ojo a las funciones split( ), reverse( ) y join( )
| navegator = hacker1.split(space); | ||
|
|
||
| console.log(navegator); | ||
|
|
There was a problem hiding this comment.
Te falta pasarlo a cadena y separar cada letra por espacios. La función split( ) convierte la cadena sobre la que estás trabajando en un array separando cada elemento en función del argumento que le des a split( )
| cadenaSinEspacios += letrasEspacios[i]; | ||
| } | ||
| } | ||
|
|
There was a problem hiding this comment.
Por qué has usado for...in y no el otro bucle for?
| // Alguna letra es distinta, por lo que ya no es un palindromo | ||
| iguales = false; | ||
| } | ||
| } |
There was a problem hiding this comment.
Para ahorrar líneas si niegas la primera condición te ahorras el else:
if(!letras[i] === letrasReves[i]){ iguales = false break }
Y deberías usar para estos casos una igualación estricta '===' en vez de una igualación simple '=='
Y cuando se cumpla la condición que no quieres, hacer un break, para salir del bucle, ya que sabes que no van a ser palíndromos
|
This pull request has been automatically marked as stale because it didn't have any recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This pull request is closed. Thank you. |
2 participants
No description provided.