JavaScript 这样写，不知道为什么，总觉有点别扭

[pinews]

if (xxx) {
ele.onclick = funA;
} else {
ele.onclick = funB;
}

总感觉哪里有问题。

[yxcoder]

xxx?funA:funB

[jowan]

ele.onclick = () => {
fn(xxx);
}

[sutang]

ele.onclick = xxx ? funA : funB

[azh7138m]

没有缩进肯定别扭。

[Mitt]

能用语法糖解决的问题都不是问题，"编译器"可以帮你优化掉的

[zjsxwc]

function Condition1(){/*do sth when condition1*/}
function Condition2(){/*do sth when condition2*/}

xxx = "Condition1";
ele.onclick = window[xxx];

[uxstone]

动态类型嘛 习惯就好

[tinycold]

你的直觉是对的，对一个 DOM 元素动态地绑定 /移除事件监听是很蠢的事情，编译器到底能不能优化到那一步我不知道，不过你的同事肯定是还没优化到那个地步。

如果你实在是有这么两个函数要重用，这个代码会写成：
```
el.onclick = clickHandler

// clickHandler
if (xxx) {
funA()
} else {
funB()
}
```

[banricho]

写的什么鬼……

```JS
const fnA = () => {}
const fnB = () => {}
const handle = () => {
isA ? fnA() : fnB()
}

ele.addEventListener('click', handle)
```

[wenzhoou]

@tinycold 这两种写法明显不等价。

[tinycold]

@wenzhoou 我理解的「等价」是运行结果一样，不知道你说的「等价」是哪一层意思。

[pinews]

@wenzhoou 是的，当 click 发生的时候，xxx 判断的结果可能已经变了

[hackfly]

函数在 js 属于第一类值，和 c 之类的有点不一样

[no1xsyzy]

Qt 信号槽，handler 就是可以变的，甚至两个的。

[no1xsyzy]

@pinews #12 也有可能需要保留那时的判断结果
单独存一个变量不如还是直接传函数。

不过如果这些个 funA funB 还会在其他地方用，我想还是存变量好，方便调试，知道是点击事件触发的。

[pinews]

@no1xsyzy 感谢，解答了我的疑惑

[2kCS5c0b0ITXE5k2]

ele.onclick: (() => {
if(xxx) {
funA()
} else {
funB()
}
})

[lraining]

onclick 本质上是一个函数指针，直接指定函数指针或者在一个函数中指定，两者没有本质上的差别，但是有的时候给 onclick 赋值的时候要去掉之前的赋值，否则有可能 onclick 会指向多个函数

[markyun]

// handleClick
let handleClick;
if (field === 'a') {
handleClick = () => {
this.a(row);
};
} else if (field === 'b') {
handleClick = () => {
this.b(row);
};
}

[learnshare]

理论上不建议 onclick，因为只能关联一个函数。应该是

target.addEventListener('click', () => {
xxx ? funA : funB;
});