我至今仍感到羞愧的代码

[d0x0b]

最近看到一篇文章写的是为自己写的代码感到羞愧，原文如下： https://www.freecodecamp.org/news/the-code-im-still-ashamed-of-e4c021dff55e/ 红迪讨论： https://www.reddit.com/r/programming/comments/5d56fo/the_code_im_still_ashamed_of/ 好奇大伙写过哪些自认为感到羞愧/恶心/骄傲的代码？一些大学食堂给每月低消费同学补贴这种算良心代码吗？

[yuhuai]

羞愧的代码肯定是设计错误难于重构的，typo 这种，开源项目一搜 git log 能有一堆的 fix ，大家都忙起来的时候忽略是很正常的

[yuhuai]

大多数情况下，错误命名，无效三元这种东西,ide 都可以帮忙批量纠正了

[dingdangnao]

let that = this
我感觉我自己写的代码都挺烂的😂能凑合用就行了
毕竟我也不是程序员

[Richared]

@atwoodSoInterest 我居然忘了 result 是不是这么写了。

[yuhuai]

我的几段恶心的代码都来自于我的懒惰，两个都是因为场景尚不复杂
第一个案例是，我知道应该用构造器模式重写，但是因为没有时间加暂时没有更多需求，选择了写一个重载
第二个案例时入行没多久，尚不熟悉设计模式时，在一个公共类函数中写了一个简单的 switch 用来分化策略，直到很久之后我再看到这个函数，才发现需求膨胀比我想象的要快得多

[DOLLOR]

想起我很久以前写过一个公共组件，叫做 pubicXXX ，
后来的同事也有样学样，用类似的方式这样命名 pubicAAA ，pubicBBB ，pubicCCC……
等项目大了之后才发现问题，但已经积重难返了

[thetbw]

if(a == true){
return true;
}else{
return false;
}

[thetbw]

@dingdangnao 这个不有点用吗，function 上下文不一样呀

[EspoirBao]

@reoah2 可不止如此哦，某个城市现在已经部署了大范围可以检测人脸的摄像头，可以以脸身搜脸，以证搜脸，开车的可以把连续好几天甚至几个月你经过哪些卡口的信息记录下来，是所有 zh 人的脸！至于哪些人可以使用该系统...

[yxisenx]

公司有一个快递下单的系统，会生成一个单号，这段代码是我几年前刚转行入职的时候写的，前面是日期，最后六位是一个随机数，我写的是，随机一个数字，检查有无使用，有使用过，继续随机, 没有使用过, 就使用这个数字, 如果一直都重复, 就会一直尝试......还好公司单量不多, 这代码也稳定运行了几年了，还没出问题

[hertzry]

def forword (self, input):

[root71370]

mysql 前缀查询百分号写到前边了
code like '%str'

[dode]

@yxisenx 现在该怎么写呢

[guanzhangzhang]

我同事的代码，就一个 groovy 里传递一个布尔值给 shell ，原本是
```
sh "xxx.sh arg1=${v1} arg2=${v2}"
```

然后他写法是：
```
if (cache=="true") {
sh "xxx.sh arg1=${v1} arg2=${v2} cache=true"
} else {
sh "xxx.sh arg1=${v1} arg2=${v2} cache=false"
}
```
类似的代码他写了好多次，说了就是屡教不改和找借口🙂

[yxisenx]

@dode #33 没改，公司每天就几百单快递，基本没影响

[jipaidian]

@thetbw 我也写过，被部门老大狠狠地检视了一番