curriculum icon indicating copy to clipboard operation
curriculum copied to clipboard

Limitamos el uso de Regex en Text Analyzer?

Open unjust opened this issue 1 year ago • 3 comments

Estamos viendo que muchas usan regex en lugar de bucles. Deberiamos poner algo en el readme ?

unjust avatar Nov 10 '23 16:11 unjust

Prohibamos el uso de ellas en el readme y quitemos cualquier recurso o referencia que hable sobre ellas, al menos en este proyecto

JuanCarlosHM avatar Nov 10 '23 16:11 JuanCarlosHM

Creo que se podría restringir en determinadas funcionalidades, por ejemplo en la que cuenta los números y sugerir que ahí usen isNaN de forma explicita, regex les puede ser útil en el futuro pero restringirlo puede darles la idea que no es útil o que no deben usarlo.

jmontes50 avatar Nov 10 '23 16:11 jmontes50

No lo veo un lugar donde hablamos de regex o compartimos recursos sobre eso. podemos meter una pauta si habia una seccion como "consideraciones tecnicas"

Seria bueno saber, en cual parte estan usando regex - para que especifica requisito? Y que estan reemplazando con el regex - un bucle o en algun metodo como getCharacterCountExcludingSpaces? Ahi cuando mencionamos el requisito podemos decir que recomendamos que no usen regex.

unjust avatar Nov 14 '23 21:11 unjust