请问写代码根据优先级返回数值有更好的写法吗?

2022-12-01 14:39:05 +08:00
 syntaxj

以车的各种状态返回不同颜色举例

// 伪代码


function(){
//正常状态
let color = '白色'

if (踩油门状态) {
color = '绿色'
}
if (汽油不够) {
color = '橘色'
}
if (车速太快状态) {
color = '红色'
}

return color

}

这样可以初步以车速太快>汽油不够>踩油门>默认 来返回颜色。但是代码多了比较难阅读?而且需要严格保持 if 的顺序
还有个想法是逻辑正常写,if 顺序也随便,但用一个栈把符合 if 条件的颜色全部推入栈中,同时颜色以一个携带优先权的数字在另一个 config 文件中维护,最后筛选 return 数字最大的那个

请问有更优雅的写法吗?

929 次点击
所在节点    问与答
5 条回复
DrakeXiang
2022-12-01 14:48:59 +08:00
既然要有权重,那按顺序来是最简单易懂的方式吧,否则你代码乱写,单看权重你都不知道你这个权重是不是最大的。
代码写法上如果逻辑都比较统一,那可以用表驱动法,另外可以把权重大的放前面,匹配到直接 return
justplaymore
2022-12-01 15:01:15 +08:00
按你的例子,车的状态是可能同时存在的。

状态对象 {
状态码,
颜色,
优先级
}

假如系统当前命中的状态是:车速太快状态, 汽油不够状态。
那么就将这些命中的状态放到一个集合中:[车速太快状态对象, 汽油不够状态对象]。
针对这个集合做排序和筛选策略,最终得到你认为优先级最高的状态对象,使用这个状态对象的颜色进行显示。
rabbbit
2022-12-01 15:12:25 +08:00
嫌状态乱难以阅读可以用 xstate
不过个人感觉必须要非常熟悉 xstate 才行,不然还不如不用。
wangxiaoaer
2022-12-01 15:27:52 +08:00
二楼的方案感觉比较合理,拿到所有状态后,选出状态里面权重值最大的,把他的颜色返回。
Sunzehui
2022-12-02 07:26:22 +08:00
我的想法和二楼差不多,看楼主返回的数值只有颜色,以后如果新增了大小或者其他,会更乱,建议为每个状态存对象到数组方便管理

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/899334

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX