PHP生成指定长度和数量随机单词的函数问题求助

原代码存在的问题 #

1. 生成逻辑错误：仅循环$words次，每次给变量赋值单个字符，完全没处理每个单词需要$chars个字符的逻辑，最终只会得到最后一次循环的单个字符。
2. 变量覆盖问题：每次生成单词时直接覆盖变量，没有保留之前生成的结果。
3. 文件读取冗余：文件仅含一行内容，用while循环读取没必要，直接读取整个文件内容更高效。

修复后的代码 #

function randomStringWords($words = 0, $chars = 0) {
    // 参数合法性校验
    if ($words <= 0 || $chars < 3 || $chars > 8) {
        return [];
    }

    // 读取单行文件内容
    $charSource = file_get_contents("myFile.txt");
    if (!$charSource) {
        return [];
    }

    $sourceLength = strlen($charSource);
    $result = [];

    // 循环生成指定数量的单词
    for ($i = 0; $i < $words; $i++) {
        $word = '';
        // 为当前单词生成指定长度的字符
        for ($j = 0; $j < $chars; $j++) {
            $word .= $charSource[rand(0, $sourceLength - 1)];
        }
        $result[] = $word;
    }

    return $result;
}

// 调用示例：生成3个4字符的单词
$myRandomStrings = randomStringWords(3, 4);
// 按示例格式输出结果
echo implode('、', $myRandomStrings);

代码说明 #

• 参数校验：增加对单词数量、字符长度的合法性判断，避免无效参数导致异常。
• 文件读取优化：用file_get_contents直接读取整个文件内容，适配单行文件的场景。
• 双重循环逻辑：外层循环控制单词数量，内层循环生成每个单词的指定长度字符，确保结果符合要求。
• 结果收集：用数组存储所有生成的单词，最后通过implode转换为示例中的分隔格式输出。

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