Javascript Jetbrains PHPStorm 5.0.4告诉我有一个重复的jQuery选择器

这是我写的jQuery的一部分

$('#editUser').click(function() {
    if ($(".selectedTR")[0]) {
        if($('.form-actions').is(':visible')) {
            $('.form-actions').slideUp('slow',function() {
                $('.form-actions > h3').text("Edit");
            }).css('display', 'none');
        }

        $('.form-actions').css('display', 'block').slideDown('slow');
    } else {
        alert("Please select a user");
    }
});

---

如何删除重复的选择器？

您可以通过将选择器放入变量来缓存它。试试这个：

$('#editUser').click(function() {
    if ($(".selectedTR").length) { 
        var $formActions = $('.form-actions');
        if ($formActions.is(':visible')) {
            $formActions.slideUp('slow', function() {
                $formActions.children('h3').text("Edit");
            }).css('display', 'none');
        }
        $formActions.css('display', 'block').slideDown('slow');
    } else {
        alert("Please select a user");
    }
});

---

为

$（'.form actions'）

声明一个变量并使用它。PHPStorm WTF。我已经厌倦了不断地告诉人们要缓存他们的查询。@ŠimeVidas，嗯？它的好phpstorm是突出低效代码，不是吗？@NimChimpsky LOL，我指的是FTW，不是WTF。可能更适合此类问题，因为您的代码中实际上没有任何错误。您甚至可以将

if（$formActions.is（':visible'））{$formActions.slideUp（

替换为

$formActions.filter（':visible'）.slideUp（