创建一个接受字符串并根据字符串是否包含重复字符返回 true 或 false 的函数

Question

var arr = [];
repeatcount = 0;

function isogram(str) {
  for (var i = 0; i < str.length; i++) {
    for (var g = 0; g < arr.length; g++) {
      if (str[i] != arr[g]) {
        arr.push(str[i]);
      }
    }
  }
  if (arr.length != str.length) {
    return false
  } else {
    return true
  }
}


document.write(isogram("jiang"));

console.log(arr);

我在使用console.log(arr)的时候发现arr是空的，不知道为什么。我使用的方法基本上是创建一个存储非重复字符的数组然后比较长度，如果长度不同则有重复字符。

Answer 1

由于开头的arr 长度等于0，因此永远不会触发内部循环。

g < arr.length // is always false

我建议你摆脱内部循环并使用一个简单的条件来代替 - 如果当前字母不在 arr 内 - 推它，如果有 - 不要。

var arr = [];
repeatcount = 0;

function isogram(str) {
  for (var i = 0; i < str.length; i++) {
    if (arr.indexOf(str[i]) === -1) {
       arr.push(str[i]);
    } else {
      return false;
    }
  }
  
  return arr.length === str.length;
}

console.log(isogram("jiang"));
console.log(isogram("jiiang"));

奖励：一条线解决方案：

const isogram = (str) => [...str].length === [...new Set(str)].length;

console.log(isogram("jiang"));
console.log(isogram("jiiang"));