遇到朋友提交的这种代码该怎么办

[aprikyblue]

受朋友委托，然后跟另一个朋友一起开发的，
一起开发的这个，之前 2 年没联系，

结果 ........
看完我一句也不想说

[tomine]

把你自己的代码贴上来看看

[NovemberEleven]

期待我就是那个朋友系列

[cqcn1991]

这个 if else 还能怎么写？不是挺正常么…

[TakanashiAzusa]

@cqcn1991 这里不应该是 switch case 么。。

[Banio]

是想说 if ( 1 == i ) 这样写才合理么？

[TakanashiAzusa]

其实我比较好奇为什么是“朋友”提交的代码。。

[harry890829]

switch case 和 if...else if 不就是写法上不一样么，还有什么不同么

[JamesRuan]

典型的数据和逻辑分离有问题的设计。

[qinglangee]

我一直觉得会开发外挂的人都很牛 X

[cqcn1991]

@TakanashiAzusa 可以，但是我觉得只是语言风格，并不算 code smell.

[zhicheng]

命名 OK ，缩进 OK ，代码风格 OK ，注释 OK 。唯一的缺点只有 magic number 。但不是正规则大项目也无所谓。
在我看来，这些代码段几乎是 perfect 了。

[learnshare]

其实应该把这么多数据放在合理的数据结构里，否则以后改到哭

[akira]

图像识别的时候，不要严格依赖颜色，这个颜色在不同电脑上是有可能不一样的。。

[LuoboTixS]

槽点是在 hardcode 像素位置吗？

[tiange]

简单明了

[jianghu52]

有缩进，命名正常，注释不少。还要啥自行车。

[fo2w]

@harry890829
a[0] = 0;
a[1] = 1;
a[2] = 2;
...
a[999] = 999;
好像和写个循环也就是写法上有点区别, 有什么不同么

[cst4you]

@akira A 卡 N 卡 2333333
N 卡压缩颜色的作弊技巧从 Geforce 开始到现在都还在用呢.

[blacktulip]

居然有注释，这已经是很给面子了，楼主还想要怎么样的代码？

[moe3000]

我觉的还是说几句吧，是说编码规范么还是那一大推 if ，还是其他？

@NovemberEleven 我也期待这个系列 doge 脸