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

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

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

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

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

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

所以说之后评审代码这种事是我完全没有勇气去做的,里面放眼望去基本都是 Yin 间代码。
15863 次点击
所在节点    程序员
144 条回复
wikinee
2021-01-28 14:30:43 +08:00
这帖子名字也太太太太太太长了。
faustina2018
2021-01-28 14:51:43 +08:00
@Vveeb 哈哈哈哈哈哈哈哈哈哈哈哈哈哈哈哈哈哈哈哈以为括起来就可以显得不那么长了吗?果然黑色显瘦( ̄▽ ̄")
Hider5
2021-01-28 14:58:55 +08:00
好帖,开启追更模式
proger
2021-01-28 15:03:16 +08:00
追更+1
pangleon
2021-01-28 15:20:40 +08:00
@tojonozomi 营销鬼才,像你这么写点击量翻 10 倍
Shazoo
2021-01-28 15:30:43 +08:00
@goodboy95 多谢多谢。
dk7952638
2021-01-28 15:34:14 +08:00
首先我肯定你的认真,但是我还是要说,一个要走的人,她并不会很在乎你的指摘,你在他眼里只是一道离职程序,你的负责在她看看来只是找茬而已,so...难得糊涂吧还是
zshneedmoney
2021-01-28 15:42:24 +08:00
你们标准也太低了 我呆的这个地方 写 bug 是要扣钱的 文档补不齐不许提测。
catsoul
2021-01-28 15:50:09 +08:00
你说这个 Yin 间 代码,我第一反应是 王某 居然把 Yin 语言 做出来了?
Vveeb
2021-01-28 16:00:28 +08:00
@dk7952638 我一直在克制自己不要用“指摘”这个词语的哈哈,看你这么说好熟悉,看来您也是 对日 /日企 相关的?
Vveeb
2021-01-28 16:02:26 +08:00
@pangleon 决定了,给老哥起的名字盗了,然后我出门左转去混晋江了,哈哈哈[滑稽]
dos7002
2021-01-28 16:06:25 +08:00
要走的没心思干活啦。拿个 lint 工具自己改吧
bxd5812127
2021-01-28 16:18:32 +08:00
难道是我前同事,心中窃喜。
SkyYu822
2021-01-28 16:25:30 +08:00
《让无论如何都是你错的同事修改代码风格是否搞错了什么》 🐶
HangoX
2021-01-28 16:30:31 +08:00
用 check-style 自动检查不就可以了?团队接了这个之后发现得很及时
Jf35jxN3fwBXyeLh
2021-01-28 16:32:19 +08:00
不得不承认,有人就是没有想把事情做好的想法,他们的想法就是尽快完成功能,告诉 leader 干完了。身边一同事,他的理念就是工程建设不重要,代码烂不烂不重要,有的代码就是写了个空壳,表明这个功能加进去了,一问他,他就说写给别人看的,汇报时别人知道有这么多功能会觉得牛逼,只要 leader 看来有正收益,满足 leader 就行,又没人来看代码。挺高的职级,写的 c/c++ 一堆 warning,随手用来测试代码就是敢直接提 master (从来没有 develop 的),代码到处是一些当前不需要的代码。呵呵,已经去祸害其他公司了
weiliangweiliang
2021-01-28 16:55:11 +08:00
追更+1
Vveeb
2021-01-28 19:55:18 +08:00
@YouKnowIt 哇这种人,功利心好强!我听我朋友也吐槽过类似的这么个人,是他的一个小 leader,总结就是“精致的利己主义者”,听他说的,我觉得用“小人”这个词来形容他的这个 leader 不为过了。甲方没钱了,项目要结束了转维护了,都不告诉手下,问还说没有的事儿。直到甲方亲自找到我朋友头上了,告诉他项目结束了,他才知道;一起出差外派,他 leader 说经费不够,只能住差点的酒店,然后 leader 自己偷偷跑去住高级酒店。总之就是各种背地里使坏踩人上位。我都惊了。
devfeng
2021-01-28 21:54:39 +08:00
我的追番键呢?
1a0ma0
2021-01-28 22:59:07 +08:00
嚯,有内味了,当啥程序员呀,建议直接写厕纸,不对,轻小说。

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

https://tanronggui.xyz/t/748956

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

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

© 2021 V2EX