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

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

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

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

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

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

所以说之后评审代码这种事是我完全没有勇气去做的,里面放眼望去基本都是 Yin 间代码。
15861 次点击
所在节点    程序员
144 条回复
impanghu
2021-01-27 22:40:53 +08:00
哈哈哈哈哈哈哈哈哈哈
bclerdx
2021-01-27 22:43:42 +08:00
LZ,你不要把你的想法和方式强加给她人吧。
dayeye2006199
2021-01-28 02:51:07 +08:00
这种都不需要人工 review,搞个 linter,要求对方先改到没有 warning 你再给她看逻辑就好了
laminux29
2021-01-28 07:14:57 +08:00
这个妹子,应该是那种曾经真实水平只是星耀,但以前的版本允许有运气能上王者,然后她上了王者,就骄傲了。后来版本变化,让运气变得无效,要靠真正实力才能上王者,妹子怎么也回不去王者段位,懵逼了,然后就暴躁了。

你要想解开她的心结,得让她明白,她掉段且上不去的真正原因,是她从头就没有王者的真正水平,而且当时游戏公司也犯了错误,让她这种人上了王者。不过你对一个成年人这么说,大概率的结局是,会不听你的,然后记恨于你。

你作为中层,应该对公司负责,把精力集中在工作上,而不是搞内斗。我如果是你,会先给她一个机会,直接告诉她,你是她的上级,你交代的任务,她必须无条件完成。她可以提建议,但听不听建议是你的事情,她不能因为你不听她的建议而反对你。接着,如果她仍然反对你,你就直接找上级,或者找 HR,把这妹子调离你的部门,原因很简单,就一条,员工不服从领导安排。
hpu423
2021-01-28 08:01:40 +08:00
标题很长
cominghome
2021-01-28 08:23:04 +08:00
草,看这标题想起了改论文的时候,初稿改格式都改了四五版。。。
Vveeb
2021-01-28 08:27:59 +08:00
@css3 之前装了个方法复杂度的插件,默认配置下,复杂度超过 3 就标红了。一个方法体,之前看过有人说他都控制在 10 行以内。但是我觉得要是一直追求这样的书写方法,应该会挺累的。
ragnaroks
2021-01-28 08:42:20 +08:00
培训班都没有这么离谱
Vveeb
2021-01-28 08:47:24 +08:00
@bclerdx 因为她的代码的设计和逻辑这块我是根本不敢审的,看都看不懂。清一色 public void 方法。所以只能抓一抓这种初高中生都能理解的大小写&拼写问题,而且这种问题上报到领导,领导也能懂。总不能从头给领导讲清楚.net 开发,然后再告诉领导她哪哪哪的代码根本就不符合规范吧~。
Marszm
2021-01-28 08:54:15 +08:00
羡慕能找到这么舒服的工作.....对比下自己,自己简直是劳模.
sigl0p
2021-01-28 08:59:30 +08:00
Code Metrices 插件,提示得丑了一点。
meepo3927
2021-01-28 09:08:42 +08:00
截个图让大家瞧瞧
peapods
2021-01-28 09:14:13 +08:00
为什么这样的人会做程序员
no1xsyzy
2021-01-28 09:17:33 +08:00
后续,搞快点 gkd
其实可以从系统理论挑几句话,也能向外行说明不对劲的地方,因为系统理论是跨学科的,经管方面也会重新发现一遍的(查一般系统论,第二个就是 MBA 智库……)
可维护性问题多半从属于耗散结构论
RlhholylightR
2021-01-28 09:20:05 +08:00
我能说我们使用手机号码注册获取验证码前要先调用校验接口校验该手机号码是否注册过吗
Bzero
2021-01-28 09:20:24 +08:00
身边就有一例,真的让人很难受。自己从来没主动过,对代码没有追求,用好吃懒做一点不过分。责备的时候,还顶嘴,不知自己哪里有问题。
xrr2016
2021-01-28 09:23:15 +08:00
@lemon94 策略模式
cheng6563
2021-01-28 09:25:04 +08:00
@cco 老大说了:把所有功能丢到一个接口里取,跟第三方宣传的时候就可以说“一个接口实现 XXX 功能”
msy45
2021-01-28 09:25:38 +08:00
看了这日系标题和 1 楼,这就是轻小说吧😂
Vveeb
2021-01-28 09:27:56 +08:00
@peapods 其实是个很拧巴的事儿,本来就一个测试部门,完了非要揽开发的活儿。因为我本科是软件专业的,就先给我派去学技术了,学成龟·来之后,再带俩人,其中一个就是这女的,据说是也有一定基础,之前做过开发,还是研究生,然后.....就这?就这?就这????

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

https://tanronggui.xyz/t/748956

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

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

© 2021 V2EX