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

[Vveeb]

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

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

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

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

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

所以说之后评审代码这种事是我完全没有勇气去做的，里面放眼望去基本都是 Yin 间代码。

@JavaFirstMaster 嗐，这更是想都不敢想。

反正就是个 30 多岁的已经有小孩儿了的大我应该一轮的一女的。
共事一年下来，发现这个人就是脑回路异于常人—— 她永远不会认为自己是错的，即使是客观事实已经摆在那里。她不仅不会承认自己的错误，别人试图指正她的时候，她还必定会反过来指责别人，说得跟别人有错一样。

就这个拼写大小写的问题，一开始都是口头指正的，来回三四次我实在是怒了，我就问她“你怎么保证你这次给我审的不会再出现同样问题？”，她反过来问我“我就是能保证，那你凭什么能保证你上次给我挑出来的错误就不是最后一个了？”。 蛤？ 我懵了一下子，合着我给她审核，给她挑错， 我还得保证她必须会犯错，我？？？？

然后我质问她“为什么你每次说全都改了没有问题了，我还是能找到问题，为什么上次那个 xxx 单词的首字母还是没有大写？” 她说”我就是看不着，你能怎么办？“ 。 蛤？ 她找不到错误， 还要质问我我怎么办？ 是我犯的错误么？ 我怎么办？ 我....我告老师....我。

后来，审核这方面我就跟她邮件交流了（抄送了领导）。又来回两次，还有错。最后她回给我的邮件写着“这一版是最终版了，如果再出现这种小问题，就不要再纠结了，我觉得不用改。如果有问题的话找领导，领导说改我就改”。 蛤？ 我又一次懵逼了，我给她做审核的，她说最终版就最终版了？？我？？？？

然后领导来找我了，我说还有错误，领导虽然不懂开发，但是起码英语还是懂的，稍微解释了一番，摆出客观事实，给领导解释明白了，然后领导就下场亲自去跟她交流了，哈哈

更新个《篇二》吧，就叫.... 《以为丈育女同事脑子开窍了向我讨教代码，结果是让我朗读课文》

大部分是之前写的，复制过来改了改。

时间要往前推移一点

一天下午， 她给我发消息 “拿着你的(笔记)本过来一下啊 ”。然后我就带着我的笔记本去她的位置（挺远，不在一个办公区）找她。

问她为啥，她说自己的笔记本太卡了，还没有最新的代码，想在我的本上看一下最新的代码。 因为她需要在她的项目里加一项功能——“检查用户的输入里是否含有非法字符串”，而这个功能我已经做过了，她想看下我的代码参考一下。 这很正常，然后她就叫我把我写的代码里逻辑的地方给她看一下。

这也很正常，我就给代码打开了定位到我写的方法上了，我说“就在这个地方，调用了个人家外国人写好的方法。” （其实就一行）

事情到这里，可能作为一个正常人，如果从她的角度来讲，可能就会说： “谢谢，等我研究研究怎么用，要是不懂了再问你”或者， “你能跟我讲讲为什么要调这个方法，这个方法是干什么的，这个方法怎么用呢？” 之类的话。

但是！你绝对！！想不到！！！接下来她说了什么！！！

她说：“哎你给我念一遍”

我听完她说这话之后我整个人愣住了！愣住了.... 在长达两秒的时间里，我的大脑是空白的，我不知道我要说什么，我要回答她什么。

我整理了下思路，有点语塞地问出“_啊？这为什么要念呢？方法名不都写的很清楚么？_” 她说“你给我念一遍，我看着费劲。” （费劲...费劲...费...劲...?） “嗯，一个一个单词地念”她补充道。

我当时的心理活动：你是看不懂英文啊？还是说我的电脑屏幕有问题，亮度太低？显示器坏了？都没有啊？为什么、为什么？我要给你念一遍呢？你是老师来检查我作业的么?让我给你念一遍？这是一个请求别人帮助，参考别人成果物时候的态度么？退一步说，你要是英文水平不好，或者盯了大半天屏幕了眼睛难受，你都可以说啊？ “你给我念一遍，一个一个地念”....是什么意思？

听到了她的这句话，

就像是，一个正跟你谈笑风生的朋友毫无征兆地给你脸上来了一拳。

就像是，七月份三十多度的天气你走在路上，一个砂锅大的冰雹砸到了你的头上。

惊喜？惊讶？震惊？费解？ 都不足以确切描述我的心情。

但是最后我还是把方法名一个单词一个单词地给她读了一遍，然后给参数列表的参数都给读了一遍，并解释了一下参数是干什么用的。 我也不知道她究竟在我讲完了这一遍之后听懂没有（我觉得是没有） 最后，她说“好，回头我再看一看这里”

我背负着沉重的心情离开了她的工位，总觉得是我自己做错了什么，感觉我在与人的沟通交流上，遇到了重大瓶颈，甚至开始了自我剖析。 以上，就是我那一天里遇到过的，最令我辗转反侧的事情了。

[JavaFirstMaster]

可能是看上你了, 想和你有更多交流

[learningman]

@JavaFirstMaster ++

[Vveeb]

Yin 间代码节选:
1.一个方法写了 400 行，113 的复杂度，放眼望去全是 if...else, switch, 这个等于 0 那个不等于 1 之类的

2.一个循环里有个变量叫`currentList`, 我想看下是个什么 List， 结果它是个别的 List 里的下角标 i 的元素。

3.然后这个`currentList` 点出来了一个 ``currentList.ArrayList`的属性，然后这个属性是个 List<string>

4.看见个叫 “NorFunc” 的属性不知道啥意思，F12 定位过去 结果是个 string 类型的属性....

冰山一角，冰山一角。。。

[wzzzx]

1 楼说的没错

[Mithril]

@Vveeb 我觉得一楼说的是对的

[sunziren]

@wzzzx 你说了我想说的

[FaustinaD]

所以为什么发帖要加书名号？

[cornetCat]

@faustina2018 日轻受害者吧

[NexTooo]

自古一楼出人才

[doveyoung]

《关于听说过名字只要够长再加上书名号就能比较突出显示主题但我还是不理解这件事》

[h82258652]

这种你让她隔一段时间再回去看，她自己都看不懂的
我司同事就是这样，一个类几千行，一个方法上千行，VS 敲个回车都要卡

[jay4497]

所以，别敲码了，写轻小说去吧 doge;

[abelmakihara]

ide 都有提示拼写错误的 我也很难忍受拼写错误

[itskingname]

说说看她的名字，免得祸害下家

[Vveeb]

@faustina2018 哈哈哈觉得标题起得太长了所以拿书名号括了起来

[shiny]

.NET 应该也有自动化的代码风格检查工具吧，交给机器就好了，不合格自动打回就行了。

[Vveeb]

@doveyoung 哈哈哈哈哈哈，好烦~

[WinnieNumberTwo]

哈哈，好奇为什么这个同事要走？离开你们公司难道找到了更好的下家？

[Rwing]

editorconfig 走起来，各种规则都 error，然后加上 EnforceCodeStyleInBuild，他本地都 build 不过

[airfling]

我这边的代码审核都是代码我觉得不合格就打回去修改，什么时候修改好了什么时候我再合，反正我不嫌烦

[caaaalabash]

哈哈哈哈哈标题是在模仿番名吗

[Vveeb]

@WinnieNumberTwo 不不不，我们本身就是个测试部门（还是黑盒），这个人本身工作能力+做人、做职场人更不行，跟她共事过的人都跟她干过架不愿意待见她，自己提的不想在我们组干了，转去别的组了。难受了一年多终于等到这个神仙走了！

[Vveeb]

@Rwing 嗐，这就涉及到我这个基本类型互相转换都要现去百度的选手的知识盲区了。知道有 .editorconfig 这么个东西，没研究过。

[Vveeb]

@airfling “什么时候修改好了什么时候我再合，反正我不嫌烦”

讲道理流程确实应该是这样的，但是这个人，为什么叫她“毒瘤”呢，因为她永远都是“理不直气也壮”，而且脑回路跟一般人着实不一样。共事的这一年多，经常被气到想要原地去世。而且在一个本不是开发的部门做开发的工作，就很难，领导都不会开发，向上反映开发的问题，费时费力。

[jones]

十年前总是追求 LZ 说的这些，现在所有命名都改拼音了

[fffang]

@Vveeb 这个复杂度怎么算的？

[forgottencoast]

@Vveeb 为什么上级领导坐视不管，是他亲人吗？

[Vveeb]

@jones 哈哈哈，我看别的公司我朋友写的代码，命名都用中文的，但是我们这个不是国内的项目，所以没招

[Vveeb]

@fffang 我用的个叫 “CodeMaid” 的插件，提供了一个很 nic 的 e 窗口，上面列出了所有成员，方法，返回值类型，方法复杂度。 我之前还用过一个叫什么 “CodeMatrix”（记不太清），是直接可以在方法前面标注出方法复杂度的。

[zypy333]

用书名号我还以为是讨论另外一个帖子呢, 这也属于不符合规范吧

我这个小公司也有这种同事, 各种下划线驼峰大小写来回穿梭用, 这边都没代码审查,当初把我跟他分一组, 找理由婉拒了,感觉只要不接手不合作,再乱由他乱去吧

[Vveeb]

@forgottencoast 也不是不管，我们这儿说白了是个外企养老公司，公司基本不会主动开你，因为开人公司还要赔偿。
这个交接工作进行了一周下来，她已经成功气坏了我 + 组长 + 科长 （除了我都不是开发）。甚至三人组成了“同仇敌忾”小组。

[lemon94]

@Vveeb 第一条，如果业务需求中就有很多条件判断的话不用 if else 有什么别的办法么

[Vveeb]

@zypy333 Sorry Sorry， 在这个社区混（ mo ）迹（ yu ）时间不长， 条条框框的还没来得及仔细了解。

[cco]

俺们一个接口能大约有 30 多种功能- -。
冰山一角，冰山一角。。。

[Vveeb]

@cco 这不是接口，怕不是个“接洞”了哈哈， 虽然我也不太懂接口是啥，没设计过

[cco]

@Vveeb 据说是为了接口复用。除了请求报文和响应报文长得一样外- -。我没觉得哪里可以复用的。这种还有很多，最近改 bug 快疯了- -。

[Vveeb]

@lemon94 Emmmm 我也是菜鸟，搞个“设计模式”啥的？ 之前还特意去查了 “优化多 if else 分支”的方法。有啥“反射+字典”的，还有啥一些我也不太懂的。反正就我们现在这个微乎其微的代码量+业务逻辑需求。

有一个事件处理方法是处理所有收到的消息，然后要判断这个消息各个属性的值是什么来进行不同的处理，我目前能想到的，就是写一个类，然后用收进来的消息构建类对象，逻辑放在类里面的方法&属性 GetterSetter 去处理。

[zypy333]

@lemon94 整理清楚,拆分到不同抽象层级里的方法里去就差不多了吧,也可以用卫语句提前 return,参考<<代码整洁之道>>,设计模式的话,有模板模式,策略模式等可以用

[lemon94]

@Vveeb 你说的映射+字典我也在用。我是做前端的，经常会根据后端返回的状态码来展示不同的文字，这种时候可以把状态码和对应的字符串以 key-value 形式保存在字典里，用的时候可以直接映射取值。但如果状态码对应的不只是字符串，而是一整段逻辑，这时我就只能用 if else 判断，虽然丑陋，但想不到更简洁优雅的方式。

[jadeborner]

这个同事长得怎么样？

[annielong]

涉及流程跳转，有时候只能 if else，没有特别合适的优化方法，

[janxin]

宁就是轻小说带文豪？

[sevenzhou1218]

没啥好吐槽的，人都要走了，干嘛还很认真的给你搞。。。

[Vveeb]

@sevenzhou1218 没错的，她就是这么个态度，东西都交出去就完了，就是一坨屎也得让我接着。

[Vveeb]

@jadeborner 长得？ 那差不多就是”亲者痛仇者快“这么个水平了

[CrazyRundong]

代码风格的问题不应该是交由 CI 吗？风格检查不通过不给 merge 不就行了 = =

[Vveeb]

@CrazyRundong @_@ CI 是个啥， 可能还没用到这么高级的东西，我们用的顶多就是个 StyleCop 检查下风格。而且 Merge 的话，也是我们这边先过一遍，最后交给外国的开发团队去 Merge 。他们那边会有一次 Code Review & Function Review 。

[lights]

和领导摊牌，要么她能持续改正，要么你走

[CrazyRundong]

@Vveeb https://en.wikipedia.org/wiki/Continuous_integration

[qping]

@Vveeb #45 指望要离职的同事给你写代码，是不明智的，可能还越改越差呢

[fiveelementgid]

Case sensitive.......
话说你这个工作性质
是不是外包？

[Vveeb]

@CrazyRundong 扫噶，持续集成啊，那么 jenkins 应该算是吧，我们的服务器有跑 jenkins，但是好像没有代码风格检查

[Vveeb]

@fiveelementgid 哈哈不是，是分公司给母公司打工这样子

[css3]

所以一个方法或者类最多写多少行合适？

[jzmws]

加个扫描 代码不符合规范的都拒绝提交

[ETiV]

好家伙，这标题，我还以为要动画化了呢

[Keyes]

@ETiV 哈哈哈哈就你最皮

[Lemeng]

她，她的话，是不是你未婚她未嫁呢。感觉怎么样呢

[no1xsyzy]

@lemon94 最重要是要把逻辑隔离出来，最完全的隔离是所有 if else 串的 if (...) 和 {...} 的部分都应当是一个具有有意义的名字的 function call 或者 method call 。尤其 if (...) 最好不要有运算，最多 if ((name = value) == another_value)

而且这样的代码运行效率会很可能变高（假设不内联）：
if (longCondition1()) {longOperation1()} else if ... else {longOperationN()}
因为你要把这些东西内联进去这段代码很可能会超过缓存大小，再从更低速的缓存甚至内存里刷进缓存的话效率就低了。

[KeepingNow]

都给我看乐了😂。

[eijuziew]

哈哈哈哈哈哈哈哈哈哈

[bclerdx]

LZ，你不要把你的想法和方式强加给她人吧。

[dayeye2006199]

这种都不需要人工 review，搞个 linter，要求对方先改到没有 warning 你再给她看逻辑就好了

[laminux29]

这个妹子，应该是那种曾经真实水平只是星耀，但以前的版本允许有运气能上王者，然后她上了王者，就骄傲了。后来版本变化，让运气变得无效，要靠真正实力才能上王者，妹子怎么也回不去王者段位，懵逼了，然后就暴躁了。

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

你作为中层，应该对公司负责，把精力集中在工作上，而不是搞内斗。我如果是你，会先给她一个机会，直接告诉她，你是她的上级，你交代的任务，她必须无条件完成。她可以提建议，但听不听建议是你的事情，她不能因为你不听她的建议而反对你。接着，如果她仍然反对你，你就直接找上级，或者找 HR，把这妹子调离你的部门，原因很简单，就一条，员工不服从领导安排。

[hpu423]

标题很长

[cominghome]

草，看这标题想起了改论文的时候，初稿改格式都改了四五版。。。

[Vveeb]

@css3 之前装了个方法复杂度的插件，默认配置下，复杂度超过 3 就标红了。一个方法体，之前看过有人说他都控制在 10 行以内。但是我觉得要是一直追求这样的书写方法，应该会挺累的。

[ragnaroks]

培训班都没有这么离谱

[Vveeb]

@bclerdx 因为她的代码的设计和逻辑这块我是根本不敢审的，看都看不懂。清一色 public void 方法。所以只能抓一抓这种初高中生都能理解的大小写&拼写问题，而且这种问题上报到领导，领导也能懂。总不能从头给领导讲清楚.net 开发，然后再告诉领导她哪哪哪的代码根本就不符合规范吧~。

[Marszm]

羡慕能找到这么舒服的工作.....对比下自己,自己简直是劳模.

[sigl0p]

Code Metrices 插件，提示得丑了一点。

[meepo3927]

截个图让大家瞧瞧

[peapods]

为什么这样的人会做程序员

[no1xsyzy]

后续，搞快点 gkd
其实可以从系统理论挑几句话，也能向外行说明不对劲的地方，因为系统理论是跨学科的，经管方面也会重新发现一遍的（查一般系统论，第二个就是 MBA 智库……）
可维护性问题多半从属于耗散结构论

[RlhholylightR]

我能说我们使用手机号码注册获取验证码前要先调用校验接口校验该手机号码是否注册过吗

[Bzero]

身边就有一例，真的让人很难受。自己从来没主动过，对代码没有追求，用好吃懒做一点不过分。责备的时候，还顶嘴，不知自己哪里有问题。

[xrr2016]

@lemon94 策略模式

[cheng6563]

@cco 老大说了：把所有功能丢到一个接口里取，跟第三方宣传的时候就可以说“一个接口实现 XXX 功能”

[msy45]

看了这日系标题和 1 楼，这就是轻小说吧😂

[Vveeb]

@peapods 其实是个很拧巴的事儿，本来就一个测试部门，完了非要揽开发的活儿。因为我本科是软件专业的，就先给我派去学技术了，学成龟·来之后，再带俩人，其中一个就是这女的，据说是也有一定基础，之前做过开发，还是研究生，然后.....就这？就这？就这？？？？

[fuwenpei]

好家伙，老二刺螈了。

[Vveeb]

@xrr2016 对对对，想起来了！是有这么个东西~

[jchencode]

我把这样的人，统一称为"老黄牛"

[morimi2026]

这其实没法沟通，所以正确做法是不要去指出，水过就好

[tutustream]

低情商：什么鬼标题


高情商:有没有考虑改行去写轻小说

[Vveeb]

@no1xsyzy 大佬所言全是我知识盲区.....
后续的话，应该是没有了，领导也没招，所以工作内容就直接拿过来了不审了。反正不是特别重要的模块，到时候如果真的有需要往国外开发的仓库提 Pull Request 的话，就直接重做就完了。

但是 《前传》的话，我估计真的能写上一本书了，共事一年多，一些惊天地泣鬼神的事迹我都用小本本记下来了。也成了日后跟朋友、同事闲暇之余的一段笑谈了属于是。

[LokiSharp]

用过好点的 IDE 都能解决啊，买个 Rider 和 ReSharper

[Vveeb]

@morimi2026 是的是的! 最后就是“闭一只眼瞎一只眼” 过去了。

[Vveeb]

@tutustream 低情商： 一个方法写了 400 行，100 多的复杂度，回去给我改！

高情商：您的这个 AI 算法写得真不错，能让我学习一下么？

[aoeui]

我只想问，🐻大么？ 🐶

[Vveeb]

@aoeui 要是🐻大我至于在这里吐槽？ 那我必然是去手把手去教她改大小写啊 🐶

[Shazoo]

好奇，什么是“方法复杂度”？时间复杂度倒是经常得评估下。

什么叫做 400 行 ，100 多的复杂度？这是什么量化标准？

[sockpuppet9527]

我有个好办法，公费让她去波兰一趟就好了。 参考 ： /t/693941
有可能波兰老哥都看了两行，都不给她 code review 了。 :)

[dswyzx]

啊，我就喜欢类里边属性一股脑小写开头，问就是 json 直接粘贴而成。warning? 又不是不能用
不过我支持楼主这么认真的对待规范

[Vveeb]

@Shazoo 可能我用词不 biu 准哈，就是那种，多一个 if else 分支就会+1 的那种复杂度。量化标准好像没什么标准，开头说了“不算正规开发团队、差不多就是打杂的”。 最后代码还是要提 PR 给外国团队审核的，但是 400 多行的一个方法，里面基本都是 if else switch，并且注释都没有， 本身做的还是写 byte bit 的操作，条件里全是 “0，1，2，3，4”的。一般人多看两眼就吐了我觉得

[JavaFirstMaster]

@Vveeb 这个人一看就是对代码没追求, 能力和责任心不强, 但是自尊心强. 你指出她错误的时候她应该是知道有问题的, 就是嘴硬+懒而已. 让领导下场是正确的做法.

哪里都有垃圾代码, 我现在参与的项目里经常见到"不判断 null", "if true return true else false"的代码. 以前我是个敲两行代码就要格式化一下的人, 现在也是见怪不怪了, 因为改出来问题了人家还埋怨你...

[Vveeb]

@dswyzx 嗐，其实本身我也就是个“少儿班”的水平，我们的工作也没什么高深的东西，所以有点强迫症的我，大部分之间都在格式化代码+想命名+写注释。甚至会照着 NET 官方英文文档的格式来写我的方法的 XML 文档注释 + 遣词造句。

[Jasonwxy]

深有同感，前一段时间我们这也来了个大神，不知道怎么招过来的。啥问题都要问，说了问题也不改，一个问题过一段时间问一次，之后我都直接翻聊天记录截图给他，反正跟他合作过的都苦不堪言。还好不久前走了，现在在收拾他留下的烂摊子
感觉这种人就是没有责任+没有能力，除非哪天突然改了性子，别人没法改变他。跟这种人合作是真的难受，我到后期反正是不怎么理这个人了，该怎么办怎么办吧，最好早点滚蛋。这种人不适合去职场跟别人沟通合作，太自我

[heaton_nobu]

你指着那个 400 行的 function 让她给你讲讲里面逻辑

[goodboy95]

@Shazoo 这个方法复杂度，更专业的叫法是圈复杂度