Discussion :

[speedev]

Bonjour

J'ai une erreur "Argument non valide" sur IE à ce niveau :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
function addToStart(fnc){
  if(!window.listStart) window.listStart = new Array();
  window.listStart.push(fnc)

}
function start(){
  var ls = window.listStart;
  if(ls){
    for(i=0; i<ls.length; i++){
      fnc = ls[i];
      if(typeof(fnc) == 'function'){
        fnc();
      } else {	  
        eval(fnc);
// L'ERREUR SE SITUE A CE NIVEAU
      }
    }
  }
}
addToStart(drawerMenu.construct);

IL s'agit d'un chargeur d'événement pour remplacer le onload du body.
Le code n'est pas de moi. Mais l'erreur qui intervient empêche un de mes scripts de s'exécuter. Je n'arrive à résoudre le bug. Vu que l'erreur est située à une ligne plutôt avancée, je peux en déduire que l'erreur apparait daans l'une des boucles du for()...

[SpaceFrog]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

window[fnc]

??

ou

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

eval(fnc+'()')

ou encore

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

window[fnc+'()']

[speedev]

OK merci je teste ça!

[speedev]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

window[fnc+'()']

Fonctionne!!
Yipaaa!
merci!

[SpaceFrog]

l'eval devrait fonctionne aussi avec les () ...

[speedev]

Ben pourtant non seule la dernière proposition fonctionne. J'ai du mal de toute façon à comprendre pourquoi ça plantait!

[sekaijin]

Bonjour

J'ai une erreur "Argument non valide" sur IE à ce niveau :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
function addToStart(fnc){
  if(!window.listStart) window.listStart = new Array();
  window.listStart.push(fnc)

}
function start(){
  var ls = window.listStart;
  if(ls){
    for(i=0; i<ls.length; i++){
      fnc = ls[i];
      if(typeof(fnc) == 'function'){
        fnc();
      } else {	  
        eval(fnc);
// L'ERREUR SE SITUE A CE NIVEAU
      }
    }
  }
}
addToStart(drawerMenu.construct);

IL s'agit d'un chargeur d'événement pour remplacer le onload du body.
Le code n'est pas de moi. Mais l'erreur qui intervient empêche un de mes scripts de s'exécuter. Je n'arrive à résoudre le bug. Vu que l'erreur est située à une ligne plutôt avancée, je peux en déduire que l'erreur apparait daans l'une des boucles du for()...

il n'y a aucune erreru dans ce script c'est dans l'usage que tu en fait qu'il y en a une.

lorsque tu fait un addToStart tu passe en parametre le nom d'une fonction. dans ce cas c'est func() qui est appelé

l'autre façon de l'utiliser est de lui passer un code javascript
addToStart("mavar=45;");

dans ce cas c'est le eval qui est utilisé
donc si tu a un erreur sur le eval c'est que dans un de test addToStart tu à un mauvais parametre

ajoute juste avant le eval un alert(func) il t'affichera le code fautif et te permettra de coriger l'appel à addToStart


Pour tous ce script permet d'ajouter des éléments au onLoad de telle sorte de pouvoir en mettre plusieurs.

je vous le recommande
dans vos fichiers js si il doit y avoir un elemeent qui se charge au démarage il vous suffit aalors d'appeler addToStart

le chargement du script suffit alors pour le rendre actif


A+JYT

[speedev]

ajoute juste avant le eval un alert(func) il t'affichera le code fautif et te permettra de coriger l'appel à addToStart

Ben c'est ce qui m'a permi de trouver le code fautif oui...^^! Tu n'as pas lu la suite du post?? Tout est résolu depuis qlq temps déjà...ce qui te perturbe sont seulement les ptis trucs de froggy qui m'ont fait modifier les lignes de code, ce dernier reste exactement le même que le premier.

tchaw

[SpaceFrog]

Heu j'ai pas trop compris mais ça à l'air de fonctionner au poil

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
<script>
var listStart = [];  
 
function addToStart(fnc) {
    listStart[listStart.length]=fnc;
}
 
 
function start() {  
    for(var i=0; i<listStart.length; i++) {
        listStart[i]();
    }
}
 
function one() {
    alert("one");
}
 
function two(xx) {
    alert(xx);
}
 
addToStart(one);
addToStart(function(){ two("hello"); });
 
window.onload = start; 
</script>

et pour simplifier, autant remplir l'array de façon homogène ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
addToStart(function(){one()};
addToStart(function(){ two("hello"); });

[speedev]

Cette ligne marche sur IE??

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

addToStart(one);

Etonnant! Mais tant mieux! C'est vrai qu'il y a nettement moyen de simplifier