Почему не срабатывает конструкция?

Question

[Kristina8787]

Здравствуйте. Я новичок в JS , подскажите почему условие не отрабатывает такое

for (var key in entity)
        {
             if(task.find(item => item.ufCrmTask[0] == entity[key].ID)){
                arrayList[entity[key].TITLE ? entity[key].TITLE : entity[key].NAME] 
                = task.filter(item => item.ufCrmTask[0] == entity[key].ID);
            }else if(task.find(item => item.ufCrmTask === false)){ 
                arrayList.withoutEntity = task.filter(item => item.ufCrmTask === false);
                
            }
            
        }

Получаю только результаты из условия if, else if как будто не отрабатывает, если меняю местами текст в условиях то тоже самое получаю результат только из if . Подскажите что я не так делаю

Comments

[Alex]

Ну, хрен знает. Закиньте код в песочницу

[Вадим]

По-вашему, все знают что такое в Вашем коде entity, task, arrayList?
А в хрустальном шаре сразу отображается ошибка, из-за которой у Вас что-то не работает?

Правила использования сервсиа

3.2. Формулировать вопрос максимально информативно и однозначно.

[Kristina8787]

Вадим, это просто массивы в которых содержатся объекта типа
массив[
{object},
{object}
];

[Вадим]

Kristina8787, и сразу всё стало понятно(нет). То, что это массив объектов - это очевидно бегло пробежавшись по коду.
Вам нужна расшифровка слов "максимально информативно"? Дайте ВСЮ информацию. Структура объектов, массивов, значения переменных, используемых методов и т.д.

В идеале закиньте всё в песочнику Jsfiddle\codepen и прикрепите к вопросу.

[Kristina8787]

Вадим, Task

9:
action: {accept: false, decline: false, complete: true, approve: false, disapprove: false, …}
favorite: "N"
id: "18"
title: "Задача без привязки к сущности"
ufCrmTask: false
__proto__: Object
10:
action: {accept: false, decline: false, complete: true, approve: false, disapprove: false, …}
favorite: "N"
id: "20"
title: "Задача для компании"
ufCrmTask: Array(1)
0: "CO_186"

entity

5:
ID: "L_780"
TITLE: "Лид #780"

6:
ID: "CO_186"
TITLE: "AutoCompany90"

arrayList это пустой массив

[Вадим]

Kristina8787, Почти отлично. Идём дальше. Что не срабатывает?
Ожидаемый результат - имеющийся результат

[Kristina8787]

Вадим, ожидаемый что после первого условия запишутся совпадения в arrayList под ключом title из массива с которым значения совпало , затем после проверки второго условия запишется в arrayList в объект с именем withouEntity совпадения в которых значение проверяемого ключа равно false. Имеющийся результат, записывается только первое условие ,если менять местами запишется так же только первое условие. Первое условие это if. Получается что метод за одно условие перебирает все элементы,записывает совпадения,а те что не совпали просто опускает

[Вадим]

Kristina8787, Такс, ну, не скажу что я чётко понял желаемый результат..
Лапша у Вас конечно знатная.. попытался разгребсти.
Получилось что типа такого(вывод результата в консоль, переключитесь на вкладку Result)


Самое критическое что Вы делали не верно в своём коде - назначали массиву свойства, а не создавали в нём новые элементы.
Например:
arrayList.withoutEntity =
Не создаёт новый элемент массива. Создаём у самого массива свойство с ключом withoutEntity

[Kristina8787]

Вадим, про критическое поняла. Но ваш метод мне возвращает просто исходный массив task

6:
id: "20"
title: "Задача для компании"
ufCrmTask: ["CO_186"]

и не включает в него элемент у которого ufCrmTask === false

[Вадим]

Kristina8787,
По-моему на исходный массив совсем не похоже.
Как я и сказал, я не очень понял что Вам нужно получить в итоге.

[Kristina8787]

Вадим, я может написала как то не так, но вот пример того что мне нужно

{
            titleEntity:{
                titleTask: title,
                id: 2,
                ufCrmTask: [D_2]
            }
        }

{
            withoutEntity:{
                titleTask: title,
                id: 4,
                ufCrmTask: [D_4]
            }
        }

[Kristina8787]

Вадим, и второе условие не выполняется всё равно у меня. Мне бы понять просто почему оно не выполняется

[Вадим]

Kristina8787, Такс, смотрите, как я понял, вам нужно соотнести таски с entity. И замапить их друг на друга, чтобы в итоге получить все таски каждой entity внутри ключа этой самой entity.
Я позволил себе немного переименовать переменные и полностью переписал алгоритм, НО что более важно, я предлагаю Вам более подходящую под эту историю структуру данных. Не массив объектов. А объект, свойствами которого являются заголовки entity.
Получилось вот так:
Вывод так же в консоль браузера при переключении на вкладку Result.
Подходит?

[Вадим]

Kristina8787, кстати предложенное решение легко модифицировать чтобы одна и та же задача попадала к разным entity, так как ufCrmTask у Вас массив.

[Kristina8787]

Вадим, отлично, да, вот оно самое. Спасибо вам большое

[Вадим]

Kristina8787, закинул в ответы, выберите решением :-)

[Вадим]

Kristina8787, а, и я там забыл удалить лишний var item

[Вадим]

Kristina8787, а ещё алгоритм можно усовершенствовать для повышения скорости работы. Если интересно - обращайтесь.

Answer 1

[Вадим]

Вывод результата в консоль:

Comments

[Kristina8787]

только теперь не могу разложить его так что бы в таблицу данные записать

[Вадим]

Таблицу как делаете? Какая структура таблицы?

[Kristina8787]

Вадим,

у меня внутри task есть свойство comment ,внутри которого обьекты comment в которых текст и время.
таблицу вот такую надо как ниже. у меня на этапе перебора либо не итерируемый объект говорит,либо выводит вложенными массивами которые в цикле не перебрать

Имя Entity | Имя Task | Время comment | Текст comment

[Kristina8787]

Вадим, а хотя смогла, чуть хитрости надо было. Спасибо вам

[Вадим]

Kristina8787, ;-)

[Kristina8787]

Вадим,

[{…}]0: {ID: "9270", TASK_ID: "18", }

 (76) [{…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}]0: 
(5) [{…}, {…}, {…}, {…}, {…}]0: {ID: "96", TASK_ID: "3",
 [{…}]
 [{…}]
 [{…}]0: {ID: "9268", TASK_ID: "16",

на последнем этапе немного не пойму как такие массивы перебрать. это уже обьекты комментариев внутри тасков

[Вадим]

Kristina8787, так, ну, получается, что эти массивы лежат внутри объекта task.
Что-то типа такого:

{
  title: 'task1'
  comments: [{...}, {...}]
}

Соответственно, чтобы их обойти, можно использовать любой подходящий метод перебора для массива.
for или forEach, например.
типа

task.comments.forEach(function(comment){
  // код
})

[Kristina8787]

Вадим, это всё лежит в одном массиве map который.
на первой стадии цикла for in вот такое получаю,на следующей то что выше прислала

[Kristina8787]

Вадим, ага,перебрала, остался последний нюанс ,не понимаю почему у меня когда кончаются комменты для task , вставляется сначала пустая строка а потом идет новый task

[Kristina8787]

Вадим, а всё, починила. еще раз спасибо вам