фишка есть какая-то?
                  
                  
                  
                  
                  
                  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 :)
Обсуждают сегодня