请问 这种写法 有什么优化的么

Question

Answer 1

这2个判断可以并成一个

if (
  type === 1 && this.collect_Status === 0 ||
  type === 2 && this.give_Status === 0
) {
  this.setLinkcollect(type);
} else {
  this.dellinkCollect(type);
}

Answer 2

使用枚举变量代替魔法数字

Answer 3

• 函数命名应遵循统一规范：小驼峰 setLinkCollect
• if/else优化：switch、三目、map、class

Example：

this[`${1==(this.collect_Status+type)?'set':'del'}LinkCollect`](type)
this[`${2==(this.give_Status+type)?'set':'del'}LinkCollect`](type)

Answer 4

个人会选择点赞和收藏处理程序分开, 内部根据状态三目运算

Answer 5

if ((type === 1 && this.collect_Status === 0) || (type === 2 && this.give_Status === 0)) {
    this.setLinkcollect(type)
  } else {
    this.dellinkCollect(type)
  }