Question

我正在尝试最小化（提高效率）我的JavaScript代码，并讨论我应该采用哪种方法。

我有10多个页面，每个页面有两个不同的视频;显示未登录用户的短视频和登录用户显示的长视频。我正在使用WordPress网站，所以我使用post-id而不是视频播放器ID（插件）来定位每个视频，因为它是动态的（每次页面刷新时都会更改）。页面上的每个视频都包含在一个类中，因此我可以在单击任一视频时进行定位。

例如，

$f('#post-1883 .short-video .player').bind({
    ready : function(e) {
        e.preventDefault();
        console.log('short video'); 
        // Do something here if it's a short video      
    }   
});

 $f('#post-1883 .long-video .player').bind({
    ready : function(e) {
        e.preventDefault();
        console.log('long video');
        // Do something here if it's a long video           
    }   
});

正如您在示例中所看到的，当您使用不同的帖子ID（例如＃post-1122，＃post-4234等）定位JavaScript文件中的10个视频时，它可能是重复且笨重的。 / p>

处理这种情况的最佳方法是什么？我应该使用switch-case语句吗？

谢谢， rolu

Answer 1

这适合你吗？

$('.player').bind('click', function (e) {
    e.preventDefault();
    var self = $(this),
        videoId = this.id;

    if(self.hasClass('long-video')) {
        console.log('long video');
    } else if(self.hasClass('short-video')) {
        console.log('short-video');
        console.log(videoId);
    }
});

您可以看到正在使用的示例：JsFiddle

JS：我应该在这种情况下使用switch-case语句还是if / else？

1 个答案: