生成每5位添加'-'的随机字母序列时代码异常问题排查

原代码的核心问题 #

• 索引计算逻辑错误：pwArray[pwArray.Count(c => c != null) + 1] = '-' 完全偏离预期。执行判断时，当前iter位置已经填充了字母，Count(c => c != null)会包含该字符，再加1会把'-'放到错误的索引位，导致数组中出现未填充的空值，最终生成的字符串出现错位空白。
• 手动修改迭代变量引发逻辑混乱：iter+=1 跳过了下一次循环的索引，使得后续iter ==9 || iter==13的判断条件完全偏离预期的分组位置，字符填充顺序被打乱。
• 硬编码判断条件易出错：直接指定iter的具体值来插入分隔符，不仅扩展性差，还容易因为迭代变量的手动修改导致判断失效。

修复方案1：用List动态构建（简洁安全） #

这种方式无需预先计算数组长度，动态添加字符，逻辑更清晰：

string letters = "qwertyuiopasdfghjklzxcvbnmQWERTYUIOPASDFGHJKLZXCVBNM";
Random r = new Random();
List<char> pwList = new List<char>();

// 生成4组，每组4个字母
for (int group = 0; group < 4; group++)
{
    // 每组添加4个随机字母
    for (int i = 0; i < 4; i++)
    {
        pwList.Add(letters[r.Next(letters.Length)]);
    }
    // 最后一组后不加分隔符
    if (group != 3)
    {
        pwList.Add('-');
    }
}

string pw = new string(pwList.ToArray());

修复方案2：用固定数组（修正原思路） #

如果坚持使用数组，通过维护当前填充索引来避免迭代变量混乱：

string letters = "qwertyuiopasdfghjklzxcvbnmQWERTYUIOPASDFGHJKLZXCVBNM";
char[] pwArray = new char[19];
Random r = new Random();
int currentIndex = 0;

for (int group = 0; group < 4; group++)
{
    // 填充4个随机字母
    for (int i = 0; i < 4; i++)
    {
        pwArray[currentIndex++] = letters[r.Next(letters.Length)];
    }
    // 非最后一组添加分隔符
    if (group != 3)
    {
        pwArray[currentIndex++] = '-';
    }
}

string pw = new string(pwArray);

两种方案都能生成符合要求的格式，比如AxDG-aGJA-fbnz-KASz，不会出现错位或空白问题。

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