Сортировка, в чем ошибка?

Question

[Name Surname]

добрый день, сортировка примерно такого листа

const arr = [
    { name: "name1", name1: "name2" },
    { name: "Egor", name1: "Fill" },
    { name: "Egorka", name1: "beter" },
    { name: "fullName", name1: "Man" }
  ];


  const filterList = arr.filter(item =>
    item.forEach(element => element.indexOf("Egor") > -1)
  );

Мне нужно по каждому элементу массива пройтись, и проверить если там искомая подстрока, затем вернуть все в filterList который я буду в дальнейшем мапить . У меня чет не получается, моя версия сверху. Ребят, наверное я чет не так делаю, можете мне помочь пожалуйста.

Answer 1

[Роман]

const arr = [
    { name: "name1", name1: "name2" },
    { name: "Egor", name1: "Fill" },
    { name: "Egorka", name1: "beter" },
    { name: "fullName", name1: "Man" }
];


const filterList = arr.filter(items =>
    Object.values(items).some(element => element.indexOf("Egor") > -1)
);

console.log(filterList);

Сори. Чуть чуть обманул, теперь все верно)))

на выходе

[ { name: 'Egor', name1: 'Fill' }, { name: 'Egorka', name1: 'beter' } ]

Comments

[Name Surname]

к сожалению ошибка(

[Роман]

Name Surname, ошибка у вас. Без обид, но эта ошибка говорит лишь о том, что поле name или name1 у вас не строка.

Чтобы убедится в этом сделайте так:

const filterList = arr.filter(items => {
    console.log(items)
    return Object.values(items).some(element => element.indexOf("Egor") > -1)
});

а еще лучше так:

const filterList = arr.filter(items => 
    Object.values(items).some(element => {
         console.log(typeof element);
         return element.indexOf("Egor") > -1
    })
);

[Роман]

исправить ситуацию можно так:

const filterList = arr.filter(items => 
    Object.values(items).some(element => {
         if( typeof element !== "string" ) return false;
         return element.indexOf("Egor") > -1
    })
);

Но лучше все таки разобраться, почему у вас одно из полей не string.

[Name Surname]

Роман, Было бы наверное глупо на вас обижаться, ведь вы мне помогаете, за что я вам искренне благодарен, да , вы правы, приходит числовое значение. Теперь я еще более запутался в том что нужно сделать

[Роман]

Name Surname, не стоит обижаться, я сегодня возможно слегка груб, признаю мой косяк)))
попробуйте так:

const filterList = arr.filter(items => 
    Object.values(items).some(element => {
         if( typeof element !== "string" ) return false;
         return element.indexOf("Egor") > -1
    })
);

[Роман]

или даже так:

const filterList = arr.filter(items => Object.values(items).join(",").indexOf("Egor") > -1);

[Name Surname]

Роман, спасибо тебе большое) работает так как мне нужно

[Роман]

Name Surname, предложенный мной вариант лишь заглушка, вам в любом случае надо разобраться почему проскакивают поля с цифрами и исправить это. Данные, которые вы не контролируете должны подвергаться тщательной проверке перед использованием, иначе вы рискуете тем, что ваше приложение будет уязвимо для различного рода атак, связанных с подсовыванием невалидных данных.

[Name Surname]

Роман, вы сделали больше чем думаете) ваш код натолкнул меня на правильное решение, поэтому я убрал костыль, и теперь все как нужно) поэтому правда, искренне вам спасибо

Answer 2

[Антон Антон]

const filterList = arr.filter(item =>
    Object.values(item).some(element => element.indexOf("Egor") > -1)
  );

Comments

[Роман]

Тут не проканает Array.forEach(...), нужен Array.some(...)

// Object.values(item).forEach(element => element.indexOf("Egor") > -1)
Object.values(items).some(element => element.indexOf("Egor") > -1)

[Name Surname]

следующее выбивает на ваш код

[Антон Антон]

Роман, согласен, поправил