JS网页消息编解码器解码返回undefined异常问题求助

核心错误：解码时字符索引计算逻辑错误 #

你的解码函数中，提取编码块的索引计算完全错误：

currentdt = currentdt + dstring.charAt(l * i);

每个编码块固定为5个字符，第l个编码块的起始位置应该是l * 5，然后依次取该块内的第i个字符（i从0到4），正确索引应为l * 5 + i。原代码的l*i会导致重复取同一个位置的字符，甚至出现索引越界，最终让find函数找不到匹配项，返回undefined混入结果。

修复后的解码函数 #

function decode() {
  decodedtext = "";
  dstring = prompt("enter a string to decode");
  // 先校验输入长度是否为5的倍数，避免无效处理
  if (!dstring || dstring.length % 5 !== 0) {
    alert("输入的编码串长度必须是5的倍数");
    return;
  }
  for (let l = 0; l < dstring.length / 5; l++) {
    currentdt = "";
    for (let i = 0; i < 5; i++) {
      // 修正索引计算，定位到当前编码块的对应字符
      currentdt += dstring.charAt(l * 5 + i);
    }
    const char = find(currentdt);
    // 处理找不到匹配项的情况，避免undefined混入结果
    decodedtext += char || "?";
  }
  alert(decodedtext);
}

其他优化建议 #

1. 变量声明规范：所有未用let/const声明的变量（如alphabet、循环中的i/l）会成为全局变量，容易引发意外问题，建议补充声明：

const alphabet = 'abcdefghijklmnopqrstuvwxyz'.split('');

同时在encode和decode的循环中，将i、l改为let i、let l。

2. 清空累积结果：全局变量encodedtext会在多次调用encode时累积之前的结果，建议每次编码前清空：

function encode() {
  estring = prompt("enter a string to encode, can only include non capital letters A-Z or SPACE");
  if (!estring) return; // 用户取消输入时直接退出
  encodedtext = ""; // 编码前清空旧结果
  for (let i = 0; i < estring.length; i++) {
    // 原有编码逻辑
  }
  alert(encodedtext);
}

3. 异常输入处理：编码时若用户取消输入，estring会是null，后续循环会报错，添加if (!estring) return可避免这种情况。

测试验证 #

用你提供的测试串soduhksdjnsoduhsdlfjpijgnpjnsedsfnn（对应"code"），修复后解码会正确返回code：

• soduh对应索引18 → 'c'
• ksdjn对应索引20 → 'o'
• sdlfj对应索引2 → 'd'
• pijgn对应索引14 → 'e'

内容的提问来源于stack exchange，提问作者Walle