CODEWITHSUNDEEP
javascriptjquery
Gurigraphics
Gurigraphics

Reputation: 331

splice row from array by value

I want splice the line with value = 3

[3,"John", 90909090]

data.json

{
"headers":[[
{"text":"Code","class":"Code"},
{"text":"Code","class":"Code"}
]],
"rows":[
[0,"Peter", 51123123],
[3,"John", 90909090],
[5,"Mary",51123123]
],
"config":[[0,0,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0]],
"other":[[13,0]]
}

I try this:

var size = data.rows.length; // number of rows

var del = 3 // Value of ID to be deleted          

for (i = 0; i < size; i++) {  

var id = data.rows[i][0];                  

    if(del==id){  // if del = id -> splice                                         

       data.rows.splice(i,1);

    }

}

Results:

Only splice or only loop this code works.

But, with both show this error:

Uncaught TypeError: Cannot read property '0' of undefined(…)

It occurs in "data.rows[i][0]"

Upvotes: 5

Views: 1997

Answers (4)

Kamuran S&#246;necek
Kamuran S&#246;necek

Reputation: 3333

You can use lodash for filter your objects or arrays. Look at the filter method for your case: 

var myObject = {
"headers":[[
{"text":"Code","class":"Code"},
{"text":"Code","class":"Code"}
]],
"rows":[
[0,"Peter", 51123123],
[3,"John", 90909090],
[5,"Mary",51123123]
],
"config":[[0,0,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0]],
"other":[[13,0]]
};

//filter by lodash
myObject.rows =  _.filter(myObject.rows,function(row){
  return row[0] !== 3;
});

Upvotes: 0

Yosvel Quintero
Yosvel Quintero

Reputation: 19070

You can iterate with Array#forEach():

var data = {"headers": [[{"text": "Code","class": "Code"}, {"text": "Code","class": "Code"}]],"rows": [[0, "Peter", 51123123],[3, "John", 90909090],[5, "Mary", 51123123]],"config": [[0, 0, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]],"other": [[13, 0]]},
    del = 3; // Value of ID to be deleted

data.rows.forEach(function(item, index) {
  item[0] === del && data.rows.splice(index, 1);
});

console.log(data.rows);
.as-console-wrapper { max-height: 100% !important; top: 0; }

ES6:

data.rows.forEach((item, index) => item[0] === del && data.rows.splice(index, 1));

Upvotes: 1

hello_world
hello_world

Reputation: 432

instead of using a for loop, id use the array filter function:

data.rows = data.rows.filter(function(row){
    return row[0] !== del;
});

Upvotes: 3

Nina Scholz
Nina Scholz

Reputation: 386654

Just add a break to the condition, because the next element, is the one you have spliced, which is not anymore in the array.

if (del == id) {  // if del = id -> splice
   data.rows.splice(i, 1);
   break; // no more to search
}

Upvotes: 2

Related Questions