Conversation
Los 4 puntos del ejercicio 1
01-JS-DOM-APIs/js/main.js
Outdated
| } | ||
| }) | ||
| .catch(error => { | ||
| document.getElementById("contenidoJson").style.background = "red"; |
There was a problem hiding this comment.
Estas referencias a estilos intentá que estén siempre en los archivos de CSS
| reject(Error("Hubo un error")); | ||
| } | ||
|
|
||
| httpRqs.send(); |
There was a problem hiding this comment.
En esta función la indentación te quedó toda desarreglada, fijate si lo podés arreglar
01-JS-DOM-APIs/js/main.js
Outdated
| configurar.url = 'http://api.icndb.com/jokes/random'; | ||
|
|
||
| reusable(configurar).then(response =>{ | ||
| var objeto = JSON.parse(response); |
There was a problem hiding this comment.
Tené más cuidado con los nombres que estás usando para las cosas. configurar es un verbo, da la idea de que es una función. objeto es demasiado general. mensajeAlerta parece más el nombre de una variable que de una función, las funciones hacen algo
01-JS-DOM-APIs/js/main.js
Outdated
| }) | ||
| }; | ||
| function traerConParametros(value){ | ||
| value = document.getElementById("valorUsuario").value; |
There was a problem hiding this comment.
Por qué recibís un parámetro value si después lo pisás?
01-JS-DOM-APIs/js/main.js
Outdated
| var fila = document.createElement('tr'); | ||
| var elemento = matriz[i].length; | ||
|
|
||
| for(n=0;n<elemento;n++){ |
There was a problem hiding this comment.
Ojo con las variables i y n. Nunca las declaraste por lo que son globales.
intii
left a comment
intii
left a comment
There was a problem hiding this comment.
En líneas generales está aprobado. Te dejé un par de comentarios que me gustaría que al menos revisaras, si querés los corregís.
Y un comentario, si podés empezar a programar en inglés, mejor. En este momento a veces tenés variables en inglés, a veces en español, y en general siempre vas a tener que programar en inglés.
También tenemos que poner foco en los nombres de las funciones y las variables
-Cambie la refencia de estilo y lo pase a css -Arregle la indentacion -Cambie el nombre de algunas variables y funciones(para intentar hacerlas mas claras) -Arregle la pisada de parametro value -Solucione la declaracion de i y n en la funcion
2 participants
Terminados todos los ejercicios, con un commit por cada uno