遵循DRY原则:如何避免重复jQuery选择器并引用对象内属性?
解决重复jQuery选择器并引用对象内部属性的问题
当然可以优化你代码里重复的jQuery选择器,但你尝试的this.placeholder写法行不通,核心原因是:在对象字面量的定义过程中,this并不指向当前正在创建的options对象——它指向的是外层函数MyFunction的上下文(非严格模式下通常是全局对象window,严格模式下是undefined)。
最优解决方案:先缓存选择器结果
最简洁高效的方式是先把重复的DOM查询结果缓存到一个变量里,再用这个变量给对象的属性赋值,这样既避免了重复查询DOM(提升性能),也消除了代码重复:
function MyFunction(){ // 缓存DOM查询结果,只执行一次选择器 const placeholderOption = $('#select option[value="-1"]'); var options = { placeholder: placeholderOption, first: placeholderOption.next() }; }
备选方案:对象初始化后再赋值
如果你一定要通过对象内部的属性来引用,也可以先初始化对象的部分属性,之后再补充赋值:
function MyFunction(){ var options = { placeholder: $('#select option[value="-1"]') }; // 对象创建完成后,就能直接引用它的属性了 options.first = options.placeholder.next(); }
不过第一种方案更推荐,因为它的代码结构更清晰,而且只执行一次DOM查询,性能更优。
内容的提问来源于stack exchange,提问作者Ivan
相关产品推荐
相关产品推荐

