五六年的程序员写出这种代码
图片

评论

最新评论(106

前端大师傅
05/10 15:53  
一般我会定义一个“”空字符串的常量及空引用常量池。而不是直接用字面量。 List<object> empties = new List<ojbect>{"",null}; if(empties.contain(l2.getName()); 当然C#里可以写得更优雅就是用?问号空接 l2?.getAdName()
胡萝卜炒肉
05/10 13:42  
@人民程序员 比较空字符串 O(n) 这里的n不就是1吗....
老菜鸟0217
05/10 11:54  
你太年轻
SoSoYouth
05/10 10:21  
就截图里面的代码来看,没毛病啊
光头卫士
05/10 10:17  
为何不用StringUtils.isNotEmpty(l2.getAdName)呢?引个包可能比较困难吧?不知道,咱也不敢问呐
笨笨小喵
05/10 09:52  
l2不为null的话没有任何毛病
爱吃猫的鱼0313
05/10 08:37  
我都是调用stringUtils包
人民程序员
05/09 19:24  
equals复杂度是O(n), 比较length是否大于0就行了(O(1))
狐狸大仙
05/09 19:10  
没看出啥问题。。。
张亦俊
05/09 16:32  
让他写kotlin去
毛笔写代码
05/09 16:26  
嘲笑别人半天,发现自己错了。。。呵呵哒
田Instance
05/09 16:04  
代码有bug, 而且有明显的更优雅的写法。
星辰点点
05/09 15:54  
变量名起的清晰脱俗, 用12 叫变量名, 213不服
codebuster
05/09 15:07  
@啸歌舞剑 &&已经避免了空指针的问题了
色魔张大妈
05/09 15:01  
楼主图样
胡建洲
05/09 14:51  
总有人觉得自己很厉害,就算是十年的,代码该怎么就怎么写
九千里
05/09 14:33  
如果l2不为空 ,这代码是不会出错的.如果l2是空才有可能会出错,不过用StringUtils的确比较好, 另外if 后面加个空格, 括号里面两边再加个空格会好看点
不吃肉夹馍 码云六周年
05/09 14:13  
我都是用 StringUtils 不过这么写我觉得也可以啊, 问题在哪呢?

没有更多内容

加载失败,请刷新页面

返回顶部
顶部