0

Всем привет! Я изучаю js и столкнулся с задачей, которая никак не укладывается в голове. Помогите понять, что я делаю не так? Заранее благодарю)

Условие:

Создать функцию которая принимает массив пользователей, поле на которое хочу проверить и значение на которое хочу проверять. Проверять что все аргументы переданы. Возвращать новый массив с пользователями соответствующие указанным параметрам.

Мой код:

users = [
  {name: 'Pavel', gender: 'male', age: 29},
  {name: 'Anna', gender: 'female', age: 18},
  {name: 'Fiona', gender: 'female', age: 19},
  {name: 'Julia', gender: 'female', age: 28},
  {name: 'Alex', gender: 'male', age: 23},
  {name: 'Sonya', gender: 'female', age: 17},
  {name: 'Frank', gender: 'male', age: 44},
  {name: 'John', gender: 'male', age: 49},
  {name: 'Greta', gender: 'female', age: 57},
];

function funcGetUsers(arrayUsers, key, keyValue) {
  let endArray = arrayUsers.slice();

  for (let user in endArray) {
    if (endArray[user][key] != keyValue) {
      endArray.splice(user, 1);
    } else {
      continue;
    }
  }
  
  return endArray;
}

let newUser = funcGetUsers(users, 'gender', 'female');
console.log(newUser);

2
  • а откуда известно что что-то не так? 21 апр 2021 в 16:55
  • 4: {name: "John", gender: "male", age: 49} 21 апр 2021 в 16:57

2 ответа 2

5

Можно сделать так попроще

function funcGetUsers(arrayUsers, key, keyValue) {
    return arrayUsers.filter(item => item[key] === keyValue);
}

let users = [
{name: 'Pavel', gender: 'male', age: 29},
{name: 'Anna', gender: 'female', age: 18},
{name: 'Fiona', gender: 'female', age: 19},
{name: 'Julia', gender: 'female', age: 28},
{name: 'Alex', gender: 'male', age: 23},
{name: 'Sonya', gender: 'female', age: 17},
{name: 'Frank', gender: 'male', age: 44},
{name: 'John', gender: 'male', age: 49},
{name: 'Greta', gender: 'female', age: 57}
];

let newUser = funcGetUsers(users, 'gender', 'female');
console.log(newUser);

PS. А ваша проблема вы мутируете массив при этом перебираете его, что не приветствуется.

const users = [
    {name: 'Pavel', gender: 'male', age: 29},
    {name: 'Anna', gender: 'female', age: 18},
    {name: 'Fiona', gender: 'female', age: 19},
    {name: 'Julia', gender: 'female', age: 28},
    {name: 'Alex', gender: 'male', age: 23},
    {name: 'Sonya', gender: 'female', age: 17},
    {name: 'Frank', gender: 'male', age: 44},
    {name: 'John', gender: 'male', age: 49},
    {name: 'Greta', gender: 'female', age: 57}
];

function funcGetUsers(arrayUsers, key, keyValue){
    let result = [];
    for (let user of arrayUsers) {
        if (user[key] === keyValue) {
            result.push(user);
        }
    }
    return result;
};

const newUser = funcGetUsers(users, 'gender', 'female');
console.log(newUser);

1
  • @Pr13rak я имел ввиду второй вариантю он не короткий 21 апр 2021 в 17:39
2

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

const users = [
    {name: 'Pavel', gender: 'male', age: 29},
    {name: 'Anna', gender: 'female', age: 18},
    {name: 'Fiona', gender: 'female', age: 19},
    {name: 'Julia', gender: 'female', age: 28},
    {name: 'Alex', gender: 'male', age: 23},
    {name: 'Sonya', gender: 'female', age: 17},
    {name: 'Frank', gender: 'male', age: 44},
    {name: 'John', gender: 'male', age: 49},
    {name: 'Greta', gender: 'female', age: 57}
];

const funcGetUsers = (arrayUsers, key, keyValue) => {
    let endArray = [];
    for (let user in arrayUsers) {
        if (arrayUsers[user][key] === keyValue) {
            endArray.push(arrayUsers[user]);
        }
    }
    return endArray;
};

const newUser = funcGetUsers(users, 'gender', 'female');
console.log(newUser);

4
  • А возможно ли реализовать это не используя стрелочные функции, зная только самые базовые, просто это урок определённого этапа, где я знаю только простые функции и базовые методы массивов, slice, splice, shift, pop?
    – Pr13rak
    21 апр 2021 в 16:54
  • да возможно это 21 апр 2021 в 16:59
  • А не подскажите как?
    – Pr13rak
    21 апр 2021 в 17:01
  • обновил ответ посмотрите выше 21 апр 2021 в 17:03

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.