看到这样的代码，你想不想杀人！

难道不能用switch吗

来自酷炫的 CNodeMD

小叮当看了想打人

666

这是煞笔写的吧？开了

哈哈还给你添了注释挺给面子了

代码已经表达了心情，全是-> : (

好看啊 多美

稍微修改一下，你会发现这样写看起来超舒服

const sg = data.content.strGoods;
const storesMainName = sg ? (
  sg === '01' ? '数码产品（手机、电脑、相机）' :
  sg === '02' ? '车（电瓶车、摩托车、自行车）' :
  sg === '03' ? '家电' :
  sg === '04' ? '摄影' :
  sg === '05' ? '珠宝' :
  sg === '06' ? '驾校' :
  sg === '07' ? '其他' :
  sg === '08' ? '医疗美容' :
  sg === '09' ? '车险' : ''
) : '';

比switch还舒服

附解法二

const sgs = {
  '01': '数码产品（手机、电脑、相机）',
  '02': '车（电瓶车、摩托车、自行车）',
  '03': '家电',
  '04': '摄影',
  '05': '珠宝',
  '06': '驾校',
  '07': '其他',
  '08': '医疗美容',
  '09': '车险',
};

const storesMainName = sgs[data.content.strGoods] || '';

这说明，面试的时候，问问“数据结构”还是有用的。

说实话我之前最开始参加工作的时候写过 现在回想看我当时确实蛮弱智的 虽然现在也不强 但至少在进步

明显就是智商欠费不够用，要补

面试咋过的

From PWA CNode

有注释已经很好了。。。我接手的代码很多都是没有注释的，命名喜欢用a b c d e f ....

这种代码看到就有强烈的重构欲...

@dislido 我就是写法2的人。2333333.能用写成配置的，绝不写死。

看起来有一种蜜汁美感，理解起来也不困难。 学到了。

@dislido 还是解法二看着舒服

来自酷炫的 CNodeMD

上面的代码还是可以看的， 那种套好多次循环的，然后判断多才不好读。

杀人是不好的，我反手就是一本《代码大全》

确实这种问题就应该表驱动，switch都不太好

switch？感觉最好是弄个 Map，说白了一个对象搞定。

From PWA CNode

为什么不能用map

var map = {
	'01' :'数码产品（手机、电脑、相机）',
	'02' : '车（电瓶车、摩托车、自行车）' ,
	.....
};
var sg = data.content.strGoods;
var name = map[sg];