code-review看到这样的if else 你怕了吗

151 阅读1分钟

上周五,我们进行了code-review,对,没想到,我们这个组有这个环节,我对这个环节充满了期待,因为我觉得这是一次可以很好的学习机会。不仅仅是了解业务,也更是技术的交流。

but,有点失望,我不做吐槽。我觉得,对于能改变的我尽力推动,改变不了的,怪我无力....我只需要保持住我的风格,继续勇往直前。不要被“世俗”所打平!

同事的可怕套娃逻辑,上截图

image.png

不好意思,我反手直接改了,真的看着很难受啊,if else else if的,头大啊,啊啊啊....

image.png

思想如下:

const action = {
  '1': ['processing','ProPage'],
  '2': ['fail','FailPage'],
  '3': ['fail','FailPage'],
  '4': ['success','SuccessPage'],
  '5': ['cancel','CancelPage'],
  'default': ['other','Index'],
}

const handleEvent = (status)=>{
  let action = action[status] || actions['default'],
     
}