Нокаут arrayForEach undefined property

Question

У меня возникли проблемы с попыткой получить число из каждого элемента в наблюдаемом массиве нокаута и добавить числа вместе и назначить его другой вычислимой переменной. Вот что я прямо сейчас ...

Semesters: ko.observableArray([ 
    { 
     semesterName: "Fall", 
     semesterCode: "300", 
     PlannedCourses: ko.observableArray([]), 
     totalCredits: ko.computed(function(){ 
      var total = 0; 
      ko.utils.arrayForEach(this.PlannedCourses, function (course) { 
       total += course.MinHours(); 
      }); 

      return total; 
     }), 
    }, 
    ... 

То, что я пытаюсь сделать, это, в переменной totalCredits, я пытаюсь перебирать массив PlannedCourses и получить переменную MinHours для каждого элемента и добавить их вместе в общей переменной. Затем я возвращаю его в элемент totalCredits в массиве Semesters. Проблема, с которой я сталкиваюсь, заключается в получении переменной PlannedCourses в части ko.utils.arrayForEach. Я получаю undefined на нем, и я не уверен, почему. Я думаю, что это простая синтаксическая ошибка, но я не вижу, что случилось.

Наблюдаемый массив PlannedCourses - это динамический объект, который правильно получает список PlannedCourses. Он определяется в контексте самого себя, но я не передаю его в вычисляемую функцию totalCredits должным образом.

Надеюсь, это достаточно ясно. Спасибо за помощь!

Примечание: вся остальная часть кода работает по назначению. Единственной частью, которая не работает, является вычисляемая функция totalCredits. Я не уверен, что что-то внутри ko.utils.arrayForEach работает, поскольку я не смог зайти так далеко.

Answer 1

Вы будете нуждаться чтобы изменить способ заселить свой Semesters наблюдаемой массив использовать функцию конструктора для того, чтобы получить ссылку на нужный объем для this:

function semester(name, code) { 
    this.Name = name; 
    this.Code = code; 
    this.PlannedCourses = ko.observableArray([]); 
    this.totalCredits = ko.computed(function(){ 
     var total = 0; 
     ko.utils.arrayForEach(this.PlannedCourses(), function (course) { 
      //Note the change to "this.PlannedCourses()" above to get the underlying array 
      total += course.MinHours(); 
     }); 

     return total; 
    }, this); //now we can pass "this" as the context for the computed 
} 

Посмотрите, как мы можем теперь перейти в n для второго аргумента для ko.computed для использования в качестве контекста для this во внутренней функции.Для получения дополнительной информации см. Документы нокаута: Managing 'this'.

Вы затем создать новые экземпляры semester при заполнении вашего массива:

Semesters: ko.observableArray([ 
    new semester("Fall", "300"), 
    new semester(...) 
]); 

Этот подход также означает, что у вас есть последовательный способ создания ваших semester объектов (вычисленный определяется только один раз для одной вещи), а чем возможно включение опечаток и т. д. в любое повторение, которое вы, возможно, первоначально имели.

Answer 2

Объем (это) не то, что вы думаете.

См http://knockoutjs.com/documentation/computedObservables.html

попробуйте добавить свой контекст, как следующее:

Semesters: ko.observableArray([ 
{ 
    semesterName: "Fall", 
    semesterCode: "300", 
    PlannedCourses: ko.observableArray([]), 
    totalCredits: ko.computed(function(){ 
     var total = 0; 
     ko.utils.arrayForEach(this.PlannedCourses, function (course) { 
      total += course.MinHours(); 
     }); 

     return total; 
    }, this), // new context passed in here 
}, 
... 

Делать это проходит в контексте самого элемента массива в вашей вычисленной функции.

Edit: вам может понадобиться для доступа к семестры объекта внутри вас цикла, и добавить какой-нибудь способ для ссылки на текущий элемент:

Semesters: ko.observableArray([ 
{ 
    semesterName: "Fall", 
    semesterCode: "300", 
    PlannedCourses: ko.observableArray([]), 
    totalCredits: ko.computed(function(){ 
     var total = 0; 
     for(var i = 0, len = Semesters().length; i < len; i++) { 
      // check current array item, possibly add an id? 
      if(Semesters()[i].semesterName === "Fall" && 
       Semesters()[i].semesterCode === "300") { 

       ko.utils.arrayForEach(Semesters()[i].PlannedCourses, function (course) { 
        total += course.MinHours(); 
       }); 
       break; // done searching 
      } 
     } 

     return total; 
    }) 
}, 

Answer 3

Как и другие уже упоминали ваш this не то, что вы думаете. В вашем случае контекст должен быть передан вычисленный следующим образом:

totalCredits: ko.computed(function() { 
    // Computation goes here.. 
}, this) 

Другой подход может заключаться в сохранении правильного this в некоторой локальной переменной во время создания объекта (бывший var self = this;, а затем использовать self вместо this.) ,

Однако ko.utils.arrayForEach не работает с наблюдаемыми массивами, но работает на чистых массивах JavaScript, поэтому вы должны разворачивать наблюдаемый массив для доступа к элементам базового массива:

ko.utils.arrayForEach(this.PlannedCourses(), function(course) { 
    // ... 
}); 

// Or 

ko.utils.arrayForEach(ko.unwrap(this.PlannedCourses), function(course) { 
    // ... 
});