[AJAX] [vos avis] fonction "générique" pour les requetes

Version imprimable

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:

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;
	}
}

:merci:

16/07/2007, 00h12
Shinuza

Le bug tracking est inexistant :aie:

Si ça se passe bien on fait ça, et si ça se passe mal on a aucun moyen de le savoir...
16/07/2007, 07h53
gorgonite

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

Code:

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"

16/07/2007, 09h41
gorgonite

tu suggères un

Code:

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

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

Code:

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;
	}
}