push()无法在javascript中运行

时间:2013-09-04 05:47:34

标签: javascript local-storage

问题:控制台中的Uncaught TypeError: Object #<Object> has no method 'push'

http://jsfiddle.net

中的代码

7 个答案:

答案 0 :(得分:3)

更改存储项目ID(购物车)并重试,看起来像之前存储的项目“购物车”ID不是json数组,因为@ dc5在评论部分建议

UPD:试试这个http://jsfiddle.net/vJkBQ/4/

HTML

<div id='cart'></div>
<input type="button" id="add" value="Add To Cart item 1" />
<input type="button" id="add2" value="Add To Cart item 2" />

Javascript

//TODO: move from globals
var storageName = 'myCART';

$(document).ready(function () {
    var item = {
        DepartmentID :333,
        CategoryID:117,
        BrandID:19,
        BrandImage:"&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;",
        BrandName:"General",
        ID:711
    };
    var item2 = {
        DepartmentID :123,
        CategoryID:321,
        BrandID:18,
        BrandImage:"&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;",
        BrandName:"Common",
        ID:712
    };

    localStorage.clear(storageName);
    $('#add').click(function(){
       addToCart(item);
    });

    $('#add2').click(function(){
       addToCart(item2);
    });
});

function addToCart(item){
    //by @slebetman   
    var items = JSON.parse(localStorage.getItem(storageName));
    if (! (items instanceof Array) ) {
        items = [];
    }

    var itemIndex = getItemIndexById(items, item.ID);    
    if(typeof(itemIndex) === 'number'){
        items[itemIndex].quantity++;
    }
    else{
        item.quantity = 1;
        items.push(item);
    }

    localStorage.setItem(storageName, JSON.stringify(items));
    console.log(localStorage.getItem(storageName));
}

//find search item index
function getItemIndexById(items, id){
    for(var i = 0; i < items.length; i++){
        if(items[i].ID == id){
            return i;
        }
    }

    return false;
}

答案 1 :(得分:3)

表达式:

JSON.parse(localStorage.getItem(storageName))

很可能不会返回数组。在这种情况下声明:

var oldStorage = JSON.parse(localStorage.getItem(storageName)) || [];

不足。

你应该做的事情是这样的:

var oldStorage = JSON.parse(localStorage.getItem(storageName));
if (! (oldStorage instanceof Array) ) {
    oldStorage = [];
}

这是一种检测数组的简单方法。还有更多高级方法,例如检查.length等存在,检测数组和数组对象,以及在Array对象被覆盖或跨iframe工作的情况下检测数组。


补充答案:

你已经改变了很多代码,但问题仍然存在。这一行:

if (items != null) {

不足以检查项目是否为数组。你应该这样做:

if ( items instanceof Array ) {

确保它确实是一个数组。

另外,在else块中:

}else{
    console.log('Cart is empty, preparing new cart array');
    items.push(item);

console.log消息表明正在准备一个新阵列。然而,这是因为代码没有初始化新数组,而是使用items变量,就好像它是一个数组一样。你应该这样做:

}else{
    console.log('Cart is empty, preparing new cart array');
    items = [];
    items.push(item);

警告:

然而,毕竟,请听取评论者的意见。如果你从头开始编写这些内容而不是做我建议的事情就可以解决你所有的问题。但是如果你正在做的是修改别人的代码,那么Cart很可能以不同于你预期的格式存储。{/ p>

请在致电console.log(localStorage['Cart'])之前JSON.parse并在此处发布您的结果。问题在于浏览器上的您的本地存储空间,通常无法在其他人的计算机上重现。

答案 2 :(得分:1)

JSON.parse(localStorage.getItem(storageName)) 

总是返回一个数组?如果是这样,问题是并非所有浏览器都支持push方法。如果缺少,您可以添加使用此代码段添加:

if(!Array.prototype.push){
    Array.prototype.push=function(x){
        this[this.length]=x;
        return true
    }
};

这段代码只是一个开始,你绝对可以改进它

答案 3 :(得分:1)

您可以使用Array.prototype.push()在对象上调用call()

JSFiddle

的JavaScript

function appendToStorage(storageName, data){
    var oldStorage = JSON.parse(localStorage.getItem(storageName)) || [];
    Array.prototype.push.call(oldStorage, data);
    localStorage.setItem(storageName,JSON.stringify(oldStorage));
}

答案 4 :(得分:1)

JSON.parse会返回一个对象或null。所以在这里你不知道你是否有一个对象或一个数组。

JSON.parse(localStorage.getItem(storageName)) || [];

您可以改用:

function appendToStorage(storageName, data){
    var oldStorage = JSON.parse(localStorage.getItem(storageName)) || {};
    if (oldStorage != null){
        $(oldStorage).extend(data);
        localStorage.setItem(storageName,JSON.stringify(oldStorage));
    } 
}

$ .extend()会将您的项目添加到另一个JSON对象。信息here

答案 5 :(得分:0)

您的'item'不是数组,也不能使用push方法。

答案 6 :(得分:0)

对象没有'push'方法,你可以这样做!

function appendToStorage( storageName, data ) {
    var oldStorage = JSON.parse( localStorage.getItem( storageName ) ) || {},
        push = Array.prototype.push;
    push.call( oldStorage, data );
    // oldStorage.data = data;
    localStorage.setItem(storageName,JSON.stringify(oldStorage));
}