Php execute code only if result meets condition

Question

I have a shop checkout function something like this:

    $min_delivery = 25;
    foreach ($shoppingCartItem as $item) {
     $insert->name = $item->name;
     $insert->price = $item->price;
     $insert->quantity = $item->qty;
     $insert->save();
     $total += $item->price*$item->qty;
    }

Is there any php function that would allow the foreach loop to happen only if ($total > $min_delivery) .

Or the only way would be to do the foreach twice, once only to calculate $total, then if ($total > $min_delivery) do a second foreach to insert into the database.

*EDIT - some details on why I want some other way instead of two loops:

The issue is that I can't trust the $item->price from the shopping cart because it comes from the user (and I don't verify it until checkout) so I need to check it against database before inserting.

So doing the loop twice would mean to query the database twice.

You answered your own question - first count total, then checkout. — u_mulder
– u_mulder, Commented Sep 5, 2018 at 15:55
@u_mulder That is one way yes, I asked if is there another way better? — Victordb
– Victordb, Commented Sep 5, 2018 at 15:56
Given that $total is calculated in the loop, I'm not sure that only executing the loop based on its value is going to get you very far. If there's a class involved that wraps the whole cart, adding a getTotal() method to it would make this a lot easier, and avoid the need to have a second dedicated loop. — iainn
– iainn, Commented Sep 5, 2018 at 15:57
Another method, would be to generate the total during add/remove, and keep it in the cart as a separate variable to directly look at when you need. But it creates variable maintenance during add/remove cart items. — IncredibleHat
– IncredibleHat, Commented Sep 5, 2018 at 15:59

Chris Lear · Accepted Answer · 2018-09-05 16:03:25Z

0

Here's a possible solution

$total = array_reduce($shoppingCartItem,
    function($carry,$item) {
        return $carry + $item->price*$item->qty;
    }
);
$min_delivery = 25;
if ($total > $min_delivery) {
    ...
}

answered Sep 5, 2018 at 16:03

Chris Lear

6,7601 gold badge20 silver badges29 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Victordb Over a year ago

I added some details in the main question on why I am looking for a solution without two loops.

Chris Lear Over a year ago

This is only one loop :). But if you don't trust $item->price, I think you should fix that in the $shoppingCartItem object before doing this calculation.

Victordb Over a year ago

I tried to avoid doing a query every time a item is added to the cart. This way I only query the database once on the checkout.

Collectives™ on Stack Overflow

Php execute code only if result meets condition

1 Answer 1

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related