pasar i como parametro de una funcion dentro de un loop

[mayid]

Estoy dando vueltas con esto, pero no lo puedo resolver. Llamo una funcion y le paso parametros desde dentro de un loop. Pero solo me toma el ultimo valor del loop!

Código Javascript:

for (var i=0; i<cantidadOpciones; i++) 
       {       
         indexOpcion = comboOriginal.options[i].index;
       
         itemCombo = document.createElement('a');
         itemCombo.id = "input"+indexOpcion;
         itemCombo.href = "#"+indexOpcion;
         itemCombo.innerHTML = comboOriginal.options[i].text;         
         
         indice = eval(indexOpcion);
         itemCombo.onclick = function(){ seleccionar( combo_id, indice ); return false; };
         divDesplegable.appendChild(itemCombo);                 
       }
 

El error es en esta linea:
itemCombo.onclick = function(){ seleccionar( combo_id, indice ); return false; };

En fin, cada vez que clickeo en uno de los links generados, el valor siempre es el ultimo. En cambio, con esta linea el valor si es variable:
itemCombo.href = "#"+indexOpcion;

Estoy trabajando con IE.

[abimaelrc]

Iniciado por mayid

Estoy trabajando con IE.

El final me lo dañaste

Ahora ¿cuál es la idea? Es para que te puedan sugerir una mejor forma o guiarte mejor.

Edito:
Trata usando la palabra clave this. Eso hace referencia al objeto, el usar las variables provoca que te seleccione el último, porque hace referencia a la variable no al valor de la variable. Mira este tema para que tengas una mejor idea http://www.forosdelweb.com/f13/claus...lacion-900897/

[mayid]

Gracias por los comentarios y el enlace.

La idea es que me ayuden a que las variables se evaluen en el loop, y se pasen como valor y no como variable. Eso es lo que esta fallando.

Antes de hacer eventos así, probe con attachEvent pero pasar variables era un lio, pero eso uso la sintaxis .onclick

EDITO: lo de this... no se como aplicarlo a mi caso. Creo que el origen esta en el envio de las variables, no en la recepcion de parametros....

[masterpuppet]

Que tal mayid,

proba con algo así, a ver si te sirve:

Código Javascript:

for (var i=0; i<cantidadOpciones; i++) 
{       
    indexOpcion = comboOriginal.options[i].index;
 
    itemCombo = document.createElement('a');
    itemCombo.id  = "input"+indexOpcion;
    itemCombo.href = "#"+indexOpcion;
    itemCombo.innerHTML = comboOriginal.options[i].text;         
         
    itemCombo.onclick = (function(e) {
      var idx   = indexOpcion;
      var combo = combo_id;
      return function() {
        seleccionar(combo, idx);
        return false;
      }
    })();
    
    divDesplegable.appendChild(itemCombo);                 
}
 

Saludos.

[mayid]

Esto por si mismo no funciono:

Código Javascript:

var id = combo_id;
var index    = indexOpcion;
itemCombo.onclick = function(){ seleccionar( id, index ); return false; };
 

Y poniendolo así no me retorna nada, o al menos no veo nada con un alert desde seleccionar():

Código Javascript:

itemCombo.onclick = function(){ 
         
                  var id = combo_id;
                  var index    = indexOpcion;
                  return function() {
                    seleccionar(id, index);
                    return false;
                  }
                  return false;
                  
                  };
 

[masterpuppet]

no es lo mismo mayid, míralo bien, en mi ejemplo es una funcion anónima que se autoejecuta

Código Javascript:

itemCombo.onclick = (function(e) {
      var idx   = indexOpcion;
      var combo = combo_id;
      return function() {
        seleccionar(combo, idx);
        return false;
      }
 })();
 

[mayid]

ok. gracias a todos.

Esto funciono:

Código Javascript:

...
itemCombo.alt = combo_id;
         itemCombo.rel = indexOpcion;
         itemCombo.innerHTML = comboOriginal.options[i].text;
        
         itemCombo.onclick = function(){                                
                                  seleccionar( this ); 
                                  return false; 
                                  };
 

Código Javascript:

function seleccionar( link ){   
    
    combo_id = link.alt;
    indexOpcion = link.rel; 
...
}