Работа с объектом и массивом?

Question

[Дмитрий Ярощук]

По заданию мне нужно написать функцию hireNewEmployee, которая будет принимать в параметры объект employee и возвращать 'You're Hired! Congrats!' или 'No hired: sorry we cannot hire you. Here is why: ', reason). Где у reason будет массив свойств по которым не подошел работник.
К примеру:
ограничение в возрасте – не моложе 25 лет на эту должность
образование: высшее
прошлый опыт: минимум 1 год.
Примечание: создавать объект employee через defineProperty с указанием дескрипторов.
Пробовал решить, код не работает, в консоль выдает ошибку.

const employee = {};
const reason = [
    {
        age: 25,
        education: 'higher',
        experience: 1
    }
]
Object.defineProperties(employee, {
    name: {
        value: 'Dmitriy',
        writable: false,
        configurable: false,
        enumerable: false
    },
    age: {
        value: 21,
        writable: true,
        enumerable: true,
        configurable: true
    },
    education: {
        value: 'higher',
        writable: true,
        enumerable: true,
        configurable: true
    },
    experience: {
        value: 0,
        writable: true,
        enumerable: true,
        configurable: true
    }
})
function hireNewEmployee (objectEmployee,objectReason) {
    for (let key in objectEmployee) {
        if (objectEmployee.key['age'] >= objectReason.key['education']
            && objectEmployee.key['education'] === objectReason.key['education']
            &&  objectEmployee.key['experience'] >= 1) {
            return console.log('You are Hired! Congrats!');
        }
    }
    return console.log(`Not hired: sorry we cannot hire you. Here is why: ${objectReason}`);
}
hireNewEmployee(employee,reason);

Answer 1

[Сергей Соколов]

for (let key in objectEmployee) {

что будет в key?

objectEmployee.key['age']

есть ли у objectEmployee свойство key? (нет)

spoiler

const filters = {
  age: v => v >= 25,
  education: v => v === 'higher',
  experience: v => v >= 1,
};

const employee = {};
const defaultDescriptor = { writable: true, enumerable: true, configurable: true };
Object.defineProperties(employee, {
  name: { ...defaultDescriptor, value: 'Dmitriy', writable: false },
  age: { ...defaultDescriptor, value: 21 },
  education: { ...defaultDescriptor, value: 'higher' },
  experience: { ...defaultDescriptor, value: 0 },
});

const hireNewEmployee = (employee, filters) => {
  const errors = [];
  Object.entries(filters).forEach(([name, func]) => {
    if (!func(employee[name])) {
      errors.push(name);
    }
  });

  return errors.length
    ? `Not hired: sorry we cannot hire you. Here is why: ${errors.join(', ')}`
    : 'You are Hired! Congrats!';
};

hireNewEmployee(employee, filters);
// "Not hired: sorry we cannot hire you. Here is why: age, experience"

Comments

[Дмитрий Ярощук]

Сергей Соколов подскажите я убрал цикл и key. Теперь возникла проблема что постоянно выводится:
return console.log(`Not hired: sorry we cannot hire you. Here is why: ${objectReason}`). А условие которое содержится в if не выполняется, не могу понять почему.

function hireNewEmployee (objectEmployee,arrayReason) {
    if (objectEmployee['age'] >= arrayReason['age']
        && objectEmployee['education'] === arrayReason['education']
        && objectEmployee['experience'] >= arrayReason['experience']) {
        return console.log('You are Hired! Congrats!');
    }
    return console.log(`Not hired: sorry we cannot hire you. Here is why: ${arrayReason}`);
}
hireNewEmployee(employee,reason);

[Сергей Соколов]

Дмитрий Ярощук, в вашем варианте не собираются обнаруженные ошибки. Надо же каждую проверку проводить, и по результатам добавлять или нет очередное свойство в список «нарушений».

написал вариант решения по-своему. Имхо лучше фильтры оформить как объект
{ ИмяСвойства: ФункцияКотораяПроверяет } (обновил ответ)

Ну и избежать повторов в дескрипторах.

[Дмитрий Ярощук]

Сергей Соколов Если вам не сложно могли бы вы более подробно расписать как работает ваш код

[Сергей Соколов]

Дмитрий Ярощук, уточните, какие именно моменты вам не понятны – постараюсь расписать.
Не хочется просто делать за вас домашку для copy-paste

[Дмитрий Ярощук]

Сергей Соколов Вот что мне не очень понятно, буду благодарен за ответ!
1. Мне не очень понятно что происходит за сравнение и как оно нам дальше способствует в коде age: v => v >= 25 (впервые вижу такое)
2.Object.entries(filters).forEach(([name, func]) Мне не очень понятно что здесь происходит. Вы выводите ключи и значения объекта filters и превращаете его в массив далее с помощью метода ForEach вы обращаетесь , а к чему я не могу понять.
3.!func(employee[name]) Не понимаю что за условие и как оно работает. Понимаю что вы обращаетесь к ключу объекта employee, при чем тут !func не понимаю

[Сергей Соколов]

Дмитрий Ярощук, постараюсь объяснить:
1. стрелочные функции вам уже знакомы? Вот, это они. Наверное, совсем уж краткая запись непривычна. Вот можно по-разному расписать, одно и то же (кроме последнего варианта с обычной function, но в данном коде это то же самое

age: v => v >= 25,
age: (v) => v >= 25,
age: (v) => {
  return v >= 25;
},
age: function (v) {
  return v >= 25;
},

В общем, в свойство age записывается значением функция, которая принимает единственный аргумент и возвращает Булево значение. true если аргумент больше или равен 25. Иначе false.

2. Object.entries() как вы знаете, возвращает массив пар (массив массивов) [ключ, значение] для итерируемых свойств объекта-аргумента. Т.е. для нашего filters получится массив

[ ["age", function1], ["education", function2], ... ]

Далее этот массив-массивов перебирается методом forEach() Используя деструктурирующее присваивание из переданного в функцию аргумента-массива-пары сразу вытаскиваем в переменные name и func. Можно было бы длиннее:

.forEach((pair) => {
  const name = pair[0];
  const func = pair[1];
  // ...
})

3. на каждой итерации name – это имя очередного фильтра: строки "age", "education", и т.д.
А func это очередная соотв. функция, которые мы там раньше в filters определили. Для age это та функция (v) => v >= 25. Функция, которая возвращает Булево значение (true или false). В нормальном случае true. Если значение НЕ соответствует ожиданиям, то false. Вот этот случай ошибки и нужен, поэтому инвертируем Булево значение: !false === true Т.е. в if () попадаем, когда функция проверки возвращает ошибку, false.

[Дмитрий Ярощук]

Сергей Соколов скажите пожалуйста можно ли данное условие !func(employee[name]) записать более понятнее и проще ? Например:

if (        filters.age() === false
            || filters.education() === false
            || filters.experience() === false) {
            errors.push(name);
        }

[Сергей Соколов]

Дмитрий Ярощук, так — нельзя, потому, что
во-первых в те функции же надо передавать значение, в каждую своё: filters.experience(employee.experience)
во-вторых, по результатам провероок в массив errors в итоге могут же попать одно, два, все, или ни одного имён свойств – так что одним errors.push() не обойтись.

[Дмитрий Ярощук]

Сергей Соколов я вас правильно понял что данное условие !func(employee[name]) никак нельзя заменить или более проще записать ?

[Сергей Соколов]

Дмитрий Ярощук, можно попробовать чуть разжевать:

if (false === func(employee[name])) {

// или вообще длинно:

const checkValue = employee[name]; // name – строка "age", потом "education", "experience"
// т.е. в первом случае employee[name] то же самое что employee.age

const checkResult = func(checkValue); // получится или true или false
if (checkResult === false) {
  // ...

[Дмитрий Ярощук]

Сергей Соколов подскажите при написание данной задачи столкнулся с тем, IDE пишет что в объекте filters методы age, experience, education не используются. Можно ли это как-то решить ? Так как мы действительно в коде конкретно к ним не обращаемся.

const filters = {
    age: function (element) {
        return element >= 25;
    },
    education: function (element) {
        return element === 'higher'
    },
    experience: function (element) {
        return element >= 1
    }
}

[Сергей Соколов]

Дмитрий Ярощук, а этот filters потом используете же? Может, IDE тупит и не видит, что всё используется. Там же явно имена свойств не присутствуют — это выше понимания IDE )

Имхо зря вы вместо стрелочных функций поставили обынчые. И именование аргумента "element" не отражает сути того, что туда передадут — очередное значение, а не DOM элемент.

[Дмитрий Ярощук]

Сергей Соколов я вот так сделал по вашему примеру и никак не получается избавиться от warning(unused property). Скажите есть ли какой-то вариант устранить данную проблему ?

const filters = {
    // age: function (a) {
    //     return a) => a >= 25,
    // }
    age:       (a) => a >= 25,
    education: (a) => a === 'higher',
    experience:(a) => a >= 1,
}
const employee = {};
const defaultDescriptors = {
    writable: true,
    enumerable: true,
    configurable: true
}
Object.defineProperties(employee, {
    name: {
        ...defaultDescriptors,
        value: 'Dmitriy',
        writable: false
    },
    age: {
        ...defaultDescriptors,
        value: 21
    },
    education: {
        ...defaultDescriptors,
        value: 'higher'
    },
    experience: {
        ...defaultDescriptors,
        value: 0
    }
})
function hireNewEmployee (employee,filters) {
    const reasons = [];
    Object.entries(filters).forEach(([name,func]) => {
        if (func(employee[name]) === false) {
            reasons.push(name);
        }
    })
    return reasons.length
        ? `Not hired: sorry we cannot hire you. Here is why: ${reasons.join(', ')}`
        : 'You are Hired! Congrats!';
}
console.log(hireNewEmployee(employee,filters));

[Сергей Соколов]

Дмитрий Ярощук, сменить IDE? :)
У меня ни VSCode ни ESlint не видят проблем.

..А я вижу потерянные точки-с-запятой после const filters.. и defaultDescriptors

[Дмитрий Ярощук]

Сергей Соколов подскажите столкнулся еще с одной проблемой никак не могу решить. Я при проверке вызываю функцию lieToEmployer которая меняет значения объекта employee. И в итоге функция hireNewEmployee пишет что я не нанят на работу из-за возраста хотя возраст позволяет принять меня на работу.

const filters = {
    age:       (a) => a >= 25,
    education: (a) => a === 'higher',
    experience:(a) => a >= 1,
};
const employee = {};
const defaultDescriptors = {
    writable: true,
    enumerable: true,
    configurable: true
};
Object.defineProperties(employee, {
    name: {
        ...defaultDescriptors,
        value: 'Dmitriy',
        writable: false
    },
    age: {
        ...defaultDescriptors,
        value: 21
    },
    education: {
        ...defaultDescriptors,
        value: 'middle'
    },
    experience: {
        ...defaultDescriptors,
        value: 0
    }
})
function hireNewEmployee (employee,filters) {
    const reasons = [];
    Object.entries(filters).forEach(([name,func]) => {
        if (func(employee[name]) === false) {
            function lieToEmployer (employee) {
                return Object.defineProperties(employee, {
                    age: {
                        value: 27
                    },
                    education: {
                        value: 'higher'
                    },
                    experience: {
                        value: 7
                    }
                });
            }
            lieToEmployer(employee);
            reasons.push(name);
        }
    });
    return reasons.length
        ? `Not hired: sorry we cannot hire you. Here is why: ${reasons.join(', ')}`
        : 'You are Hired! Congrats!';
}
console.log(hireNewEmployee(employee,filters));

[Сергей Соколов]

Дмитрий Ярощук, у соврать-то соврали, но поздно: в список косяков уже занесли одну проблему.