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

[LeroyMooney]

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

楼下好像蛮多人指责我的
1 、关于标题中的“外包”一词：并没有歧视的意思，跟公司其他外包同事相处的很愉快。标题中的“外包”是不是戳着某些人伤口了？
2 、关于说我没有当面指出的：这是我跟他说了两次之后修改的版本

[Leonard]

哈哈哈哈哈哈

[quan01994]

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

[legiorange]

？

[mazai]

脱裤子放屁，多此一举😂

[Guidoo]

哈哈 这代码。。 强行封装

[totoro52]

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

[bzw875]

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

[liuxey]

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

这样还能说得过去一点

[zhuangzhuang1988]

正常啊，这代码

[wxsm]

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

[huage2580]

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

[murmur]

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

[CalistaK]

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

[shiji]

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

[jwenjian]

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

[AA5DE3F034ACCB9E]

学到了

[zhuweiyou]

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

[liudaolunhuibl]

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

[wangwei1025]

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

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

[zhuweiyou]

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

[codiy]

牛逼了

[ily433664]

if (true) {
xxx
}

[anonydmer]

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

[vicalloy]

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

[kop1989]

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

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

[drydiy]

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

[viator42]

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

[66beta]

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

[Felldeadbird]

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

[lagoon]

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

[sutra]

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

[yamasa]

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

[CodeGroup]

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

[arthas2234]

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

[lonelymarried]

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

[LeroyMooney]

我随便再贴几行吧
```
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]

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

[LeroyMooney]

其他太长的就不贴出来了

[lwlizhe]

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

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

好像圆不下去~~~

[liudaolunhuibl]

@sutra 对 哦，忘记了

[rocksolid]

这是按代码量算工资么

[liudaolunhuibl]

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

[xinyang5302677]

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

[zzzmh]

你给他改成

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

互相伤害
[doge]

[xuanbg]

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

[atonku]

不然一天一行代码也没有

[sevenzhou1218]

个人觉得没问题 getter?

[no1xsyzy]

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

——

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

[zhongrs232]

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

[endlessJ219]

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

[xuanbg]

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

[SurfaceView]

这代码有 bug 吗

[limuyan44]

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

[lvxiaomao]

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

[Azer]

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

[sonxzjw]

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

[msg7086]

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

[SyncWorld]

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

[fengpan567]

又不是不能用！！

[lskjdfgl]

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

[mahone3297]

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

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

[ljzxloaf]

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

[webcoder]

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

[dengji85]

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

[q9OxQg]

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

[celeron533]

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

[daimen]

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

[bleepbloop]

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

[thtznet]

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

[tinyuu]

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

[liuxey]

@anonydmer #23
@liudaolunhuibl #18

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

[LiubaiQ]

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

[daimen]

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

[iceneet]

？？？迷惑主义

[cat007]

没有 bug 的代码就是好代码

[jianpeng957]

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

[QRan]

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

[UBcai]

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

[wennew]

这么闲吗，我从来不看别人代码，如果你是他上司就让他 checkstyle 下呗

[VictorJing94]

字母数计 kpi?

[treizeor]

很正常啊，定义私有变量 issueStatistics，然后编辑器生成的 setter/getter 基本都是这样

[Blanke]

应该是插件自动做的处理

[MineDog]

楼上说这么写正常的都是认真的？这明显是无效代码啊

[zhoulifu]

面向代码量编程

[sjzzz]

有什么理由要进入这个方法，为啥不直接拿。这个代码的价值在哪？多了 3 行代码么？

[0x663]

@66beta
低情商：脱裤子放屁的封装方法
高情商：留下可以扩展的口子，是个高手！

[Oktfolio]

见的多了，转岗过来半年了吧，他的工资应该至少是我的两倍。

https://sm.ms/image/LDUw4SofCHXZ9Bi
https://sm.ms/image/qrhiKj19LI6MmG4
https://sm.ms/image/y2dx15UNubhknvf

[Oktfolio]

他们写的代码全是飘黄的，感觉 IDE 的提示对他们来说根本没有用

[idealhs]

有问题吗

[tairan2006]

说没问题的建议转行

[Asuka0947]

这行代码会影响到你吗？会牵连到你吗？如果没有，大可不必管它，打工人没必要看不起打工人

[Sydney7106]

@ily433664 曾经有那么一段时间， 喜欢这样写. 只是为了分隔代码又不想提成方法~~

[hodur]

@sjzzz 阿里的朋友?

[fenghengzhi]

@vicalloy 应该只是想深拷贝一下吧

[treizeor]

@sjzzz 私有变量，外部必须通过 public 方法拿有问题吗

[Godikov]

刚入行时也经常瞧不上别人代码，时间久了这种代码都不会让我觉得蠢，甚至觉得可爱。

[LeroyMooney]

@drydiy
@endlessJ219
@limuyan44
@daimen
@wennew
统一回复一下，不包括 pojo 和 mapper 层，600 多行代码写了一星期，规范啥的先不提了，出现了 5 个问题，匿了

[janus77]

可能第一个 null 以前不是 null 是一个空 list，后面需求变动，为了方便就把他改成 null 了。问题不大

[leavic]

这 tm 写成 if else 还能多几行呢。

[lozzow]

原地 TP