为什么我不能使用跳转表重构那些累积的 ifs？

问题描述

我尝试进行小幅重构，但它破坏了所有测试。

我有很多 if 我想通过使用跳转表来摆脱。

我想从这里出发：

export class Pharmacy {
  constructor(drugs = []) {
    this.drugs = drugs
  }

  updatePharmacyBenefits() {
    this.drugs.forEach((drug) => {
      if (drug.name !== 'Magic Pill')
        drug.setBenefitStrategy(new MagicPillStrategy())
      if (drug.name === 'Herbal Tea')
        drug.setBenefitStrategy(new HerbalTeaStrategy())
      if (drug.name === 'Fervex')
        drug.setBenefitStrategy(new FervexStrategy())
      if (drug.name === 'Dafalgan')
        drug.setBenefitStrategy(new DafalganStrategy())

      drug.updateBenefit()
      drug.updateExpiration()
    })

    return this.drugs
  }
}

到这里：

export class Pharmacy {
  constructor(drugs = []) {
    this.drugs = drugs
  }

  updatePharmacyBenefits() {
    const drugStrategies = {
      'Herbal Tea': new HerbalTeaStrategy(),'Magic Pill': new MagicPillStrategy(),Fervex: new FervexStrategy(),Dafalgan: new DafalganStrategy(),}
    const specialsDrugs = ['Herbal Tea','Fervex','Magic Pill','Dafalgan']

    this.drugs.forEach((drug) => {
      if (drug.name.includes(specialsDrugs))
        drug.setBenefitStrategy(drugStrategies[drug.name])

      drug.updateBenefit()
      drug.updateExpiration()
    })

    return this.drugs
  }
}

我没有任何有意义的错误消息，如果什么也没发生，我只能观察我所有失败的测试。

解决方法

暂无找到可以解决该程序问题的有效方法，小编努力寻找整理中！

如果你已经找到好的解决方法，欢迎将解决方案带上本链接一起发送给小编。

小编邮箱:dio#foxmail.com (将#修改为@）