I am trying to improve bits of code on different repos I've worked on and noticed this block

Object.values(data).forEach((value) => {
      for (let i = 0; i < todosList?.length; i  ) {
        if (
          value === todosList[i].title ||
          value === todosList[i].description
        ) {
          setData({
            ...data,
            error: "A todo exists with that title or description",
          });
          match = true;
          return;
        }
      }
    });
    if (match) {
      return;
    }

It's running a forEach and a for loop but is there a way to cut out either one so it becomes more efficient? This should basically be checking the todosList object and see if there are any matching titles or descriptions and if there are, it will let the user know it has been used already.