JavaScript中的Caesars Cipher-为什么“ A”在这里变成“ [”?

时间:2019-03-08 06:48:40

标签: javascript encryption caesar-cipher rot13

我目前正在经历一个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”的原因是什么?

对我的代码的任何评论或提示也将不胜感激。

1 个答案:

答案 0 :(得分:2)

这是一个简单的解决方法,将>更改为>=。不要忘记数组是零索引的。 if((alphabet.indexOf(String.fromCharCode(charCodes[i])) + 13) >= alphabet.length)