JavaScript“firstNonRepeatingLetter”函数返回未定义而不是字符？

Question

这里是问题陈述：

编写一个名为 firstNonRepeatingLetter† 的函数，该函数接受字符串输入，并返回字符串中不重复的第一个字符。

例如，如果输入 'stress'，函数应该返回 't'，因为字母 t 在字符串中只出现一次，并且在字符串中首先出现。

作为一个额外的挑战，大写和小写字母被认为是相同的字符，但函数应该返回首字母的正确大小写。例如，输入“sTreSS”应返回“T”。

下面是我的代码，它返回未定义。我无法捕捉到错误。

function firstNonRepeatingLetter(s) {
  var x = s.replace(" ", "");
  for (var i = 0; i < x.length; i++) {
    if (x.charCodeAt(i) < 96) {
     
      var y = x.replace(x[i], "");
      var z = String.fromCharCode(x.charCodeAt(i) + 32);
      if (y.indexOf(x[0]) > -1 || y.indexOf(z) > -1) {
        continue;
      } else {
        var m = x[i];
      }
      return m;
    } else if (x.charCodeAt(i) > 96) {
      
      var y = x.replace(x[i], "");
      var z = String.fromCharCode(x.charCodeAt(i) - 32);
      if (y.indexOf(x[0]) > -1 || y.indexOf(z) > -1) {
        continue;
      } else {
        var m = x[i];
      }
      return m;
    }
  }
}

Answer 1

您的条件y.indexOf(x[0]) > -1 不正确。您不需要比较 first 字符，而是 ith 字符。将它们都替换为y.indexOf(x[i]) > -1。

---

一些提示：

• 请避免使用单字母变量名。它们使调试和理解代码变得异常困难。
• 块内的var 声明，如for、if 等，没有任何用处。您已重新声明 var m 两次。这只会增加混乱。
• .replace(" ", "") 仅删除第一个空格，即使您可能打算删除所有空格。要么使用正则表达式（也可用于添加更多案例，或​​将输入限制为特定字符），要么使用replaceAll（注意浏览器兼容性和提案状态）。使用正则表达式可能看起来像.replace(/\s/g, "") 来删除所有空格，或.replace(/[^a-z]/gi, "") 来删除任何非拉丁字母。
• 不要在 charcode 中添加或减去 32，只需使用 .toUpperCase 和 .toLowerCase。
• 同样，比较 小于或大于 96 的字符代码不会处理具有此精确字符代码的字符，即 ` 字符。相反，请考虑将整个字符串转换为大写或小写，而不是分别处理这两种情况。
• 最后，根据分配是否允许这样做，考虑使用Object 和Array 方法。

---

这是在 ECMAScript 2015+ 中编写函数的两种方式：

const getFirstUniqueLetter = (string) => {
  string = [...string.replace(/\s/g, "")];

  const {
    map,
    unique
  } = string.reduce((result, character) => {
      result.map[character.toLowerCase()] = (result.map[character.toLowerCase()] || 0) + 1;

      if(!result.unique.includes(character)){
        result.unique.push(character);
      }

      return result;
    }, {
      map: {},
      unique: []
    });

  // You can add the `|| ""` in the next line to get a default output of the empty string, if no character is unique. Otherwise, `undefined` is returned.
  return unique.find((character) => map[character.toLowerCase()] === 1) /* || "" */;
};

const getFirstUniqueLetter = (string) => {
  string = string.replace(/\s/g, "");

  // You can add the `|| ""` in the next line to get a default output of the empty string, if no character is unique. Otherwise, `undefined` is returned.
  return [...string].find((character) => string.match(new RegExp(character, "gi")).length === 1) /* || "" */;
};

Answer 2

这并不能解决您编写的代码，但我想我会提供一种替代方法。

    function firstNonRepeatingLetter(s){
      var allCharacters = s.split('');
      var character;
      var characterMap = {};
      //get all character counts
      for(var i=0;i<allCharacters.length;i++){
        character = allCharacters[i];
        if(typeof(characterMap[character]) == 'undefined'){
          characterMap[character] = 0;
        }
        characterMap[character]++;
      }
      //remove any duplicates
      for(var c in characterMap){
        if(characterMap[c] > 1){
          delete characterMap[c];
        }
      }
      //find the first character in your string that is *still* in the map
      for(var i=0;i<allCharacters.length;i++){
        character = allCharacters[i];
        if(typeof(characterMap[character]) != 'undefined'){
          return character;
        }
      }
      //all characters in the string were duplicated (e.g. "ABBA")
      return 'No non repeating letters';
    }
    
    console.log(firstNonRepeatingLetter("stress"));
    console.log(firstNonRepeatingLetter("doodle"));
    console.log(firstNonRepeatingLetter("abba"));
    console.log(firstNonRepeatingLetter("supercalifragilisticexpialidocious"));