Problemas con scripts dinámicos

[gorani]

Un saludo a todos los foreros y paso al tema en cuestión.

Necesito cargar scripts dinamicamente en aras de ahorrar recursos innecesarios en la carga de datos que quizás no se utilicen.

Hasta ahora cargaba los datos en objetos JSON en en HEAD de la página y todo funciona estupendamente, he pensado que como el tema que trato va de resultados futbolísticos y los ejercicios anteriores no son importantes a menos que se quiera conocer algún dato, esos datos los puedo cargar bajo demanda.
He dividido los objetos en varios archiovos .js para cargarlos cuando se necesiten, el problema que tengo es el mismo que en este [URL="http://www.forosdelweb.com/f13/cargar-scripts-dinamicamente-622349/"]post[/URL] antiguo.

La función que utilizo es esta:

Código:

function cargaSRC (src,callback) {

  var scripts = document.getElementsByTagName('script');
  var cargado = false;
    for (var i = 0, script; script = scripts[i]; i++) {
     if (script.src == src) {
        cargado = true;
        break;
      }
    }
    
  if (!cargado) { 
      var elemento = document.createElement('script');
      elemento.setAttribute('type', 'text/javascript');
      
  if (elemento.readyState){ // Internet Explorer
    elemento.onreadystatechange = function(){
      if (elemento.readyState == "loaded" || elemento.readyState == "complete") {
        elemento.onreadystatechange = null;
        callback ();
      }
    };
  } else { // Otros navegadores
    elemento.onload = function(){
    callback();
  };
}      
      
      elemento.setAttribute('src', src);
      document.getElementsByTagName("head")[0].appendChild(elemento);
  }
}

Como tengo funciones dependientes que llaman a esos datos me da error la primera vez que la utilizo (tengo try catch en las funciones), en cambio la segunda vez es correcta.

El caso es que si anulo callback y coloco la función de la siguiente manera funciona correctamente.

Código:

    };
  } else { // Otros navegadores
    elemento.onload = function(){
    MIFUNCION (Param1,Param2,Param3);
  };

¿ A que se puede deber?

de esta forma llamo a la función así: (la que funciona)

cargaSR (el archivo .js);

de la otra forma llamo a la función así: (funciona la segunda vez)

cargaSRC (el archivo .js, MIFUNCION (Param1,Param2,Param3));

Espero haberme explicado correctamente, un saludo y gracias.

PD: Con la función dynamicLoader de Caerolus me pasa exactamente lo mismo, sólo que esta vez funciona a la segunda de las dos formas.

[AlvaroG]

Código Javascript:

cargaSRC (el archivo .js, MIFUNCION (Param1,Param2,Param3));
 

En vez de pasar una función como callback, estás pasando el resultado de ejecutar esa función.
Es decir, a menos que MIFUNCION devuelva un objeto de función, este código intenta ejecutar algo que no es ejecutable. Si tu función es algo como esto:

Código Javascript:

function MIFUNCION(....) { ..... return 1; }
 

la llamada a callback termina siendo "1()" (llamar a un entero como función) lo cual es claramente incorrecto.

Para que funcione como querés, tenés que "envolver" la llamada en una función, tal y como estás haciendo en el segundo bloque de código que pusiste.

Para saber por qué funciona la segunda vez, hay que ver qué es lo que hace MIFUNCION

La solución simple es escribir:

Código Javascript:

cargaSRC('archivo.js', function() { MIFUNCION(...); });
 

Saludos.

[IsaBelM]

el problema es callback() no es una función. prueba así

cargaSRC ('x.js', function() {fnc()});

uppssss, perdona AlvaroG

[AlvaroG]

jeje choque frontal, no problemo

[gorani]

Muchas gracias IsaBelM y AlvaroG por vuestra pronta y acertada respuesta, si era eso, ahora funciona perfecto.
Tambien quisiera conocer, si existe la forma de saber si la carga del script es fallida desde un servidor, para por ejemplo cargarlo desde otro, un evento tipo onerror.

Muchas gracias y un saludo.

[IsaBelM]

prueba a usar el operador typeof(). pasale un tercer parámetro (nombre de la función)

elemento.onload = function(){
if(typeof(window[foo])=='function') {callback();} else {alert('función no cargada')}
};

con esto sabrás si existe la función o no

[gorani]

Iniciado por IsaBelM

prueba a usar el operador typeof(). pasale un tercer parámetro (nombre de la función)

Código:

elemento.onload = function(){
if(typeof(window[foo])=='function') {callback();} else {alert('función no cargada')}
};

con esto sabrás si existe la función o no

Entiendo que lo quieres decir es que compruebe si está cargada una función alojada en el script que quiero inluir dinamicamente y que foo tengo que sustituirlo por el nombre de la función esperada, pero como ya he citado anteriormente yo en los scripts sólo incluyo objetos JSON (datos), las funciones las cargo en el head y he probado a sustituir ...

elemento.onload = function(){
if(typeof(window[OBJETO JSON])=='object') {callback();} else {alert('función no cargada')}
};

.... y no funciona, supongo que el fallo será por window.

Gracias por la ayuda y un saludo.