@syb2024

Функция увеличивает ЗП, если ошибка то уведомление админу?

функция не проходит одну проверку

✓ функция должна возвращать промис (5ms)
✓ должна корректно увеличивать ЗП сотрудника с наименьшей ЗП (3ms)
✓ должна отправлять уведомление сотруднику
✕ в случае ошибки увеличения ЗП должна отправлять уведомление администратору, но не пользователю (3ms)

function increaseSalary() {
 return api.getEmployees()
  .then(employeeData => {
    const [minSalaryEmployee] = employeeData.reduce(([minEmployee, minSalary], employee) => {
      const {salary} = employee;
      return (salary < minSalary
        ? [employee, salary]
        : [minEmployee, minSalary]
      );
    }, [null, Infinity]);
    const {id, salary: oldSalary} = minSalaryEmployee;
    const newSalary = oldSalary * 1.2;
    return {id, salary: newSalary};
  })
   .then(({id, salary}) => api.setEmployeeSalary(id, salary))
   .then(({name, id, salary}) => api.notifyEmployee(id, `Hello, ${name}! Congratulations, your new salary is ${salary}!`))
   .catch(e => api.notifyAdmin(e));
}

const api = {
  _employees: [
    { id: 1, name: 'Alex', salary: 120000 },
    { id: 2, name: 'Fred', salary: 110000 },
    { id: 3, name: 'Bob', salary: 80000 },
  ],

  getEmployees() {
    return new Promise((resolve) => {
      resolve(this._employees.slice());
    });
  },

  setEmployeeSalary(employeeId, newSalary) {
    return new Promise((resolve) => {
      this._employees = this._employees.map((employee) =>
        employee.id !== employeeId
          ? employee
          : {
            ...employee,
            salary: newSalary,
          }
      );
      resolve(this._employees.find(({ id }) => id === employeeId));
    });
  },

  notifyEmployee(employeeId, text) {
    return new Promise((resolve) => {
      resolve(true);
    });
  },

  notifyAdmin(error) {
    return new Promise((resolve) => {
      resolve(true);
    });
  },

  setEmployees(newEmployees) {
    return new Promise((resolve) => {
      this._employees = newEmployees;
      resolve();
    });
  },
};
  • Вопрос задан
  • 210 просмотров
Решения вопроса 1
grantur5707
@grantur5707
Full Stack Web Developer
Не проще отделить шаг уведомления сотрудника и отправлять его только в случае успешного завершения всех предыдущих операций?

function increaseSalary() {
  return api.getEmployees()
    .then(employeeData => {
      const [minSalaryEmployee] = employeeData.reduce(([minEmployee, minSalary], employee) => {
        const { salary } = employee;
        return (salary < minSalary
          ? [employee, salary]
          : [minEmployee, minSalary]
        );
      }, [null, Infinity]);
      const { id, salary: oldSalary } = minSalaryEmployee;
      const newSalary = oldSalary * 1.2;
      return { id, salary: newSalary };
    })
    .then(({ id, salary }) => api.setEmployeeSalary(id, salary)
      .then(updatedEmployee => {
        return api.notifyEmployee(updatedEmployee.id, `Hello, ${updatedEmployee.name}! Congratulations, your new salary is ${salary}!`);
      })
    )
    .catch(e => {
      return api.notifyAdmin(e);
    });
}
Ответ написан
Пригласить эксперта
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы