AG-Grid中IF语句优化用法：设置指定列不可编辑

这里有几种更简洁、易维护的方式来重构你的代码，解决一堆||判断的冗余问题：

方法1：使用数组+includes方法 #

把需要设置为不可编辑的列ID统一放到一个数组里，然后用includes快速判断当前列是否在这个列表中：

// 把需要禁用编辑的列ID集中管理，注意移除了原代码里重复的'yy'
const nonEditableColIds = ['xxx', 'yyy', 'xx', 'yy', 'dd', 'mm', 'cc', 'vv'];

if (nonEditableColIds.includes(event.column.colId)) {
  event.colDef.editable = false;
}

方法2：使用Set提升查找性能 #

如果需要禁用的列数量较多，用Set的has方法会比数组includes更快（时间复杂度O(1) vs O(n)）：

// 定义不可编辑列的Set集合
const nonEditableColIds = new Set(['xxx', 'yyy', 'xx', 'yy', 'dd', 'mm', 'cc', 'vv']);

if (nonEditableColIds.has(event.column.colId)) {
  event.colDef.editable = false;
}

为什么这两种写法更好？ #

• 可读性更强：一眼就能看清哪些列被设置为不可编辑，不用在一堆||里找列ID
• 维护成本低：后续要添加/移除禁用列，只需要修改集合里的元素，不用调整条件判断的结构
• 避免重复错误：原代码里重复写了'yy'，集中管理的方式能轻松避免这类问题

另外，建议把这个列ID集合定义在事件处理函数的外部（比如组件的全局变量或常量），不用每次触发点击事件都重新创建集合，能小幅度提升性能。

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