生成固定长度唯一ID的JS脚本异常：块长度不足问题排查

我来帮你分析下问题所在，其实是两个小细节导致的：

1. 数字类型引发的拼接错误 #

你的randomNumber()函数返回的是数字类型（比如5而不是"5"），当你用+把多个数字连在一起时，JavaScript会执行算术加法，而非字符串拼接。举个例子：如果四个randomChar()都返回数字，比如1、2、3、4，那代码里的randomChar() + randomChar() + randomChar() + randomChar()就会变成1+2+3+4=10，转成字符串后就是"10"——这就是你看到2-3字符短块的核心原因！

2. 递归调用未返回有效结果 #

在blockBuilder()里，当你检测到block.length < 4时，虽然调用了blockBuilder()重新生成，但你没有把递归生成的正确结果返回或赋值给当前的block变量。最后return的还是最初那个有问题的短块，相当于递归白跑了一趟。

修复方案 #

第一步：让randomNumber()返回字符串 #

把数字转成字符串，确保所有拼接操作都是字符串层面的：

function randomNumber() {
  return Math.floor(Math.random() * 10).toString();
}

第二步：修复递归的返回逻辑 #

当递归调用时，直接返回新生成的正确block，替换掉原来的错误值：

function blockBuilder() {
  var block = randomChar() + randomChar() + randomChar() + randomChar();
  if (block.length < 4) {
    return blockBuilder(); // 直接返回递归生成的有效块
  }
  return block;
}

更稳妥的循环替代方案（可选） #

递归在这里不会有栈溢出风险，但用循环生成4字符块会更直观，也能避免递归的潜在问题：

function blockBuilder() {
  let block = '';
  // 循环直到凑够4个字符
  while (block.length < 4) {
    block += randomChar();
  }
  return block;
}

完整修复后的代码 #

var alphabet = "a,b,c,d,e,f,g,h,i,j,k,l,m,n,o,p,q,r,s,t,u,v,w,x,y,z";
var alphaArray = alphabet.split(",").map(function(l) { return l.toUpperCase() });

function randomLetter() {
  var index = Math.floor(Math.random() * 26);
  return alphaArray[index];
}

function randomNumber() {
  return Math.floor(Math.random() * 10).toString(); // 转成字符串
}

function randomChar() {
  return Math.floor(Math.random() * 2) == 0 ? randomLetter() : randomNumber();
}

function blockBuilder() {
  let block = '';
  while (block.length < 4) {
    block += randomChar();
  }
  return block;
}

function idBuilder() {
  return [
    blockBuilder(),
    blockBuilder(),
    blockBuilder(),
    blockBuilder(),
    blockBuilder(),
    blockBuilder(),
    blockBuilder(),
    blockBuilder()
  ].join("-"); // 用数组join拼接更简洁
}

for (var i = 0; i < 100; i++) {
  var v = idBuilder();
  document.write(v + "<br>");
}

现在运行这个脚本，就能稳定生成每个块都是4字符、总长度39位的标准ID了。

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