Как можно сократить код?

Question

[Deman1]

Здравствуйте, подскажите пожалуйста как можно этот код сократить?

<?$invo = 0;
foreach($invoices as $item): ?>
<? if (!($item['invoice_status'] == 1)) {
continue;
}?>
<? $invo=$invo+$item['invoice_ammount']; ?>
<?endforeach; echo $invo; ?>

Answer 1

[Arman]

<?= array_sum(array_map(function ($item) {
    return $item['invoice_status'] == 1 ? $item['invoice_ammount'] : 0;
}, $invoices)) ?>

Answer 2

[Сергей Герасимов]

<?=array_reduce($invoices, function(&$sum, $item) {
    return $sum + ($item['invoice_status'] == 1) ? $item['invoice_ammount'] : 0; 
})?>

Comments

[Stalker_RED]

прибавлять нужно invoice_ammount, и статус должен быть не равен 1

[Сергей Герасимов]

Stalker_RED, ещё раз посмотрите исходный код.

[Stalker_RED]

Сергей Герасимов, да, про != я гоню, но к invoice_ammount вы не обращаетесь. А там может быть больше единицы, внезапно.

[Сергей Герасимов]

Stalker_RED, внезапно оно попадёт под continue в исходном коде, а у меня прибавит ноль...

[Stalker_RED]

Сергей Герасимов, а invoice_ammount?

[Сергей Герасимов]

Stalker_RED, ну да, что-то я не внимательно посмотрел...

Answer 3

[Кайрат Убукулов]

Можно использовать оператор: +=

Подробнее тут:

<?$invo = 0;
foreach($invoices as $item): ?>
<? if (!($item['invoice_status'] == 1)) {
continue;
}?>
<? $invo += $item['invoice_ammount']; ?>
<?endforeach; echo $invo; ?>

Comments

[Одиночка Айс]

И в чем ваше сокращение то? В одном операторе?

[Кайрат Убукулов]

Да. Можно дальше наверно сократить.

[edward_freedom]

Кайрат Убукулов, бред

[DevMan]

edward_freedom, огня ... требует жопа твоя!