JS实现URL slug生成器时includes()校验错误与replace替换不全问题

• String.prototype.includes()的参数要求是单个待匹配的子字符串，代码直接传入数组forbiddenChars时，JS会自动将数组转为字符串"&',,"，实际判断逻辑变成了「查找字符串中是否包含&',,这串连续字符」，和需求完全不符。
• includes()方法的返回值是布尔值（true/false），将布尔值和-1做比较时，true > -1和false > -1的结果都为true，所以无论输入是否包含禁用字符，都会触发日志打印，产生误报。

• 虽然你遍历了所有禁用字符，但每个禁用字符仅执行了一次replace()操作。String.replace()传入字符串作为匹配规则时，只会替换第一个命中的匹配项，所以当同一个禁用字符出现多次时，除第一个之外的都会被保留。
• 针对&的特殊处理逻辑存在局限性：你仅匹配替换-&，如果出现&前没有-的场景，对应&就会被遗漏；且多个&出现时，也只会替换第一个命中的-&。

直接使用带全局修饰符的正则完成替换，逻辑更简洁，也不会出现匹配遗漏的问题：

function slugifyString(str) {
  return str.toLowerCase()
    .replace(/\s+/g, '-') // 所有空格替换为-，连续空格合并为单个-
    .replace(/[&',]/g, '') // 全局删除所有禁用字符&、'、,
    .replace(/-+/g, '-'); // 可选：清理替换后可能产生的连续-，保证slug格式规范
}

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