Как запушить массив в объект по условию?

Question

[ura2rist]

Пишу условие, если == true то выполнить следующие действия, если нет то просто записать объект, в итоге в консоль выводится menuOtd, но наполнен он только в тех местах в которых срабатывает else, т.е часть после if выполняется но не пушится в общий массив. Я выводил переменную val в консоль но там все ок, т.е код выполнен но он не пушится. В чем дело?

pnd.findOne({where: {id: 1}})
			.then(pnds => {
				pnds.getOtds()
				.then(otds =>{
					var menuOtd = [];
					for(ot of otds){
						var val = {};
						if(ot.pndOtd.sub == 'true'){
							otd.findOne({where: {id: ot.id}})
								.then(ots=>{
									ots.getSubOtds()
										.then(sub => {
											var q =[];
											for(a of sub){
												//console.log('Отделения ' + ots.name + '!Под ' +a.name)
												q.push(a.name);
											}
											val = {
												name: ots.name,
												sub: 'true',
												pndId: 1,
												subm: q
											}
											menuOtd.push(val);
										})
								})
						}else{
							val = {
								name: ot.name,
								sub: 'false'
							}
							menuOtd.push(val);
						}
					}

					console.log(menuOtd)
					response.render('index',{
						pndli: pndli,
						pnd: info,
						table: table,
						otd: menuOtd
					});
				})
			});

Answer 1

[hzzzzl]

попробуй так, если я правильно понял структуру

.then(sub => {
/*
                      var q =[];
                      for(a of sub){
                        //console.log('Отделения ' + ots.name + '!Под ' +a.name)
                        q.push(a.name);
                      }
*/
                      val = {
                        name: ots.name,
                        sub: 'true',
                        pndId: 1,
                        subm: sub.map(s => s.name)
                      }
                      menuOtd.push(val);
                    })

Comments

[Игорь]

menuOtd.push({
          name: ots.name,
          sub: 'true',
          pndId: 1,
          subm: sub.map(s => s.name)
        });

Можно записать короче.

[ura2rist]

все равно выводит по итогу консоль лог как-будто выполняется код только после else

[Игорь]

ot.pndOtd.sub == 'true' // точно строка в sub ?

[ura2rist]

Игорь, да

Answer 2

[Andrey Suha]

У Вас внутри if ассихронный код, а внутри else нет, вот и получается что вывод в консоль срабатывает до того как выполнится код внутри if

Comments

[ura2rist]

Как в этом случае тогда лучше сделать?

[Andrey Suha]

ura2rist, не знаю, я не понимаю что Вы делаете, на мой взгляд то как Вы работаете с промисами это ужасно. Почитайте статью. Используйте promise all что бы посмотреть что будет после выполнения всех промисов.

[ura2rist]

Так тут не используется вообще промис

[Andrey Suha]

ura2rist, а .then это что? ots.getSubOtds() разве под капотом не промис? pnd.findOne не промис? pnds.getOtds() тоже не промис?

[Andrey Suha]

ura2rist, вообще
ot.pndOtd.sub == 'true'

Зачем булевое значение храните как строчное? Имно но все же лучше использовать === а не ==

[Игорь]

Andrey Suha, Скорее всего в ot.pndOtd.sub именно строка, которая имеет значение true/false

[Andrey Suha]

Игорь, это понятно, как я понимаю это запись из монго. Я не понимаю зачем хранить булевое значение как строку

[Игорь]

pnd.findOne({where: {id: 1}})
      .then(pnds => {
        pnds.getOtds()
        .then(otds =>{
          var menuOtd = [];
          for(ot of otds){
           console.log(ot);  
          }
        })
      });

Нам нужно знать что здесь

[Игорь]

Andrey Suha, Есть вероятность, что это внешняя API, которая в свою очередь передаёт такое значение.
Да, я согласен это неприемлемо!
Это ужасно!

[Andrey Suha]

Игорь, Окей, даже если это внешнее api и автор на него не имеет влияния сути вопроса не меняет. Он вызывает консоль лог или что то пробует сделать с menuOtd до того как выполнится

otd.findOne({where: {id: ot.id}})
                .then(ots=>{
                  ots.getSubOtds()
                    .then(sub => {

Если внимательно посмотрите то увидите что этот код внутри if, а

console.log(menuOtd)
          response.render('index',{
            pndli: pndli,
            pnd: info,
            table: table,
            otd: menuOtd
          });

Не попадает не в один из then'ов поэтому и в консоли нету того что должно попасть туда внутри ifa, ни на страницу эти данные не попадают. Я Вас часто вижу в комментариях и думаю Вам не нужно рассказывать про асинхронность и как она работает))

[Игорь]

Andrey Suha, Я полагаю, что автор так и задумывал.
Ему нужно было посмотреть что в menuOtd

Но, он не учитывает, что внутри цикла код асинхронный.

Andrey Suha вы правы.

После того как завершится цикл, последует вывод в консоль, пустого menuOtd
Пуш вmenuOtd не произойдёт!
Пуш произойдёт после resolve.

Автору нужно искусственно создать промис и подождать выполнение

Вот некоторые картинки.

Answer 3

[Игорь]

Коллега, ваша ошибка заключается в том, что внутри цикла, код асинхронный.
Вывод в консоль произойдёт раньше чем вы ожидаете.

Вот тут я неаккуратно изобразил анимацию, но суть верна и явно отражает вашу проблему.
Курсором я показываю очерёдность выполнения кода.
Следите за курсором.


Итог:
На момент вывода в консоль menuOtd пуст
Тип ошибки: логическая