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

[LeroyMooney]

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

[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