Removing an empty array from an object with JavaScript

Question

I have this JSON object myFilters:

{"filters":
      {"role":"","jobs":[]}
}

I can correctly remove the empty object from it with this function clean (myFilters):

function clean(obj) {
  for (var propName in obj) {
    if (
      obj[propName] === null ||
      obj[propName] === undefined ||
      obj[propName] === ""
    ) {
      delete obj[propName];
    }
  }

So now, my myFilters object becomes:

{"filters":
    { "jobs":[] }
}

How can I now remove the empty array and the key from my JSON object?

svltmccc · Accepted Answer · 2019-12-17 12:49:20Z

1

You should add one more condition like

function clean(obj) {
  for (var propName in obj) {
    if (
      obj[propName] === null ||
      obj[propName] === undefined ||
      obj[propName] === "" ||
      Array.isArray(obj[propName]) && obj[propName].length === 0
    ) {
      delete obj[propName];
    }
  }
}

answered Dec 17, 2019 at 12:49

svltmccc

1,38612 silver badges25 bronze badges

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

Comments

Community · Accepted Answer · 2020-06-20 09:12:55Z

1

You should check the type of property before check its value by the typeof

The jobs property is an object and you can check its value by its length. it is empty if its length equals to 0.

function clean(obj) {

    for (var propName in obj) {

        if (typeof (obj[propName]) == 'object') {

            if (obj[propName].length == 0) {
                delete obj[propName];
            }

        } else {

            if (
                obj[propName] === null ||
                obj[propName] === undefined ||
                obj[propName] === ""
            ) {
                delete obj[propName];
            }

        }
        
    }

}

edited Jun 20, 2020 at 9:12

CommunityBot

11 silver badge

answered Dec 17, 2019 at 13:06

Hamid Javadi

2112 silver badges7 bronze badges

Comments

norbitrial · Accepted Answer · 2019-12-17 13:52:56Z

I like Saveli Tomac's solution, so I upvoted that. Let me show you an additional shortening on the original solution also.

As it's been stated that you need to check 2 more things if you are looking for an empty array. So what about checking null, undefined and '' values easier?

if (!undefined) { console.log('undefined needs to be deleted') };
if (!null) { console.log('null needs to be deleted') };
if (!'') { console.log(`'' needs to be deleted`) };

Checking Array.length if it has 0 value can be also shorter just like the following:

const array1 = [];
const array2 = [1,2,3];
if (!array1.length) { console.log('array1 has 0 length') };
if (!array2.length) { console.log('array2 has 0 length') };

So based on those code snippets you can have an additional shortening just like the following:

// extended with other types for the demo
let myObject = { "filters": { "role": "", "jobs": [], "nullValue": null, "undefinedIsHere": undefined, "arrayWithValue": [1,2,3], "stringValue": "hello", "numberishere": 123 } };

const clean = (obj) => {
  for (let propName in obj) {
    if (
      !obj[propName] ||
      Array.isArray(obj[propName]) && !obj[propName].length
    ) { delete obj[propName] };
  }
}

clean(myObject.filters);
console.log(myObject);

Or with a 1️⃣ liner:

// extended with other types for the demo
let myObject = { "filters": { "role": "", "jobs": [], "nullValue": null, "undefinedIsHere": undefined, "arrayWithValue": [1,2,3], "stringValue": "hello", "numberishere": 123 } };

const clean = (obj) => {
  Object.keys(obj).forEach(propName => (!obj[propName] || Array.isArray(obj[propName]) && !obj[propName].length) && delete obj[propName]);
}

clean(myObject.filters);
console.log(myObject);

Read further here:

I hope this helps!

Rohìt Jíndal · Accepted Answer · 2019-12-18 06:03:36Z

1

Try this :

var filterObj = {
	"filters": {
		"role": "",
		"jobs": []
	}
};

for (var i in filterObj) {
  for (var j in filterObj[i]) {
  	if ((filterObj[i][j] === null) ||
        (filterObj[i][j] === undefined) ||
        (filterObj[i][j].length === 0)) {
        delete filterObj[i][j];
    }
  }
}

console.log(filterObj);

answered Dec 18, 2019 at 6:03

Rohìt Jíndal

27.4k16 gold badges80 silver badges133 bronze badges

Comments

Umair · Accepted Answer · 2019-12-17 13:16:24Z

0

Saveli's answer should work fine. Here's an alternative approach you can use to achieve the same result.

const object = {
  "filters": {
    "role": "",
    "jobs": [],
    "foo": undefined,
    "baz": null,
    "bar": {},
    "moreJobs": ['1', '2']
  }
}

const result = {
  filters: Object.keys(object.filters).reduce((acc, key) => {
    if (
      object.filters[key] !== null &&
      object.filters[key] !== undefined &&
      object.filters[key] !== '' &&
      typeof object.filters[key] === 'object' && Object.keys(object.filters[key]).length > 0
    ) {
      acc[key] = object.filters[key];
    }

    return acc;
  }, {})
};

console.log(result);

answered Dec 17, 2019 at 13:16

Umair

6,2967 gold badges31 silver badges49 bronze badges

Comments

ylmz · Accepted Answer · 2019-12-17 13:20:24Z

0

It should be like this:

function clean(obj) {
  for (var propName in obj) {
    if (obj.hasOwnProperty(propName) && 
      obj[propName] === null ||
      obj[propName] === undefined ||
      obj[propName] === "" || 
      (Array.isArray(obj[propName]) && obj[propName].length <= 0)
    ) {
      delete obj[propName];
    }
  }
}

edited Dec 17, 2019 at 13:20

answered Dec 17, 2019 at 12:52

ylmz

463 bronze badges

Collectives™ on Stack Overflow

Removing an empty array from an object with JavaScript

6 Answers 6

Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related