Discusión

Consejo para que no les suceda como a mi xD

@alexword·26/3/2026Explicación

No lo habia notado, pero incluso cuando tienes un comentario con un numero como ejemplo /100 es interpretado como numero magico y te dara el error jajaja, si pueden borrar los comentarios default pienso que es lo mejor

8respuestas
Respuestas

Si, estoy trabajando en eso, porque tengo expresiones regulares para evaluar las cosas, pero estos ejercicios de refactorización tengo que trabajarlos más, pero estoy en eso :)

Gracias por todo el feedback

JL

Estimado Profesor Fernando,

Reciba un cordial saludo.

En relación con el ejercicio asignado, procedí a implementar la solución utilizando el patrón Strategy, conforme a lo indicado. Como referencia, utilicé el siguiente recurso: https://refactoring.guru/es/design-patterns/strategy/typescript/example.

Sin embargo, al aplicar dicha solución, el sistema de evaluación arrojaba dos observaciones: el uso de números mágicos y que la función principal excedía las 40 líneas. En el caso de los números mágicos, tomé la precaución de eliminarlos, incluyendo aquellos presentes en comentarios, ya que aparentemente también son considerados por la validación.

Posteriormente, al refactorizar la solución sin aplicar el patrón Strategy y limitándome a extraer la lógica en funciones, el ejercicio logró pasar satisfactoriamente todos los tests.

En ese sentido, considero que podría existir alguna limitación o inconsistencia en las pruebas automatizadas respecto a la validación del uso de patrones de diseño. Tal vez sería conveniente complementar dichas pruebas con algún mecanismo que verifique explícitamente la implementación del patrón requerido (por ejemplo, validando la estructura de interfaces o clases asociadas a Strategy).

Saludos cordiales.

Gracias Juan,

Estoy analizando qué otras formas tengo para realizar estos ejercicios de refactorización y encajarlos en el proceso que quiero, porque sí, esto de los números mágicos es un problema y poder detectar patrones deseados también estará retador, pero estoy analizándolo.

Gracias por el aporte

@eltamales20/4/2026

Apenas lo iba a comentar y me encuentro que también tuviste el mismo problema jajaja, estuve como 20 mins intentando encontrar el dichoso numero mágico, ojala hubiera leído esto antes jaja

Si, luego de terminar un trabajo, quiero pulir esta revisión de ejercicios de refactorización, porque tengo que adaptarlos mejor.

Hola, gracias a todos por los comentarios

Yo estaba teniendo problemas con este tipo de ejercicios porque no acabo de entender qué estoy haciendo mal. Seguiré intentándolo ya que alguno ya lo ha resuelto. Si existe alguna manera de paserle el código a alguien para saber qué hago mal estaría genial también. No tengo mucha práctica y menos con el tipado de TS.

Saludos y gracias de antemano

@estouchedisindo9/5/2026editado

Hola de nuevo

Gracias a todos los que habéis compartido el código, sigo sin tener claro cual es el problema. Mi solución no es como la vuestra, pero sí he extraído todos los números a constantes (me faltaba alguno) y he cambiado las expresiones de función (con const) a declaraciones de función. En fin, a este paso ya debo tener mal hasta el cálculo de la tarifa final de darle tantas vueltas. Ja, ja, ja. ¡¡Gracias por vuestra generosidad!! Suerte y saludos

ACTUALIZACIÓN: AÚN ESTABA USANDO UN PAR DE NÚMEROS, el mínimo de distancia y de pasajeros. Se me queda pequeño el editor en línea para ver estos casos. También me decía a mí que el mínimo de líneas de la función principal tiene que ser 40; el único problema era ese par de números. Gracias, Fernando, buen trabajo en cualquier caso, no debe ser fácil los test de los ejercicios de refactorización.

Ya estoy en esos de refactorización y arreglando unos de async/await, esto habilitará otro panorama de ejercicios pronto :)

Escribir un comentario

Debes iniciar sesión para publicar un comentario.