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

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

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

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

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

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

所以说之后评审代码这种事是我完全没有勇气去做的,里面放眼望去基本都是 Yin 间代码。
15863 次点击
所在节点    程序员
144 条回复
YoRolling
2021-01-28 10:57:57 +08:00
你同事逛 v2 么
LittleYangYang
2021-01-28 11:00:31 +08:00
歪个楼,突然想到了

《关于我对审核同事文档的标准已经降低到“只要别一段正文有三个字体、两个底色、三个字号、一堆错别字、大纲级别乱标、乱写标点符号、缩进混乱就行了”然而她还是做不到的这件事》
tojonozomi
2021-01-28 11:17:52 +08:00
好家伙,看得我的低血压都治好了
《傻逼女同事想让我血压飙升——代码规范编码风格大战》
hideinbush
2021-01-28 11:31:49 +08:00
@doveyoung 《身为男子程序员的我,被年上女同事嘴炮缠斗这件事》
preach
2021-01-28 11:34:13 +08:00
大部分人的工作只是为了养家糊口
northisland
2021-01-28 11:49:03 +08:00
没啥行动上的建议。。。

我更在意实现逻辑通顺 + 接口不 2B 。

####

工具上的建议:
1. 团队编程规范写好,知会全组。
2. case-style 检查器。
Vveeb
2021-01-28 11:55:55 +08:00
@YoRolling 并不可能,我甚至觉得我整个公司里的人都不会有人知道 V2EX 这么个地方,好吧保守估计不超 10/500
Vveeb
2021-01-28 12:01:41 +08:00
@tojonozomi 附言 01 也只是摘了最近的一个事儿吐槽了一下,在之前的工作交流中,她就是总能以常人根本想象不到的方式,直接拿高压气泵插我颈动脉给我抬血压。所以打算晚上下班了没事儿再更个《篇二》 [滑稽]
Vveeb
2021-01-28 12:02:51 +08:00
@hideinbush 哈哈哈哈 HHHHHHH,电脑和键盘都邮给你,你来写~
Vveeb
2021-01-28 12:05:15 +08:00
@Jasonwxy 私以为我在公司发现了个新物种,没想到这种人还不占少数,哈哈哈哈
Vveeb
2021-01-28 12:11:21 +08:00
@heaton_nobu 嗯,她在我 Review 之前已经跟我打招呼了,说“我那里面有个挺大的方法,那个你不用纠结,没法再缩了,都是根据实际情况来的,而且已经在实机环境下验证过了,功能都没问题”。所以,以我跟她共事一年多的经验来看,如果让她给我讲里面的逻辑,她大概率会回怼我“都是些 if else 你就自己接上实机环境看呗?这么简单的东西是你看不懂么?”
xuboying
2021-01-28 12:22:44 +08:00
建议用 lint 和风格检查器,这种风格和拼写的事情是机器的强项。本身就不应该出现人与人的沟通交流。
吵架的前提本身就不应该出现,所以也没有必要纠结谁对谁错。
flashrick
2021-01-28 12:38:56 +08:00
搁这写日轻呢
wr516516
2021-01-28 13:46:24 +08:00
写的不错,期待《篇二》
newmoyupoi
2021-01-28 13:57:33 +08:00
我有代码强迫症和洁癖,所以一直一个人搞一整套系统,接手公司 20 多个大小项目,工作总比人多,拿的比别人少,所以还是不是特别严重的洁癖,还是不要像我一样...
alanhe421
2021-01-28 13:59:00 +08:00
可以贴出来一个 code snippet 看看,确实这种人不在少数,暴露出来,自己推不动,领导推,领导推不动或者不想推,你可能就是呆错儿地儿,换部门或者换公司。
bk201
2021-01-28 14:02:10 +08:00
逻辑没问题,能跑得起来就让人离开把,折腾别人干嘛
Vveeb
2021-01-28 14:03:21 +08:00
@18519017871 怎么贴 Code 呢?
Vveeb
2021-01-28 14:04:08 +08:00
@wr516516 哈哈哈好那我抓紧找时间更~
EvilDevilJin
2021-01-28 14:26:47 +08:00
@faustina2018 《关于一段话加上书名号看起来就像变成轻小说这件事情》

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

https://tanronggui.xyz/t/748956

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

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

© 2021 V2EX