search

HiginioUrbano / practica_dwec_gestor_presupuesto

Práctica de Gestión de Presupuestos para el módulo de Desarrollo Web en Entorno Cliente
GNU General Public License v3.0
0 stars 0 forks source link

Problema en la funcion filtrarGastoWeb #13

Closed HiginioUrbano closed 2 years ago

HiginioUrbano commented 2 years ago

Buenas @pedroprieto estaba realizando la función filtrar gastoWeb y me he atascado, puesto que tengo un error, pero con los datos que me da el cypress y tal no consigo saber en qué lugar tengo exactamente el error, me podrías echar un cable?

HiginioUrbano commented 2 years ago

He actualizado el código agregando un limpiador como hago en la función repintar para vaciar la lista que ya estaba agregada de forma predefinida. Me ha costado verlo puesto que la función en la guía no lo requería, pero casualmente en siempre era el doble y lo he visto, pero me sigue sin pasar el test de manera completa, se me para a mitad, concretamente en la parte que comprueba las descripciones, pero no se porque.

HiginioUrbano commented 2 years ago

@pedroprieto la reabro que la habia cerrado sin querer

pedroprieto commented 2 years ago

Buenas,

Ábrelo en el navegador y mira la consola a ver si hay algún error. Envíame captura.

HiginioUrbano commented 2 years ago

He vuelto a actualizar el código, solucionando lo de las descripciones, ahora el error parece estar en las etiquetas.

image

HiginioUrbano commented 2 years ago

Te adjunto la captura de pantalla

HiginioUrbano commented 2 years ago

Sigo dándole vueltas al código, he detectado el error, está a la hora de filtrar las etiquetas, me quedo con 2 y debería quedarme con 3, pero nada lo que he probado soluciona el problema y estoy desesperado ya :(

pedroprieto commented 2 years ago

Buenas,

https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/blob/a5b4b61b351d75dfb0160b34ac3acf89cd1834e3/js/generarDatosEstaticos.js#L18

El gasto que creas aquí tiene la etiqueta Transporte y no transporte. Como la búsqueda distingue mayúsculas de minúsculas, no devuelve ese gasto en el filtrado.

Fallo difícil de pillar... :wink:

Un saludo!!

pedroprieto commented 2 years ago

Igual sería buena idea modificar la expresión regular para que no distinga mayúsculas de minúsculas... Me lo apunto para el año que viene!!

HiginioUrbano commented 2 years ago

Pero en teoría tengo puesto /gi, que la i es lo que se usa para quitar está distinción, no?

El mar., 23 nov. 2021 18:54, Pedro Prieto @.***> escribió:

Buenas,

https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/blob/a5b4b61b351d75dfb0160b34ac3acf89cd1834e3/js/generarDatosEstaticos.js#L18

El gasto que creas aquí tiene la etiqueta Transporte y no transporte. Como la búsqueda distingue mayúsculas de minúsculas, no devuelve ese gasto en el filtrado.

Fallo difícil de pillar... 😉

Un saludo!!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/issues/13#issuecomment-976938075, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUM3WNOTZCTCHKOIEVIKO5TUNPIMNANCNFSM5ITIAA3A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

pedroprieto commented 2 years ago

Sí, la expresión regular la detecta. Y le pasa a la función de filtrado un array de etiquetas que incluye Transporte. Pero el problema no está ahí, sino en la función de filtrado: https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/blob/a5b4b61b351d75dfb0160b34ac3acf89cd1834e3/js/gestionPresupuesto.js#L225

Aquí usas un operador de comparación de igualdad, así que los textos tienen que ser iguales. Y "transporte" != "Transporte".

Saludos!

HiginioUrbano commented 2 years ago

Y que hago? modificar esa función?

El mar, 23 nov 2021 a las 19:02, Pedro Prieto @.***>) escribió:

Sí, la expresión regular la detecta. Y le pasa a la función de filtrado un array de etiquetas que incluye Transporte. Pero el problema no está ahí, sino en la función de filtrado:

https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/blob/a5b4b61b351d75dfb0160b34ac3acf89cd1834e3/js/gestionPresupuesto.js#L225

Aquí usas un operador de comparación de igualdad, así que los textos tienen que ser iguales. Y "transporte" != "Transporte".

Saludos!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/issues/13#issuecomment-976948644, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUM3WNP5KK3GNPFWIHYY423UNPJL3ANCNFSM5ITIAA3A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

HiginioUrbano commented 2 years ago

Y en ese caso, como lo hago? porque solo me queda eso, tengo hasta la práctica de almacenamiento realizada

HiginioUrbano commented 2 years ago

He pasado el test primero añadir .toLowerCase en ambos comparadores, pero cuando mete (transporte, casa) me da error también

pedroprieto commented 2 years ago

No, no... Cambia el texto de la etiqueta para que ponga transporte con minúscula. No te compliques.

HiginioUrbano commented 2 years ago

Que lo hago en el archivo de test?

El mar., 23 nov. 2021 20:36, Pedro Prieto @.***> escribió:

No, no... Cambia el texto de la etiqueta para que ponga transporte con minúscula. No te compliques.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/issues/13#issuecomment-977070554, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUM3WNI4TWGWMEYTU3WCXY3UNPULLANCNFSM5ITIAA3A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

HiginioUrbano commented 2 years ago

Y de ser asi, en que archivo de test está ese?

pedroprieto commented 2 years ago

No hombre. En tu archivo: https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/blob/a5b4b61b351d75dfb0160b34ac3acf89cd1834e3/js/generarDatosEstaticos.js#L18

Ponlo en minúscula, tal como indicaba el enunciado.

HiginioUrbano commented 2 years ago

Vale, no me acordaba de esa parte de la práctica ya, cambiado y solucionado todo, gracias.

El mié, 24 nov 2021 a las 9:35, Pedro Prieto @.***>) escribió:

No hombre. En tu archivo:

https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/blob/a5b4b61b351d75dfb0160b34ac3acf89cd1834e3/js/generarDatosEstaticos.js#L18

Ponlo en minúscula, tal como indicaba el enunciado.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/issues/13#issuecomment-977649432, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUM3WNJ5ZZETP6JOBNILPG3UNSPWHANCNFSM5ITIAA3A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

pedroprieto commented 2 years ago

A mandar! :wink:

HiginioUrbano commented 2 years ago

Una última pregunta aprovechando este hilo, de cara al exámen las expresiones regulares son importantes?

El mié, 24 nov 2021 a las 10:27, HIGINIO URBANO OCHOA (< @.***>) escribió:

Vale, no me acordaba de esa parte de la práctica ya, cambiado y solucionado todo, gracias.

El mié, 24 nov 2021 a las 9:35, Pedro Prieto @.***>) escribió:

No hombre. En tu archivo:

https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/blob/a5b4b61b351d75dfb0160b34ac3acf89cd1834e3/js/generarDatosEstaticos.js#L18

Ponlo en minúscula, tal como indicaba el enunciado.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/issues/13#issuecomment-977649432, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUM3WNJ5ZZETP6JOBNILPG3UNSPWHANCNFSM5ITIAA3A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

pedroprieto commented 2 years ago

No. Puse un mensaje en el foro al respecto. Tienes lo que va a entrar en Aules. Y en el vídeo de la tutoría de esta semana.

Un saludo!

HiginioUrbano commented 2 years ago

Lo he visto justo después de preguntarte -.-´

El mié, 24 nov 2021 a las 10:40, Pedro Prieto @.***>) escribió:

No. Puse un mensaje en el foro al respecto. Tienes lo que va a entrar en Aules. Y en el vídeo de la tutoría de esta semana.

Un saludo!

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/HiginioUrbano/practica_dwec_gestor_presupuesto/issues/13#issuecomment-977703528, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUM3WNNUOG46PXGE2KETHZLUNSXJLANCNFSM5ITIAA3A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.