Puis-je utiliser des fermetures Javascript ici au lieu d'une variable globale?

Question

configuration actuelle:

var placeId; 
function selectPlace(place) { 
    $('#selectPlace').html('Selected Place: <b>' + place.Name + '</b>'); 
    $('#map').hide(400); 
    placeId = place.Id; 
} 

$(document).ready(function() 
{ 
    $('#postMessage').click(function() { 
     alert("PlaceId: " + placeId); 
    }); 
}); 

Can/devrais-je utiliser des fermetures?

Answer 1

Sur la base de vos commentaires, il semble que ce que vous « cherchez est le suivant:

function selectPlace(place) { 
    if(!place){ 
    return selectPlace.placeId; 
    }else{ 
    $('#selectPlace').html('Selected Place: <b>' + place.Name + '</b>'); 
    $('#map').hide(400); 
    selectPlace.placeId = place.Id; 
    } 
} 

$(document).ready(function(){ 
    $('#postMessage').click(function() { 
    alert("PlaceId: " + selectPlace()); 
    }); 
}); 

ce n'utilise pas une fermeture, juste stocke le dernier ID attribué sur l'objet de la fonction. Vous devez ensuite renvoyer la valeur si elles n'utilisent pas la fonction en tant que setter. Si vous souhaitez utiliser une fermeture pour faire la même chose, il ressemblerait beaucoup à l'exemple ci-dessus:

(function(){ 
    var placeId; 

    window.selectPlace = function(place) { 
    if(!place){ 
     return placeId; 
    }else{ 
     $('#selectPlace').html('Selected Place: <b>' + place.Name + '</b>'); 
     $('#map').hide(400); 
     placeId = place.Id; 
    } 
    } 
})(); 

Par ailleurs, la meilleure façon de repérer une fermeture est si une fonction a des variables dans ce que n'a pas été déclaré avec var à l'intérieur de la fonction en cours, mais a été dans une autre fonction, il se trouve à l'intérieur. Comme vous pouvez le voir ci-dessus, la variable placeId n'est pas déclarée à l'intérieur de la fonction selectPlace, ce qui signifie que la fonction selectPlace est une fermeture qui utilise la variable placeId.

Answer 2

Il semble une chose raisonnable à faire, en fonction du contexte, vous pouvez facilement le faire en remplaçant votre code avec une expression de fonction à la:

(function(){ 
    var placeId; 
    // It looks like you want selectPlace to be a global function? 
    // hence i'm using assignment of a function expression here 
    selectPlace = function (place) { 
     $('#selectPlace').html('Selected Place: <b>' + place.Name + '</b>'); 
     $('#map').hide(400); 
     placeId = place.Id; 
    } 

    $(document).ready(function() 
    { 
     $('#postMessage').click(function() { 
      alert("PlaceId: " + placeId); 
     }); 
    }); 
})();