Discussion :

[Air P-E]

Bonjour à tous,

Je fais actuellement une requête ajax vers mon serveur, retourne les valeurs au format JSON, et construit un tableau suivant les valeurs avec DOM...
A côté de ça mon problème a l'air ridicule mais je n'arrive pas à le résoudre !

Je vous montre d'abord mon code

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
48
49
50
51
52
53
54
55
56
57
param_tests_jmeter.Ajax = {
 
    jspFolder : '/ParamTestJMeter-portlet/jsp',
 
    getTests : function() {
        var xhr = this.getXhr();
        if (xhr === null) {
            alert("Your browser doesn't support XMLHttpRequest!");
            return;
        }
 
        var url = this.jspFolder+'/dynamic_request/getJMeterTests.jsp';
 
        // fonction appelée lors d'un changement d'état de la requête (de traitementMethods.jsp)
        xhr.onreadystatechange = function() {
            param_tests_jmeter.Ajax.stateChanged(xhr);
        };
        xhr.open("GET",url,true);
        xhr.send(null);
    },
 
    stateChanged : function(xhr) {
        // On vérifie que le requête s'est bien effectuée avant de continuer
        if (xhr.readyState === 4 &&
            xhr.status === 200) {
            // évaluation du message JSON (eval())
            var tableauValeurs = eval("(" + xhr.responseText + ")");
 
            //Construction de mon tableau
            param_tests_jmeter.Table.createTable('div_edit_table_jmeter');
            var imax = tableauValeurs.length;
            for (var i = 0; i < imax; i++)
                param_tests_jmeter.Table.addRow(tableauValeurs[i].name,
                                                tableauValeurs[i].environment,
                                                tableauValeurs[i].description,
                                                tableauValeurs[i].fileName,
                                                tableauValeurs[i].isActive);
        }
    },
 
    // récupération de l'objet XmlHttpRequest (ajax) permettant d'échanger des informations
    getXhr : function() {
        var xhr;
        try {
            // Firefox, Opera 8.0+, Safari
            xhr = new XMLHttpRequest();
        } catch (e) {
            // Internet Explorer
            try {
                xhr = new ActiveXObject('Msxml2.XMLHTTP');
            } catch (e) {
                xhr = new ActiveXObject('Microsoft.XMLHTTP');
            }
        }
        return xhr;
    }
};

mon problème donc et que j'aimerais faire un return de tableauValeurs pour pouvoir construire mon tableau hors de la fonction statechanged...
Le problème vous l'aurez deviné, c'est que si je fais ceci, ma construction du tableau s'exécute avant même d'avoir reçu les résultats.

J'ai tenté en synchrone avec le paramètre à false, mais c'est pas convaincant !

J'avais cru voir qu'on avait pas trop le choix et qu'on devait faire nos actions dans la fonction, mais je n'y crois pas ! Il y a forcément une solution !? Nan ?

[gwyohm]

Bonjour,

Il faut que tu passes une fonction en paramètre à stateChanged, qui pointe vers la fonction qui fait les traitements particuliers à la requete ajax. C'est le principe du callback :

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
 
var dessineTableau = function(xhr) {
  // évaluation du message JSON (eval())
  var tableauValeurs = eval("(" + xhr.responseText + ")");
  //Construction de mon tableau
  param_tests_jmeter.Table.createTable('div_edit_table_jmeter');
  var imax = tableauValeurs.length;
  for (var i = 0; i < imax; i++)
  param_tests_jmeter.Table.addRow(tableauValeurs[i].name,
    tableauValeurs[i].environment,
    tableauValeurs[i].description,
    tableauValeurs[i].fileName,
    tableauValeurs[i].isActive);
}
// ...
stateChanged : function(xhr, callback) {
  if (xhr.readyState === 4 &&
            xhr.status === 200) {
            callback(xhr);
  }
}
// ...
getTests : function() {
        var xhr = this.getXhr();
        if (xhr === null) {
            alert("Your browser doesn't support XMLHttpRequest!");
            return;
        }
 
        var url = this.jspFolder+'/dynamic_request/getJMeterTests.jsp';
 
        // fonction appelée lors d'un changement d'état de la requête (de traitementMethods.jsp)
        xhr.onreadystatechange = function() {
            param_tests_jmeter.Ajax.stateChanged(xhr, dessineTableau );
        };
        xhr.open("GET",url,true);
        xhr.send(null);
    }

Tu gardes ainsi une encapsulation de xhr qui reste générique

[Air P-E]

Bonjour gwyohm,

Merci de ta réponse.

J'ai mis en place ton code (en le modifiant un tout petit peu) et ça fonctionne !

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
getTests : function() {
        var xhr = this.getXhr();
        if (xhr === null) {
            alert("Your browser doesn't support XMLHttpRequest!");
            return;
        }
 
        var url = this.jspFolder+'/dynamic_request/getJMeterTests.jsp';
 
        // fonction appelée lors d'un changement d'état de la requête
        xhr.onreadystatechange = function() {
            jmeter_param_tests.Ajax.stateChanged(xhr, jmeter_param_tests.Ajax.displayTable);
        };
        // les paramètres sont envoyés en GET : inutile de passer en POST étant donné
        // que ces valeurs seront de toute façon affichées à l'utilisateur.
        xhr.open("GET",url,true);
        xhr.send(null);
    },
 
    stateChanged : function(xhr, callback) {
        // On vérifie que le requête s'est bien effectuée avant de continuer
        if (xhr.readyState === 4 &&
            xhr.status === 200) {
 
            // évaluation du message JSON (eval())
            callback(eval("(" + xhr.responseText + ")"));
        }
    },
 
    displayTable : function(tableValue) {
        //Construction de mon tableau
        jmeter_param_tests.Table.createTable('jmeter_div_edit_table');
 
        var imax = tableValue.length;
        for (var i = 0; i < imax; i++)
            jmeter_param_tests.Table.addRow(tableValue[i].name,
                                            tableValue[i].environment,
                                            tableValue[i].description,
                                            tableValue[i].fileName,
                                            tableValue[i].isActive);
    },

la fonction displayTable est bien indépendante du reste !

Cependant la fonction getTests me dérange un petit peu... Parce qu'en l'état actuel, si je veux faire une autre requête Ajax il faut que je créé une autre fonction et donc remettre tout le contenu de getText, il faut juste modifier l'url et la fonction... Ne peut-on pas faire encore plus générique ? Tu vois ce que je veux dire ?

Sinon concernant le code en lui même as-tu des suggestions ?

[gwyohm]

et bien getTests peut prendre en paramètres l'url et la fonction...

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
 
getTests : function(url, callback) {
        var xhr = this.getXhr();
        if (xhr === null) {
            alert("Your browser doesn't support XMLHttpRequest!");
            return;
        }
 
        var url = this.jspFolder+url;
 
        // fonction appelée lors d'un changement d'état de la requête
        xhr.onreadystatechange = function() {
            jmeter_param_tests.Ajax.stateChanged(xhr, callback);
        };
        // les paramètres sont envoyés en GET : inutile de passer en POST étant donné
        // que ces valeurs seront de toute façon affichées à l'utilisateur.
        xhr.open("GET",url,true);
        xhr.send(null);
    }

[Air P-E]

Oui justement en cherchant un peu je suis tombé sur :
http://www.developpez.net/forums/d65...pel-generique/

Donc ça revient un peu à ce qu'on est en train de faire mais il reste un petit problème sur la fonction passé en paramètre... Peut-on passer les paramètres de la fonction passé en paramètre en même temps ?

Parce qu'à l'appel de la fonction callback, il se peut que je veuille utiliser plusieurs fonction avec plusieurs paramètres...

[gwyohm]

Tu vas finir par t'écrire un framework... c'est peut-être plus simple d'en utiliser un existant... (et je ne dis pas ça parce que je n'ai fait que regarder comment c'était fait dans prototype...)

Le but est en fait de définir des paramètres à une fonction sans l’exécuter tout de suite: on veut obtenir une nouvelle fonction qui contient la première pré-paramétrer et passer le(s) dernier(s) paramètre(s) au dernier moment...

alors on peut wrapper notre fonction dans une autre, mais de façon plus générique, tu peux faire quelque chose comme ça :

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
 
var getPresetFunction = function() {
 // si on n'a aucun parametre, ca ne va pas...
 if(arguments.length < 1) throw "pas de fonction à préparer...";
 // on recupere la fonction
 var _function  = arguments[0];
 // on fait une copie des paramètres sauf le premier qui est la fonction
 // on doit passer par Array.prototype.slice car arguments n'est pas un vrai Array
 var args = Array.prototype.slice.call(arguments, 1);
 // on renvoie une nouvelle fonction qui lors de son appel concatène 
 // les paramètres reçus à ceux reçus par getPresetFunction 
 return function() {
   var params = args.concat(Array.prototype.slice.call(arguments, 0));
   return _function.apply(this, params);
 };
}
// teste:
var myAlert=function() {
  for(var i=0, length=arguments.length; i<length; i++) {
    alert(arguments[i]);
  }
}
myAlert("a"); //=> "a"
myAlert("a", "b"); //=> "a" puis "b"
 
var myPreparedAlert = getPresetFunction(myAlert, "c", "d");
myPreparedAlert("e", "f"); // => "c" puis "d" puis "e" puis "f"

Après tu peux faire quelque chose de plus pratique :

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
 
Function.prototype.prepare = function() {
 // si on n'a aucun parametre,on renvoie la fonction... c'est this
 if(arguments.length==0) return this;
 // on recupere la fonction
 var _function  = this;
 // on fait une copie des paramètres 
 // on doit passer par Array.prototype.slice car arguments n'est pas un vrai Array
 var args = Array.prototype.slice.call(arguments, 0)
 return function() {
   var params = args.concat(Array.prototype.slice.call(arguments, 0));
   return _function.apply(this, params);
 }
};
// teste:
var myAlert2=function() {
  for(var i=0, length=arguments.length; i<length; i++) {
    alert(arguments[i]);
  }
}
myAlert2("a"); //=> "a"
myAlert2("a", "b"); //=> "a" puis "b"
 
var myPreparedAlert2 = myAlert2.prepare("c", "d");
myPreparedAlert("e", "f"); // => "c" puis "d" puis "e" puis "f"

De cette façon, tu peux préparer tes fonctions de retour avec des paramètres, la transmettre à ton encapsulation de xhr qui ajoutera le dernier parametre : la réponse texte