javascript - Jetbrains PHPStorm 5.0.4 は、重複した jQuery セレクターがあることを教えてくれます

Question

これは私が書いた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");
    }
});

複製されたセレクターを削除するにはどうすればよいですか?

Answer 1

セレクターを変数に入れることで、セレクターをキャッシュできます。これを試して：

$('#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");
    }
});