Check if a big object have null value

Question

I'm trying to make a function to go through an object and if is there any null values. Here is my code:

  async function validateObjectField(course) {
  let error = new Error();
  const keys = Object.keys(course);
  keys.forEach(k => {
    const obj = course[k];
    if (typeof obj === "object") {
      if (Array.isArray(obj)) {
        if (Object.size(obj) === 0) {
          error.message = `${obj} is required!`;
          return error;
        }
        for (const o of obj) {
          error = validateObjectField(o);
          if (error) return error;
        }
      } else {
        error = validateObjectField(obj);
        if (error) return error;
      }
    }
    if ((typeof obj === "string" && obj.length === 0) || obj === null) {
      error.message = `${obj} is required!`;
      return error;
    }
  });
}

I'm using console.log() to observe and see that sometimes the code goes to return statement but the remaining code is still executed. Anyone know why? I got stuck and have no idea what to do now.

In this block of codes:

if (obj.length === 0) {
    error.message = `${obj} is required!`;
    return error;
}

It did not return the error and break the function.

`return`ing from a forEach won't do anything except terminate the *current* iteration early. Use something like `for..of` instead, or `.find`. Could also use `JSON.stringify` - feels kind of weird, but would trim down the code a LOT — CertainPerformance, Oct 08 '19 at 07:17
You were already using `forEach`, but the fundamental problem is that you can't break out of a `forEach` — CertainPerformance, Oct 08 '19 at 07:24
So is there any solution? I want to return a custom error but not the false or true like in the answer — Nguyen Hoang Vu, Oct 08 '19 at 07:26
I change it, and sorry but I updated the question, can you view it again? thank you — Nguyen Hoang Vu, Oct 08 '19 at 07:30
You are still using `forEach`, but you cannot break out of a `forEach` - see the linked question. (unless you `throw` an error, which is a bad idea) — CertainPerformance, Oct 08 '19 at 07:34
Ah, sorry. I changed to for..of, but when it reach the block of codes as I asked, it did not return the error — Nguyen Hoang Vu, Oct 08 '19 at 07:36
You're still using `forEach`, which you can't break out of: `keys.forEach(k => {` Use `for..of` instead there — CertainPerformance, Oct 08 '19 at 07:37
You are *still* using `keys.forEach(k => {`, which is the problem. — CertainPerformance, Oct 08 '19 at 07:39
ah, I did not notice that. Sorry too much, thanks for your patient — Nguyen Hoang Vu, Oct 08 '19 at 07:39

Check if a big object have null value

0 Answers0