请问下大家这种代码还能优化吗

Question

        if (resState == 8) {//下
            li_one.appendTo("#haved .ul-list");
            li_one.find(".status").html("完场");
        } else if (resState == 9) {
            li_one.appendTo("#haved .ul-list");
            li_one.find(".status").html("推迟");
        } else if (resState == 10) {
            li_one.appendTo("#haved .ul-list");
            li_one.find(".status").html("中断");
        } else if (resState == 11) {
            li_one.appendTo("#haved .ul-list");
            li_one.find(".status").html("腰斩");
        } else if (resState == 12) {
            li_one.appendTo("#haved .ul-list");
            li_one.find(".status").html("取消");
        } else if (resState == 13) {
            li_one.appendTo("#haved .ul-list");
            li_one.find(".status").html("待定");
        }

如上,这种代码执行效率怎么样?萌新求指教蟹蟹

Answer 1

执行效率没什么问题，就是代码简化些吧

var textObj = { 8: "完场", 9: "推迟", 10: "中断", 11: "腰斩", 12: "取消", 13: "待定" }
if (textObj[resState]) {
  li_one.appendTo("#haved .ul-list")
  li_one.find(".status").html(textObj[resState])
}

Answer 2

判断较多条件单一情况 效率肯定是switch case 较强
写法简洁优化可以考虑 object结构 map结构
优化角度考虑问题
https://segmentfault.com/a/11...

Answer 3

先不说switch，为什么li_one.appendTo("#haved .ul-list");不提出来放到外面。。。

Answer 4

搬运工：JavaScript 复杂判断的更优雅写法

Answer 5

newMenthods (resState) {
    const dataList = {
        8: '完场',
        9: '推迟',
        10: '中断',
        11: '腰斩'
    }
    li_one.appendTo("#haved .ul-list")
    li_one.find(".status").html(dataList[resState])
}

Answer 6

执行效率没什么问题，主要是问题是代码缺乏可读性，比如说8，9，10这些太死板。当然可以也优化下常说的优雅问题

Answer 7

var STATE_MAP = {
  8: '完场',
  9: '推迟',
  10: '中断',
  11: '腰斩',
  12: '取消',
  13: '待定'
}

var stateLabel = STATE_MAP[resState]
            
if (stateLabel) {
  li_one.appendTo("#haved .ul-list");
  li_one.find(".status").html(stateLabel);
}

Answer 8

表驱动就可以 可以看下代码大全 表驱动模块介绍。其楼上几个用的就是

Answer 9

这么多判断影响效率吧

Answer 10

jquery该放弃了..走向vue和react的怀抱吧

Answer 11

你这也太丑了