过早客
  • 首页
  • 节点
  • 成员
  • 广告投放
  • 登录
  • 注册

同事这个代码什么水平

IT技术 • charliecw84 • 发表于 1 年前 • 最后回复来自 woyouzui • 1 年前

if (container.has(x)) {
return true;
}
return false;

加入收藏 新浪微博 分享到微信 ❤赞 4617 次点击 0 人赞 1 人收藏

打开微信“扫一扫”,打开网页后点击屏幕右上角分享按钮

共收到39条回复
lujunx 1 年前 湖北省 #1 赞 14

可读性非常强,老司机水平

eventloop 1 年前 湖北省 #2 赞 2

你就说能不能跑吧? 跑的对不对吧?

czz_ls 1 年前 湖北省 #3 赞 0

这个container连个对象都没有,确定能跑,这是个错误代码

lingjing88 1 年前 湖北省 #4 赞 4

没毛病,可以跑,反而是那些一大堆的 lambda匿名方法 .Where().Select(). 看的人头疼,又不好调试

shang53880 1 年前 湖北省 #5 赞 2

一行就可以搞定的,一定要多写几行,不然怎么能体现工作量呢

hajkmw 1 年前 湖北省 #6 赞 1

我觉得这个代码到还真没啥问题

coast1986 1 年前 湖北省 #7 赞 6

return container.has(x);
less is more,不过如果KPI里有代码行数要求,当我没说。

bawanglong 1 年前 湖北省 #8 赞 5

如果代码能跑,而且不涉及性能问题,完全不用改,怎么写都行,越复杂你越重要

andbreak 1 年前 湖北省 #9 赞 1

写的非常好,让Ta写

sanmaoguo 1 年前 湖北省 #10 赞 1

没啥问题,ide会给出优化建议

shadow_2019 1 年前 北京市 #11 赞 1

有些公司用代码行数统计代码量,然后计算产能的

letiankaimen 1 年前 湖北省 #12 赞 0

我记得关于注释的第一要义是代码本身就是注释。

jaysonguo 1 年前 湖北省 #13 赞 0

可读性很强 不一定越少越好

kiwivip 1 年前 湖北省 #14 赞 0

就像吃热干面,好吃而且不拉肚子就行了,你管他是好看还是不好看呢

dengwoxi001 1 年前 湖北省 #15 赞 0

挺好的,没毛病。
写个return 后面跟一大坨判断式脑子还要转转到底各个场景返回的是啥。

yellowbug 1 年前 湖北省 #16 赞 0

看了一堆回复,也不难理解我们的it水平为什么落后别人一大截

fhehdwdnv 1 年前 湖北省 #17 赞 1

感觉这个没啥问题,方便以后加别的代码。谁知道后面又会有什么需求,导致返回结果之前还需要别的操作?

sueandedward 1 年前 上海市 #18 赞 0

没毛病啊

zhangyu123 1 年前 江苏省 #19 赞 0

挺好,业务代码就是要在不影响性能情况下,写的越容易理解越好
当然,要是故意不想让人看懂另说

Phony 1 年前 湖北省 #20 赞 0

现在都喜欢统计代码量,这样没毛病

moses 1 年前 湖北省 #21 赞 0

if (container.has(x)) {
return true;
} else {
return false;
}

这样可读性更好

zespri 1 年前 湖北省 #22 赞 0

如果是考虑到后期需要扩展,为处时需要做点事情,这样写没啥问题。否则不就是一行代码搞定

kaka3511 1 年前 湖北省 #23 赞 0

boolean flag = container.has(x);
if (true == flag ) {
return true;
}
if (false == flag ) {
return false;
}

这样拓展性更好。

pipi 1 年前 湖北省 #24 赞 0

直接返回 bool

xxxxxy 1 年前 湖北省 #25 赞 2

return container.has(x) ? true : false;
这样可读性差吗?

dark 1 年前 湖北省 #26 赞 0

专家水平。

Jiucai 1 年前 湖北省 #27 赞 1

@xxxxxy 你这才是。。。。

pass9wen33 1 年前 湖北省 #28 赞 0

通俗易懂,没毛病

benjw 1 年前 湖北省 #29 赞 0

这种是强行凑代码行数KPI的搞法,而且有点不伦不类
这种风格基本上在 java 里面 sonar 都要归结为坏味道
很简单的一行代码就能清晰理解的写法不用:

return container.has(x);

非要搞些半 if (即没有 else)部分来凑

BusyFox 1 年前 缅甸 #30 赞 0

有没有可能原本return true之前有其他的事情要做,现在删了 ?

tacoTuesday 1 年前 湖北省 #31 赞 0

return container.has(x);

Aoo 1 年前 湖北省 #32 赞 0

断章取义的水平

hexaislqb 1 年前 湖北省 #33 赞 0

@moses else 不加更好

tracymcladdy 1 年前 湖北省 #34 赞 0

教育起别人来,个个写代码像写诗。
自己写起来,能跑就行了。

Dirk 1 年前 广东省 #35 赞 0

你就说这代码清不清晰吧

jasontsai 1 年前 广东省 #36 赞 0

@kaka3511 java style

zclzhangcl 1 年前 上海市 #37 赞 0

见多了这种代码

phynos 1 年前 湖北省 #38 赞 1

没毛病,要是计较这个代码的,我才感觉有毛病,起码太小气了
包括上面某个人,这都能扯到 it 水平,也是他妈醉了

woyouzui 1 年前 湖北省 #39 赞 0

上面的container of,变量只要定义一次,却可以被n多不同实例引用,架构复杂但实现灵活。uboot里面把对象直接固化到程序固定section的,完全别想动态扩展,但程序的可控性最大化。两种截然不同的风格,挺有意思的。

请绑定手机号后,再发言,点击此处
Guozaoke.com—源自武汉的高端交流分享社区
相关主题
有多少人知道华为在AI算力的基础建设上已经超越英伟达了?
EDA断供
最近时间少了, 隔了这么久, 才做了一个网站
求推荐一个AI智能体客服
Android手机推荐
请教机器学习人工智能的一个技术问题
搞了个AI 生图的网站,不需要登录,永久免费
[第二波送码]动动嘴皮,AI秒出图!2025最炸裂图标神器:免费生成+手机实时预览
我没房子的时候,每年赚的钱比不上房子每年涨的钱。等我咬牙买了,每年赚的钱,比不上房子每年亏的钱
开发了一个管理 Linux 服务器的桌面可视化管理工具,求蹂躏

过早客微信公众号:guozaoke • 过早客新浪微博:@过早客 • 广告投放合作微信:fullygroup50 鄂ICP备2021016276号-2 • 鄂公网安备42018502001446号