jQuery - Ajouter cliquez sur l'événement à Div et aller sur le premier lien trouvé

Je pense que j'ai été trop de temps à regarder cette fonction et juste coincé à essayer de comprendre le bien propre façon de le faire.

C'est une fonction jQuery qui ajoute un événement click pour tout div qui a un click classe CSS. Lorsque que div.click est cliqué, il redirige l'utilisateur vers le premier lien qui s'y trouvent.

function clickabledivs() {
    $('.click').each(
        function (intIndex) {
            $(this).bind("click", function(){
                window.location = $( "#"+$(this).attr('id')+" a:first-child" ).attr('href');
            });
        }
    );
}

Le code fonctionne tout simplement bien que je suis assez sûr qu'il y est assez de meilleure façon de l'accomplir, spécialement le sélecteur, je suis en utilisant: $( "#"+$(this).attr('id')+" a:first-child" ). Tout semble long et lent. Des idées?

S'il vous plaît laissez-moi savoir si vous avez besoin de plus de détails.
Merci!

PS: j'ai trouvé quelques très belles jQuery analyse comparative de référence de Project2k.de ici:
http://blog.projekt2k.de/2010/01/benchmarking-jquery-1-4/

InformationsquelleAutor Xavi Esteve | 2010-04-15
  1. 6

    En fonction du nombre de ces div.click éléments que vous avez, vous pouvez utiliser l'événement de la délégation pour gérer ces clics. Cela signifie en utilisant un seul gestionnaire d'événements pour tous les divs qui ont le click classe. Puis, à l'intérieur de ce gestionnaire d'événement, votre de rappel des actes fondés sur le qui div.click l'événement provient de. Comme ceci:

    $('#div-click-parent').click(function (event)
{
    var $target = $(event.target); //the element that fired the original click event
    if ($target.is('div.click'))
    {
        window.location.href = $target.find('a').attr('href');
    }
});

    Moins de gestionnaires d'événements, meilleure mise à l'échelle - plus div.click éléments qui ne ralentit pas votre gestion des événements.

    • Wow, merci pour la grande et des réponses rapides. La fonction pourrait certainement être optimisé beaucoup plus. Je vais rester avec l'un Ours mangent vous suggère. Un événement pour tous les div clics. Merci!!!!!
    • Juste un petit tweak, "si ($cible.est('div.cliquez sur')" besoin d'un supplément ) à la fin. Merci encore!
    • Oups, merci! Je vais corriger ça.
    • merci pour cette solution simple est d'essayer de faire des div cliquable basé sur h3 titre a href intérieur et cela fonctionne!
    InformationsquelleAutor Matt Ball
  2. 5

    optimisé délégation avec jQuery 1.7+

    $('#div-click-parent').on('click', 'div.click', function () {
   window.location.href = $(this).find('a').attr('href');
});
    InformationsquelleAutor Bastian Rang
  3. 2

    Au lieu de lier tous les clics sur la charge, pourquoi ne pas les lier au clic? Devrait être beaucoup plus optimal.

    $(document).ready(function() {
   $('.click').click(function() {
        window.location = $(this).children('a:first').attr('href');
        return false;
    });
});
    • Où voyez-vous quelque chose à propos de la liaison de l'clique sur charger?
    InformationsquelleAutor Jeriko
  4. 0

    Je serais probablement faire quelque chose comme;

    $('.click').click(function(e){
  window.location.href = $(this).find('a').attr('href');
});
    • +1 parce que vous avez été le premier à ne pas supposer qu'il a été immédiate d'un enfant!
    • Je suis assez sûr qu'il signifie simplement 'a' au lieu de a.
    InformationsquelleAutor David Yell