具有公共 getter 和 setter 方法的 JavaScript 对象

Question

昨天，我发布了关于prototypal inheritance and constructors 的信息。我最终决定采用一种方法，使代码保持整洁，并以可能对性能产生轻微影响为代价让原型保持独立：

function Card(value) {
    // setValue is a public instance method
    this.setValue = function (val) {
        if (!range[val]) {
            val = drawRandom();
        }

        // value is a public instance variable
        this.value = val;

        return val;
    };

    this.setValue(value);
}

然而，我对这种方法的问题是，每当我想设置 Card 实例的值以获得验证时，我都必须调用 setValue 方法。我想做的是有一个自定义的setter方法。到目前为止，这是我所拥有的：

function Card(val) {
    // value is a private instance variable
    var value;

    // This is a private instance method
    // It's also self-invoking, but that's beside the point
    (function (x) {
        if (!range[x]) {
            x = drawRandom();
        }

        value = x;
    }(val));

    this.__defineGetter__("value", function () {
        return value;
    });

    // Some code duplication
    this.__defineSetter__("value", function (x) {
        if (!range[x]) {
            return false;
        }

        value = x;

        return x;
    });
}

这很好用。调用var card = new Card() 给了我一个随机值的实例，调用card.value = null 失败，因为它超出了范围。

我的问题是，除了它明显更长之外，我似乎在复制一些代码。如果 setter 方法与构造函数一起被调用，那就太好了。这样，我可以消除整个自调用私有实例方法。

Answer 1

首先，您应该始终将value 设置为obj.value = newValue，即使在内部也是如此，因为它会调用您的验证。在构造函数中表示：

this.value = val;

但是，如果您在声明 setter 和 getter 之前 这样做，那将不起作用。所以把它移到之后，这样setter函数就存在了。

---

这里的工作示例：http://jsfiddle.net/8tCjm/4/

var drawRandom = function () {
    return Math.floor(Math.random() * 3) + 1;
};

var range = {
    1: 'Ace',
    2: 'Two',
    3: 'Three'
};

function Card(val) {
    var value;

    this.__defineGetter__('value', function () {
        return value;
    });

    this.__defineSetter__('value', function (x) {
        if (range[x]) {
            value = x;
        } else {
            value = drawRandom();
        }

        return value;
    });

    this.value = val;
};

console.log(new Card(1).value); // 1
console.log(new Card(2).value); // 2
console.log(new Card(3).value); // 3

console.log(new Card(987).value); // not 987 (1-3)
​

Answer 2

函数是 Javascript 中的第一类对象，所以你完全可以这样做来消除重复：

function setter (x) {
    if (!range[x]) {
        return false;
    }
    return x;
}
(function (x) {
    value = setter(x);
    if (!value) {
        value = drawRandom();
    }

}(val));

this.__defineGetter__("value", function () {
    return value;
});

// Some code duplication
this.__defineSetter__("value", setter);