Javascript Remove from array if empty

Question

I have an array looking like this:

var testArray = [
    {"cid": "1234567"},
    {"cid": "892345"},
    {"cid": ""},
    {"cid": "8267783"},
    {},
    {"cid": "096873"},
];

How do I remove, either before a for loop or when looping, where cid = "" and where is empty {}

I tried this:

for(var i = 0; testArray.length; i++){

    if(testArray.cid && testArray.cid != ""){

    }

}

This didn't work :-/ Got this error: Cannot read property "cid" from undefined

Hope this makes sense and thanks in advance :-)

You just missing the index of item if( testArray[i].cid ) — tam.dangc
– tam.dangc, Commented Mar 5, 2019 at 7:00

holydragon · Accepted Answer · 2019-03-05 06:42:44Z

4

Use filter() to filter out undesired data.

var testArray = [
    {"cid": "1234567"},
    {"cid": "892345"},
    {"cid": ""},
    {"cid": "8267783"},
    {},
    {"cid": "096873"},
];
console.log(testArray.filter(arr => arr.cid))

answered Mar 5, 2019 at 6:42

holydragon

6,7687 gold badges48 silver badges76 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Maheer Ali Over a year ago

You code will remove the objects with cid will be true,0,undefined,null. And OP only asked for ''

Mansa Over a year ago

Super, but just in case... Is there a way I can accomplish this without the use of filter?

Maheer Ali Over a year ago

@Mansa see my answer I also showed the way with filter() using simple loops

Basit · Accepted Answer · 2019-03-05 06:51:55Z

3

If you need to remove ALL empty values ("", null, undefined and 0):

arr = arr.filter(function(e){return e});

To remove empty values and Line breaks:

arr = arr.filter(function(e){ return e.replace(/(\r\n|\n|\r)/gm,"")});

Example

arr = ["hello","",null,undefined,1,100," "]  
arr.filter(function(e){return e});

return

["hello", 1, 100, " "]

answered Mar 5, 2019 at 6:51

Basit

8901 gold badge12 silver badges34 bronze badges

Comments

Maheer Ali · Accepted Answer · 2019-03-05 06:52:54Z

2

When you will use splice() during the loop. You need to decrease i by 1 A better way of doing this using filter(). Below I showed both methods.
And for checking empty object {} you should compare Object.key(obj).length with 0

var testArray = [
    {"cid": "1234567"},
    {"cid": "892345"},
    {"cid": ""},
    {"cid": "8267783"},
    {},
    {"cid": "096873"},
];
//doesnot mutates the original array.
let result = testArray.filter(x => Object.keys(x).length !== 0 && x.cid !== '');

//original array will be change after this loop
for(let i = 0;i<testArray.length;i++){
  if(Object.keys(testArray[i]).length ===0 || testArray[i].cid === ''){
    testArray.splice(i,1);
    i--;
  }
}
console.log(testArray)
console.log(result);

edited Mar 5, 2019 at 6:52

answered Mar 5, 2019 at 6:47

Maheer Ali

36.5k8 gold badges53 silver badges83 bronze badges

Comments

A.T. · Accepted Answer · 2019-03-05 06:51:31Z

2

var testArray = [
    {"cid": "1234567"},
    {"cid": "892345"},
    {"cid": ""},
    {"cid": "8267783"},
    {},
    {"cid": "096873"},
];

testArray = testArray.filter(item=> Object.keys(item).length && item["cid"]);
console.log(testArray)

answered Mar 5, 2019 at 6:51

A.T.

26.7k8 gold badges50 silver badges68 bronze badges

Comments

Bibberty · Accepted Answer · 2019-03-05 06:56:53Z

0

Just added a post filter function for completeness.Vote for HolyDragon tho.

var testArray = [
    {"cid": "1234567"},
    {"cid": "892345"},
    {"cid": ""},
    {"cid": "8267783"},
    {},
    {"cid": "096873"},
];

testArray.filter(i => i.cid)
         .forEach(elem => {
  // do my code post filter
  console.log(elem.cid);
});

answered Mar 5, 2019 at 6:56

Bibberty

4,7762 gold badges10 silver badges23 bronze badges

Collectives™ on Stack Overflow

Javascript Remove from array if empty

5 Answers 5

3 Comments

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

3 Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related