problemas con seTimeout()

[galorn]

Hola:

Estoy intentando utilizar la funcion setTimetout() con parametros y me esta dando problemas, ya he revisado varios post con soluciones, las he intentado aplicar pero no consigo solucionarlo. El problema es que no me pasa la variable que deseo.

Código Javascript:

//funcion que printa en la web (genera codigo html) de los productos añadidos
function play(){
  var msg_text = document.getElementById("msg_text");
  
  //removing the list
  if ( msg_text.hasChildNodes() ){
      while ( msg_text.childNodes.length >= 1 ){
          msg_text.removeChild( msg_text.firstChild );       
      } 
  }
    
    //volvemos a crear la lista
    var ul = document.createElement("ul");
    
    //rellenamos la lista entera al completo
    for (i; i<listaGeneral.length; i++){
        setTimeout("rellenar("+ul+")",1000);
            //rellenar(ul);
        }
    msg_text.appendChild(ul);
}
 
function rellenar(ul){
    
        var li = document.createElement("li");
        
        if(listaGeneral[i].tipo=="message"){
            
            var mensajeMostrado = "<span class=message>"+listaGeneral[i].numero+": "+listaGeneral[i].nombre1+" le dice a "+listaGeneral[i].nombre2+" : "+listaGeneral[i].mensaje+"</span>";
        }else{
            var mensajeMostrado = "<span class=order>"+listaGeneral[i].numero+": "+listaGeneral[i].mensaje+"</span>";
        }
        
        li.innerHTML = mensajeMostrado;
    
        ul.appendChild(li);
        
}
 

Si en vez de setTimeout("rellenar("+ul+")",1000); pongo rellenar(ul); si que me funciona, pero claro no me lo va rellenando poco a poco (cada segundo). he probado también de poner setTimeout("rellenar("'+ul+'")",1000); y todo tipo de posiblidades, comillas simples todo, comillas simples donde estan las dobles y al reves... pero no me sale.

Muchas gracias por la ayuda, un saludo!!!!!!!!!!

[zerokilled]

lo que sucede es que setTimeout, al igual que otras funciones, ejecutan el codigo en el contexto global. entonces, cuando intentas usar una variable local de una funcion en setTimeout, el interprete intentara buscar esa variable en el ambito global y por ende, al no estar definida como variable global, se genera error. lo que tienes que hacer es crear una funcion anonima en setTimeout para que se cree un closure de la variable local.

Código:

setTimeout(function(){
// aqui invocas tu funcion con los parametros necesario;
}, 1000);

[galorn]

La verdad que no se si lo estoy poniendo bien (he probado bastantes maneras) pero me sigue sin ir...

Código Javascript:

for (i; i<listaGeneral.length; i++){
        setTimeout(function(){'rellenar('+ul+')';}, 1000);
    }
 

[zerokilled]

la invocacion a tu funcion no se hace en formato de string... invoca tu funcion de manera comun y corriente, tal como harias con cualquier otra funcion.

[galorn]

He probado de manera común y tampoco me funciona:

Código Javascript:

for (i; i<listaGeneral.length; i++){
        
        setTimeout(function(){rellenar(ul);}, 1000);  //asi no me funciona
        //rellenar(ul); ---------> asi si que me funciona
    }
 

Muchas gracias por la ayuda!

[zerokilled]

se supone que tal como lo tienes en la linea 3 te debe funcionar, no creo que ahi es donde reside el problema. verifica la consola de errores a ver que te indica.

[galorn]

Si, estaba bien como decias, el problema era otro.
La cuestion estaba en como trataba la llamada a la funcion, que con un for no estaba bien ya que iva llamando y aumentando el valor de i, entonces cuando queria printar ya no tenia la i valor maximo y aun no se habia ejecutado en ningun momento (ya que tenia puesto 1000 como valor, y un bucle while va mucho mas rapido).

ahi va la solución:

Código Javascript:

//funcion que printa en la web (genera codigo html) de los productos añadidos
function play(){
    var msg_text = document.getElementById("msg_text");
    //removing the list
    if ( msg_text.hasChildNodes() ){
        while ( msg_text.childNodes.length >= 1 ){
        msg_text.removeChild( msg_text.firstChild );       
        } 
    }
    
    //volvemos a crear la lista
    var ul = document.createElement("ul");
    
    //rellenamos la lista entera al completo
    rellenar(ul);
    
    msg_text.appendChild(ul);
}
 
function rellenar(ul){
    
     if(i < listaGeneral.length) {
            i++;
            var li = document.createElement("li");
            
            if(listaGeneral[i].tipo=="message"){
                
                var mensajeMostrado = "<span class=message>"+listaGeneral[i].numero+": "+listaGeneral[i].nombre1+" le dice a "+listaGeneral[i].nombre2+" : "+listaGeneral[i].mensaje+"</span>";
            }else{
                var mensajeMostrado = "<span class=order>"+listaGeneral[i].numero+": "+listaGeneral[i].mensaje+"</span>";
            }
            
            li.innerHTML = mensajeMostrado;
            
            ul.appendChild(li);
            setTimeout(function(){rellenar(ul);}, 1000);
     }
}