我目前正在经历一个FreeCodeCamp挑战,它要求您创建一个ROT13密码(一种非常简单的密码,它将每个字母转换为永无休止的字母前的13个字母)。我的代码如下:
function rot13(str) {
let lettersRegex = /[A-Z]/;
let alphabet = 'ABCDEFGHIJKLMNOPQRSTUVWXYZ'.split('');
let charCodes = [];
for(let i = 0; i < str.length; i++) {
if(str.charAt(i).match(lettersRegex)) {
charCodes.push(str.charCodeAt(i));
} else {
charCodes.push(str.charAt(i));
}
}
let ret = '';
console.log(`charCodes is currently ${charCodes}`);
for(let i = 0; i < charCodes.length; i ++) {
let temp = 0;
if(lettersRegex.test(String.fromCharCode(charCodes[i]))) {
if((alphabet.indexOf(String.fromCharCode(charCodes[i])) + 13) > alphabet.length) {
temp = charCodes[i] + alphabet.indexOf(charCodes[i]) - 12;
}
else {
temp = charCodes[i] + 13;
}
ret += String.fromCharCode(temp);
} else {
ret += charCodes[i];
}
}
console.log(ret);
return ret;
}
rot13("GUR DHVPX OEBJA SBK WHZCF BIRE GUR YNML QBT.");
//THE QUICK BROWN FOX JUMPS OVER THE L[ZY DOG.
基本上,除“ A”外,每个字母在密码后都会转移到正确的答案。导致此代码中的“ A”变成“ [”而不是“ N”的原因是什么?
对我的代码的任何评论或提示也将不胜感激。
答案 0 :(得分:2)
这是一个简单的解决方法,将>
更改为>=
。不要忘记数组是零索引的。
if((alphabet.indexOf(String.fromCharCode(charCodes[i])) + 13) >= alphabet.length)