JavaScript Pig Latin转换器代码优化及实现思路求助

首先你的实现思路完全正确，作为新手能写出可正常运行的版本已经非常不错，以下是具体的优化方向和其他实现方案：

现有代码可优化的细节点 #

• 变量声明规范：当前元音数组用var声明，且循环变量i未显式声明会造成全局变量污染，建议固定值用const声明，可变值用let声明
• 元音判断冗余：当前数组同时存储大小写元音，其实可以将待判断字符统一转小写/大写后匹配，数组只需要存5个小写元音即可，减少冗余
• 无效逻辑删除：末尾的!vowels.includes(str) === true判断逻辑无效，vowels数组存储的是单个字符，不可能匹配完整字符串，循环遍历完未找到元音时直接加ay返回即可
• 查询效率优化：数组的includes方法时间复杂度是O(n)，如果用Set存储元音，查询时间复杂度可以降到O(1)，数据量小的时候差异不大，但属于更优的实现习惯
• 边界场景补充：当前代码没有处理空字符串、多单词、带特殊字符的输入，有对应需求可以补充适配逻辑

简化版循环实现 #

function pigLatin(str) {
  const vowels = new Set(['a', 'e', 'i', 'o', 'u'])
  // 首字母是元音直接加way返回
  if (vowels.has(str[0].toLowerCase())) {
    return `${str}way`
  }
  // 遍历找第一个元音位置
  for (let i = 0; i < str.length; i++) {
    if (vowels.has(str[i].toLowerCase())) {
      return `${str.slice(i)}${str.slice(0, i)}ay`
    }
  }
  // 全辅音场景直接加ay
  return `${str}ay`
}

正则版实现（代码更简洁） #

利用正则直接匹配开头的所有辅音串，不需要手动写循环遍历：

function pigLatin(str) {
  // 匹配开头连续的辅音，忽略大小写
  const leadingConsonants = str.match(/^[bcdfghjklmnpqrstvwxyz]+/i)
  return leadingConsonants 
    ? `${str.slice(leadingConsonants[0].length)}${leadingConsonants[0]}ay`
    : `${str}way`
}

进阶优化方向 #

• 支持多单词转换：通过split(' ')拆分字符串为单个单词数组，逐个转换后再用join(' ')拼接即可
• 大小写保留：比如输入Apple输出Appleway，输入Hello输出Ellohay，可以通过判断原字符的大小写状态调整转换后的大小写
• 特殊字符适配：如果输入带标点、数字，可以先提取非字母内容，单词转换完成后再放回原位置
• 入参校验：增加非字符串、空字符串的判断逻辑，避免代码抛出异常

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