Javascript OOP private functions

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 на объекте, я получаю сообщение об ошибке в функции qryTempArray в json.length в течение цикла (не определено). Зачем?

Answer 1

Поскольку код написан выше, я удивлен, что вы даже попадаете в цикл. Казалось бы, вы получите undefined при вызове 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());