Paufiner mon Ajax [Résolu]

[okoweb]

Bonjour,
Comment jugez vous ce code :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
$.ajax({
    url: "staff.php", 
    data: null,
    dataType: "json",
    beforeSend: function(xhr){
        console.log(xhr);
    },
    success: function(data, textStatus, jqXHR){
        console.log(data, textStatus, jqXHR);
        // succès de la transaction, on doit traiter le contenu de data  
    },
    error: function(jqXHR, textStatus, errorThrown){
        console.log(jqXHR, textStatus, errorThrown);
        // échec de la transaction, gérer la catastrophe  
    },
    complete: function(jqXHR, textStatus){
        console.log(jqXHR, textStatus); 
        // la transaction s'est terminée par un succès ou par un échec  	
    }
});

NB. D'avance il marche très bien. Y'a-t-il mieux à faire?

Merci d'avance...

[MrPringle]

Ca me paraît bien, le callback sur complete est redondant si tu as déjà "success" et "error" non ?

[galerien69]

salut,

je dirais que ton code est inutile.
Inutile, pourquoi?
Ben si tu écris du code qui ne répond à aucun problème, c'est du code inutile.

Ici, tu dev les callbacks appelés par .ajax, avec un corps vide. Donc autant ne pas les écrire.
Tu recopies tous les arguments (enfin j'imagine) passés dans tes méthodes, mais si tu t'en sers pas, ils ne font que polluer les yeux du lecteur.
Dans la même idée, tu as mis des console.log, je pense pour dire que c'était bien exécuté. Evidemment en prod faut les enlever .

Et enfin, il faut mieux mettre une url absolue pour /staff.php, ou la récupérer par un qq moyen qui ne lie pas la page que tu appèles dans le même dossier que la page ou est exécuté ton script !

rq : les paragraphes "ici...' et 'Tu...' ont été commentés alors que mes yeux ont filtré d'eux même les console.log...

[okoweb]

merci pour vos remarques...