Discussion :

[Kazumi]

Bonjour,

Avant de faire l'action d'un formulaire qui renvoie vers une page html,

Code html :

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

<form name="saisie_periode" action="index.html" onsubmit="return ecriture()">

j'ai du code javascript qui fait patienter l'utilisateur tant qu'un fichier n'est pas créé,

Code :

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

1
2
3
4
5
6
7
8
//Tant que le fichier xml n'est pas créé, on patiente
var fichier_xml = new ActiveXObject("Scripting.FileSystemObject");
while(!fichier_xml.FileExists(url_XML)){
	var start = new Date().getTime();
	for (var i = 0; i < 1e7; i++) {
		if ((new Date().getTime() - start) > 1000){ //On sleep le processeur pendant 1secondes
			break;
		};}}

Tout marche, cependant quand le fichier est long à se créer, j'ai un message d'alert qui m'affiche :

Un script présent sur cette page ralentit Internet Explorer. S'il continue à être exécuté, votre ordinateur ne pourra peut-être plus réagir. Voulez-vous interrompre ce script ?

Ce qui gêne énormément mon client, des idées pour éviter ce message ?

Merci d'avance

[SylvainPV]

Une boucle sur 1 milliard d'élément, ce n'est pas un sleep ! C'est sans aucun doute la pire façon possible d'attendre un délai, le CPU ne va jamais dormir. Utilise setTimeout

[Kazumi]

Code :

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

1
2
3
while(!fichier_xml.FileExists(url_XML)){
	setTimeout(function(){},1000);
}

Et maintenant j'ai une fenêtre windaub me signalant le message qui suit :

Une exception de type 'Erreur d'exécution n'a pas été gérée.
Voulez-vous déboguer l'application ?

Des idées ? Car je suis obligé de répondre au boite de dialogue...

[Bovino]

une fenêtre windaub

Assez douteuse l'insinuation, parce que si on compare au code présenté dans ton premier message, Windows est quand même une merveille...

Bref, manifestement, tu ne maitrises pas du tout les timers en JavaScript.
Un setTimeout met dans la pile d'exécution une fonction à lancer après un délai donné, ce qui signifie que le script continue à se dérouler entre temps, ce qui explique très probablement le message affiché.
Il faut donc réorganiser ton code de façon à lancer le timer une seule fois et qu'à chaque callback, tu vérifies la présence du fichier. S'il existe, tu continues le traitement, sinon tu relances le timer.

Dans ce que tu as tenté, tu fais une boucle while qui ajoute des fonctions (vides en plus...) à la pile d'appels tant que le fichier n'existe pas, il ne faut pas s'étonner que "windaube" n'aime que moyennement... je doute d'ailleurs que Linux réagirait différemment...

[Kazumi]

En effet je ne m'y connais pas du tout en timer js :/

Code :

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

1
2
3
4
5
6
7
8
9
10
var timer = 3000;
function test_fichier(){
	if(!fichier_xml.FileExists(url_XML)){
		timer=3000;
	}
	else {
		alert('Traitement fini');
	}
}
setTimeout(test_fichier(),timer);

mon code après modification, qui ne donne toujours rien :/

Merci pour ta piste Bovino

[Kazumi]

Code :

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

1
2
3
4
5
6
7
8
9
setTimeout(test_fichier(),20000);
function test_fichier(){
	if(!fichier_xml.FileExists(url_XML)){
		setTimeout(test_fichier(),10000);
	}
	else {
		alert('Traitement fini');
	}
}

Et avec ce code j'ai une erreur mémoire :/
Help !

[SylvainPV]

Code :

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

1
2
3
4
5
6
7
8
9
10
 
function test_fichier(){
  if(!fichier_xml.FileExists(url_XML)){
    setTimeout(test_fichier,10000);
  }
  else {
    alert('Traitement fini');
  }
}
test_fichier();

[Kazumi]

Le code que tu viens de me fournir ne fonctionne pas. Le formulaire s'envoie tant bien même que le fichier n'existe pas encore :/

[Bovino]

C'est normal : l'envoi du formulaire dépend du bouton submit, or à aucun moment tu ne demandes d'empêcher le comportement par défaut.
Donc lors du click, tu lances ton timer, mais le code rend la main en attendant le délai du setTimeout et le formulaire est soumis...

Code html :

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

<form name="saisie_periode" action="index.html" onsubmit="return ecriture()">

Code :

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

1
2
3
4
function ecriture(e){
    e.preventDefault();
    // suite du code
}

Ensuite, attention à la syntaxe de setTimeout : Les fonctions de rappel (callback) ou les utilisations cachées de eval()