фишка есть какая-то?
employeesMapping(employees: Array<WfmEmployeesResponseModel>): object {
const employeesMap: object = {};
employees && employees.length && employees.forEach(item => {
if (!employeesMap[item.idRole]) {
employeesMap[item.idRole] = {};
}
employeesMap[item.idRole][item.id] = item;
});
return employeesMap;
}
зачем использовать оператор && в строке employees && employees.length && employees.forEach для проверки массива? не проще просто написать в начале функции !isArray { return } и уже ниже писать перебор массива forEach?
проверка на нулл и андефайнд
Возможно, код писал человек, который не знает про isArray :)
Обсуждают сегодня