Revisar #22
Revisar #22
Conversation
…demas indicadores
There was a problem hiding this comment.
Sean más descriptivas en los comentarios de los commits, bien por el proyecto final, muy interesante y los filtros complejos, puedo ver muy poco trabajo de Abi, no sé si trabajaron juntas y después subió Ori, pero se percibe más trabajo de un de ustedes. Recuerden poner todo en ingles de ahora en adelante.
|
|
||
| newdataViolencia = countryData.indicators.filter(indicators=> | ||
| { | ||
| return (indicators.indicatorName.includes("mujeres")|indicators.indicatorName.includes("Mujeres")) && (indicators.indicatorName.includes("violencia") |
There was a problem hiding this comment.
Muy bien el trabajo con la discriinación de data
| }) | ||
|
|
||
| // función que retorna los nombres de los indicadores para Brasil "Mujeres y violencia" | ||
| for (var i = 0; i < newdataViolencia.length; i++) |
There was a problem hiding this comment.
Preferible usar forEach o map
| } | ||
|
|
||
| //Filtro que proporciona los indicadores de "Mujeres y Negocios" | ||
| function negocios(countryData){ |
There was a problem hiding this comment.
Nombres de funciones en ingles
| //Filtro que proporciona los indicadores de "Mujeres y Empleo" | ||
| function empleo (countryData){ | ||
| let newdataEmpleo=[]; | ||
| let indicatorEmpleo=[]; |
There was a problem hiding this comment.
Los nombres de variables, fnuciones, id, clases ya no serán tolerables en español en el próximo proyecto.
src/data.js
Outdated
| } | ||
| for (let i = 0; i < newdataBrasil.length; i++) { | ||
| let indicatorList = document.getElementById("indicatorsSelectBrasil"); | ||
| let indicator= document.createElement("option"); |
There was a problem hiding this comment.
Muy bien por crear las opciones dinámicas
src/data.js
Outdated
|
|
||
| //) | ||
| //console.log(newdataMexico); | ||
| //) | ||
| //console.log(newdataMexico); | ||
|
|
||
| //Filtro que prporciona los ndicadores que contienen la palabra mujer en Brasil | ||
| //Filtro que prporciona los ndicadores que contienen la palabra mujer en Brasil | ||
|
|
||
| //let newdataBrasil = data.BRA.indicators.filter(indicator=> | ||
| // { | ||
| //let newdataBrasil = data.BRA.indicators.filter(indicator=> | ||
| // { | ||
| // return indicator.indicatorName.includes("mujeres") | ||
| //} | ||
|
|
||
| //) | ||
| //console.log(newdataBrasil); | ||
| //) | ||
| //console.log(newdataBrasil); | ||
|
|
||
| //Filtro que prporciona los ndicadores que contienen la palabra mujer en Chile | ||
| //Filtro que prporciona los ndicadores que contienen la palabra mujer en Chile | ||
|
|
||
| //let newdataChile = data.CHL.indicators.filter(indicator=> | ||
| //let newdataChile = data.CHL.indicators.filter(indicator=> | ||
| //{ | ||
| //return indicator.indicatorName.includes("mujeres") | ||
| //return indicator.indicatorName.includes("mujeres") | ||
| //} | ||
|
|
||
| //) | ||
| //console.log(newdataChile); | ||
| //) | ||
| //console.log(newdataChile); | ||
|
|
||
| //Para visualizar que hay en data | ||
| //console.log(datos(data)); | ||
| //Para visualizar que hay en data | ||
| //console.log(datos(data)); |
There was a problem hiding this comment.
Todo el código comentado mejor eliminarlo
No description provided.