V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
Vveeb
V2EX  ›  程序员

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

  •  5
     
  •   Vveeb · 36 天前 · 12836 次点击
    这是一个创建于 36 天前的主题,其中的信息可能已经有所发展或是发生改变。
    前提: .NET 开发,不是正规开发团队,帮国外分公司的开发打杂的,人家写好的框架我们在上面实现个小功能或者修 BUG 啥的。
    本人水平也只是熟练使用 Console.WriteLine() 的水平,但是差不多也能看出来代码写的漂亮不漂亮。

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

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

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

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

    所以说之后评审代码这种事是我完全没有勇气去做的,里面放眼望去基本都是 Yin 间代码。
    第 1 条附言  ·  36 天前
    @JavaFirstMaster 嗐,这更是想都不敢想。

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

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

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

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

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

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

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

    时间要往前推移一点

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

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

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

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

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

    她说:“哎你给我念一遍

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

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

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

    听到了她的这句话,

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

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

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

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

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

    144 条回复    2021-01-29 15:17:21 +08:00
    1  2  
    YoRolling
        101
    YoRolling   35 天前
    你同事逛 v2 么
    LittleYangYang
        102
    LittleYangYang   35 天前
    歪个楼,突然想到了

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

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

    ####

    工具上的建议:
    1. 团队编程规范写好,知会全组。
    2. case-style 检查器。
    Vveeb
        107
    Vveeb   35 天前
    @YoRolling 并不可能,我甚至觉得我整个公司里的人都不会有人知道 V2EX 这么个地方,好吧保守估计不超 10/500
    Vveeb
        108
    Vveeb   35 天前
    @tojonozomi 附言 01 也只是摘了最近的一个事儿吐槽了一下,在之前的工作交流中,她就是总能以常人根本想象不到的方式,直接拿高压气泵插我颈动脉给我抬血压。所以打算晚上下班了没事儿再更个《篇二》 [滑稽]
    Vveeb
        109
    Vveeb   35 天前
    @hideinbush 哈哈哈哈 HHHHHHH,电脑和键盘都邮给你,你来写~
    Vveeb
        110
    Vveeb   35 天前
    @Jasonwxy 私以为我在公司发现了个新物种,没想到这种人还不占少数,哈哈哈哈
    Vveeb
        111
    Vveeb   35 天前
    @heaton_nobu 嗯,她在我 Review 之前已经跟我打招呼了,说“我那里面有个挺大的方法,那个你不用纠结,没法再缩了,都是根据实际情况来的,而且已经在实机环境下验证过了,功能都没问题”。所以,以我跟她共事一年多的经验来看,如果让她给我讲里面的逻辑,她大概率会回怼我“都是些 if else 你就自己接上实机环境看呗?这么简单的东西是你看不懂么?”
    xuboying
        112
    xuboying   35 天前
    建议用 lint 和风格检查器,这种风格和拼写的事情是机器的强项。本身就不应该出现人与人的沟通交流。
    吵架的前提本身就不应该出现,所以也没有必要纠结谁对谁错。
    flashrick
        113
    flashrick   35 天前   ❤️ 1
    搁这写日轻呢
    wr516516
        114
    wr516516   35 天前
    写的不错,期待《篇二》
    newmoyupoi
        115
    newmoyupoi   35 天前   ❤️ 1
    我有代码强迫症和洁癖,所以一直一个人搞一整套系统,接手公司 20 多个大小项目,工作总比人多,拿的比别人少,所以还是不是特别严重的洁癖,还是不要像我一样...
    18519017871
        116
    18519017871   35 天前
    可以贴出来一个 code snippet 看看,确实这种人不在少数,暴露出来,自己推不动,领导推,领导推不动或者不想推,你可能就是呆错儿地儿,换部门或者换公司。
    bk201
        117
    bk201   35 天前
    逻辑没问题,能跑得起来就让人离开把,折腾别人干嘛
    Vveeb
        118
    Vveeb   35 天前
    @18519017871 怎么贴 Code 呢?
    Vveeb
        119
    Vveeb   35 天前
    @wr516516 哈哈哈好那我抓紧找时间更~
    EvilDevilJin
        120
    EvilDevilJin   35 天前
    @faustina2018 《关于一段话加上书名号看起来就像变成轻小说这件事情》
    wikinee
        121
    wikinee   35 天前
    这帖子名字也太太太太太太长了。
    faustina2018
        122
    faustina2018   35 天前
    @Vveeb 哈哈哈哈哈哈哈哈哈哈哈哈哈哈哈哈哈哈哈哈以为括起来就可以显得不那么长了吗?果然黑色显瘦( ̄▽ ̄")
    Hider5
        123
    Hider5   35 天前
    好帖,开启追更模式
    proger
        124
    proger   35 天前
    追更+1
    pangleon
        125
    pangleon   35 天前
    @tojonozomi 营销鬼才,像你这么写点击量翻 10 倍
    Shazoo
        126
    Shazoo   35 天前
    @goodboy95 多谢多谢。
    dk7952638
        127
    dk7952638   35 天前
    首先我肯定你的认真,但是我还是要说,一个要走的人,她并不会很在乎你的指摘,你在他眼里只是一道离职程序,你的负责在她看看来只是找茬而已,so...难得糊涂吧还是
    zshneedmoney
        128
    zshneedmoney   35 天前
    你们标准也太低了 我呆的这个地方 写 bug 是要扣钱的 文档补不齐不许提测。
    catsoul
        129
    catsoul   35 天前
    你说这个 Yin 间 代码,我第一反应是 王某 居然把 Yin 语言 做出来了?
    Vveeb
        130
    Vveeb   35 天前
    @dk7952638 我一直在克制自己不要用“指摘”这个词语的哈哈,看你这么说好熟悉,看来您也是 对日 /日企 相关的?
    Vveeb
        131
    Vveeb   35 天前
    @pangleon 决定了,给老哥起的名字盗了,然后我出门左转去混晋江了,哈哈哈[滑稽]
    dos7002
        132
    dos7002   35 天前
    要走的没心思干活啦。拿个 lint 工具自己改吧
    bxd5812127
        133
    bxd5812127   35 天前
    难道是我前同事,心中窃喜。
    SkyYu822
        134
    SkyYu822   35 天前
    《让无论如何都是你错的同事修改代码风格是否搞错了什么》 🐶
    HangoX
        135
    HangoX   35 天前
    用 check-style 自动检查不就可以了?团队接了这个之后发现得很及时
    YouKnowIt
        136
    YouKnowIt   35 天前
    不得不承认,有人就是没有想把事情做好的想法,他们的想法就是尽快完成功能,告诉 leader 干完了。身边一同事,他的理念就是工程建设不重要,代码烂不烂不重要,有的代码就是写了个空壳,表明这个功能加进去了,一问他,他就说写给别人看的,汇报时别人知道有这么多功能会觉得牛逼,只要 leader 看来有正收益,满足 leader 就行,又没人来看代码。挺高的职级,写的 c/c++ 一堆 warning,随手用来测试代码就是敢直接提 master (从来没有 develop 的),代码到处是一些当前不需要的代码。呵呵,已经去祸害其他公司了
    weiliangweiliang
        137
    weiliangweiliang   35 天前
    追更+1
    Vveeb
        138
    Vveeb   35 天前
    @YouKnowIt 哇这种人,功利心好强!我听我朋友也吐槽过类似的这么个人,是他的一个小 leader,总结就是“精致的利己主义者”,听他说的,我觉得用“小人”这个词来形容他的这个 leader 不为过了。甲方没钱了,项目要结束了转维护了,都不告诉手下,问还说没有的事儿。直到甲方亲自找到我朋友头上了,告诉他项目结束了,他才知道;一起出差外派,他 leader 说经费不够,只能住差点的酒店,然后 leader 自己偷偷跑去住高级酒店。总之就是各种背地里使坏踩人上位。我都惊了。
    devfeng
        139
    devfeng   35 天前 via Android
    我的追番键呢?
    1a0ma0
        140
    1a0ma0   35 天前   ❤️ 1
    嚯,有内味了,当啥程序员呀,建议直接写厕纸,不对,轻小说。
    kalluwa
        141
    kalluwa   34 天前
    函数内几百行,几千行都不是什么问题。问题是你这个同事压根就不想好好做,换句话说,养老也得装个好的态度,情商低无疑
    no1xsyzy
        142
    no1xsyzy   34 天前
    @Vveeb “精致的利己主义者”,既不精致,也不利己,更不是个人(“者” = XXX 的人)。(伏尔泰点了个赞并 @ 神圣罗马帝国)
    lp7631010
        143
    lp7631010   34 天前
    人家都离职了 跟你费这劲呢
    heaton_nobu
        144
    heaton_nobu   34 天前
    @Vveeb #111 你:“我还真看不懂,您受累给我讲讲。”
    1  2  
    关于   ·   帮助文档   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   1257 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 19ms · UTC 22:45 · PVG 06:45 · LAX 14:45 · JFK 17:45
    ♥ Do have faith in what you're doing.