程序员“迷惑代码”大赏

谈到程序员，对于外行人来说一贯的印象就是格子衫大裤衩外加人字拖，蓬头（秃头）垢面黑眼圈，还有就是“人傻钱多死得快”，这是外界对程序员固有的思想，但是作为新世纪的程序员，我们可没那么邋遢，白衬衫西装裤黑皮鞋，整洁的发型，这才是新一代码农

那对于我们内行人来说呢每天接触最多的就是代码，代码是程序员的一生挚友，一入代码深似海，所以代码也是评判一个程序员水平的标准。代码的简洁程度，API使用，代码逻辑，注释等都是评定代码的好坏...代码让人看了舒服就是好代码。

但是不同的程序员在职业生涯的不同阶段可能会写出不同水平程度的代码，今天我们就来欣赏一下几段令人迷惑的代码，无关品评，博你一笑，有则改之无则加勉。

No.1

这段代码从redis获取一个set数据类型的集合，拿到所有集合元素smembers，然后迭代，使用一个int型的count计数器记录元素个数，然后返回。

乍一看，这段代码没什么问题，对吧？仔细一看，就发现这段代码这样写好傻啊！

image

最明显的就是使用迭代的方式计算元素个数，怕是不知道java中set集合有一个叫做size()的方法吧，直接就能返回元素个数啦。然后呐？还有没有问题，熟悉redis的读者都知道redis的set数据类型也有自己的API啊，我们想获得元素个数直接jedis.scard(key)就OK啦。晕，既然用redis了，怎么不想着把redis的基本使用搞搞清楚呢。加油啊，老弟

No.2

再来看下第二段代码，这段代码的大致意思是记录某个业务功能的每月的用户访问量，首先接收一个前端上送的月份值，拼接成一个key，然后获取当前用户的用户ID，把用户ID存到redis的set数据类型中（自动去重），然后再通过scard返回用户访问量（在被告知有scard这个命令后学会用啦）。

image

乍一看，这段代码没什么问题吧，该用的API都用了，再也不是傻傻的循环统计用户数了。没那么简单吧，把业务逻辑先和产品经理确认清楚吧老弟。相同的月份每年都有，第二年怎么办呢？月度访问量一直都能查看吗还是过一段时间看不了之前月份的了？这些都是要确认清楚的。

No.3

继续看第三段代码，大致意思是要从 itemList 中剔除掉一些元素，哪些元素呢，就是元素的 code 属性不在 codeSet 中的元素，通过两个for循环，首先遍历itemList取出每个元素，拿出来元素的code属性再遍历 codeSet ，判断相同时把元素添加到新的list中 。

image

。。。内层循环没必要，codeSet.contains(code);一个API就搞定了，而且判空什么的也没加。

image

上面三个是本人亲身经历，公司新来的同事（两个多月了）写完代码让我进行代码检查发现了几个比较低级的错误，来时经理告诉我这是高级人员让我带带，代码写的可不高级啊。

No.4

第四段这个吧，之前在网上流传过一段时间，说是去面试做的笔试题，题目是对给定的int数组进行排序。

image

这样写的哥们脑洞真的大，关键是还实现了

image

哈哈。。。不知是段子还是真的。。

No.5

第五段是前端的HTML，这段代码有一种哲学在里面。逻辑毫无漏洞，为什么说我代码不行。兄弟应对客户提出的【清缓存】的需求，怪不得你写代码快呢。

image

哈哈，欺负客户不懂代码吗？

image

No.6

image

这段代码思路也是异常清奇，大佬是在佩服！

image

No.7

还有人说没有人在注释里对话吗？比如一个人说:

//这一段是谁写的？

然后另一个人在下面接:

//是我，怎么了

然后故事就开始了…

image

No.8

听女朋友讲，他们公司有一女开发，做个一签到送积分的活动，写了俩接口给前端调，一个签到接口，一个加积分的接口，签到成功后告诉前端成功了，前端再调加积分的接口。

image

END

项目组中各个开发的水平不一，看到“迷惑”的代码大家也不要惊慌，如果知道是谁写的就偷偷告诉他吧。你还知道哪些搞笑代码呢，留言评论吧。

以上就是今天的全部内容了，本文无关品评，博你一笑，有则改之无则加勉。希望各位都写出优质代码，走向人生巅峰！！

---

往期热文：

• Redis的各种数据类型到底能玩出什么花儿？

• Redis使用指南

• Redis分布式锁实战

• 联合索引在B+树上的存储结构及数据查找方式

• MySQL索引那些事

• Redis缓存穿透，缓存击穿，缓存雪崩，热点Key

欢迎关注，谢谢支持

image