代码之家 › 专栏 › 技术社区 › Mouad Ennaciri

JavaScript/Angular1-重构算法/逻辑条件

lodash ecmascript-6 angularjs javascript

Mouad Ennaciri · 技术社区 · 7 年前

我有两个条件来比较两个模型是否存在,以显示不同的消息。 this.realEstateProjectCurrentProduct 保持不变,这是属性 housingTax 或 propertyTax 不同的是,else的情况保持不变。

if (this.realEstateProjectCurrentProduct.housingTax) {
  return this.housingTax = `${this.realEstateProjectCurrentProduct.housingTax} ${this.$translate.instant('currencySymbols.euro')}`
} else {
  return this.housingTax = 'No data'
}
if (this.realEstateProjectCurrentProduct.propertyTax) {
  return this.propertyTax = `${this.realEstateProjectCurrentProduct.propertyTax} ${this.$translate.instant('currencySymbols.euro')}`
} else {
  return this.propertyTax = 'No data'
}

例如,由于这两种情况与Lodash或ecmaScript 2015有太多相似之处,我如何改进这两种情况

2 回复 | 直到 7 年前

dark_ruby 7 年前

您可以使用属性分解,例如:

let { housingTax, propertyTax } = this.realEstateProjectCurrentProduct;
housingTax = housingTax ? `${housingTax} ${this.$translate.instant('currencySymbols.euro')}`: 'no data'
propertyTax = propertyTax ?  `${propertyTax} ${this.$translate.instant('currencySymbols.euro')}` : 'no data'

UncleDave 7 年前

虽然不是最安全的方法(魔术字符串通常很糟糕),但您可以尝试提取实际功能,并输入您想要更改的属性的名称。

function processTax(taxType) {
  if (this.realEstateProjectCurrentProduct[taxType]) {
    this[taxType]= `${this.realEstateProjectCurrentProduct[taxType]} ${this.$translate.instant('currencySymbols.euro')}`
  } else {
    this[taxType]= 'No data'
  }
}

processTax('housingTax');
processTax('propertyTax');

如果您希望将来有更多,请考虑使用阵列:

['housingTax', 'propertyTax'].forEach(tax => processTax(tax));

推荐文章

Softly · 单选按钮未按预期取值

1 年前

Jason Owens · 我的html/javascript播放按钮无法播放嵌入的youtube视频

1 年前

SlickRed · 我无法使用JS关注HTML元素

1 年前

Mayron Guedes · 如果登录名不在本地存储中,如何重定向页面?

1 年前

assembler · Nextjs没有处理发布请求

1 年前

PavelB · 如何使用Seleniumwebdriver和mocha只运行一个测试文件?

1 年前

BADRUM · 执行两个获取功能后,如何导航回页面?

1 年前

Toniq · javascript为php保存多维数组

1 年前

Natalie Smyth · 隐藏类只在我试图切换的两个部分中的一个部分上工作

1 年前

SkyWalker · 在JavaScript中,合并日期-时间序列的有效算法是什么?

1 年前