CLASSE AJAX COMPLÈTE

Commentaire de kankrelune le 03/06/2007 12:18:24

Dans la lignée de Naixn

for(k in arr)
{
    switch (k)
    {
         case "url" : this.url = arr[k]; break;
... .. .

Comme ça tu peux passer soit un tableau associatif soit un objet... c'est bien plus simple... .. .

Sinon est ce que tu connais escape(), encodeURI(), encodeURIComponent() parce que tu réinvente la roue avec ton getCaller()... .. .

Personellement je pense qu'un approche objet en utilisant le prototypage serait plus propre... .. .

Ton isset() ne sert à rien si ce n'est à faire joli... entre faire

if(isset(toTest))

ou directement faire

if(typeof toTest != "undefined")

autant prendre la deuxième solution... .. .

Sinon c'est plutôt bien... .. .

@ tchaOo°

Commentaire de Arto_8000 le 03/06/2007 16:18:20

Pour le isset, c'était justement pour faire plus beau, maintenant que je fais davantage de PHP je trouve ça plus pratique et visuellement plus facile à reconnaître que le traditionnel typeof.

Naixn -> Pour cette source je n'en avais jamais entendu parler, mais je vais sûrement jeter un coup d'oeil.

Pour la façon de passer les paramètres, je vais corriger pour que ça soit un tableau associatif et pour le parseDataBeforeSend c'était un vieille habitude que j'avais pris d'utiliser ça au lieu du escape.

Aussi pour rajouter des events, le onComplete c'est la même chose que le callback et le onFailure c'est une assez bonne idée que je vais rajouter. Sinon je doutes de l'utilité du onInteractive et onCreate.

Commentaire de kankrelune le 03/06/2007 16:51:27

"Sinon je doutes de l'utilité du onInteractive et onCreate"

bah ça peut toujours servir genre pour une barre de chargement... mais bon c'est pas indispensable... .. .

par contre je maintien tu devrais utiliser le prototypage parce que la c'est bof bof... toi qui fais du php... bien que le model objet de javascript n'ai d'objet que le nom... imagine ton code en php... .. .

function Ajax()
{
$thisObj = (object)array();
$thisObj->callback = null;
$thisObj->asyn = true;
$thisObj->data = "";
$thisObj->url = "";
$thisObj->method = "GET";
$thisObj->obj = httprequest();
$thisObj->returnFormat = "txt";

if ($thisObj->obj->txt !== null)
return $thisObj->obj;

$thisObj->execute = create_function('', 'execRequest();');
$thisObj->setParam = create_function('$param', 'setParam($param);');
$thisObj->setParamFromForm = create_function('$param', 'setParamFromForm($param);');

return $thisObj;
}

Comprend tu... au final tu crée des référence sur des fonctions à chaque instanciation... c'est couteux en perf et pô beau... .. . ;o)

Alors qu'avec le prototypage tous tes objets héritent de tes méthodes...

Ajax.prototype.execute = function()
{
    // code de ta fonction execRequest()
};

Ajax.prototype.setParam = function(param)
{
    // code de ta fonction setParam()
};

Ajax.prototype.setParamFromForm = function(param)
{
    // code de ta fonction setParamFromForm()
};


Sinon... tu utilise beaucoup de fonction qui ne sont pas à proprement parlé des méthode (ou alors statique) tu devrait plus travailler en OO... cf code à la fin de mon post... .. .

Pour finir tu risque d'avoir des problèmes d'instanciation avec certaines version de IE... tiens un bout de code tiré du fin fond de la poubelle qui me sert de pc... lol... .. . ;o)

Ajax.prototype.init = function()
{
    this.obj = null;
    if (window.XMLHttpRequest)
        this.obj = new XMLHttpRequest();
    else if (window.ActiveXObject)  // if IE
    {
        var ieversions = ['Msxml2.XMLHTTP','Microsoft.XMLHTTP','Msxml2.XMLHTTP.5.0','Msxml2.XMLHTTP.4.0','Msxml2.XMLHTTP.3.0'];

        for(var i=0; !this.obj && i<ieversions.length; i++)
        {
    try
    {
        this.obj = new ActiveXObject(ieversions[i]);
    }
    catch(e)  { }
        }
    }
}

Voili voilou... .. .

@ tchaOo°

Commentaire de manouille le 04/06/2007 10:20:49

hello, super ta class, mais ...

j'ai un soucis, j'obtiens systématiquement une erreur  que voici :

document.content has no properties
onreadystatechange()ajax.js (ligne 194)
[Break on this error] eval (_tempAJAX_Reference_.onComplete);

PLEASE HELP ;-)

Commentaire de Arto_8000 le 04/06/2007 14:29:41

Manouille -> Cela vient fort probablement du code que tu as mis dans le onComplete. J'imagine que tu utilisais un String dans lequel tu avais mis ton code. Post moi la déclaration que tu as fait et je vais pouvoir t'aider davantage.

Commentaire de Evangun le 11/08/2007 00:47:58

Hello,

bravo pour cette source ! C'est vrai que prototype est excellent pour Ajax, mais j'ai la flemme d'extraire les fonctions Ajax qui sont au milieu d'un framework de 100ko :/ Donc cette classe-ci fera très bien l'affaire en attendant pour une utilisation commune, et je rajouterai onCreate si besoin, car j'aime beaucoup l'idée.

Quelques problèmes quand même :

- comme le dit Cyrano24, il y a un problème avec inArray(). Ca me rajoute une case à tous mes tableaux, et cette case contient "inArray" !? Mes tableaux sont multi-dimensionnels et associatifs, je ne sais pas si cette info peut t'aider à régler le problème ? Enfin bref, en en faisant une fonction normale et pas une méthode prototypée, tout est revenu dans l'ordre.

- personne ne l'a évoqué jusqu'ici, alors je ne sais pas si c'est moi mais : en GET ça ne marche pas :/ Pour POST tu envoies les données avec .send, OK, mais avec GET tu devrais les concaténer dans l'URL, et tu ne le fais pas il me semble. Il manque donc cette ligne au tout début de Ajax.prototype.execute, et là ça marche : if (this.method == 'GET') this.url += '?'+this.data;

- "syntaxe" ne prend pas de "h" :)

Merci encore :D

Commentaire de Arto_8000 le 11/08/2007 05:39:23

Pour la case ajoutée à tous tes array, le problème est juste avec le foreach je crois ? Quand tu fais les foreach il prend en compte que tout ce qui est ajouté avec prototype est une donnée, un peu bizarre, mais ça le fait. Pour éviter cela rajoute un condition du genre dans tes boucles où il y a le problème :

if (typeof arr[k] == "function")
continue;

Pour le GET c'est sûrement un oublie de ma part.

Aussi le projet est loin d'être mort. Personnellement j'utilise souvent cette classe et de temps en temps je vois des choses que je vais rajouter pour un prochain update. Pour l'instant j'ai pris un peu d'avance sur le commencement d'un autre projet, mon chat PHP, et celui-ci va prendre un peu de temps avant de revenir sur ma table de travaille.

Commentaire de Evangun le 10/09/2007 13:51:09

Hello,
je reviens aujourd'hui avec 2 choses assez importantes à modifier, qui se trouvaient dans le même bloc :

à la place de

if (this.data != "")
this.data += "&";
this.data += j + "=" + escape(arr[k][j]);

mettre :

if(typeof arr[k][j] != "function"){
if (this.data != "")
this.data += "&";
this.data += j + "=" + escape(arr[k][j]);
}

car là on se retrouve avec le problème de l'énumération des méthodes de prototypage : si on ne met pas if(typeof arr[k][j] != "function"), il va envoyer toutes les méthodes prototypées de Array dans la requête, ce qui est totalement inutile et génère plus de trafic.

L'autre chose à modifier :
à la place de :

this.data += j + "=" + escape(arr[k][j]);

mettre

this.data += j + "=" + encodeURIComponent(arr[k][j]);

Comme le dit le Mozilla devguide : "Les fonctions escape et unescape ne fonctionnent pas correctement pour les caractères non-ASCII et sont donc déconseillées". Concrètement ce qui se passe, c'est que le UTF8 est viré et que le caractère "&" fait planter. Il faut utiliser encodeURIComponent.

Il y a encore un autre truc que j'ai remarqué, mais que je n'ai pas encore résolu, alors je repasserai plus tard :)

Commentaire de phpajax le 23/10/2007 03:45:55

tres bonne source, comment fait on pour y ajouter oncreate pour charger un gif de loading ??

Commentaire de phpajax le 23/10/2007 04:12:36

Ajax.prototype.init = function()
{

    this.onCreate();

    .......

}

Ajax.prototype.onComplete = function (response)
{
document.getElementById('loading').style.visibility='hidden';
}

Ajax.prototype.onCreate = function ()
{
   document.getElementById('loading').style.visibility='visible';
}

elle est ou la faute? ou sinon, je doit procéder comment ?
merci pour ta réponse très rapide :)

Commentaire de Arto_8000 le 24/10/2007 01:25:05

Pour le onComplete c'est quand tu mets les paramètres. Désolé de ne pas avoir été assez clair.

var AjaxOBJ = new Ajax();
ObjAJAX.setParam ({
    onComplete :  "enlever_le_loading()",
    onFailure :  "enlever_le_loading()",
    [... tes autres paramètres ...]
});

Commentaire de phpajax le 27/02/2008 15:31:41

Ce que je veux dire par plante c'est que le navigateur (IE) est vraiment scotche a jamais comme si il y avait un deadlock.