Pro Javascript 设计模式勘误表？

Question

任何人都可以确认《Pro Javascript 设计模式》第 3 章中的这些示例是否有缺陷，如果有的话，从根本上讲，它们距离生成“类”常量的预期目标是否超过一两个拼写错误？在 JavaScript 中？谢谢。

var Class = (function() {

  // Constants (created as private static attributes).
  var UPPER_BOUND = 100;

  // Privileged static method.
  this.getUPPER_BOUND() {//sic
    return UPPER_BOUND;
  }

  ...

  // Return the constructor.
  return function(constructorArgument) {
    ...
  }
})();

/* Usage. */

Class.getUPPER_BOUND();

/* Grouping constants together. */

var Class = (function() {

  // Private static attributes.
  var constants = {
    UPPER_BOUND: 100,
    LOWER_BOUND: -100
  }

  // Privileged static method.
  this.getConstant(name) {//sic
    return constants[name];
  }

  ...

  // Return the constructor.
  return function(constructorArgument) {
    ...
  }
})();


/* Usage. */

Class.getConstant('UPPER_BOUND');

Answer 1

我认为，这是错误的。正如前面提到的，“this”指的是窗口对象，代码也存在语法错误。以下代码应该实现所需的目标：

var Class = (function () {

    // Private static attributes.

    var constants = {
        UPPER_BOUND: 100,
        LOWER_BOUND: -100
    };            

    var sc = function (constructorArgument) {

    };

    // Privileged static method.
    sc.getConstant = function (name) {
        return constants[name];
    };

    // Return the constructor.
    return sc;
})();

alert(Class.getConstant('UPPER_BOUND'));

Answer 2

该代码可以简单地修复，因为

var Class = {
  UPPER_BOUND: 100
};

代码的其余部分是过度设计的或完全错误的，应该被忽略。

如果您关心只读，请将可写标志设置为 false（注意默认值为 false）。

var Class = {};
Object.defineProperty(Class, "UPPER_BOUND", {
  value: 100,
  enumerable: true,
  configurable: true
});

Answer 3

警惕任何声称是“Pro”的东西。我没有读过这本书，但我对代码的看法如下：

> var Class = (function() {
> 
>   // Constants (created as private static attributes).

“属性”这个词是错误的，它应该是“属性”或“变量”，因为它们是变量，也可以描述为属性本地激活/变量对象。

>   var UPPER_BOUND = 100;
> 
>   // Privileged static method.  
>   this.getUPPER_BOUND() {//sic

该代码将在全局上下文中执行，其中 this 是窗口/全局对象。因此，如果存在全局 *getUPPER_BOUND* 函数，则将不带参数调用该函数。它后面是一个大括号 ({)，它在不能出现块的位置打开一个块，因此这是一个语法错误。

我假设以下内容是有意的：

    this.getUPPER_BOUND = function() {

它创建全局/窗口对象的 getUPPER_BOUND 属性，该属性在代码运行时分配 RHS 上的匿名函数。

>     return UPPER_BOUND;   }
> 
>   ...
> 
>   // Return the constructor.
>   return function(constructorArgument) {

这是分配给全局变量“Class”的函数。

>     ... 
>   }
>  })();

经过修复，它可能会“工作”，但并不优雅。任何在代码中出现如此明显错误的书都没有经过仔细编写，并且在出版之前肯定也没有经过适当的审查。

使用信誉良好的在线资源，并继续就您不理解或认为有错误的任何问题提出问题。还有其他讨论 javascript 的论坛可以为技术问题提供更详细的答案。

Answer 4

看看这个作为一个不错的选择： http://www.klauskomenda.com/code /javascript-programming-patterns/

对于不可变的公共属性，正如建议的那样，使用 Object.freeze 和 John Resig 的好建议：http://ejohn.org/blog/ecmascript-5-objects-and-properties /

并且为了不破坏全局范围，请将命名空间添加到 jQuery：是否可以在 jQuery 中创建命名空间？

Answer 5

让它工作，但不确定这是否是作者的意图。

var Class = (function()
{
    // Constants (created as private static attributes).
    var constants =
    {
        UPPER_BOUND: 100,
        LOWER_BOUND: -100
    };

    // Return the method(s).
    return {
        getConstant: function(name)
        {
            return constants[name];
        }
    }
}());

console.log(Class.getConstant('UPPER_BOUND')); // shows "100" in console

Answer 6

这样做怎么样？

/* Grouping constants together. */
var Class = (function() {
  // Private static attributes.
  var constants = {
     UPPER_BOUND: 100,
     LOWER_BOUND: -100
  }

  // Return the constructor.
  return new function(constructorArgument) {
     // Privileged static method.
     this.getConstant = function(name) {//sic
       return constants[name];
     }
   }
})();

console.log(Class.getConstant("LOWER_BOUND"));