这样的代码要进行重构吗？
代码如下：
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、太占行数了,修改不便
[解决办法]
代码的重构，也有很多问题需要考虑，比如时间，比如人力，比如技术支持，比如……太多的决定因素，既然是公司的代码，要充分考虑重构的成本
[解决办法]
没必要重构。虽然改成switch-case会显得规范点。
一般来说：只要已经是稳定运行的东西，就不要去浪费时间修改。难保不准改多了改出问题。
如果是要对原有模块进行性能或功能上的整体优化，那才有重构的必要性。
[解决办法]
是应该改称 if(...){} else if(...){}.用switch刻度行更高,不过效率上和else if没有差别.
实际工作中,多一事不如少一事,运行的好好的,动它干嘛?
楼上的代码，加上改称else if，心里爽了，实际上也感觉不到效率提升，呵呵....
不顾，自己写出这样代码，会觉得比较丢脸
