Discussion :

[gorgonite]

Salut,



Je me sers relativement peu d'Ajax et je souhaiterais avoir vos avis quant à la qualité de cette fonction

nb: elle est censée compléter une bibliothèque pour scripts XUL... donc je suis sur que le navigateur utilise Gecko, et donc inutile de tester pour se servir de msxml activex & cie

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
 
function send(url,data,method,sendXML,objectAsync) {
        var xhr = new XMLHttpRequest();
	var res;
 
        if (sendXML)
        {
                xhr.overrideMimeType("text/xml");
        }
 
	if (objectAsync) {
		xhr.onreadystatechange = function() {
			// On ne fait quelque chose que si on a tout reçu et que le serveur est ok
			if(xhr.readyState == 4 && xhr.status == 200){
        			if (sendXML) {
			                res = xhr.responseXML;
			        } else {
			                res = xhr.responseText;
			        }
				objectAsync.do(res);
			}
		}
	}
 
        if (method == "POST") {
                xhr.open("POST",url,false);
                xhr.setRequestHeader("Content-type", "application/x-www-form-urlencoded");
                xhr.send(data);
        } else {
                if (data == "") {
                        xhr.open("GET",url,false);
                } else {
                        xhr.open("GET",url+'?'+data,false);
                }
                xhr.send(null);
        }
 
	if (objectAsync == null) {
		if (sendXML) {
                	res = xhr.responseXML;
	        } else {
        	        res = xhr.responseText;
	        }
		return res;
	}
}

[Shinuza]

Le bug tracking est inexistant

Si ça se passe bien on fait ça, et si ça se passe mal on a aucun moyen de le savoir...

[gorgonite]

peux-tu détailler un peu plus ?
que suggères-tu à la place ?

[Shinuza]

Code :

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

1
2
3
4
5
6
7
8
9
// On ne fait quelque chose que si on a tout reçu et que le serveur est ok
			if(xhr.readyState == 4 && xhr.status == 200){
        			if (sendXML) {
			                res = xhr.responseXML;
			        } else {
			                res = xhr.responseText;
			        }
				objectAsync.do(res);
			}

Tu fais quelque chose si le serveur à répondu et que tu as tout reçu, mais tu peux avoir un problème de liaison pendant le chargement, ou un problème de liaison tout court, et la bonjour les dégats pour trouver "pourquoi ça ne marche pas"

[gorgonite]

tu suggères un

Code :

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

else {alert("problem "+xhr.readyState+" "+xhr.status)}

[gorgonite]

et si je propose une personnalisation du traitement de l'erreur ce ne sera pas trop lourd (je ne suis pas habitué à l'Ajax )

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
function send(url,data,method,sendXML,objectAsync) {
        var xhr = new XMLHttpRequest();
	var res;
 
        if (sendXML)
        {
                xhr.overrideMimeType("text/xml");
        }
 
	if (objectAsync) {
		xhr.onreadystatechange = function() {
			// On ne fait quelque chose que si on a tout reçu et que le serveur est ok
			if(xhr.readyState == 4 && xhr.status == 200){
        			if (sendXML) {
			                res = xhr.responseXML;
			        } else {
			                res = xhr.responseText;
			        }
				objectAsync.do(res);
			} else {
				objectAsync.error(xhr);
			}
		}
	}
 
        if (method == "POST") {
                xhr.open("POST",url,false);
                xhr.setRequestHeader("Content-type", "application/x-www-form-urlencoded");
                xhr.send(data);
        } else {
                if (data == "") {
                        xhr.open("GET",url,false);
                } else {
                        xhr.open("GET",url+'?'+data,false);
                }
                xhr.send(null);
        }
 
	if (objectAsync == null) {
		if (sendXML) {
                	res = xhr.responseXML;
	        } else {
        	        res = xhr.responseText;
	        }
		return res;
	}
}