新来的外包同事，这代码，我？？?

186 条回复 • 2021-03-22 08:56:52 +08:00

1 2

❮

❯

1

Leonard

2021-03-19 09:25:45 +08:00

哈哈哈哈哈哈

2

quan01994

2021-03-19 09:27:03 +08:00

7

别问，问就是按代码量计费的。

3

legiorange

2021-03-19 09:27:16 +08:00 via Android

？

4

mazai

2021-03-19 09:27:16 +08:00

脱裤子放屁，多此一举😂

5

Guidoo

2021-03-19 09:27:21 +08:00

哈哈这代码。。强行封装

6

totoro52

2021-03-19 09:27:41 +08:00

笑死我了这就是脱裤子放屁啊

7

bzw875

2021-03-19 09:28:38 +08:00

可能是不知不觉摸了 1 天鱼，随便写几行代码提交吧

8

liuxey

2021-03-19 09:28:58 +08:00

public List<IssueStatistics> getIssueStatistics() {
return issueStatistics == null ? new ArrayList<>() : issueStatistics;
}

这样还能说得过去一点

9

zhuangzhuang1988

2021-03-19 09:29:35 +08:00 via Android

正常啊，这代码

10

wxsm

2021-03-19 09:30:39 +08:00

谁都有脑抽的时候，如果只是一行还好

11

huage2580

2021-03-19 09:31:07 +08:00

问一下他，怎么写出这么秀的代码的

12

murmur

2021-03-19 09:31:17 +08:00

getter/setter 而已。函数的封装并没有问题

13

CalistaK

2021-03-19 09:31:26 +08:00

这代码不报错呀，逻辑上好像也没问题（手动狗头

14

shiji

2021-03-19 09:31:45 +08:00

4

有的时候也许 null 的返回值还没有沟通好，先这么放着，等定下来了直接改。

15

jwenjian

2021-03-19 09:32:40 +08:00 via iPhone

反正也会被 JVM 优化掉，不影响😁

16

AA5DE3F034ACCB9E

2021-03-19 09:32:51 +08:00

学到了

17

zhuweiyou

2021-03-19 09:33:50 +08:00

3

有可能之前还有别的条件和代码, 然后删掉了, 只剩下这一行.

18

liudaolunhuibl

2021-03-19 09:34:21 +08:00

2

@liuxey 1 、新建 ArrayList 的时候最好指定容量，这里就表示一个空的所以 new Arraylsit （ 1 ）就可以了，2 、new 一个 arrayList 的时候最好静态方法，用谷歌的 guava：Lists.newArrayListwithCatity(1)

19

wangwei1025

2021-03-19 09:35:06 +08:00

我还写过
if (condition){
A=B;
}else{
A=B
}

哈哈哈哈，有时候确实会脑子抽了

20

zhuweiyou

2021-03-19 09:35:28 +08:00

比如 return issueStatistics == null || issueStatistics.length == 0 ? null : issueStatistics;
后来不需要 issueStatistics.length == 0 了, 只删掉了一部分. 忘记改成返回了

21

codiy

2021-03-19 09:35:31 +08:00

牛逼了

22

ily433664

2021-03-19 09:38:56 +08:00

if (true) {
xxx
}

23

anonydmer

2021-03-19 09:39:13 +08:00

11

@liuxey @liudaolunhuibl 我们一直是要求 null 时候返回 Collections.emptyList(), 这个方法返回的是不可变对象，无法向里面写内容；不然返回一个 ArryList() 别人往里面写内容了可能会引起难以排查的 bug

24

vicalloy

2021-03-19 09:39:49 +08:00

和这个比起来怎么样
#!/bin/python3
def toJSON(self):
　 xxx = []
　...
　 return json.loads(json.dumps(xxx))

25

kop1989

2021-03-19 09:42:05 +08:00

get/set 在默认情况下本来也只是返回 this.xxx ，逻辑上我个人认为并没有错误。
而且更可能的情况下是这块代码有过改动或者未来有预计的改动，所以和 ide 的默认写法不同。

这就像是：同样表达去哪玩，有的人说“你去 xxx 玩”，而有的人说“你来 xxx 玩”。这俩表达的意思一致，只不过是表达者主体的情景不同。

26

drydiy

2021-03-19 09:42:24 +08:00

1

本来外包就处于鄙视链的底端，你这还强调一下。欣慰得是，评论区都很很友善。
我在现同事的代码中也看到这种脱裤子放屁的赋值，我默默得修复了。
工作中难免会有脑子短路的时刻。

27

viator42

2021-03-19 09:42:46 +08:00

代码写的还挺讲究，虽然没啥用

28

66beta

2021-03-19 09:43:49 +08:00

16

留下可以扩展的口子，是个高手！

29

Felldeadbird

2021-03-19 09:44:21 +08:00

不明白这什么有的？写代码总有脑抽的时候。
至少得再列出别得。

30

lagoon

2021-03-19 09:44:42 +08:00

考虑到我自己也经常写垃圾代码，再考虑到这人的垃圾代码写的还挺工整。
我决定默不作声。

31

sutra

2021-03-19 09:45:01 +08:00

@liudaolunhuibl 不应该是 Collections.emptyList() 吗？哈哈

32

yamasa

2021-03-19 09:46:49 +08:00

如果想规避 npe，直接 Optional.ofNullable(issueStatistics).orElse(new ArrayList<>(1)); 就好了。

33

CodeGroup

2021-03-19 09:48:41 +08:00

这算什么，我这还有 list.size()>=0 的判断，我寻思数组的长度也没有负的吧

34

arthas2234

2021-03-19 09:49:39 +08:00

@shiji +1
有可能是还不确定要如何处理，先原路返回了，在这里先留个位置等后面在改
不过应该要打个 TODO，怕后面忘记了

35

lonelymarried

2021-03-19 09:50:11 +08:00

这是写 oc 写多了，怕 null 崩溃所以判断一下？

36

LeroyMooney

OP

2021-03-19 09:50:18 +08:00

我随便再贴几行吧
```
private String nodeName;
private int issueTotal ;
private int countersign;

xxService.execl(projectNum);

xxService.selectid();

public String s(int a, int b) {
return (Math.round((((a - b) * 1.0f) / a) * 100) + "%");
}

public String D(int a, int b) {
return (Math.round(a * 1.0f / b * 100) + "%");
}
```

37

LeroyMooney

OP

2021-03-19 09:51:34 +08:00

回复没用到 Markdown，大家将就看下吧

38

LeroyMooney

OP

2021-03-19 09:53:43 +08:00

其他太长的就不贴出来了

39

lwlizhe

2021-03-19 09:54:00 +08:00

容老夫想想怎么圆这块……

或许这块是参考双重检测的单例模式呢，只不过少了关键字和双重检测而已

好像圆不下去~~~

40

liudaolunhuibl

2021-03-19 09:54:34 +08:00

@sutra 对哦，忘记了

41

rocksolid

2021-03-19 09:55:54 +08:00

这是按代码量算工资么

42

liudaolunhuibl

2021-03-19 09:56:13 +08:00

@anonydmer 但是可能这个方法返回的 list 外部还要继续 add 呢，难免会有这种业务的，或者说现在没有后面加上了，然后哪个人没有看这个方法的代码不就报错了吗，

43

xinyang5302677

2021-03-19 09:56:33 +08:00

你怎么戾气这么大？你是 985 吗？

44

zzzmh

2021-03-19 09:57:34 +08:00

5

你给他改成

public List<IssueStatistics> getIssueStatistics() {
return issueStatistics == null ? issueStatistics == null ? issueStatistics == null ? issueStatistics == null ? issueStatistics == null ? null : issueStatistics : issueStatistics : issueStatistics : issueStatistics : issueStatistics;
}

互相伤害
[doge]

45

xuanbg

2021-03-19 09:57:44 +08:00

有可能不是手写的，沙雕生成器生成的。

46

atonku

2021-03-19 09:58:27 +08:00

不然一天一行代码也没有

47

sevenzhou1218

2021-03-19 10:00:03 +08:00

个人觉得没问题 getter?

48

no1xsyzy

2021-03-19 10:00:09 +08:00

@anonydmer 那我就随便地不负责任地推荐一下 Haskell：想改？ unsafe 吧。

——

说起来，也可能存在显式表示此处可返回 null 的情况

49

zhongrs232

2021-03-19 10:01:04 +08:00

2

这代码有个解释：如果我和女票意见一致，那么听我的，否则听女票的
然而我并没有女票，sigh~

50

endlessJ219

2021-03-19 10:01:25 +08:00

感觉楼主有点攻击，这个人水平不高不能赖外包啊

51

xuanbg

2021-03-19 10:01:25 +08:00

@LeroyMooney 这！！！ s d 实锤了……

52

SurfaceView

2021-03-19 10:02:04 +08:00

这代码有 bug 吗

53

limuyan44

2021-03-19 10:03:00 +08:00

13

我刚毕业的时候就喜欢吐槽别人代码，后来我发现所有人的代码里都有垃圾包括我自己，就淡然多了，没有一个程序员的代码经得起吐槽，如果谁觉得有，请把 github 贴出来让大家看看。

54

lvxiaomao

2021-03-19 10:06:47 +08:00

你就知足吧，起码不是
if (list == null) {return null;} else {return list;}

55

Azer

2021-03-19 10:08:14 +08:00

看过一个
if(checkbox.checked == true && i == 1)

56

sonxzjw

2021-03-19 10:10:36 +08:00

要是我，可以的话我会加 100 个空行。那我的代码量行数就蹭蹭蹭...想想就美

57

msg7086

2021-03-19 10:12:47 +08:00

我见过 if (IsXXX() == true) { return true; } else { return false; } 的。

58

SyncWorld

2021-03-19 10:13:46 +08:00

@limuyan44 垃圾代码谁都写过~ 我现在回头改我以前的代码都得看半天

59

fengpan567

2021-03-19 10:15:19 +08:00

又不是不能用！！

60

lskjdfgl

2021-03-19 10:15:34 +08:00

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

61

mahone3297

2021-03-19 10:17:05 +08:00

1

好简洁的写法。。。应该这样写

```
public List<IssueStatistics> getIssueStatistics() {
// 如果为 null，则返回 null
if (issueStatistics == null) {
return null;
} else {
// 如果不为 null，则返回本来的值
return issueStatistics;
}
}
```

62

ljzxloaf

2021-03-19 10:20:46 +08:00

pojo 不都这样吗？如果不是 pojo，他之前可能是写 c/c++的

63

webcoder

2021-03-19 10:21:48 +08:00

1

@SyncWorld 回去看半年前的代码，这是哪个傻 x 写的垃圾代码，哦是我啊，那没事了。

64

dengji85

2021-03-19 10:23:58 +08:00

真的高深莫测。你们这些凡人看不懂吧？还笑别人

65

q9OxQg

2021-03-19 10:24:57 +08:00 via Android

我不懂编程，不知道大家笑什么。但是我也还是笑了

66

celeron533

2021-03-19 10:27:45 +08:00

可能是为了未来的扩展性（逃

67

daimen

2021-03-19 10:27:57 +08:00

不出 bug 的代码就是好代码，再说了，编译器会优化的

68

bleepbloop

2021-03-19 10:28:55 +08:00

也许是被资本家剥削得太狠，精神状态不大好呢？

69

thtznet

2021-03-19 10:29:02 +08:00

从经济学角度看，这个社会大部分的问题其本质都是经济问题

70

tinyuu

2021-03-19 10:29:04 +08:00

2

挺好的逻辑清晰，封装良好，设计充分，重构方便，拥抱未来。

71

liuxey

2021-03-19 10:29:49 +08:00

@anonydmer #23
@liudaolunhuibl #18

感谢提醒，写惯了 immutable 语言，Java 都快成伪代码了

72

LiubaiQ

2021-03-19 10:30:58 +08:00

@daimen 代码不规范，用不了半年就难以维护了。。。

73

daimen

2021-03-19 10:33:03 +08:00

@LiubaiQ 楼主就是来找相对于外包的优越感的，一两行代码能看出来个啥，我也是搞不懂了。

74

iceneet

2021-03-19 10:33:04 +08:00

？？？迷惑主义

75

cat007

2021-03-19 10:33:43 +08:00

没有 bug 的代码就是好代码

76

jianpeng957

2021-03-19 10:37:36 +08:00

有种为了炫技而炫技的感觉

77

QRan

2021-03-19 10:39:13 +08:00

@lagoon 我也是这样想的，我自己菜我也就不发言了 ^_^哈哈哈

78

UBcai

2021-03-19 10:39:40 +08:00

贴的第一个代码，就是按照我这里需求来说，有些 list 是要返回 null，有些需要 new arraylist<>()(看了上面大佬的写法，我准备改为 Collections.emptyList()),这样写就需要看前端需要什么了，然后好改。贴的第二个代码估计我太菜才一年多 java，我还是没看到哪里有问题？各位大佬有知道的给我分析分析么。