How to for loop check array in javascript?

Question

I have two array:

var listOfObjects = [{ "name": "A", "data": [{'value1':'1','value2':'2'}] },
                     { "name": "B", "data": [{'value1':'','value2':''}] }];
var markedImage = ["A","B","C"];

and this is my code:

function checkData() {
        var invalidDoc = [];

        for (var i = 0; i < listOfObjects.length; i++) {
            for (var j = 0; j < markedImage.length; j++) {
                if (listOfObjects[i].name== markedImage[i]) {
                    for (var k = 0; k < listOfObjects[i].data.length; k++) {
                        if (listOfObjects[i].data[k] == "") {
                            var invalidfile = markedImage.indexOf(listOfObjects[i].name);
                            invalidDoc.push(invalidfile);
                            break;
                        } else {
                            var valid = markedImage.indexOf(listOfObjects[i].name);
                            validDoc.push(valid);
                        }

                    }

                } else {
                    var invalidfile = markedImage.indexOf(listOfObjects[i].name);
                    invalidDoc.push(invalidfile);
                }
            }
        }
    }

What I want to do is check both array.

First check is : If item in markedImage is not exist in listOfObjects, push the index of markedImageinto invalidDoc.

Second check is : the data of listOfObjects cannot be empty, if it is empty push the index of markedImageinto invalidDoc.

For example,

Expected result for invalidDoc is [1,2]. Because data of "B" in listOfObjects is empty, "C" is not exist in listOfObjects. 1 and 2 is the index of "B" and "C" in markedImage.

a expected output array is better than words below: invalidDoc will have 1 and 2 inside. Because data of "B" in listOfObjects is empty, "C" is not exist in listOfObjects. 1 and 2 is the index of "B" and "C" in markedImage. — xianshenglu
– xianshenglu, Commented Feb 7, 2018 at 2:40
@xianshenglu he did. For example, invalidDoc will have 1 and 2 inside. — CozyAzure
– CozyAzure, Commented Feb 7, 2018 at 2:41

Kirill Simonov · Accepted Answer · 2018-02-08 04:46:31Z

1

Check this example. I used the reduce function over the markedImage array to find elements that are not present in the listOfObjects and check the data arrays.

isDataEmpty is a function which does the validation of every data array. You can change its logic according to the desired result.

var listOfObjects = [{ "name": "A", "data": [{'value1':'1','value2':'2'}]}, //not empty
                     { "name": "B", "data": [{'value1':'','value2':''}]},   //empty
                     { "name": "D", "data": []},                            //empty
                     { "name": "E", "data": [{'value1':'1','value2':'2'}]}, //not empty
                     { "name": "G"},                                        //empty
                     { "name": "H", "data": [{'value1':'','value2':''}, 
                                             {'value1':'1','value2':'2'}]}];//empty
var markedImage = ["A","B","C", "D", "E", "F", "G", "H"];

function isDataEmpty(data) {
    // You can add any validation conditions here.
    // I assume that your data is an array and that by empty
    // data you mean that array may be empty, undefined,
    // or have some objects with empty 'value1' or 'value2'
    return !data ||
           data.length === 0 || 
           data.some(function(d){return d.value1 === "" || d.value2 === ""});
};

function find(array, name) {
    for (var i in array) {
        if (array[i].name === name) return array[i];
    }
    return null;
}

var invalidDoc = markedImage.reduce(function(acc, cur, i) {
    var found = find(listOfObjects, cur);
    if (!found || isDataEmpty(found.data)) {
        acc.push(i);
    }
    return acc;
},[]);

console.log(invalidDoc);

edited Feb 8, 2018 at 4:46

answered Feb 7, 2018 at 3:11

Kirill Simonov

8,4913 gold badges23 silver badges47 bronze badges

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

12 Comments

Xion Over a year ago

for some reason.. i can't use the => arrow function.

Kirill Simonov Over a year ago

Then you can replace it with a regular function, eg (data) => { becomes function(data) {

Kirill Simonov Over a year ago

@Xion Well, then you can change them to var, all logic stays the same. Check my updated code.

Xion Over a year ago

The find methos is not supported in IE...and what if my listOfObjects is like this [{ "name": "A", "data": [{'value1':'1','value2':'2'},{'value1':'','value2':''}] }]; .

Kirill Simonov Over a year ago

@Xion should this object be considered emty?

|

Collectives™ on Stack Overflow

How to for loop check array in javascript?

1 Answer 1

12 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

12 Comments

Your Answer

Sign up or log in

Post as a guest

Related