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

186 replies • 2021-03-22 08:56:52 +08:00

1 2

❮

❯

Leonard

Mar 19, 2021

1

哈哈哈哈哈哈

quan01994

Mar 19, 2021

7

2

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

legiorange

Mar 19, 2021 via Android

3

？

mazai

Mar 19, 2021

4

脱裤子放屁，多此一举😂

Guidoo

Mar 19, 2021

5

哈哈这代码。。强行封装

MIUIOS

Mar 19, 2021

6

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

bzw875

Mar 19, 2021

7

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

auin

Mar 19, 2021

8

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

这样还能说得过去一点

zhuangzhuang1988

Mar 19, 2021 via Android

9

正常啊，这代码

wxsm

Mar 19, 2021

10

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

huage2580

Mar 19, 2021

11

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

murmur

Mar 19, 2021

12

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

CalistaK

Mar 19, 2021

13

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

shiji

Mar 19, 2021

4

14

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

jwenjian

Mar 19, 2021 via iPhone

15

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

AA5DE3F034ACCB9E

Mar 19, 2021

16

学到了

zhuweiyou

Mar 19, 2021

3

17

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

liudaolunhuibl

Mar 19, 2021

2

18

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

wangwei1025

Mar 19, 2021

19

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

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

zhuweiyou

Mar 19, 2021

20

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

codiy

Mar 19, 2021

21

牛逼了

lovelive1024

Mar 19, 2021

22

if (true) {
xxx
}

anonydmer

Mar 19, 2021

11

23

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

vicalloy

Mar 19, 2021

24

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

kop1989

Mar 19, 2021

25

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

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

drydiy

Mar 19, 2021

1

26

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

viator42

Mar 19, 2021

27

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

66beta

Mar 19, 2021

16

28

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

Felldeadbird

Mar 19, 2021

29

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

lagoon

Mar 19, 2021

30

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

sutra

Mar 19, 2021

31

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

yamasa

Mar 19, 2021

32

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

CodeGroup

Mar 19, 2021

33

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

arthas2234

Mar 19, 2021

34

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

lonelymarried

Mar 19, 2021

35

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

LeroyMooney

OP

Mar 19, 2021

36

我随便再贴几行吧
```
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) + "%");
}
```

LeroyMooney

OP

Mar 19, 2021

37

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

LeroyMooney

OP

Mar 19, 2021

38

其他太长的就不贴出来了

lwlizhe

Mar 19, 2021

39

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

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

好像圆不下去~~~

liudaolunhuibl

Mar 19, 2021

40

@sutra 对哦，忘记了

rocksolid

Mar 19, 2021

41

这是按代码量算工资么

liudaolunhuibl

Mar 19, 2021

42

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

xinyang5302677

Mar 19, 2021

43

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

zzzmh

Mar 19, 2021

5

44

你给他改成

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

互相伤害
[doge]

xuanbg

Mar 19, 2021

45

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

atonku

Mar 19, 2021

46

不然一天一行代码也没有

sevenzhou1218

Mar 19, 2021

47

个人觉得没问题 getter?

no1xsyzy

Mar 19, 2021

48

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

——

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

zhongrs232

Mar 19, 2021

2

49

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

endlessJ219

Mar 19, 2021

50

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

xuanbg

Mar 19, 2021

51

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

SurfaceView

Mar 19, 2021

52

这代码有 bug 吗

limuyan44

Mar 19, 2021

13

53

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

lvxiaomao

Mar 19, 2021

54

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

Azer

Mar 19, 2021

55

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

sonxzjw

Mar 19, 2021

56

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

msg7086

Mar 19, 2021

57

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

SyncWorld

Mar 19, 2021

58

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

fengpan567

Mar 19, 2021

59

又不是不能用！！

lskjdfgl

Mar 19, 2021

60

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

mahone3297

Mar 19, 2021

1

61

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

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

ljzxloaf

Mar 19, 2021

62

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

webcoder

Mar 19, 2021

1

63

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

yangwa

Mar 19, 2021

64

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

q9OxQg

Mar 19, 2021 via Android

65

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

celeron533

Mar 19, 2021

66

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

daimen

Mar 19, 2021

67

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

bleepbloop

Mar 19, 2021

68

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

thtznet

Mar 19, 2021

69

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

tinyuu

Mar 19, 2021

2

70

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

auin

Mar 19, 2021

71

@anonydmer #23
@liudaolunhuibl #18

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

LiubaiQ

Mar 19, 2021

72

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

daimen

Mar 19, 2021

73

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

iceneet

Mar 19, 2021

74

？？？迷惑主义

cat007

Mar 19, 2021

75

没有 bug 的代码就是好代码

jianpeng957

Mar 19, 2021

76

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

QRan

Mar 19, 2021

77

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

floki

Mar 19, 2021

78

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