Как написать функцию, которая будет увеличивать зарплату?

Question

[kuramaa922]

Привет, описание задачи ниже. Мой вопрос состоит в том, что не проходит последний тест

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

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

Нужно:

Получает данные по всем работникам
Находит работника с наименьшей зарплатой
Отправляет запрос на повышение зарплаты этому сотруднику на 20%
Если запрос прошел успешно - отправить сотруднику уведомление об увеличении ЗП тектом: Hello, <имя>! Congratulations, your new salary is <новая ЗП>!
Если запрос завершился неудачей - отправить данные об ошибке администратору
Должна всегда возвращать resolved промис с boolean значением:

true если увеличение прошло успешно
false если нет
Все функции для получения/изменения данных асинхронны и возвращают промисы.

api.getEmployees(); // Возвращает массив с объектами {id: 343, name: 'Alex', salary: 20000}
api.setEmployeeSalary(employeeId, newSalary); // Принимает id сотрудника и новую зарплату. Возвращает новые данные по сотруднику.
api.notifyEmployee(employeeId, text); // Принимает id сотрудника и текст уведомления
api.notifyAdmin(error); // Принимает ошибку

function increaseSalary() {
  try {
    return api.getEmployees()
        .then(emp => {
            const [minSalaryEmployee] = emp.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(error)
  }
}

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();
    });
  },
};

Comments

[Юрий Матрос]

Проблема с последним тестом заключается в том, что обработка ошибок в функции increaseSalary не возвращает отклоненный промис, а вместо этого отправляет уведомление администратору. Чтобы исправить это, нужно добавить return Promise.reject(e) в блок catch:

function increaseSalary() {
  return api.getEmployees()
    .then(emp => {
      const [minSalaryEmployee] = emp.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(error => {
      api.notifyAdmin(error);
      return Promise.reject(error); // добавляем отклоненный промис
    });
}

Также, в этой функции нет обработки ситуации, когда массив сотрудников пуст, что может вызвать ошибку. Рекомендуется добавить проверку на пустой массив перед использованием reduce:

function increaseSalary() {
  return api.getEmployees()
    .then(emp => {
      if (emp.length === 0) { // проверяем, что массив не пустой
        throw new Error('No employees found');
      }
      const [minSalaryEmployee] = emp.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(error => {
      api.notifyAdmin(error);
      return Promise.reject(error);
    });
}

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

Отличный заголовок вопроса! Заглянул, полный любопытства и надежды : )

[kuramaa922]

Юрий Матрос, к сожалению, не проходит

[kuramaa922]

решил