Question

我几天创建了这段代码，但我相信有可能改进它，有人可以帮助我创建一个更聪明的方法吗？

// Hide registered or customized field if not checked.
function checkUserType(value) {
  if (value == 2) {
    $('#registered').hide();
    $('#customized').show();
  } else if (value == 1) {
    $('#registered').show();
    $('#customized').hide();
  }
}

checkUserType($('input:radio[name="jform[place_type]"]:checked').val());

$('#jform_place_type').on('click', function () {
  checkUserType($('input:radio[name="jform[place_type]"]:checked').val());
});

演示：http://jsbin.com/emisat/3

Answer 1

// Hide registered or customized field if not checked.
function checkUserType(value) {

}

var t = function () {
  var value = $('input:radio[name="jform[place_type]"]:checked').val();
  if (value == 2) {
    $('#registered').hide();
    $('#customized').show();
  } else if (value == 1) {
    $('#registered').show();
    $('#customized').hide();
  }
};

$('#jform_place_type').on('click', t);

Answer 2

您可以通过存储DOM元素并缓存其余部分来改进Jquery（性能）。这是我猜你能达到的最大值。

function checkUserType(value) {
  var r = $("#registered");
  var c = $("#customized");
  if (value == 2) {
    r.hide();
    c.show();
  } else if (value == 1) {
    r.show();
    c.hide();
  }
}

var func = function () {
  checkUserType($('input:radio[name="jform[place_type]"]:checked').val());
};

$('#jform_place_type').on('click', func);

如需进一步阅读，请查看此JQuery Performance

特别阅读文件的第三段

缓存jQuery对象

养成将jQuery对象保存到变量的习惯（就像上面的例子一样）。例如，never（eeeehhhhver）执行此操作：

$('#traffic_light input.on').bind('click', function(){...});
$('#traffic_light input.on').css('border', '3px dashed yellow');
$('#traffic_light input.on').css('background-color', 'orange');
$('#traffic_light input.on').fadeIn('slow');

相反，首先将对象保存到本地变量，然后继续操作：

var $active_light = $('#traffic_light input.on');
$active_light.bind('click', function(){...});
$active_light.css('border', '3px dashed yellow');
$active_light.css('background-color', 'orange');
$active_light.fadeIn('slow');

提示：由于我们要记住我们的局部变量是一个jQuery包装集，我们使用$作为前缀。请记住，永远不要在应用程序中多次重复jQuery选择操作。

Answer 3

http://api.jquery.com/toggle/

    $('#jform_place_type').on('click', function () {
//show is true if the val() of your jquery selector equals 1
// false if it's not
      var show= ($('input:radio[name="jform[place_type]"]:checked')
        .val()==1);
//set both divs to visible invisible / show !show(=not show)
// (not show) means that if show=true then !show would be false
      $('#registered').toggle(show);
      $('#customized').toggle(!show);
    });

如果您不止一次需要一个选择器然后缓存它我认为它被称为对象缓存，因为Claudio已经提到过，这就是为什么你会看到很多：

$this=$(this);
$myDivs=$("some selector");

保存jquery函数（jquery对象）结果的变量的约定是它们以$开头，但由于它只是一个变量名，你可以把它称为任何你想要的，下面的代码也可以这样做：

me=$(this);
myDivs=$("some selector");

有一个更好的方法隐藏和显示div与收音机检查？

3 个答案: