Почему некорректно работает код?

Question

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

По заданию при проверке объекта на условие ( if (func(employee[name]) === false)) мне нужно вызывать функцию (lieToEmployer) которая меняет значения объекта employee.
Я предполагаю что проблема заключается в том что данные в объекте не меняются или меняются с опозданием поэтому основная функция hireNewEmployee работает еще со старыми данными объекта employee и выводит некорректный результат опираясь на старые значения объекта employee.

const filters = {
    age: function (a) {
        return a >= 25;
    },
    education: function (a) {
        return a === 'higher';
    },
    experience: function (a) {
        return 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) {
                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));

Comments

[Lynn «Кофеман»]

Я уже видел этот вопрос.
Чем вас не устроили ответы на предыдущую версию?

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

Lynn «Кофеман» не помогло, человек предлагал решение проблемы без использования функции lieToEmployer. Данную функцию нужно обязательно использовать при основной проверки.

[Вадим]

В чём некорректность работы объяснить не хотите?

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

Lynn «Кофеман», причем от другого аккаунта. Т.е. у парня их несколько?

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

Вадим некорректность работы заключается в том что при основной проверке func(employee[name]) === false я вызываю функцию lieToEmployer которая должна менять значения объекта employee, но она либо их не меняет или меняет с опозданием. Таким образом основная функция hireNewEmployee работает со старыми значениями объекта employee. А мне нужно что после того как функция lieToEmployer поменяла значения объекта employee, функция hireNewEmployee могла сразу с ними работать.

[Вадим]

Дмитрий Ярощук, Отлично. Всё так и работает.
Может стоит научиться дебагу?
https://learn.javascript.ru/debugging-chrome

Первая проверка по возрасту func(employee[name]) === false меняет объект и дальше функция работает с изменённым объектом. Но в reasons уже записывается не соответствие по возрасту.

Answer 1

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

Приведите точную формулировку задания насчёт "обмана". Несколько разных вариантов решения возможны..

Например, переназначить свойства в объекте: установить геттер свойства age, и возвращать всегда валидное значение.

Или в том виде как у вас, просто переназначить свойство length у массива reasons, чтобы он всегда был "пустой".

Comments

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

Сергей Соколов подскажите вы вот так имели ввиду сделать ?

function hireNewEmployee (employee,filters) {
    const reasons = [];
    let lieCounter = 0;
    Object.entries(filters).forEach(([name,func]) => {
        if (func(employee[name]) === false) {
            function lieToEmployer (employee) {
                Object.defineProperties(employee, {
                    age: {
                        value: 27
                    },
                    education: {
                        value: 'higher'
                    },
                    experience: {
                        value: 7
                    }
                });
                function checkEmployee (employee) {
                    if (employee.age >= 25
                        && employee.education === 'higher'
                        && employee.experience >= 1
                        && lieCounter < 1) {
                        lieCounter++;

                        return lieToEmployer(employee);
                    }
                    else {
                        return false;
                    }
                }
                reasons.splice(0);
                checkEmployee(employee);
            }
            lieToEmployer(employee);
            reasons.push(name);
        }
    });
    return reasons.length > 2
        ? `Not hired: sorry we cannot hire you. Here is why: ${reasons.join(', ')}`
        : 'You are Hired! Congrats!';
}
console.log(hireNewEmployee(employee,filters));

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

Дмитрий Ярощук, дубль-два:

Приведите точную формулировку задания насчёт "обмана".

Нет, я имел в виду другое. Первый вариант это заранее, не внутри проверки if (func(employee[name]) === false) { ... } хакнуть объект employee, чтобы все его свойства проходили проверку. Перезаписать свойства, например. Или назначить геттеры, которые вернут нужное.

Внутри блока, где уже обнаружена проблема, скорее всего, поздно пытаться что-то поменять.
Ну, базовая же логика, что идёт за чем.
Поэтому, ещё раз — что там с формулировкой задания?

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

Сергей Соколов Задание: Нужно извлечь пользу из дескрипторов. При проверке работника пытайтесь вызвать функцию lieToEmployer, в которой будете пытаться напрямую изменить свойства объекта работника (employee). Эмулируйте случаи, когда соврать удалось, а если нет.

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

Дмитрий Ярощук, благодарю. Ну тут можно так понять:
"При проверке работника" — это в простом варианте сразу в начале функции hireNewEmployee() тут же вызвать lieToEmployer(employee), которая "напрямую изменить свойства":

function lieToEmployer(emp) {
  emp.age = 27;
  emp.education = 'higher';
  emp.experience = 7;
}