【神代码】下午review看到的一段经典代码

[china (0x80🛡️)]

这有个老实人

[rimutuyuan]

嘻嘻

[Keyman]

哈哈，我也没想到有什么问题，看了评论才知道答案，看来我不是java的料

[kzhsw]

dto.getSearch().intern() != ""

[ruoxijun (始皇与成都九男七日那些事儿)]

这里用 Optional 可读性差，还是用 StringUtils.hasText 之类的工具类来判断好点。

[ikunkun (一起打篮球吗🏀)]

我觉得没啥问题啊。。。应该怎么写

[nitan]

dto.getSearch().length > 0

[asdf24678]

mybatis的xml写多了

[Kenneth_Yu (YUhome)]

大概率是你没付费。一般付费就会显示了

[flyhub123 (F0RK)]

内心毫无波澜，早在我司的代码里见识过类似的超神代码， if 与 else 里分别有大几百行代码，且两个分支里只有几行代码不一样。

[kotlef (花开花落)]

都是人才啊，你们

[uupp (tempvalue)]

有啥问题吗？C++看没问题哈，还是说数据量很大，这个if语句是防御编程，命中这个条件概率很低，他这么写判断的过程太多导致时间会很长？？ 求解答啊佬

[dfg]

解释一下 @little

[icbc]

调用了两次 dto.getSearch 性能开销问题吧

[little (A4纸杀手)]

乍一看感觉没啥问题，我java学得烂

[East_Duan (东东)]

if (dto.getSearch() != null && !dto.getSearch().equals(“”)) {
// 此处省略一万行
}

[clg (白色风车)]

if dto.getSearch() != “”{} 直接这样？

[mufanGuo (Jadon😈)]

好好好，null 都不判断了

[mrloong]

正确答案：

//此处省略一万行代码

[Aliyun]

python写多了只感觉写得太冗余了，差点没发现问题，建议让他看看自己能不能找到问题