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

資深大佬 : LeroyMooney 4

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

大佬有話說 (100)

資深大佬 : 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 行代码么？
資深大佬 : liushaokang

@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