Foros del Web » Programando para Internet » Javascript »

Lio con condiciones y arrays

Estas en el tema de Lio con condiciones y arrays en el foro de Javascript en Foros del Web. Buenas, estoy intentando hacer un código que compruebe si el array tiene comprueba si el array está vacío, si no está vacío comienza a recorrerlo ...
  #1 (permalink)  
Antiguo 05/07/2013, 16:59
 
Fecha de Ingreso: noviembre-2010
Mensajes: 234
Antigüedad: 13 años, 5 meses
Puntos: 2
Lio con condiciones y arrays

Buenas, estoy intentando hacer un código que compruebe si el array tiene comprueba si el array está vacío, si no está vacío comienza a recorrerlo y a almacenar datos en cada puesto del array, dejo el código para ver que es lo que debo de ir arreglando ya que sé que no me funciona y que tengo bastantes errores:

Código Javascript:
Ver original
  1. function recoger(){
  2.     var qv = new Array(5);
  3.    
  4.     if(qv[0]=null){
  5.         for(i=0; i<5; i++){
  6.             for(j=0; j<5; j++){
  7.                 for(z=0; z<5; z++){
  8.                     localStorage.setItem("qv[i]", "{{video.nombre}}");
  9.                     localStorage.setItem("qv[j]", "{{video.foto}}");
  10.                     localStorage.setItem("qv[z]", "{{video.link}}");
  11.                 }
  12.             }
  13.         }
  14.     }
  15.     else if(qv[1]=null){
  16.         for(i=0; i<5; i++){
  17.             for(j=0; j<5; j++){
  18.                 for(z=0; z<5; z++){
  19.                     localStorage.setItem("qv[i]", "{{video.nombre}}");
  20.                     localStorage.setItem("qv[j]", "{{video.foto}}");
  21.                     localStorage.setItem("qv[z]", "{{video.link}}");
  22.                 }
  23.             }
  24.         }
  25.     }
  26.     else if(qv[2]=null){
  27.         for(i=0; i<5; i++){
  28.             for(j=0; j<5; j++){
  29.                 for(z=0; z<5; z++){
  30.                     localStorage.setItem("qv[i]", "{{video.nombre}}");
  31.                     localStorage.setItem("qv[j]", "{{video.foto}}");
  32.                     localStorage.setItem("qv[z]", "{{video.link}}");
  33.                 }
  34.             }
  35.         }
  36.     }
  37.     else if(qv[3]=null){
  38.         for(i=0; i<5; i++){
  39.             for(j=0; j<5; j++){
  40.                 for(z=0; z<5; z++){
  41.                     localStorage.setItem("qv[i]", "{{video.nombre}}");
  42.                     localStorage.setItem("qv[j]", "{{video.foto}}");
  43.                     localStorage.setItem("qv[z]", "{{video.link}}");
  44.                 }
  45.             }
  46.         }
  47.     }
  48.     else if(qv[4]=null){
  49.         for(i=0; i<5; i++){
  50.             for(j=0; j<5; j++){
  51.                 for(z=0; z<5; z++){
  52.                     localStorage.setItem("qv[i]", "{{video.nombre}}");
  53.                     localStorage.setItem("qv[j]", "{{video.foto}}");
  54.                     localStorage.setItem("qv[z]", "{{video.link}}");
  55.                 }
  56.             }
  57.         }
  58.     }else
  59.         {
  60.         alert("Ya completo");
  61.     }
  62. }

Gracias!!!
  #2 (permalink)  
Antiguo 06/07/2013, 15:35
Avatar de Aijoona
Colaborador
 
Fecha de Ingreso: mayo-2011
Ubicación: Buenos Aires
Mensajes: 779
Antigüedad: 12 años, 11 meses
Puntos: 343
Respuesta: Lio con condiciones y arras

Divide y conquistaras.

Trata de que tus funciones sean sencillas y hagan una unica cosa, de esa manera es mas facil entender el codigo y componerlo. Además, tus variables son poco explicitas (qv?) y estas obviando el keyword var en tus loops, con lo cual pasan a ser variables globales.

En tu caso estas repitiendo muchas veces el mismo codigo (el loop), codigo que podes abstraer en una funcion.

PD: No tengo idea de que estás queriendo hacer con esa función.
__________________
blog | @aijoona

Etiquetas: condiciones, funcion, lio
Atención: Estás leyendo un tema que no tiene actividad desde hace más de 6 MESES, te recomendamos abrir un Nuevo tema en lugar de responder al actual.
Respuesta




La zona horaria es GMT -6. Ahora son las 17:43.