array.push()重复相同的值

时间:2018-04-07 05:18:43

标签: javascript jquery arrays object

这是我试图创建的一个对象来管理列表,但是某些地方有些错误,每当我第二次或第三次使用list.addItem();时,list.list;中的所有值(数组)更改为最终添加的值(数组)。

var list = {
  list : [],
  eligible: function(item){
    var status = true;
    var dw = (item.w*item.h*item.l)/169;
    if(dw > item.m){
      status = true;
    }else{
      status = false;
    }
    return status;
  },
  addItem : function(item){
    if(this.eligible(item)){
      this.list.push(item);
      console.log(this.list);
      this.refresh();
      alertify.success('Item Added to List');
    }else{
      alertify.warning('Item not eligible for Freight Calculation');
    }
  },
  removeItem : function(item){
    if(this.inList(item)){
      var itemIndex = this.list.indexOf(item);
      if(itemIndex > -1){
        this.list.splice(itemIndex,1);
        alertify.success('Item Removed from List');
        this.refresh();
      }
    }else{
      alertify.error('Item NOT in List');
    }
  },
  inList: function(item){
    var bool = false;
    if (this.list.filter(function(e) { return e.id === item.id; }).length > 0) {
      bool = true;
    }else{
      bool = false;
    }
    return bool;
  },
  findItem: function (id) {
    for (var i = 0; i < this.list.length; i++) {
      if (this.list[i].id === id) {
        return this.list[i];
      }
    }
  },
  refresh: function(){
    if(this.list.length > 0){
      $('.items .table tbody').empty();
      var itemNo = 0;
      $.each( this.list, function( key, item ) {
        $('.items .table tbody').append('</tr><tr><td>' + ++itemNo + '</td><td>' + item.qty + '</td><td>' + item.m + '</td><td>' + item.h + '</td><td>' + item.w + '</td><td>' + item.l + '</td><td><button data-item=\"' + item.id + '\" class=\"btn btn-remove btn-block\"><i class=\"far fa-minus-square\"></i></button></td><td>Price</td></tr>')
      });
    }else{
      $('.items .table tbody').html('<tr><td colspan=\"8\" style=\"text-align:center;\"> No Items Added.</td></tr>')
    }
  }
};

我找不到什么错,也许是因为我整天都在尝试这个。顺便说一下,我是编程新手。

更新:这就是我调用addItem的方式:

    var id=0; //just for reference     
    $('.btn-add-item').click(function(){
        var item = [];
        item.id = ++id;
        item.qty = parseInt($('input[name=qty]').val());
        item.m = parseFloat($('input[name=weight]').val()).toFixed(3);
        item.w = parseFloat($('input[name=width]').val()).toFixed(2);
        item.h = parseFloat($('input[name=height]').val()).toFixed(2);
        item.l = parseFloat($('input[name=length]').val()).toFixed(2);
        item.country = $('#countrySelect').val();
        list.addItem(item);
    });

1 个答案:

答案 0 :(得分:1)

在您的点击处理程序中,您将每个输入元素的值分配给变量item,就像item是对象一样。不幸的是,您已将item初始化为数组。您应该将item初始化为对象。然后,您的list数组将包含list个对象。

更改

var item = [];

var item = {};

由于您不熟悉编程,并且使用此代码以奇怪的方式排序Javascript,请让我解释一下为什么没有抛出任何错误让您知道这一点。

在JavaScript中,数组实际上是对象。因此,赋值(item.blah)实际上将该属性作为属性放置在项Array对象上,但不知道您的意图是将值添加到Array元素列表中。 Javascript执行它认为是您的意图。