大家工作中都遇到过哪些神奇的代码

代码

��

JSON

Obj

88 条回复 • 2019-08-26 16:02:55 +08:00

1

nikandaoleshenme

2019-08-23 12:07:02 +08:00

24

对不起，没碰到过，周边的同事写代码都流弊，代码有注释，文档清晰，源代码有版本追朔，每周 codeview，有 checkstyle，作为一个螺丝钉码农，随时可以被替换，

没啥存在感，所以最近准备换工作，换到 lz 说的这样的环境中去锻炼锻炼

2

xujinkai

OP

2019-08-23 12:15:36 +08:00 via Android

1

@nikandaoleshenme 别闹🤣

3

fenddddddda

2019-08-23 12:18:09 +08:00

2

bool isCan
if(isCan) {

}
else if(!isCan) {
}
我前几天刚看到的。

4

cabing

2019-08-23 12:18:56 +08:00

@nikandaoleshenme 规范的大厂哦~ =。=

5

cabing

2019-08-23 12:19:49 +08:00

你这还行吧。
你能想象一个类 1w 多行？

6

nikandaoleshenme

2019-08-23 12:32:48 +08:00

@xujinkai 想到了一句话，成功的结果只有一个，失败的理由千千万万（没记清楚，大概这个意思 {表情}）
拿到我们码农来说，就是，规范的代码就一种，不规范的千千万万（手动滑稽）

7

tigerfyj

2019-08-23 12:35:22 +08:00 via Android

@cabing 10 年时候进的一个外企，一个系统 50 多个类，平均 12k 行，最多的三个都是 18k，少的也都 7，8k 行。当时修 bug 真的是爽到了。

8

kxy

2019-08-23 13:46:07 +08:00

不知道咋想的，变量能命名成这样，大写的服：
Map<XXX，XXX> SOME_ANOTHER_MAP = new HashMap<>();
Map<String,Integer> Map = new HashMap<>();

9

pipixia

2019-08-23 14:05:58 +08:00 via Android

@fenddddddda 我还真见过人都傻了

10

Orenoid

2019-08-23 14:09:19 +08:00

昨天还在一篇博客看到这种代码：
if someCondition:
return True
else:
return False

11

trustbutverify

2019-08-23 14:10:16 +08:00 via iPhone

@Orenoid 可以很严谨

12

firefffffffffly

2019-08-23 14:13:15 +08:00

for(int i=0;i<3;i++) {
switch(i) {
case 0:
doSth();
break;
case 1:
doSth2();
break;
case 2:
doSth3();
break;
}
}

13

v2hh

2019-08-23 14:13:44 +08:00

@Orenoid 可以，逻辑很清晰

14

argc

2019-08-23 14:14:36 +08:00

bool isCan
if(isCan==true) {

}
看到许多人这样写觉得有点神奇。。

15

augustheart

2019-08-23 14:14:50 +08:00

看到命名那块我以为是在说我……

16

augustheart

2019-08-23 14:16:08 +08:00

@Orenoid 这种代码有个好处是方便下断点：）

17

lc1450

2019-08-23 14:20:37 +08:00

1

第一次见到这么用 rollback 的

record = XX.query.get(id)
if record:
db.session.delete(record)
else:
db.session.rollback()

18

Lin0936

2019-08-23 14:26:27 +08:00

var someObj = someBool ? false : true

19

Justin13

2019-08-23 14:32:48 +08:00 via Android

const key = someList.find(item => item.key === FOO).key

20

no1xsyzy

2019-08-23 14:47:48 +08:00

@Orenoid 可以处理为真但不是真本身的值，尤其如果某个语言没有提供恰当的任意值的布尔转换的内置函数的情况下。
不过哪个语言没有呢？不然写个 bool_cast 也可以的

21

Michaelssss

2019-08-23 14:52:33 +08:00

@Orenoid。。。。。。我这里的一个新手就爱这么写，我都懒得说他了。。

22

drafter

2019-08-23 14:55:24 +08:00

楼主说的...和我们公司好像哈哈哈哈

23

taotaodaddy

2019-08-23 14:56:07 +08:00 via Android

@Orenoid 又严谨，行数又多，满足按代码行数 kpi 要求

24

Orenoid

2019-08-23 15:05:31 +08:00

@no1xsyzy #20 他那个 someCondition 就是个百分之一百的 bool 值。。

25

autoxbc

2019-08-23 15:08:46 +08:00

对于局部变量，当定义过程和调用过程紧邻的时候，简化命名是可以接受甚至是推荐的

26

nanmian

2019-08-23 15:43:10 +08:00

i = i++; 交接的时候前人交给我的，至今没想明白他要干什么.

27

TencentCEO

2019-08-23 16:32:06 +08:00

看到过一段过滤非法字符的，其实也就是把 ASCII 码值是从 100 到 255 的替换成空格。
string s;
遍历字符串里的每个字节，把每个字节再跟 100 到 255 每个对比一下……大概是这样的

foreach( char c in s)
{

for(char a=100; a<256; a++)
{

if(c==a) {c=' ';}

}

}

大家看出这段代码的问题了吗？只要把里面那层循环替换成

if(c>100) {c=' ';}

就行了……

28

TencentCEO

2019-08-23 16:34:02 +08:00

@TencentCEO 而且我们这段代码是出现在大量数据提取时候用的，将近百万条的数据，每条数据有好几个字段要被这样过滤……

29

5yyy

2019-08-23 16:35:00 +08:00

CTO 很注重代码质量，刚入职的时候被教育了好多次ಥ_ಥ

30

nealwx

2019-08-23 16:38:52 +08:00

我有一个同事专写各种奇怪的代码，只管是否能编过，测试没问题即可。

31

shadeofgod

2019-08-23 17:20:17 +08:00

return a <= 0 ? true : false;

32

virus94

2019-08-23 17:21:56 +08:00

ob_start();
ob_clean ();
然后就没了

33

MMMMMMMMMMMMMMMM

2019-08-23 17:26:37 +08:00

34

wangchonglie

2019-08-23 17:36:29 +08:00

@nanmian #26 堆砌代码量[滑稽]

35

111qqz

2019-08-23 17:46:47 +08:00

```
if(label < 0 || label > 4)
{
exit(233);
}
```

36

Cyshall

2019-08-23 19:07:51 +08:00 via iPhone

@111qqz 一时没看懂，这是要表达啥…，0-4 的值都不要？

37

way2create

2019-08-23 19:10:25 +08:00

我变量有时也会命名 data 的就表示是数据有什么不妥吗？

38

ys759206502

2019-08-23 19:10:36 +08:00 via Android

系统设计成访问到 404 就强制退出系统，然后注销登录按钮索性调了个不存在的接口

39

hmxxmh

2019-08-23 19:27:02 +08:00 via Android

@way2create 😂对啊倒是说为什么不规范啊

40

monkeylyf

2019-08-23 19:31:45 +08:00

final boolean isValid = true;

...

if (isVaild == true) { // do something. }

41

inwar

2019-08-23 19:32:44 +08:00 via Android

@Orenoid 可能前后都是一样的形式复制粘贴,只不过恰好这个是布尔

42

loveToMy1

2019-08-23 19:44:22 +08:00 via Android

哈哈我写的都是不规范的代码😅

43

sudri

2019-08-23 20:03:08 +08:00

boolean bool = a == b ? true : false;

44

cabing

2019-08-23 20:04:07 +08:00

@tigerfyj 这些坨垃圾代码通常没有单元测试。因为没法测试。业务逻辑耦合验证，很容易牵一发动全身。

最要命的是业务需求还在堆积。。

45

learningman

2019-08-23 21:56:57 +08:00 via Android

@nanmian 可能是觉得直接 i++不优雅

46

ddup

2019-08-23 22:02:03 +08:00 via Android

某人写了一个 12MB 的 SQL 存储过程

47

xuanbg

2019-08-23 22:11:26 +08:00

我也是经常给变量命名为 data、json、obj。。。反正只是方法内使用一次，搞个变量不过是为了不至于一行代码太长不好看，也方便调试的时候有变量可以看到值罢了。

48

Varobjs

2019-08-23 22:35:29 +08:00 via Android

有，就是我
var obj = varobj.com
/逃

49

ffeii

2019-08-23 22:45:05 +08:00

滥竽充数

50

90d0n

2019-08-23 23:00:27 +08:00

某央企建设集团的项目, 全集团在用
整个项目里一个实体类都没有, 全靠 Map<String, Object>.
没交接, 没文档, 没注释, 维护起来真爽

51

dvaknheo

2019-08-23 23:00:50 +08:00

@cabing web MVC 的 Controller 不都是长这么长么 (逃

都怪你们说 MVC，业务处理要单独放一层，Controller 层只是作为外部参数处理的啊

52

Xbluer

2019-08-23 23:05:29 +08:00

1

总的来说，代码写得好的都在刷 V2，而写的不好的都在哼次哼次写代码。。。

53

doublie

2019-08-23 23:11:27 +08:00 via iPhone

有用过 phplist 的同学么

54

111qqz

2019-08-23 23:28:14 +08:00 via Android

@Cyshall 认为合理值是[0，4] 槽点是没有任何 log 直接 exit … 接手的时候找了好久

55

icekingcy

2019-08-24 00:06:43 +08:00 via iPhone

时间排序法

56

ooee2016

2019-08-24 00:12:07 +08:00 via iPhone

string s1, string s2, string s3, string s4...

57

chairuosen

2019-08-24 00:48:29 +08:00

mvc，c 里面改 v 的状态

58

jakezh

2019-08-24 03:13:34 +08:00

@111qqz #54 不是有 233 嘛全局搜索打断点就是了

59

cabing

2019-08-24 06:16:42 +08:00

@dvaknheo 写过单元测试就不会这么长啊。这么长根本没法测试。。条件都没法覆盖到。

60

wangsongyan

2019-08-24 07:10:00 +08:00 via iPhone

@nanmian 单纯为了留个坑

61

printese

2019-08-24 07:41:54 +08:00

int a = 0, b = 0;
if (属于第一类别) {
a |= 1;
}
if (属于第二类别) {
b |= 1;
}

string str = a.ToString()+b.ToString();
if (str.equals("00")) {
//不属于第一类别也不属于第二类别
}
if (str.equals("01")) {
//属于第二类别但不属于第一类别
}
if (str.equals("10")) {
//属于第一类别但不属于第二类别
}
if (str.equals("11")) {
//属于第一类别也属于第二类别
}

62

gavindexu

2019-08-24 07:51:36 +08:00 via iPhone

在我司数据库中看到了一个 dajevu

63

unicloud

2019-08-24 07:51:43 +08:00 via iPhone

为了给下次优化提供条件，写成这样：
...
doSomething();
Thread.sleep(3000);
...

64

GeruzoniAnsasu

2019-08-24 07:52:54 +08:00 via Android

不觉得每次这个话题回复都差不多吗

无非是没用的 if else，变量名，“超动态”类型

http://siliconframework.org/
来看看这种真的挺有意思的代码

65

tab16360

2019-08-24 08:05:31 +08:00 via Android

审一个工作 3 年 javaer 的代码，登录方法命名为 tuichu()😓

66

ys0290

2019-08-24 08:17:12 +08:00 via iPhone

简单的 if else 可能是为了以后扩充方便

67

ethanSong

2019-08-24 09:10:17 +08:00

@argc 估计是 C/C++写多了?

68

aitaii

2019-08-24 09:17:04 +08:00 via iPhone

java web 功能，没有定义实体对象，前后台传值用数组。

69

hoyixi

2019-08-24 09:19:24 +08:00

领导眼里，能快速的上线，跑起来就行了，至于维护、后期重构，可以靠 996 加班解决，反正涌入这行的小年轻多的是，低价+加班就行，写出来的质量本来也没指望

70

aaronhua

2019-08-24 09:20:45 +08:00

中文变量、过程写法 1800+行

71

sherlockwhitebb

2019-08-24 09:21:09 +08:00

function car_id($car,$No,$No_totle){
$tomorrow=strtotime(date('Y-m-d 23:59:59'));
$No=0;//今天第几个注册
$No_totle=0;//总数
。。。
}
这是我这辈子看过最秀的

72

dvaknheo

2019-08-24 09:32:01 +08:00

@cabing 哈，单元测试什么鬼，产品定任务，做完测试测 OK 上线，上线完。
“本来好好的，你把他动坏了，你说该怎么办吧”

73

hewelzei

2019-08-24 09:35:16 +08:00

```javascript
if (objExp.test(url) == true) return true;
else return false;
```
严谨到完美无缺

74

cabing

2019-08-24 09:37:16 +08:00

@dvaknheo 弄坏了，你就是背锅侠。。

75

piaochen0

2019-08-24 09:42:11 +08:00

我之前接手了一个项目，大部分变量名都在二十个字以上，基本上就是一句话...
还有另外一个非常资深的哥们，写的代码自带混淆，所有的命名都是 a1,a2,a3 这样...写游戏引擎的大牛...关键问题是，别人都完全看不懂他的代码，他自己隔了好几个月后，立马就能看懂。

76

xdtr

2019-08-24 09:46:00 +08:00

int v1 = 0，v2 = 0;
(v1 v2 被赋值)
string s = v1.ToString() + v2.ToString();
if(s == "1122"){...}
else if(s == "3344"){...}
...

77

lonelygo

2019-08-24 10:14:08 +08:00

莫名其妙的 if else 大概率是先写着占位，准备以后扩展，但是，没等到以后就跑路了。

78

qwertyegg

2019-08-24 10:31:12 +08:00

do{
***
}while(true)

79

xy90321

2019-08-24 11:13:08 +08:00 via iPhone

@Orenoid
如果是业务逻辑封装那完全不会觉得奇怪

80

Orenoid

2019-08-24 11:18:55 +08:00 via Android

@xy90321 主要是那篇文章里头就是个小的不能再小的 demo 代码，我放的基本就是文里的原代码了。。

81

111qqz

2019-08-24 17:27:08 +08:00 via Android

@jakezh 那是看到了代码才知道有 233 啊……

82

sumika

2019-08-24 18:23:05 +08:00

```javascript
var detailsValue1 = "";
var detailsValue2 = "";
var detailsValue3 = "";
var detailsValue4 = "";
var detailsValue5 = "";
var detailsValue6 = "";
var detailsValue7 = "";
var detailsValue8 = "";
var detailsValue9 = "";
var detailsValue10 = "";
var detailsValue11 = "";
var detailsValue12 = "";
```

以上代码是我从项目里复制出来的

83

EscYezi

2019-08-24 22:43:27 +08:00 via iPhone

最近进了一个项目，说说其中的两个槽点：

1. 出现拼音+英文混合命名，比如 ListOfYeMian （页面）

2.连着六七行声明 List<Object> xxList
然后里面存 Integer ……

剩下的命名风格不一致、Typo、没有注释、没有文档等等就不说了......听老员工说这项目换了好几批人🌚

84

EscYezi

2019-08-24 22:44:23 +08:00 via iPhone

@fenddddddda #3 这个我也遇到了🌚

85

EscYezi

2019-08-24 22:47:59 +08:00 via iPhone

@90d0n #50 Java 当成 PHP 写🤣

86

linvaux

2019-08-24 22:53:47 +08:00

for i in xxx:
for j in xxx:
for k in xxx:
for t in xxxx:
xxxx
别笑，真有人这么写的，毫无意义的变量名

87

aguesuka

2019-08-25 10:04:42 +08:00 via Android

int number0=0;这样的代码写到了 47

88

NDMXXSMX

2019-08-26 16:02:55 +08:00

@sumika 笑死我了，魔鬼数字啊