Как оптимизировать функцию фильтрации массива по вхождению?

[dc65k]

Всем привет. Исходный массив:

let arr = [
    {
        id: 1,
        firstName: 'John',
        lastName: 'Smith'
    },
    {
        id: 2,
        firstName: 'Brady',
        lastName: 'Dennis'
    },
    {
        id: 3,
        firstName: 'Sandoval',
        lastName: 'Pratt'
    },
    {
        id: 4,
        firstName: 'Davis',
        lastName: 'Andrews'
    }
]

Свойства, по которым нужно отфильтровать:

const filterBy = ['firstName', 'lastName'];

Вхождения (условия) по которым фильтруем:

const filterVal = ['joh', 'den'];

Результат:

/*
output
[
  { id: 1, firstName: 'John', lastName: 'Smith' },
  { id: 2, firstName: 'Brady', lastName: 'Dennis' }
]

 */

Решение:

function solution(a) {
    return a.filter(item => {
        return filterBy.some(prop => {
            return filterVal.some(i => {
                return item[prop].toLowerCase().includes(i)
            })
        })
    })
}

console.log(solution(arr));

Решение в плане сложности алгоритма не оптимальное. Подскажите, пожалуйста, как его оптимизировать?

[Vlasenko Fedor]

const arrayFilter = (arr, rules) => arr.filter(el => Object.keys(rules).some(key => rules[key](el[key])));
const rules = {
    firstName: v => v.toLowerCase().includes('joh'),
    lastName: v => v.toLowerCase().includes('den'),
};

console.log(arrayFilter(arr, rules));

Вариант

const rules = {
    firstName: v => /joh/i.test(v),
    lastName: v => /den/i.test(v),
};

[Aetae]

Вполне себе оптимальное. Перебирается ровно столько сколько нужно.
Разве что вынести значение в переменную, чтоб лишний раз toLowerCase не делать:

function solution(a) {
    return a.filter(item => filterBy.some(prop => {
       const value = item[prop].toLowerCase();
       return filterVal.some(i => value.includes(i));
    }));
}

console.log(solution(arr));