[MAD][WEB-PT0418][Alejandro González] - #121
Conversation
| normalString = normalString + palindromeString[i]; | ||
| } | ||
| } | ||
|
|
There was a problem hiding this comment.
Y cómo harías para quitar las comillas de la última frase? porque la última es palíndrome si quitas las comillas y no evalúas las comillas, y es más, cómo quitarías cualquier símbolo que te pudiese llegar a poner para comprobar el suceso? (Lo que has hecho está bien, pero cómo lo pensarías para que te valiese para más casos?)
Tip: dale un repaso a cómo harías el ejercicio de localeCompare sin usar dicho método
| } | ||
| } | ||
| console.log("Number of words: " + wordCount); | ||
|
|
There was a problem hiding this comment.
No es mala idea ir contando los espacios para saber el número de palabras, pero si ya has utilizado la función split(), por qué no mejor:
nWords = text.split(' ').length
| } | ||
| console.log("Number of words: " + wordCount); | ||
|
|
||
| let wordAux = text.split(" ").filter(item => item === 'et'); |
There was a problem hiding this comment.
Está bien y es correcto, mi recomendación es que lo escribas de la siguiente manera por ahora:
var et = text.split(' ').filter(function(item) { return item === 'et' })
No sé cuál es tu conocimiento de js, pero las arrow functions son de ES6 e implican algo más que una semántica más limpia que versiones anteriores de js. Hasta que lleguemos a ver ES6 te recomiendo utilizar 'function()'
|
Como revisión final, a veces pones ';' al final de las sentencias y a veces no. Has utilizado alguna vez var y la mayoría let para definir variables. Puedes poner o no ';' pero debes estandarizar si ponerlos o no, no mezclarlos, y en cuanto a la definición de las variables, utilliza por ahora var, cuando lleguemos a ES6 ya utilizarás let. Por lo demás, buen trabajo y sigue así! 👍 |
|
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.