Ajax jquery succès de la portée

J'ai cet appel ajax à un doop.php. 

    function doop(){
        var old = $(this).siblings('.old').html();
        var new = $(this).siblings('.new').val();

        $.ajax({
            url: 'doop.php',
            type: 'POST',
            data: 'before=' + old + '&after=' + new,
            success: function(resp) {
                if(resp == 1) {
                    $(this).siblings('.old').html(new);
                }
            }
        });

        return false;
    }

Mon problème est que le $(this).siblings('.old').html(new); ligne n'est pas ce qu'il est censé faire.

merci..
tous les commentaires/réponses sont votées.

Mise à jour: il semble que la moitié du problème est dans la portée (merci pour les réponses qui m'ont aidé à préciser que), mais l'autre moitié, c'est que je suis en train d'utiliser ajax dans une manière synchrone. J'ai créé un nouveau post

InformationsquelleAutor Chris | 2009-10-15
  1. 26

    Tout d'abord new est un mot réservé. Vous devez renommer cette variable.

    Pour répondre à votre question, Oui, vous avez besoin pour économiser de l' this dans une variable en dehors de la réussite de rappel, et de référence à l'intérieur de votre succès gestionnaire de code:

    var that = this;
$.ajax({
    //...
    success: function(resp) {
        if(resp == 1) {
            $(that).siblings('.old').html($new);
        }
    }
})

    Ce qui est appelé un fermeture.

    • Hmm, bizarre, j'ai essayé quelque chose de très similaire à ce que vous avez fait, mais je n'ai var saveit = $(this); n'a pas fonctionné. Je vais essayer maintenant.. Aussi ne vous inquiétez pas nouvelle, elle est appelée à autre chose dans mon code.
    • re: new, j'ai compris beaucoup de choses. 🙂
    • re la fermeture ne fonctionne pas, assurez-vous que doop est lui-même en se référant à la attendu this. Si vous appelez doop() par exemple, this pointe simplement vers la window objet.
    • Ok, après la fermeture, je suis en mesure de cibler le droit de la ce/que (je l'ai testé avec afficher/masquer et je sais que c'est le ciblage de la chose). Cependant, ce n'est pas la mise à jour de l'html.. je suis en train de faire un dépannage et de mise à jour de ma réponse ci-dessus, s'il vous plaît revenez quand vous obtenez une chance, merci.
    • Maintenant que j'ai fait de mon dépannage, j'en suis venu à la conclusion que la seconde moitié du problème, c'est que je dois utiliser ajax synchrone. pas asynchrone. Updaing ma question ci-dessus et poser une nouvelle question.
    • Vous avez raison sur le new mot-clé, mais cela ajoute beaucoup trop augmentation du code dans une longue liste à laquelle il est ajouté dynamiquement, chaque var devrait être unique, où, comme dans la réponse ci-dessous, à l'aide de context: this t-il une fois et a beaucoup plus souple des cas d'utilisation.

    InformationsquelleAutor Crescent Fresh
  2. 48

    Vous devez utiliser le contexte paramètre comme dans http://api.jquery.com/jQuery.ajax/

    function doop(){
    var old = $(this).siblings('.old').html();
    var newValue = $(this).siblings('.new').val();

    $.ajax({
        url: 'doop.php',
        type: 'POST',
        context: this,
        data: 'before=' + old + '&after=' + newValue,
        success: function(resp) {
            if(resp == 1) {
                $(this).siblings('.old').html(newValue);
            }
        }
    });

    return false;
}

    "ce" sera de transfert pour la réussite de la portée et agira comme prévu.

    • Vraiment vous ne devriez pas utiliser des mots réservés comme new comme noms de variable.
    • D'accord avec Nick - c'est la bonne approche. Considérer que le "enregistrer dans une autre variable" approche ne fonctionnera pas si vous avez la fonction "doop" appelée lorsque l'un des multiples utilisateur clique sur les boutons, ou quelque autre de duplication.
    • +1 Beaucoup plus souple réponse que celle sélectionnée, dans notre cas, le var devrait être unique pour chaque élément unique dans une très longue liste d'éléments ajout d'augmentation du code, c'est un peu court personnages et donne beaucoup plus de souplesse dans l'utilisation des cas.
    InformationsquelleAutor Guillaume Bois
  3. 6

    this est lié à l'objet pour lequel l'exécution de la fonction a été appliquée. Que pourrait être une réponse AJAX objet, ou l'objet global (window), ou quelque chose d'autre (en fonction de la mise en œuvre de $.ajax.

    Ai-je besoin pour capturer $(this) dans une variable avant d'entrer dans l' $.appel ajax, et ensuite de le passer en paramètre à l' $.appel ajax? ou ai-je besoin de passer à la anonyme succès fonction? Si ça va résoudre le problème, où dois-je passer à l' $.ajax?

    Vous n'avez en effet besoin d'un moyen de capturer la valeur de this avant de définir les success fonction. La création d'une clôture est la façon de le faire. Vous devez définir une variable distincte (par exemple,self): 

    function doop() {
    var old = $(this).siblings('.old').html();
    var new = $(this).siblings('.new').val();

    var self = this;

    $.ajax({
        url: 'doop.php',
        type: 'POST',
        data: 'before=' + old + '&after=' + new,
        success: function(resp) {
            if(resp == 1) {
                $(self).siblings('.old').html(new);
            }
        }
    });

    return false;
}

    La success fonction de conserver la valeur de self lorsqu'il est invoqué, et doit se comporter comme prévu.

    • Merci +1. C'est le même que crescentfresh de réponse et il résout une partie du problème".. sera mise à jour de la question avec plus de résolution des problèmes.
    InformationsquelleAutor harto