我的代码写的不好,收集烂代码不是为了嘲讽,而是为了提醒自己,需要写出这种被人嘲讽的代码。

2016年05月17日

1
2
3
4
5
6
7
8
9
...
$status = $work['work_status'];
if($status == '0001' || $status == '0010'){
}else if( $status == '0011' ||$status == '0100' || $status == '0101' ||
$status == '0110' || $status == '0111' ||$status == '1000' ||
$status == '1001') {
}else if($status == '1010'){
}
...

搜索了全部代码,没看到常量定义,有的只是0011这种代码。。。。更气人的是,这个字段在数据库中:

1
`work_status` varchar(5) DEFAULT NULL COMMENT '工单状态',

明明都用varchar了,还用0010是什么鬼。。。