《关于我对审核同事代码的标准已经降低到“只要把所有成员命名按照驼峰规则+单词拼写没有错误就行了”然而她还是做不到的这件事》

2021-01-27 16:21:08 +08:00
 Vveeb
前提: .NET 开发,不是正规开发团队,帮国外分公司的开发打杂的,人家写好的框架我们在上面实现个小功能或者修 BUG 啥的。
本人水平也只是熟练使用 Console.WriteLine() 的水平,但是差不多也能看出来代码写的漂亮不漂亮。

OK,正文:组里的毒瘤同事终于要走了,走之前要给她之前的工作交接给我。因为这些成果最终还是要提交给国外开发团队去 Merge 的,所以之前需要我评审一下,尽量符合人家的标准。

拿过来先看她做的类图,基本都是“望文生疑”这么个状态。我让她:“这里,这里,方法名单词首字母大写,这里拼写不对,你回去改一下,然后从头到尾自己查一下,没问题了再给我”。

过会儿,告诉我改好了没问题了,我随便看,还有同样问题,又让她改+自己查。

就这样来回来去三四遍了,一个类图,里面总共列出来才不到 200 个成员,每次说改好了没问题了,我都还能发现“大小写”&“拼写”问题。

所以说之后评审代码这种事是我完全没有勇气去做的,里面放眼望去基本都是 Yin 间代码。
15863 次点击
所在节点    程序员
144 条回复
fuwenpei
2021-01-28 09:28:37 +08:00
好家伙,老二刺螈了。
Vveeb
2021-01-28 09:29:07 +08:00
@xrr2016 对对对,想起来了!是有这么个东西~
LiuJiang
2021-01-28 09:29:59 +08:00
我把这样的人,统一称为"老黄牛"
morimi2026
2021-01-28 09:31:10 +08:00
这其实没法沟通,所以正确做法是不要去指出,水过就好
tutustream
2021-01-28 09:31:32 +08:00
低情商:什么鬼标题


高情商:有没有考虑改行去写轻小说
Vveeb
2021-01-28 09:35:39 +08:00
@no1xsyzy 大佬所言全是我知识盲区.....
后续的话,应该是没有了,领导也没招,所以工作内容就直接拿过来了不审了。反正不是特别重要的模块,到时候如果真的有需要往国外开发的仓库提 Pull Request 的话,就直接重做就完了。

但是 《前传》的话,我估计真的能写上一本书了,共事一年多,一些惊天地泣鬼神的事迹我都用小本本记下来了。也成了日后跟朋友、同事闲暇之余的一段笑谈了属于是。
LokiSharp
2021-01-28 09:36:37 +08:00
用过好点的 IDE 都能解决啊,买个 Rider 和 ReSharper
Vveeb
2021-01-28 09:38:01 +08:00
@morimi2026 是的是的! 最后就是“闭一只眼瞎一只眼” 过去了。
Vveeb
2021-01-28 09:39:23 +08:00
@tutustream 低情商: 一个方法写了 400 行,100 多的复杂度,回去给我改!

高情商:您的这个 AI 算法写得真不错,能让我学习一下么?
aoeui
2021-01-28 09:40:10 +08:00
我只想问,🐻大么? 🐶
Vveeb
2021-01-28 09:44:53 +08:00
@aoeui 要是🐻大我至于在这里吐槽? 那我必然是去手把手去教她改大小写啊 🐶
Shazoo
2021-01-28 09:49:41 +08:00
好奇,什么是“方法复杂度”?时间复杂度倒是经常得评估下。

什么叫做 400 行 ,100 多的复杂度?这是什么量化标准?
sockpuppet9527
2021-01-28 09:52:00 +08:00
我有个好办法,公费让她去波兰一趟就好了。 参考 : /t/693941
有可能波兰老哥都看了两行,都不给她 code review 了。 :)
dswyzx
2021-01-28 10:01:15 +08:00
啊,我就喜欢类里边属性一股脑小写开头,问就是 json 直接粘贴而成。warning? 又不是不能用
不过我支持楼主这么认真的对待规范
Vveeb
2021-01-28 10:11:44 +08:00
@Shazoo 可能我用词不 biu 准哈,就是那种,多一个 if else 分支就会+1 的那种复杂度。量化标准好像没什么标准,开头说了“不算正规开发团队、差不多就是打杂的”。 最后代码还是要提 PR 给外国团队审核的,但是 400 多行的一个方法,里面基本都是 if else switch,并且注释都没有, 本身做的还是写 byte bit 的操作,条件里全是 “0,1,2,3,4”的。一般人多看两眼就吐了我觉得
JavaFirstMaster
2021-01-28 10:19:28 +08:00
@Vveeb 这个人一看就是对代码没追求, 能力和责任心不强, 但是自尊心强. 你指出她错误的时候她应该是知道有问题的, 就是嘴硬+懒而已. 让领导下场是正确的做法.

哪里都有垃圾代码, 我现在参与的项目里经常见到"不判断 null", "if true return true else false"的代码. 以前我是个敲两行代码就要格式化一下的人, 现在也是见怪不怪了, 因为改出来问题了人家还埋怨你...
Vveeb
2021-01-28 10:33:36 +08:00
@dswyzx 嗐,其实本身我也就是个“少儿班”的水平,我们的工作也没什么高深的东西,所以有点强迫症的我,大部分之间都在格式化代码+想命名+写注释。甚至会照着 NET 官方英文文档的格式来写我的方法的 XML 文档注释 + 遣词造句。
Jasonwxy
2021-01-28 10:46:02 +08:00
深有同感,前一段时间我们这也来了个大神,不知道怎么招过来的。啥问题都要问,说了问题也不改,一个问题过一段时间问一次,之后我都直接翻聊天记录截图给他,反正跟他合作过的都苦不堪言。还好不久前走了,现在在收拾他留下的烂摊子
感觉这种人就是没有责任+没有能力,除非哪天突然改了性子,别人没法改变他。跟这种人合作是真的难受,我到后期反正是不怎么理这个人了,该怎么办怎么办吧,最好早点滚蛋。这种人不适合去职场跟别人沟通合作,太自我
heaton_nobu
2021-01-28 10:49:27 +08:00
你指着那个 400 行的 function 让她给你讲讲里面逻辑
goodboy95
2021-01-28 10:52:51 +08:00
@Shazoo 这个方法复杂度,更专业的叫法是圈复杂度

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://tanronggui.xyz/t/748956

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX