PHP实现带行ID的库存扣减计算遇问题，求助修正

问题根源 #

原代码中，foreach ($stock as &$item) 里的$item在数组结构调整后变为包含id和amount的二维数组，但你仍直接对$item赋值为数字（delta > 0 ? delta : 0），这会直接把整个子数组替换成数字，导致id字段丢失，第一个元素被覆盖为0。

修正后的代码 #

调整逻辑，仅操作子数组的amount字段，保留id标识：

$items = 45;
$stockRows = [
    ['id' => 1, 'amount' => 30],
    ['id' => 2, 'amount' => 40],
    ['id' => 3, 'amount' => 50]
];

$newStock = function ($items, $stock) {
    foreach ($stock as &$row) {
        // 仅针对amount字段计算扣减
        $delta = $row['amount'] - $items;
        $row['amount'] = $delta > 0 ? $delta : 0;
        $items = $delta < 0 ? abs($delta) : 0;

        // 扣减完成后提前终止循环，优化性能
        if ($items === 0) {
            break;
        }
    }
    unset($row); // 解除foreach的引用绑定，避免后续操作污染数组
    return $stock;
};

print_r($newStock($items, $stockRows));

输出结果 #

Array
(
    [0] => Array
        (
            [id] => 1
            [amount] => 0
        )

    [1] => Array
        (
            [id] => 2
            [amount] => 25
        )

    [2] => Array
        (
            [id] => 3
            [amount] => 50
        )

)

关键说明 #

• 核心修改：不再覆盖整个子数组，仅更新子数组中的amount值，保留id字段用于后续数据库更新。
• 解除引用：unset($row)是必要操作，避免PHP foreach循环的引用残留导致后续数组操作异常。
• 性能优化：当需要扣减的商品数量$items变为0时，直接终止循环，无需遍历剩余库存行。

内容的提问来源于stack exchange，提问作者DataConnect