Ko.Computed() не обновляется с помощью наблюдаемого массива

У меня есть следующий код:

// First we define our gift class, which has 2 properties:
// a Title and a Price.
// We use knockout js validation to ensure that the values input are suitable/ 
function Gift(item)
{
    var self = this;
    self.Title = ko.observable(item.Title);

    // attach some validation to the Title property courtesy of knockout js validation
    self.Title.extend({
        required: true,
        minLength: 3,
        pattern: {
            message: 'At least ',
            params: '^[a-zA-Z]+\s?[a-zA-Z]*'
        }
    });
    self.Price = ko.observable(item.Price);
    self.Price.extend({required:true,number:true,min:0.1,max:1000});
};


var viewModelForTemplated =
{
    gifts: ko.observableArray(),        // gifts will be an array of Gift classes

    addGift: function ()
    {
        this.gifts.push(new Gift({ Title: "", Price: "" }));
    },
    removeGift: function (gift)
    {
        this.gifts.remove(gift);
    },

    totalCost: ko.computed(function () {
        if (typeof gifts == 'undefined')
            return 0;

        var total = 0;

        for (var i = 0; i < gifts().length; i++)
        {
            total += parseFloat(gifts()[i].Price());
        };
        return total;
    })
}


$(document).ready(function ()
{
    // load in the data from our MVC controller 
    $.getJSON("gift/getdata", function (allGifts)
    {
        var mappedgifts = $.map(allGifts, function (gift)
        {
            return new Gift(gift);
        });

        viewModelForTemplated.gifts(mappedgifts);
    });

    ko.applyBindings(viewModelForTemplated, $('#templated')[0]);
}

а затем (выше script)

<div id="templated">

<table >
    <tbody data-bind="template: { name: 'giftRowTemplate', foreach: gifts }"></tbody>
</table>

<script type="text/html" id="giftRowTemplate">
    <tr>
        <td>Gift name: <input data-bind="value: Title"/></td>
        <td>Price: \$ <input data-bind="value: Price"/></td>           
        <td><a href="#" data-bind="click: function() { viewModelForTemplated.removeGift($data) }">Delete</a></td>
    </tr>
</script>

<p>Total Cost <span data-bind="text: totalCost"></span> </p>    

<button data-bind="click: addGift">Add Gift</button> 

<button data-bind="click: save">Save</button>

</div>

Метод totalCost работает только один раз, когда массив подарков пуст, и я могу нажимать или удалять элементы на наблюдаемом массиве() без проблем, но ничего не срабатывает.

Как мне получить диапазон, ссылающийся на totalCost для обновления? Держу пари, что-то просто:)

Спасибо за вашу помощь.

Ответ 1

Вам нужно развернуть видимое:

totalCost: ko.computed(function () {
    //also, you forgot typeof below
    if (typeof gifts == 'undefined')
       return 0;

    var total = 0;  //here \/
    for (var i=0; i < gifts().length; i++)
    {                //and here  \/
        total += parseFloat(gifts()[i].Price());
    };
    return total;
})

Причина, по которой он не обновляется, заключается в том, что

gifts.length

всегда оценивает 0 и никогда не входит в цикл. И даже если бы это произошло,

gifts[i].Price()

не будет работать по той же причине; вам нужно развернуть наблюдаемое.

Обратите внимание, что причина, по которой длина оценивается до нуля, когда вы ее не разворачиваете, заключается в том, что вы получаете длину фактической наблюдаемой функции массива. Все наблюдаемые в нокаут реализуются как регулярные функции; когда вы его не разворачиваете, вы нажимаете фактическую функцию, а не базовый массив.

Изменить

Кроме того, вам нужно ссылаться на подарки с помощью this.gifts, так как это свойство объекта. Вот почему это не работает; подарки всегда undefined.

Тем не менее, вам также нужно сделать еще одну работу, чтобы вычислить ko для работы из литерала объекта. Читайте здесь для получения дополнительной информации:

http://dpruna.blogspot.com/2013/09/how-to-use-kocomputed-in-javascript.html

Вот как бы я сделал свою модель просмотра:

function Vm{
    this.gifts = ko.observableArray();        // gifts will be an array of Gift classes

    this.addGift = function () {
        this.gifts.push(new Gift({ Title: "", Price: "" }));
    };

    this.removeGift = function (gift)
    {
        this.gifts.remove(gift);
    };

    this.totalCost = ko.computed(function () {
        var total = 0;

        for (var i = 0; i < this.gifts().length; i++)
        {
            total += parseFloat(this.gifts()[i].Price());
        };
        return total;
    }, this);
}

var viewModelForTemplated = new Vm();

Ответ 2

Вам нужно развернуть observableArray с помощью () при применении индексатора к нему. Обновите totalCost следующим образом:

totalCost: ko.computed(function () {
    if (this.gifts == 'undefined')
       return 0;

    var total = 0;
    for (var i=0; i<this.gifts().length;i++)
    {
        total += parseFloat(this.gifts()[i].Price());
    };
    return total;
})