这样的代码要进行重构吗？

Question

代码如下： if(DeviceID==10)
{
    //do something;
}
if(DeviceID==20)
{
    //do something;
}
if(DeviceID==30)
{
    //do something;
}
if(DeviceID==40)
{
    //do something;
}
if(DeviceID==50)
{
    //do something;
}
if(DeviceID==60)
{
    //do something;
}
if(DeviceID==70)
{
    //do something;
}
if(DeviceID==80)
{
    //do something;
}
if(DeviceID==90)
{
    //do something;
}
if(DeviceID==100)
{
    //do something;
}
个人观点：1、代码的功能一目了然   2、太占行数了,修改不便

Answer 1

重不重构 不敢说，但是你的代码至少也得 加上else if 啊 ，每次 进来都要判断每个if条件 ~~

Answer 2

你的//do something;里做的啥事？

Answer 3

没必要重构。虽然改成switch-case会显得规范点。一般来说：只要已经是稳定运行的东西，就不要去浪费时间修改。难保不准改多了改出问题。
如果是要对原有模块进行性能或功能上的整体优化，那才有重构的必要性。

Answer 4

另外如果dosomthing内容很多，可以抽象出去调用，不然你这方法太大了

Answer 5

是应该改称 if(...){} else if(...){}.用switch刻度行更高,不过效率上和else if没有差别.实际工作中,多一事不如少一事,运行的好好的,动它干嘛? 楼上的代码，加上改称else if，心里爽了，实际上也感觉不到效率提升，呵呵....不顾，自己写出这样代码，会觉得比较丢脸