JavaScript mécanisme destiné à maintenir sur une valeur d'une action de l'utilisateur

Question

Je l'ai créé pour conserver une valeur définie par un utilisateur contrôle d'un checbox dans un ColorBox .

Je suis relativement nouveau à jQuery et programmation JavaScript « la bonne façon » et je voulais être sûr que le mécanisme ci-dessous pour capturer les utilisateurs vérifier l'action est une bonne pratique pour JavaScript en général. De plus, puisque j'emploie jQuery est-il une méthode plus simple de conserver leur action que j'UTILISERA?

function Check() {
    this.Checked = false;
}

obj = new Check;

$(document).ready(function() {
    $('.cboxelement').colorbox({ html: '<input id="inactivate" type="checkbox" name="inactivatemachine">&nbsp;<label for="inactivate">Inactivate Machine</label>' });

    $(document).bind('cbox_cleanup', function() {
        obj.Checked = $.fn.colorbox.getContent().children('#inactivate').is(':checked');
    });

    $(document).bind('cbox_closed', function() {
        if ($($.fn.colorbox.element()).attr('id').match('Remove') && obj.Checked) {
            var row = $($.fn.colorbox.element()).parents('tr');
            row.fadeOut(1000, function() {
                row.remove();
            });
        }
    });
});

Answer 1

Personnellement, je fixer la valeur (s) à un objet directement en utilisant la méthode de data() intégré jQuery. Je ne suis pas tout à fait sûr de ce que vous essayez de faire, mais, vous pouvez, par exemple, fixer des valeurs dans les DOM pour une utilisation ultérieure un à un « espace de noms ».

$('body').data('colorbox.checked',true);

Ensuite, vous récupérerez la valeur par la suite:

var isChecked = $('body').data('colorbox.checked');

Vous exécutez la méthode data() sur un objet jquery. Je dirais que cela est plus pratique dans la mesure où jQuery va.

Answer 2

Vous pouvez saisir la référence dans une fermeture, ce qui évite les données globales et le rend plus facile d'avoir de multiples contrôles. Cependant, dans ce cas, il semble se lier au colorbox seul, donc je ne sais pas que vous pourriez utilement avoir plusieurs instances.

function Check() {
    this.Checked = false;

    var obj = this; // 'this' doesn't get preserved in closures

    $(document).ready(function() {
       ... as before
    )};
}

var check = new Check; // Still need to store a reference somewhere.

Answer 3

• $($.fn.colorbox.element()) est redondant. $.fn.colorbox.element() est déjà un élément jquery.

• Il est d'usage courant (dans les exemples que je regardais, au moins) préfixer un $ aux variables faisant référence à des éléments de jquery.
  Ainsi, var $rows = $.fn.colorbox.element().parents('tr'); donne instantanément l'idée que l'on fait référence élément jquery (s).

• Je suis fadeOut peur ne fonctionnera pas sur les lignes dans IE6 (si je me souviens bien). Vous devriez être en mesure de se cacher tout le contenu à l'intérieur du <tr> avant de le retirer.

ne peut pas aider à la chose « simplifier » parce que je ne sais pas les meilleures utilisations du colorbox.