Как упростить данный код?

Question

[vetsmen]

Как упростить данный код?

if(recoupment > 0) {
		let items = await getItems(id, price, true);
		if(!items) {
			let items = await getItems(id, price, false);
		}
	} else {
		let items = await getItems(id, price, false);
	}

Comments

[Ярослав Иванов]

recoupment > 0 ? let items = await getItems(id, price, true) !items ? let items = await getItems(id, price, false) : let items = await getItems(id, price, false); : false

[vetsmen]

Ярослав Иванов, так если recoupment будет <= 0, тогда условие в вашем примере вообще не сработает

[Ярослав Иванов]

vetsmen,

recoupment > 0 ? let items = await getItems(id, price, true); !items ? items = await getItems(id, price, false); : false : items = await getItems(id, price, false);

Answer 1

[0xD34F]

Я так понял, вам нужно что-то вроде этого:

let items = (recoupment > 0 && await getItems(id, price, true)) || await getItems(id, price, false);

Answer 2

[profesor08]

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

Comments

[vetsmen]

А как правильнее сделать? Заранее определить item все if'a или использовать var?

[neeil]

извините, а почему не var ?

[Виталий]

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

Answer 3

[Coder321]

let items = await getItemsFunc();

function getItemsFunc() {
    if (recoupment > 0) return getItems(id, price, true);
    return getItems(id, price, false);
}