前提：这个判断应该是个业务逻辑，业务逻辑是客观存在的，这个判断是难以避免的
所以这个判断不是以现在的写法，就是统一写在一个函数里面，在新的函数里面进行判断

那么就要分两种情况：
1. 这段代码是需要多次执行的，那些外面和写里面性能没区别
2. 这段代码是一次性执行的，那写在外面在性能上而言甚至优于写里面

至于 onClick 还是 addEventListener，看业务情况：
1. 需要兼容 ie 较低版本，或者这个 dom 只需要绑定一个 click 回调，onClick 无疑是最方便的，既不需要兼容也不需要在再次绑定时卸载事件
2. 多个点击回调，肯定用 addEventListener，不过就上面这段代码已经存在而言，应该不是这类情况

结论：目前业务逻辑下没啥问题，但是你改成自己喜欢的写法也没问题。只能说对业务和性能没啥产出，不过自己维护的代码让自己舒服这点很重要~