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

[Vveeb]

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

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

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

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

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

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

[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 你说了我想说的

[faustina2018]

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

[cornetCat]

@faustina2018 日轻受害者吧

[NexTooo]

自古一楼出人才

[doveyoung]

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

[h82258652]

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

[jay4497]

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

[abelmakihara]

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

[itskingname]

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

[Vveeb]

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

[shiny]

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

[Vveeb]

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

[WinnieNumberTwo]

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

[Rwing]

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

[airfling]

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