Использование оператора await в условии if. Не грязновато ли?

Question

[STTRBL]

Привет, тостер! У меня микровопрос по код стайлу.

Допустим, необходимо выполнить определенные действия в зависимости от того, вернет ли true асинхронная функция. Всё это дело происходит внутри асинхронного обработчика события. Довольно удобно написать это следующим образом:

async function handler(e) {
	if (await trueOrFalseFoo( ) ) {
		// do something
	}
}

Но при использовании await прямо в условии меня не покидает чувство, будто бы занимаюсь какой-то грязью. Чисто стилистически - это нормальная практика?

Comments

[Yan Markov]

Попробуй на досуге промисы. Говорят, хорошая вещь

[FabiBoom]

Yan Markov, но ведь await юзают только с промисами)

[FabiBoom]

А по коду все нормально, если хочешь можно в переменную сохранить:

let isOpen = await foo();

if(isOpen) {

}

[STTRBL]

FabiBoom, да вот в том то и вопрос: корректно ли (стилистически) обходиться без вспомогательной переменной и вызывать функцию с await сразу в условии?) Или в мире БоЛьШоЙ РаЗрАбОтКи на этот счет нет каких-то особых триггеров?)

[FabiBoom]

STTRBL, боже, да если по названию функции все понятно, что она именно проверяет, то можно и сразу в условие закинуть - почему вопрос именно в await? Много функций возращают boolean.

[Ihor Bratukh]

FabiBoom, не только, с любыми then объектами.

[Ihor Bratukh]

STTRBL, если вы ошибку обработали, то почему нет?

[FabiBoom]

Ihor Bratukh, да конечно знаю про thenable, но видно же что человек не понимает await вовсе)

[STTRBL]

FabiBoom, я не понимаю, чувак, ты детали вопроса-то читал или просто по заголовку глазами пробежался?) Булевое/не булевое значение - какая разница? Акцент в вопросе поставлен на СТИЛЕ НАПИСАНИЯ КОДА. Если точнее - не является ли *if (await чтонибудь( ) ){ }* грязным приёмом в плане читаемости/элегантности/прочих красивых слов. Спасибо, конечно, за ответ и внимание к вопросу, но я с гениев на Тостере всё больше угараю - один мне зачем-то промисы рекомендует, другой в каком-то там выдуманном невежестве упрекает)0

[STTRBL]

Ihor Bratukh, благодарю!

[FabiBoom]

STTRBL, я не упрекаю тебя не в чем. Мне просто непонятный был ход твоих мыслей, что меняет ключевое слово await в стилистике и все.

[STTRBL]

FabiBoom, лады, проехали!

Answer 1

[demonarxs1]

Чисто стилистически использовать функцию в условии - не очень, на мой взгляд.
Вынесите ее перед условием в переменную и затем переменную в условие.

const condition = await thueOrFalse();
if (condition) { ; }

Answer 2

[Сергей Краснов]

Если ты знаешь, что в функции точно не будет ошибки, которые нужно обработать, и если ты поймешь этот код через неделю, то можно писать такое условие без проблем.

Comments

[STTRBL]

Ништяк, спасибо за ответ! Вероятные ошибки обрабатываются в функции, да.