Как правильно и покороче составить условие?

Question

[Антон]

<?
if (($itemid[PROPERTY_SKLAD2_VALUE] != 'Y') &&
($itemid[PROPERTY_FILTER_COLOR_ENUM_ID] == $FCOLOR))
{?><!--мой код--><?}
?>

Если выполнены оба условия, то выполняется код.
Как правильно сделать (не городить многоэтажность):
учитывать второе условие, если $FCOLOR не пусто,
если пусто, то выполнить код, учитывая только первое условие?

Comments

[Дмитрий]

Битрикс что ли?

[Антон]

Дмитрий, ну, да

Answer 1

[Vitsliputsli]

if ($itemid[PROPERTY_SKLAD2_VALUE] != 'Y' &&
(empty($FCOLOR) || $itemid[PROPERTY_FILTER_COLOR_ENUM_ID] == $FCOLOR))

Answer 2

[ipokos]

я сделал бы вложенный if

Comments

[Антон]

пока так и написал, но мне кажется должно быть более изящно

[Дмитрий]

Антон, загони условие в переменную, проверяй переменную в условии