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

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

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

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

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

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

所以说之后评审代码这种事是我完全没有勇气去做的,里面放眼望去基本都是 Yin 间代码。
15863 次点击
所在节点    程序员
144 条回复
kalluwa
2021-01-29 10:15:28 +08:00
函数内几百行,几千行都不是什么问题。问题是你这个同事压根就不想好好做,换句话说,养老也得装个好的态度,情商低无疑
no1xsyzy
2021-01-29 10:27:39 +08:00
@Vveeb “精致的利己主义者”,既不精致,也不利己,更不是个人(“者” = XXX 的人)。(伏尔泰点了个赞并 @ 神圣罗马帝国)
lp7631010
2021-01-29 11:05:57 +08:00
人家都离职了 跟你费这劲呢
heaton_nobu
2021-01-29 15:17:21 +08:00
@Vveeb #111 你:“我还真看不懂,您受累给我讲讲。”

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

https://tanronggui.xyz/t/748956

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

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

© 2021 V2EX