执行 javascript“方法”；从对象调用其他方法时默认情况下

Question

我读到，在 javascript 中不要使用太多全局变量是一个很好的做法，因此我尝试将一些与我的应用程序的导航按钮相关的功能打包在一个变量中，如下所示：

var navs = {
  projects:   function(){
    main.removeClass().addClass('showing_projects');
    //... other code
  },
  line_items: function(){
    main.removeClass().addClass('showing_line_items');
    //... other code
  },
  media:      function(){
    main.removeClass().addClass('showing_media');
    //... other code
  }
}

这样我就可以

navs.projects()

显示项目。我的问题是，我有一个 showNotify() 函数，我需要在调用任何 navs.projects()、navs.line_items() 或 navs.media() 后运行它。我觉得将这个 showNotify() 行添加到这三个属性/方法中并不干燥。有没有一种方法可以让 showNotify() 在每次调用这三个方法时运行？

谢谢

Answer 1

您可以参数化每个函数的不同部分：

var navs = {
    addClassAndNotify: function(className, fn) {
        main.removeClass().addClass(className);
        fn();
        showNotify();

    }
}

并像这样使用：

navs.addClassAndNotify('showing_media', function() {
    // showing media code
});

您还可以创建一个函数生成器，如下所示：

function create(className, fn) {
    return function() {
        main.removeClass().addClass(className);
        fn();
        showNotify();
    }
}

然后像这样创建函数：

var navs = {
    projects: create(function(){
        //... other code
    }),
    line_items: create(function(){
        //... other code
    }),
    media: create(function(){
        //... other code
    }),
}

但我不认为调用 showNotify< /code> 在每个函数中都是一件大事。

Answer 2

将 showNotify() 添加到每个方法的末尾并不违反 DRY。违反 DRY 的行为是复制/粘贴 showNotify() 的内容而不是调用它。

据我所知，没有比添加调用来实现目标更简洁的方法了。即使有，这样做也是一个非常糟糕的主意，因为调用这些函数会产生一些阅读代码的人根本看不到的效果。