这段 if...else 有优雅的写法吗？

作者：waiaan 发布时间：2024-4-30 18:04:04

function onSortChange({ order, prop }) {
   let sortType = 0
   if (order === 'descending') {
      if (prop === 'thisYearIncome') {
      sortType = 1
      }
      if (prop === 'lastYearIncome') {
      sortType = 3
      }
      if (prop === 'rate') {
      sortType = 5
      }
   } else {
      if (prop === 'thisYearIncome') {
      sortType = 2
      }
      if (prop === 'lastYearIncome') {
      sortType = 4
      }
      if (prop === 'rate') {
      sortType = 6
      }
   }
   this.fetchData(sortType)
}
谢谢。

Ericcccccccc 2024-4-30 18:04:34

我感觉最多搞个 map 套一下 prop 和 sortType
外层的 if else 逻辑清晰不用改吧。

ma836323493 2024-4-30 18:05:07

简单枚举套枚举

ajaxgoldfish 2024-4-30 18:05:40

没事，无伤大雅，大道至简，要是再多的话，switch case 走一个

emeab 2024-4-30 18:06:26

```
function onSortChange({ order, prop }) {
const sortMapping = {
descending: {
thisYearIncome: 1,
lastYearIncome: 3,
rate: 5
},
ascending: {
thisYearIncome: 2,
lastYearIncome: 4,
rate: 6
}
};
// Get the sort type from the mapping based on the order and property
const sortType = sortMapping[order][prop] || 0; // Default to 0 if not found
this.fetchData(sortType);
}
```

pkokp8 2024-4-30 18:07:08

map? 不会 js ，大概这个意思
sortTypeMap[order][prop]=0-6
fetchData(sortTypeMap[order][prop])

pkokp8 2024-4-30 18:07:42

另外，第二层的第 2/3 个 if 少了 else ，会多两次判断，js 应该也支持 else if

tomatocici2333 2024-4-30 18:08:35

switch case 呗 = =看着不累

lingyi95 2024-4-30 18:09:07

你去问 gpt ，它可以给你优化成 10 行代码

mowen1992 2024-4-30 18:09:51

通义灵码来了：
function onSortChange({ order, prop }) {
// 使用映射对象来简化条件判断逻辑
const sortOrders = {
'thisYearIncome': { descending: 1, ascending: 2 },
'lastYearIncome': { descending: 3, ascending: 4 },
'rate': { descending: 5, ascending: 6 },
};
try {
// 检查 prop 是否存在于映射对象中
if (!sortOrders.hasOwnProperty(prop)) {
throw new Error(`Invalid property: ${prop}`);
}
// 检查 order 是否有效
if (!sortOrders[prop].hasOwnProperty(order)) {
throw new Error(`Invalid order: ${order}`);
}
// 直接通过映射对象获取 sortType
const sortType = sortOrders[prop][order];
// 调用 fetchData 方法更新数据
this.fetchData(sortType);
} catch (error) {
console.error('Error in onSortChange:', error.message);
// 可以在这里添加额外的错误处理，例如通知用户或记录日志
}
}

这段 if...else 有优雅的写法吗？

热门主题

大米总不可能有科技与狠活吧？

想基于容器搞个one-man idc，有两点重要问

搜狗的蜘蛛IP变了

阿里云的push过户限制新限制

请教下源码特征找网站、怎么大批量找到用这

中秋假期出行人数超6.2亿人次，我是没出门

通缩成啥样子了？美元降息对zg没啥用吧？

希望有人能为我发电

看到论坛里很多人买永久机我还是不敢买，不

大家现在用的宽带多少钱一年，多少兆

热门板块

公告

网站帮助 - Yoo趣儿

我们的愿景

在 Yoo趣儿投放广告

Yoo趣儿网站用户应遵守规则

这段 if...else 有优雅的写法吗？

热门主题

大米总不可能有科技与狠活吧？

想基于容器搞个one-man idc，有两点重要问

搜狗的蜘蛛IP变了

阿里云的push过户限制新限制

请教下源码特征找网站、怎么大批量找到用这

中秋假期出行人数超6.2亿人次 ，我是没出门

通缩成啥样子了 ？美元降息对zg没啥用吧？

希望有人能为我发电

看到论坛里很多人买永久机我还是不敢买，不

大家现在用的宽带多少钱一年，多少兆

热门板块

公告

网站帮助 - Yoo趣儿

我们的愿景

在 Yoo趣儿 投放广告

Yoo趣儿网站用户应遵守规则

中秋假期出行人数超6.2亿人次，我是没出门

通缩成啥样子了？美元降息对zg没啥用吧？

在 Yoo趣儿投放广告