公共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); 
} 

我的這種方法的問題，然而，就是我要調用的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);