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

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

lodash ecmascript-6 angularjs javascript

Mouad Ennaciri · 技术社区 · 8 年前

我有两个条件来比较两个模型是否存在,以显示不同的消息。 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 回复 | 直到 8 年前

dark_ruby 8 年前

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

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 8 年前

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

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));

推荐文章

Shibaji Biswas · 如何在Template literal的双引号内的html div id中编写JavaScript变量?

1 年前

GulgDev · 为什么不将属性设置为无效值?

1 年前

Koffee · 动画继续使用update()重新启动,而不是在Phaser 3中正常播放

2 年前

Monnt · 如何查看数组是否包括具有特定值的嵌套数组?

2 年前

JsCoder · 用ECDSA对C语言中的数据进行加密#

2 年前

dsi · 排序功能可实现多列的自定义差异

2 年前

huz3y · 如何在JavaScript中基于另一个变量值构建变量?如何重新分配它的值并对该变量执行方法?复制

2 年前

Darigan · clearTimeout将一个函数中的计时器与另一个函数(React.js)一起超时

2 年前

Joseph · 仅在React和JS中生成深色

2 年前

Joseph · 在Javascript中检索一些文本

2 年前