pangram是一个包含字母表中每个字母至少一次的句子。
到目前为止,我的代码如下:
const isPangram = (string) => {
let alpha = string.toUpperCase().split("");
for (let beta = 65; beta < 65 + alpha.length; beta++) {
let gamma = String.fromCharCode(beta);
if (alpha.includes(gamma)) {
continue;
}
else {
return false;
}
}
return true;
}
console.log(isPangram("Detect Pangram"));
为什么“检测Pangram”返回true?
发布于 2018-11-28 12:00:28
您可以使用.every
以非常简单的方式做到这一点,如下所示。
alphabets = 'abcdefghijklmnopqrstuvwxyz'.split("");
const isPangram = (string) => {
string = string.toLowerCase();
return alphabets.every(x => string.includes(x));
}
console.log(isPangram("Detect Pangram"));
console.log(isPangram("abcd efgh ijkl mnop qrst uvwx yz"));
您可以通过以下链接了解有关every
的更多信息。
发布于 2018-11-28 12:49:38
您的错误在于,您运行循环直到字符串的长度,而不是A-Z。
希望这能有所帮助。
const isPangram = (string) => {
let alpha = string.toUpperCase().split("");
for (let beta = 65; beta < 91; beta++) {
let gamma = String.fromCharCode(beta);
if (alpha.includes(gamma)) {
continue;
} else {
return false;
}
}
return true;
}
console.log(isPangram("Detect Pangram"));
console.log(isPangram("abcd efgh ijkl mnop qrst uvwx yz"));
https://jsfiddle.net/xjys9dat/ -一个相同的工作示例
发布于 2018-11-28 12:57:48
您的方法是遍历字母表,并验证目标字符串中是否存在每个字母a-z。但是,除非输入字符串长度为26,否则您的循环条件65 + alpha.length
不会覆盖字母表的大小。迭代从65到91 (包括65和91)将解决此问题。
这是另一个解决方案,它提取字母表字符,将它们放入一个集合中,并检查集合大小是否为26。
const isPangram = s => new Set(s.toUpperCase().match(/[A-Z]/g)).size === 26;
[
"Detect Pangram",
"abcd efgh ijkl mnop qrst uvwx yz",
"abcd efgh ijkl mnop qrst uvwx y",
"bcd efgh ijkl mnop qrst uvwx yz",
"abcdefghijklmnopqrstuvwxyy",
"AbCdEfGhIjKlM zYxWvUtSrQpOn",
"How quickly daft jumping zebras vex."
].forEach(test => console.log(`${isPangram(test)}\t${test}`));
https://stackoverflow.com/questions/53511692
复制相似问题