PHP MVC收银系统：完成订单时折扣失效问题求助

问题场景 #

选择1份单价100美元的Pasta，添加10%折扣后总价显示为90美元（功能正常），但点击「Complete Order」完成订单时，系统传入数据库的是折扣前的100美元总价，折扣值丢失。

问题原因 #

1. 折扣未持久化存储：折扣值仅存在于单次POST请求中，未存入Session。点击「Complete Order」发起新POST请求时，若未重新提交折扣参数，calculateDiscountedPrice方法会默认取0折扣。
2. 视图字段设计冗余：折扣输入框使用数组格式discount[]，但实际是单个值，增加了参数处理复杂度。
3. 完成订单时硬编码折扣值：insert_bill方法中硬传入10作为折扣值，而非实际计算的折扣比例。

解决方案 #

1. 修改视图：调整折扣输入字段 #

将折扣输入框从数组格式改为普通字段，并默认显示Session中保存的折扣值：

<form method="post" action="">
    <!-- 其他原有字段保持不变 -->
    <div class="mb-4">
        <label class="block text-gray-700 text-sm font-bold mb-2" for="discount">Discount:</label>
        <input class="w-full border rounded py-2 px-3" type="number" name="discount" id="discount" min="0" value="<?php echo isset($_SESSION['discount']) ? $_SESSION['discount'] : 0; ?>">
    </div>
    <!-- 其他原有按钮保持不变 -->
</form>

2. 修改控制器：持久化折扣到Session #

构造函数中初始化折扣Session

public function __construct($db)
{
    $this->db = $db;
    $this->cashRegister = new CashRegisterModel();

    session_start();

    if (!isset($_SESSION['order'])) {
        $_SESSION['order'] = [];
    }
    $this->order = $_SESSION['order'];
    
    // 初始化折扣Session
    if (!isset($_SESSION['discount'])) {
        $_SESSION['discount'] = 0;
    }
}

处理POST请求时保存折扣到Session

在index方法的POST处理逻辑中新增折扣保存代码：

if ($_SERVER['REQUEST_METHOD'] === 'POST') {
    if (isset($_POST['submitItems']) || isset($_POST['completeOrder'])) {
        // 保存折扣到Session
        $_SESSION['discount'] = isset($_POST['discount']) ? (int)$_POST['discount'] : 0;
        
        $selectedMenuItems = isset($_POST['menuItemsSelect']) ? $_POST['menuItemsSelect'] : [];
        $quantities = isset($_POST['quantities']) ? $_POST['quantities'] : [];

        // 原有添加订单项的代码保持不变...

        $totalPrice = array_sum(array_column($this->order, 'price'));
        $result = $this->calculateDiscountedPrice();

        $_SESSION['order'] = $this->order;
        $order = $this->order;
        if (isset($_POST['completeOrder'])) {
            $this->completeOrder();
        }
        $this->printBill($order,$result);
        $this->viewItems($order,$result);
    }
}

修改折扣计算方法：从Session读取折扣

private function calculateDiscountedPrice()
{
    $totalPrice = array_sum(array_column($this->order, 'price'));
    // 从Session读取持久化的折扣值
    $discount = $_SESSION['discount'];
    $discountAmount = $totalPrice * ($discount / 100);
    $discountedPrice = $totalPrice - $discountAmount;

    return [
        'discountedPrice' => $discountedPrice,
        'discountAmount' => $discount,
    ];
}

完善订单完成逻辑：重置Session并传入正确折扣值

public function completeOrder()
{                    
    $result = $this->calculateDiscountedPrice();
    $discountedPrice = $result['discountedPrice'];
    $discount = $result['discountAmount'];
    
    $orderID = $this->cashRegister->insert_order($discountedPrice);

    // 保存订单项到数据库
    foreach ($this->order as $orderItem) {
        $menuItemID = $orderItem['id'];
        $quantity = $orderItem['quantity'];
        $itemPrice = $orderItem['price'];

        $this->cashRegister->insert_order_items($orderID, $menuItemID, $quantity, $itemPrice);
    }
    // 传入实际折扣值，替换硬编码的10
    $this->cashRegister->insert_bill($orderID, $discount, $discountedPrice);
    
    // 清空订单和折扣Session
    $this->order = [];
    $_SESSION['order'] = $this->order;
    $_SESSION['discount'] = 0;
}

取消订单时重置折扣Session

if (isset($_GET['cancelOrder']) && $_GET['cancelOrder'] === 'true') {
    $this->order = [];
    $_SESSION['order'] = $this->order;
    $_SESSION['discount'] = 0; // 重置折扣
    $totalPrice = 0;
    echo 'Order has been canceled.';
}

验证逻辑 #

修改后，折扣值会随订单一起保存在Session中，点击「Complete Order」时，系统会从Session读取正确的折扣比例计算最终价格，并将实际折扣值存入数据库；同时在取消订单或完成订单后重置折扣Session，避免影响下一次订单。

内容的提问来源于stack exchange，提问作者Gary Y Hurley