Как сделать if else внутри promise then?

Question

[nezzard]

Добый вечер, есть

return 	Promise.all(myRequests)
.then((arrayOfHtml) => arrayOfHtml.map(result => result.data))
.then((arrayOfHtml) => {

тут if else

}).catch(function(e){
//error
});

Суть в чем, у меня в if идет проверка если есть элемент в json массиве, если нету тогда else, так вот если if отдает ошибку, то скрипт сразу прыгает на catch
У меня в if else есть переменная, допустим test, мне нужно если все хорошо присвоить true в противном случае false

Answer 1

[Stalker_RED]

если if отдает ошибку

Какую ошибку, что переменная не существует? Так проверяйте на существование еще одним if'ом.

Увлекся, написал формочку поиска по артисту с кучей проверок. Не уверен что покрыл все случаи отсутствующих ключей, но вроде работает. https://jsfiddle.net/Stalk/cv3h0yrb/

Comments

[nezzard]

Так дело в том, что как бы я не писал проверку, если ошибку отдает даже в if(someVar.name) и если name в json не существует, то сразу переходит к catch и не выполняет else

[tex0]

nezzard: похоже что у вас проблема не в name, а в корневой someVar. Возможна эта переменная у вас undefined.

[nezzard]

tex0: Там вообще идет путь примерно такой promiseResult[1].someArtist.lastflm.artist.img[0]

Вот допустим доходит оно до artist а картинки нету, и он выдает ошибку, и сразу прыгает на catch и как бы я не писал что-то на подобии if(json(который выше) === 'underfind') и так далее, ничего не помогает

[Stalker_RED]

nezzard: а можно пример данных? Или что твы там из lastfm запрашиваете?

[tex0]

nezzard:

доходит оно до artist а картинки нету, и он выдает ошибку, и сразу прыгает на catch

говорю же - похоже что у вас в цепочке не картинка(img) undefined, а переменная artist(или какой-либо объект-родитель выше по цепочке).
Посмотрите что объект 'e' в блоке catch промиса говорит вам...

[tex0]

nezzard: кстати, судя по цепочке есть вероятность что img действительно undefined и вы у него пытаетесь вызвать оператор [ ]. В таком случае тоже вывалится эксепшн и управление перейдёт в блок catch.

[nezzard]

tex0: Именно так, мне нужно отловить, когда img нету, потому что иногда такое бывает что нету такого артиста на ласт фм либо нету картинки, вот в таком случае мне нужно сделать переменную внутри иф var some = false

Но чтобы он не переходил в catch а выполнил функцию дальше уже с использованием переменной some

[Stalker_RED]

nezzard: Можете привести пример удачного и неудачного запросов?

[Stalker_RED]

nezzard: Добавил в пост выше ссылку на пример с кучей разных проверок. Также, вместо пустых строк можно подставить заглушки по умолчанию.

[tex0]

nezzard: так сначала проверяйте существование артиста, а потом, если он существует - проверяйте img
как-то так:
if (promiseResult[1].someArtist.lastflm.artist && promiseResult[1].someArtist.lastflm.artist.img)
do_something();
else
do_else();

[nezzard]

Stalker_RED: Но получиться по сути тоже самое, если он дойдет до пункта где будет ошибка, он перейдет в catch я ведь делал проверку в if

[nezzard]

Stalker_RED: В вашем примере если убрать все и оставить только data[0].artist и ввести неправильный запрос, то переходит к else а у меня игнорирует его

[nezzard]

tex0: Stalker_RED: https://toster.ru/answer?answer_id=957603#answers_...

[nezzard]

Stalker_RED: Вот как у меня https://jsfiddle.net/cv3h0yrb/1/

[tex0]

nezzard: что за ошибка в catch прилетает?

[Stalker_RED]

nezzard: Вы проверяете существование

if (arrayOfHtml[0].results.artistmatches.artist[0]['image'][2]['#text']) { /* ... */ }

, но если "обрыв" где-то раньше, к примеру отсутствует image, то естественно вылетает ошибка.
Можно переписать так:

if (arrayOfHtml 
&& arrayOfHtml[0]
&& arrayOfHtml[0].results
&& arrayOfHtml[0].results.artistmatches
&& arrayOfHtml[0].results.artistmatches.artist
&& arrayOfHtml[0].results.artistmatches.artist[0]
&& arrayOfHtml[0].results.artistmatches.artist[0]['image']
&& arrayOfHtml[0].results.artistmatches.artist[0]['image'][2]
&& arrayOfHtml[0].results.artistmatches.artist[0]['image'][2]['#text']) {
//  ...
}

Если вам кажется что это как-то неправильно, ну, такая ситуация вероятно возникла потому, что выше по коду никаких проверок не было. Если были - можно проверять не всю цепочку.
Можно написать отдельную функцию, которая будет проходить по всей цепочке
https://jsfiddle.net/Stalk/cv3h0yrb/3/ (см. ф-цию has())

Answer 2

[nezzard]

return 	Promise.all(myRequests)
.then((arrayOfHtml) => arrayOfHtml.map(result => result.data))
.then((arrayOfHtml) => {
  //Пытаемся получить картинку от ласт фм, если нету, выдаем пустую строку
  if(arrayOfHtml[0].results.artistmatches.artist[0]['image'][2]['#text']){
     last = arrayOfHtml[0].results.artistmatches.artist[0]['image'][2]['#text'];
   }else {
      last = false;
   }
   
  if(last) {
  	some();
  }
  
  })
  .catch(function(e){
     //Ловим ошибку 
     console.log(chalk.red("Ошибка   " +e));
  });