Javascript OOPプライベート関数

Question

jsonファイルでインスタンス化することができるコンストラクタを作成したいと思います。最後にその結果をプロトタイプのパブリック関数に渡すいくつかのプライベート関数によって使用されます。これは正しいアプローチですか？

ここで、より具体的なコード：

//constructor 
function queryArray(json){ 
    this.json = json; 

    //init qry template with default values 
    function qryInit() { 
     var qryTemplate = { 
      //some stuff 
     } 
     return qryTemplate; 
    } 

    //generate array of request templates 
    function qryTempArray(json){ 
     var template = qryInit(); 
     var qryTempArray1 = []; 
     for(var i = 0; i < json.length; i++){ 
      qryTempArray1.push({ 
       'SearchIndex': json[i].SearchIndex, 
       'Title': json[i].Title, 
       'Keywords': json[i].Keywords, 
       'MinimumPrice': json[i].MinimumPrice, 
       'MaximumPrice': json[i].MaximumPrice, 
       'ResponseGroup': template.ResponseGroup, 
       'sort': template.sort 
      }); 
     } 
     return qryTempArray1; 
    } 
} 

//function for finally building all the queries 
queryArray.prototype.qryBuilder = function(){ 
    var qryTempArray1 = []; 
    qryTempArray1 = qryTempArray(this.json); 
    //other stuff 
} 

私は、オブジェクト上qryBuilder関数を呼び出す場合、私は、forループ（未定義）でjson.lengthでの関数qryTempArrayにエラー を取得します。 なぜですか？

Answer 1

コードが上記のように書かれているので、私はループに遭遇することにも驚いています。あなたがqryBuilder（）を呼び出したときにあなたは未定義になると思われます。 私は以下の行に沿った何かが働くことを期待しています。

//constructor 
function queryArray(json) { 
    var self = this; 
    self.json = json; 

    //init qry template with default values 
    self.qryInit = function() { 
     var qryTemplate = { 
      //some stuff 
     } 
     return qryTemplate; 
    } 

    //generate array of request templates 
    self.qryTempArray = function(json) { 
     var template = self.qryInit(); 
     var qryTempArray1 = []; 
     for (var i = 0; i < json.length; i++) { 
      qryTempArray1.push({ 
       'SearchIndex': json[i].SearchIndex, 
       'Title': json[i].Title, 
       'Keywords': json[i].Keywords, 
       'MinimumPrice': json[i].MinimumPrice, 
       'MaximumPrice': json[i].MaximumPrice, 
       'ResponseGroup': template.ResponseGroup, 
       'sort': template.sort 
      }); 
     } 
     return qryTempArray1; 
    } 
    return self; 
} 
queryArray.prototype.qryBuilder = function() { 
    var qryTempArray1 = []; 
    qryTempArray1 = this.qryTempArray(this.json); 
    return qryTempArray1; 
} 
var q = new queryArray([{ 
    'SearchIndex': 0, 
    'Title': 'foo', 
    'Keywords': 'testing', 
    'MinimumPrice': 20, 
    'MaximumPrice': 40 
}]); 
console.log(q); 
console.log(q.qryBuilder());