return false ne bloque pas le lien

Version imprimable

Bonjour les amis,

j'ai fais un programmes qui me permet d'éviter de supprimer des éléments tant qu'il exister d'autres éléments qui leur sont associe.

le problème c'est que dans l'élément est toujours supprimé alors que j'ai fait des return false après l'alerte

voici le code

Code:

<td><a class="supprim" id="15" href="updateProgram.php?action=delete&id=15"><img src="croix.jpg" alt="delete" title="Supprimer ce programme" /></a></td>

et voici le js
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 function testerSuppr(){ $(document).ready(function(){ $('.supprim').click(function(){ var val=$(this).attr('id'); var xhr = getXhr(); xhr.onreadystatechange = function(){ if(xhr.readyState == 4 && xhr.status == 200){ reponse = xhr.responseText; if(reponse=="true"){ alert("les elements associes à des programmes ne peuvent pas etres supprimees"); return false; } else{ if (!(confirm('voulez vous vraiment supprimer cet element ?'))){ return false; } } } } //ENVOI DE LA VALEUR AU SERVEUR xhr.open("POST","ajaxTestOperAssocieDomaine.php",true); xhr.setRequestHeader('Content-Type','application/x-www-form-urlencoded'); xhr.send("idDomaine="+val); }); }); } testerSuppr();
Merci de votre aide

05/06/2013, 22h01
Bovino

Bon... je ne suis ni politicien ni diplomate, donc je vais te dire la vérité même si elle est désagréable...
Ton code est une bouillie infâme qui n'a ni queue ni tête.
Tu as manifestement collé des portions de code sans comprendre ce que tu fais.
Tu mélanges du code jQuery et du JavaScript natif de façon incohérente.
Tu ne comprends manifestement pas le mécanisme des échanges client/serveur.

Bref, en l'état, il n'y a que deux alternatives : te faire ton travail, mais ça, ce n'est pas le but du forum ou te laisser te noyer dans tes turpitudes...

Lorsque l'on veut faire des choses un peu avancées en JavaScript (donc aller un peu plus loin qu'un alert('Coucou') déclenché sur un attribut onclick d'une balise, la moindre des choses est d'apprendre les bases. Commence donc par cette étape essentielle si tu ne veux pas que tes développements soient une suite d'échecs successifs.
06/06/2013, 10h02
Xeuch

Merci beaucoup Bovino pour la réponse .

Citation:

Ton code est une bouillie infâme qui n'a ni queue ni tête.

je n'ai pas posté tout mon code, je n'ai posté que la partie qui me pause problème.

Citation:

Tu as manifestement collé des portions de code sans comprendre ce que tu fais.

là vraiment je ne suis pas du tout d'accord avec toi car ce code je l'ai pondu tout seul. :nono:
Peut être ce n'est pas très propre vu que je débute avec jQuery j'ai pensé que vu que c'est du JavaScript les mélanger ne pause aucun problème et d'ailleurs cette cette question je l'avais déjà posé dans un forum ou les gens m'ont fait comprendre que c'était bien possible, donc pardonnez mon ignorance si ça ne se fait pas, je ne le savais pas ça :roll:

Je ne comprend vraiment pas ce que vous entendez par ne pas comprendre les mécanismes d'échange client/serveur car j'ai bien lu le tuto sur Ajax/JavaScript et l'échange des données avec Ajax sur un autre site a moins que je n'ai rien pigé de ce que j'ai lu :roll:

Citation:

Bref, en l'état, il n'y a que deux alternatives : te faire ton travail, mais ça, ce n'est pas le but du forum ou te laisser te noyer dans tes turpitudes...

J'ai suivi le mécanisme expliqués dans les tutots et j'ai fais mon propre code, j'accepte que je ne suis pas un expert et c'est pour ça que si je ne comprend pas quelque choses je viens auprès des amis qui en sache plus que moi d'où mon post .Je ne demande jamais et jamais à ce que quelqu'un me fasse mon travail je voudrais juste avoir des remarque et suggestion sur ce qui ne vas pas.

si vous pouvez me dire en quoi mon code

Citation:

est une bouillie infâme

je suis vraiment preneur car les bonnes pratiques s'acquiert avec le temps.

Merci
06/06/2013, 10h19
SpaceFrog

Citation:

l'échange des données avec Ajax sur un autre site

Tout s'explique...

http://ajax.developpez.com/cours/
06/06/2013, 10h38
getz85
Mélanger JQuery et pure javascript n'est pas vraiment un souci, mais c'est plutôt sur la manière dont tu le fais.

Tu utilises JQuery pour quoi dans ton code? Ajouter l’événement click et récupérer l'attribut id. C'est tout. De plus, utiliser le $(document).ready dans une fonction javascript pour ensuite appeler cette fonction n'est pas vraiment une bonne pratique.

Je te conseille plutôt de faire:
Code:

1 2 3 4 5 6 7 $(document).ready(function(){ $('.supprim').click(function(){ //Ton code }); });
De plus vu que tu ajoutes une telle bibliothèque, je te suggère d'utiliser les fonctionnalités qu'elle propose; c'est à dire utiliser la fonction .ajax() :

http://api.jquery.com/jQuery.ajax/

En suivant les examples de la doc, tu devrais réussir à sortir un code plus élégant, et si tu as un souci, tu peux revenir le partager ici ;)
06/06/2013, 10h46
Xeuch

Merci beaucoup getz85 pour la réponse et les remarques,

je vais suivre tes conseils et je reviendrais si j'ai un soucis.
Merci encore une fois.
06/06/2013, 10h59
Bovino

Ensuite, tu dis que tu fais un return false... certes, mais ce return est renvoyé à quoi selon toi ?
A une fonction qui a terminé depuis longtemps son exécution et qui ne peut donc plus recevoir aucune donnée...

Merci,

c'est vrai que le contexte de la fonction n'est plus.. au moment ou j'effectue le return.

j'ai pensé à 2 solution l'une je viens de l'implémenté mais j'ai un petit souci la dessus.
Enfait si la réponse est true (élément est associé à d'autres éléments donc nedois pas être supprimé) ou bien l'user a répondu non au confirm() je supprime le lien.

Cependant après il faut que je réattribue le lien mais c'est là ou se trouve le problème car je reçois la réponse du serveur après

voici le code
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 $(document).ready(function(){ $('.supprim').click(function(){ var val=$(this).attr('id'); var attri=$(this).attr('href'); var xhr = getXhr(); xhr.onreadystatechange = function(){ if(xhr.readyState == 4 && xhr.status == 200){ reponse = xhr.responseText; alert("la reponse est " +reponse); if(reponse=="true"){ alert("les domaines associes à des types d'operation ne peuvent pas etres supprimees"); $(this).removeAttr('href'); } else{ if (!(confirm('voulez vous vraiment supprimer ce domaine ?'))){ $(this).removeAttr('href'); } } } } //ENVOI DE LA VALEUR AU SERVEUR xhr.open("POST","ajaxTestOperAssocieDomaine.php",true); xhr.setRequestHeader('Content-Type','application/x-www-form-urlencoded'); xhr.send("idDomaine="+val); //reattribueer le href $(this).attr('href',attri); }); });
la deuxième solution dont j'ai pensé est de coller un attribut data à l'élément et de mettre sa valeur initiale à false par exemple et quand je reçois la réponse du serveur, je modifie cette valeur en true comme ça à la fin je regarde la valeur de cet attribut data pour réattribuer mon Href à l'élément mais cependant vu de loin, je crains que je ne saurais pas si la réponse du serveur arrivera à temps.

existe t'il un moyen pour attendre la réponse du serveur avant la réattribution de mon href?

Merci de votre aide

J'ai trouvé ,

Au click, j'enlève la valeur du Href du lien que je stock dans une variable, lorsque la réponse du serveur arrive, si je dois supprimer , je fais une redirection vers la page de suppression avec la valeur du Href du lien, sinon je remet le lien.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
var val=$(element).attr('id');
var attri=$(element).attr('href'); //stocker la valeur du lien
$(element).removeAttr('href'); // supprimer le line
 
var xhr = getXhr();
xhr.onreadystatechange = function(){
   if(xhr.readyState == 4 && xhr.status == 200){
     reponse = xhr.responseText;
     if(reponse=="true"){
       alert("les domaines associes à des types d'operation ne peuvent pas etres supprimees");
       $('#'+val).attr('href',attri); //remttre le lien
     }
     else{
     if (confirm('voulez vous vraiment supprimer ce domaine ?')){
	document.location.href=attri;//rediriger vers la page de suppression avec le lien
    }
    else{
    $('#'+val).attr('href',attri); //remettre le lien
    }
  }
 }
}

En passant que c'est une solution propre.
Si vous avez des avis ils sont les bienvenus.

Merci