如何在PHP中按指定百分比与顺序分配数组元素到三个数组？

你的代码主要存在三个核心问题，导致分配结果不符合预期：

1. 循环索引越界且逻辑错误 #

原代码的循环条件for ($i = 0; $i <= $count; $i++)中，$count是numbers数组的长度（10），但数组索引范围是0~9，$i <= $count会导致访问不存在的$numbers[10]。同时，你在每个if块中手动自增$i，导致索引跳步，无法按顺序逐个分配元素。

2. 分配逻辑不符合轮询规则 #

你期望的是按Caller1→Caller2→Caller3的轮询顺序分配元素，直到对应Caller的配额已满，再跳过它继续分配。但原代码的逻辑是每轮强制给三个Caller各分配一个元素，完全忽略了配额已满的情况，也没有遵循轮询优先级。

修正后的代码 #

下面是符合需求的实现，采用轮询分配逻辑，同时优化了代码结构（使用返回值而非公共属性存储结果，更符合封装原则）：

class CallDistributor
{
    private $callerPercentages = [
        'caller1' => 50,
        'caller2' => 30,
        'caller3' => 20
    ];

    public function splitCalls(array $numbers)
    {
        $total = count($numbers);
        // 计算每个Caller的分配配额
        $quotas = [];
        foreach ($this->callerPercentages as $caller => $percent) {
            $quotas[$caller] = (int)floor(($percent / 100) * $total);
        }

        // 初始化结果数组与已分配计数
        $result = array_fill_keys(array_keys($quotas), []);
        $assignedCounts = array_fill_keys(array_keys($quotas), 0);
        $callerOrder = array_keys($quotas);
        $currentCallerIdx = 0;

        foreach ($numbers as $number) {
            // 找到下一个还有配额的Caller
            while (true) {
                $currentCaller = $callerOrder[$currentCallerIdx];
                if ($assignedCounts[$currentCaller] < $quotas[$currentCaller]) {
                    break;
                }
                // 当前Caller配额已满，切换到下一个
                $currentCallerIdx = ($currentCallerIdx + 1) % count($callerOrder);
            }

            // 分配当前数字
            $result[$currentCaller][] = $number;
            $assignedCounts[$currentCaller]++;
            // 切换到下一个Caller，准备下一次分配
            $currentCallerIdx = ($currentCallerIdx + 1) % count($callerOrder);
        }

        return $result;
    }
}

// 使用示例
$distributor = new CallDistributor();
$splitResult = $distributor->splitCalls([1,2,3,4,5,6,7,8,9,10]);

var_dump($splitResult['caller1']); // 输出: [1,4,7,9,10]
var_dump($splitResult['caller2']); // 输出: [2,5,8]
var_dump($splitResult['caller3']); // 输出: [3,6]

代码说明 #

1. 配额计算：根据百分比计算每个Caller应分配的元素数量，使用floor确保取整。
2. 轮询分配：通过循环索引遍历Caller顺序，每次找到还有配额的Caller分配当前元素，分配完成后切换到下一个Caller。
3. 边界处理：当某个Caller配额已满时，自动跳过并切换到下一个Caller，直到所有元素分配完毕。

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