使用onChange事件传参获undefined，下拉框控文本框maxlength遇错求助

嘿，我来帮你搞定这个问题！看起来你遇到的核心问题是事件绑定的参数传递错误，以及函数内部对参数的处理不匹配，导致拿到undefined值。我一步步给你拆解并修正：

问题根源分析 #

1. 参数传递类型不匹配
   你后台绑定事件时写的是cambiarLength(this, txtDoc)：

   • this确实是下拉框的DOM元素，但你的函数里却用document.getElementById(drop)去查找，相当于把DOM元素当成ID字符串去查询，肯定找不到结果，option变量会变成null。
   • txtDoc是文本框的ID，但你没加引号，JS会把它当成一个变量，而页面上根本没有这个变量，所以textbox参数会是undefined，后续getElementById(undefined)也会报错。

2. 可选优化：用value判断比文本更可靠
   依赖选项的文本（text）判断可能有隐患，比如文本前后有空格、大小写不一致，建议给下拉选项设置value属性（比如<option value="RUC">RUC</option>），然后通过value判断，稳定性更高。

修正方案（两种可选） #

方案1：直接传递DOM元素，简化函数逻辑

这种方式不需要再通过ID查找元素，更高效：

function cambiarLength(dropElement, textboxElement) {
  // 直接用传递过来的DOM元素
  var selectedText = dropElement.options[dropElement.selectedIndex].text;
  // 或者用value更可靠：var selectedValue = dropElement.value;
  if (selectedText.trim() === 'RUC') { // 加trim防止空格干扰
    textboxElement.maxLength = 3;
  } else {
    textboxElement.maxLength = 6;
  }
}

后台绑定代码修改为（注意txtDoc要加引号，因为要获取这个ID的元素）：

TipoDoc.Attributes.Add("onChange", "cambiarLength(this, document.getElementById('txtDoc'));");

注：去掉了javascript:前缀，因为事件属性里不需要这个前缀

方案2：保持传递ID字符串，修正函数参数处理

如果更习惯用ID传递，那函数里要确保接收的是ID字符串：

function cambiarLength(dropId, textboxId) {
  var option = document.getElementById(dropId);
  var texto = option.options[option.selectedIndex].text;
  var field = document.getElementById(textboxId);
  if (texto.trim() === 'RUC') {
    field.maxLength = 3;
  } else {
    field.maxLength = 6;
  }
}

后台绑定代码修改为（两个参数都加引号，传递ID字符串）：

TipoDoc.Attributes.Add("onChange", "cambiarLength('TipoDoc', 'txtDoc');");

额外调试小技巧 #

如果还是不确定哪里错了，可以在函数开头加console.log(drop, textbox)，打开浏览器控制台（F12）看输出的参数是什么，就能快速定位是参数没传对还是元素没找到。

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