JavaScript 揭示模块模式重构建议

Question

我在当前的项目中经常使用 JavaScript 的揭示模块模式来组织我的代码，但我想知道我是否可以以更好的方式使用它，因为我似乎创建了很多私有方法，然后以线性方式调用它们，即当一个方法执行完其操作后，调用另一个方法，然后调用另一个方法，依此类推。

该代码可以工作，但想看看它是否可以更好、更高效或使用更少的代码来实现相同的结果。

下面是一个示例模块，我们需要：

1. 将点击事件绑定到选项卡
2. 设置一些元数据值
3. 调用第 3 方跟踪函数

4. 将元数据重置为其原始值

   icisSite.tabPageView = 函数($){
   
       var $tabs = $('.tbc .tbc-tabs > div'),
           页面名称 = $('元[名称=gwa_pageName]'),
           sec3 = $('元[名称=gwa_siteSection3]'),
           sec4 = $('元[名称=gwa_siteSection4]'),
           页名长度 = 页名.长度,
           sec3Len = sec3.长度，
           sec4Len = sec4.长度，
           TXT，
           pName内容，
           pName构造，
           部分，
           元内容，
           元构造；
   
       函数初始化（）{   
           bindClickHandlerToTabs();
       };  
   
       函数bindClickHandlerToTabs() {
           if (pageNameLen !== 0 && (sec3Len !== 0 || sec4Len !== 0)){
               $tabs.bind('点击', updateMetaData(){
                   txt = $(this).find('span').text(),
                   txt='| ' + 文本；
                   更新元数据（）；
               }); 
           } 别的 {
               返回假；
           }           
       };  
   
       函数更新元数据（）{
           pNameContent = pageName.attr('内容'),
           pNameConstruct = pNameContent + txt;
           if (sec3Len !== 0 && sec4Len !== 0){
               部分 = 'gwa_siteSection4',
               metaContent = sec4.attr('内容'),
               元构造=元内容+txt；
           } 别的 {
               部分 = 'gwa_siteSection3'
               元内容 = sec3.attr('内容'),
               元构造=元内容+txt；
           }   
           返回调用GWATrackingFucntions();  
       };
   
       函数调用GWATrackingFucntions() {
           gwa_SetMetaValue(节，metaConstruct); 
           gwa_trackPageView(pNameConstruct);
           返回重置元数据（）；
       };
   
       函数重置元数据（）{ 
           pageName.attr('内容', pNameContent);
           $('元').each(函数(索引){
               var name = $(this).attr('name');
               if (部分 === 名称){
                   $(this).attr('内容', 元内容);   
               }
           });     
       };
   
       返回 {
           初始化：初始化  
       };
   
   }(jQuery);
   
   icisSite.tabPageView.init();
   

Answer 1

如果您要创建仅调用一次的函数，则需要询问为什么它们是函数而不仅仅是顺序代码。也许是为了可扩展性或模块化。额外的函数调用并不是特别重要。

您可以通过更改以下内容中固有的多个函数调用来极大地提高效率（尽管可能不明显）：

$(this).attr('content', metaContent); 

使用直接属性访问：

this.content = metaContent;

除非您的目的是修改 HTML 内容属性（这不太可能）而不是 DOM 属性（我有不知道在这种情况下，attr 方法实际上会改变哪一个）。

jQuery 即将对饱受诟病的 attr 方法进行一些重大更改，除非您确实需要（几乎从来没有），否则不要使用它始终是一个好主意。