jQuery colorbox代码审查和最佳实践

Question

我有这个颜色盒代码，但我不确定这是不是最好的方法。我有另一个因此而无效的应用程序。所以我想，下面的代码可能有问题。你看到了问题吗？感谢。

var $bootSelectPopup = $('#wraper-boot-select-structures-popup');
            $('#bootstrap-grid-popup').on("click", function(e){
            $bootSelectPopup.show();
            e.preventDefault();                
            $(function(){
            $.colorbox({
                href: $bootSelectPopup,
                transition: 'fade',
                speed: 150,
                fadeOut : 0,
                inline: true,                    
                close : 'x',
                onClosed: function() {
                    $bootSelectPopup.hide();                    
                }

            });

        });
        });