第一段引用上面的摘要：
本文旨在解决在使用PHP编写Secret Santa脚本时，当用户数量为奇数时出现的匹配问题。通过分析问题代码，找出导致最后一个用户出现重复"No Pair"的原因，并提供修改后的代码，确保每个用户都能正确匹配，同时还提供了一种更优雅的循环匹配方案。
问题分析
原始代码在处理奇数用户时，由于递归调用generateUsers函数的方式，导致在只剩下一个用户时，会重复添加 "No Pair" 的匹配项。 具体来说，当用户数量为奇数时，经过几轮匹配后，$users数组只剩下一个元素。此时，代码会进入if (count($users) === 1)的分支，将该用户与 "No Pair" 进行匹配。然而，在之前的递归调用中，generateUsers函数在unset掉两个用户后，会再次调用自身。这次调用中，虽然$newUsers可能只有一个用户，但之前的函数调用仍然会执行到if (count($users) === 1)的分支，从而导致重复添加 "No Pair" 的匹配项。
解决方案
为了解决这个问题，需要在递归调用generateUsers之前，判断$newUsers数组中是否还存在至少两个用户。只有当用户数量大于1时，才进行递归调用。
立即学习“PHP免费学习笔记（深入）”；
修改后的代码如下：
function generateUsers(array $defaultUsers, array $updatedUsers = []): bool
{
$users = (!empty($updatedUsers)) ? $updatedUsers : $defaultUsers;
if (count($users) > 1) {
if ($this->matchedUsers !== count($this->startingUsers)) {
// Pick random user and match with current user. Reset array and repeat until 1 or no users left.
$randomUserIndex = rand(1, count($users) - 1);
$this->pairs[] = [$users[0], $users[$randomUserIndex]];
unset($users[$randomUserIndex]);
unset($users[0]);
// Remove pair from list so they can't be assigned again. Reset array for 0 based index $users[0]
$newUsers = array_values($users);
// Check that there are two or more users
if(count($newUsers) > 1){
$this->generateUsers($this->startingUsers, $newUsers);
}
$this->matchedUsers += 2;
}
}
// If only one user remains can't allocate a Pair
if (count($users) === 1) {
$orderedUsers = array_values($users);
$this->pairs[] = [$orderedUsers[0], "No Pair"];
return true;
}
return true;
}关键的修改在于添加了以下代码：
if(count($newUsers) > 1){
$this->generateUsers($this->startingUsers, $newUsers);
}这确保了只有当$newUsers数组中至少有两个用户时，才会进行递归调用，从而避免了重复添加 "No Pair" 的匹配项。
更优雅的解决方案：循环匹配
另一种更优雅的解决方案是避免使用 "No Pair"，而是创建一个循环匹配，即让最后一个用户匹配到第一个用户。 这种方法可以确保每个人都有匹配对象，避免了有人落单的情况。
实现循环匹配的代码如下：
<?php
class Match
{
protected $startingUsers = [
"User1",
"User2",
"User3",
"User4",
"User5",
];
protected $pairs = [];
function getPairs(): array
{
$users = $this->startingUsers;
$count = count($users);
for ($i = 0; $i < $count; $i++) {
$giver = $users[$i];
$receiver = $users[($i + 1) % $count]; // Use modulo to loop back to the beginning
$this->pairs[] = [$giver, $receiver];
}
return $this->pairs;
}
}
$match = new Match();
foreach ($match->getPairs() as $pair) {
echo "$pair[0] gets $pair[1]";
echo "\n";
}
?>这段代码使用取模运算符 % 来实现循环匹配。 ($i + 1) % $count 确保了当 $i 是最后一个用户的索引时，接收者会是第一个用户，从而形成一个循环。
总结
本文针对PHP Secret Santa脚本在处理奇数用户时出现的问题，提供了两种解决方案：
- 修正递归调用： 通过在递归调用generateUsers之前，判断用户数量是否大于1，避免重复添加 "No Pair" 的匹配项。
- 循环匹配： 使用取模运算符 % 实现循环匹配，确保每个人都有匹配对象，避免有人落单。
选择哪种方案取决于具体的需求。 如果需要明确标识出没有匹配对象的用户，可以使用第一种方案。 如果希望确保每个人都有匹配对象，可以使用第二种方案。 循环匹配方案通常更受欢迎，因为它更公平，也更符合Secret Santa的初衷。
