V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
liangkang1436
V2EX  ›  职场话题

被同事气死了

  •  1
     
  •   liangkang1436 · 2022-04-26 19:48:43 +08:00 via Android · 9210 次点击
    这是一个创建于 701 天前的主题,其中的信息可能已经有所发展或是发生改变。
    - 变量命名用拼音,不用英语

    - 不分离代码,一个 JS 写 3000 多行,一个类 2000 多行

    - 拒绝格式化,不改

    - 到处复制粘贴,修改代码的方式就是把原来的代码复制一份和之前的逻辑用 if-else 分开,这也是导致他经受过的类代码量陡增的原因



    最终导致,他的代码没法调试,一个小 bug ,让我调了两个小时,最终通过打日志才解决。

    气得我边骂边捶桌子调式。
    74 条回复    2022-05-01 18:03:57 +08:00
    wd
        1
    wd  
       2022-04-26 19:49:48 +08:00 via iPhone
    他的代码为什么不是他自己调试?
    fnd
        2
    fnd  
       2022-04-26 19:50:11 +08:00
    liangkang1436
        3
    liangkang1436  
    OP
       2022-04-26 19:51:55 +08:00 via Android
    @wd 因为他去别的项目组了,我留下来殿后
    liangkang1436
        4
    liangkang1436  
    OP
       2022-04-26 19:54:12 +08:00 via Android
    同一段代码到处复制,愣是不抽象出一个工具类,结果导致逻辑修改的时候到处找,我不止一次跟项目经理说,重构吧,项目经理说没时间,凑活用
    liangkang1436
        5
    liangkang1436  
    OP
       2022-04-26 19:56:52 +08:00 via Android
    项目最开始的原班人马大部队已经去进行别的项目的开发了,但是现有的项目需要有开发继续维护,我就是那个留下来的人,经过长时间的迭代,代码已经不满足现有的需求,甚至还成为了阻碍,急需一次重构升级,但是没有时间搞,就这么拧巴着,最终,今天晚上爆发了
    liangkang1436
        6
    liangkang1436  
    OP
       2022-04-26 19:58:27 +08:00 via Android
    算了,摆烂吧,下班回家自己玩自己的项目的时候上点心就行了
    sprite82
        7
    sprite82  
       2022-04-26 20:17:55 +08:00   ❤️ 22
    代码和你有一个能跑就行[doge]
    v2eb
        8
    v2eb  
       2022-04-26 20:20:28 +08:00 via Android   ❤️ 1
    我的同事临离职前写了好多 bug.
    imzcg2
        9
    imzcg2  
       2022-04-26 20:20:41 +08:00 via Android
    您确定是重构?不是重头开始?技术债,哪那么容易还,尤其是加需求的时候,让原来开发的人过来都想跑路,更何况你
    liangkang1436
        10
    liangkang1436  
    OP
       2022-04-26 20:21:40 +08:00 via Android
    @imzcg2 别扎了
    arthas2234
        11
    arthas2234  
       2022-04-26 20:30:10 +08:00
    他还在项目组?那不是继续留后患?
    贴脸输出:代码截图,发邮件,抄送领导
    echo1937
        12
    echo1937  
       2022-04-26 20:32:16 +08:00
    @liangkang1436 #5 我司的项目历来如此。
    potatowish
        13
    potatowish  
       2022-04-26 20:35:19 +08:00 via iPhone
    去别的项目组继续祸害
    ericls
        14
    ericls  
       2022-04-26 20:48:37 +08:00 via iPhone
    没有人教吗?谁 merge 的? CI 怎么过的 review 怎么过的?

    有没有可能你们公司奖励这种你唾弃的代码呢? 至少应该跟团队谈谈 看看大家是怎么想的
    InDom
        15
    InDom  
       2022-04-26 20:50:44 +08:00
    发帖是靠通灵嘛 🐶
    liangkang1436
        16
    liangkang1436  
    OP
       2022-04-26 20:54:17 +08:00 via Android
    @ericls 功能是没问题的,只是维护起来很麻烦,前期为了项目进度,领导也默许了
    ericls
        17
    ericls  
       2022-04-26 20:58:21 +08:00 via iPhone
    @liangkang1436 吸取经验教训吧 下次用这个为理由 让团队争取给自己更多时间去优化代码结构 理由就是为了以后更好维护
    littlewing
        18
    littlewing  
       2022-04-26 21:24:19 +08:00
    能跑吗?能跑就行
    github2020
        19
    github2020  
       2022-04-26 23:05:10 +08:00
    团队负责人不管吗?
    FreshOldMan
        20
    FreshOldMan  
       2022-04-26 23:06:50 +08:00
    格式化上 eslint ,代码问题上 code review 啊
    feather12315
        21
    feather12315  
       2022-04-26 23:10:43 +08:00 via Android
    说 review 的,需求来了催上线,匆匆忙忙写好,整理一堆文档,谁都想早上线早省事,不想持续加班几天整整这堆即将上线的代码,写代码的人烦,code review 的也烦。
    micean
        22
    micean  
       2022-04-26 23:31:34 +08:00 via Android   ❤️ 8
    为了项目进度 = 没有能力重构
    都是借口而已,写个漂亮的业务代码能花多少时间?一直写垃圾代码就是没有那个能力而已,见多了
    buffzty
        23
    buffzty  
       2022-04-27 00:21:47 +08:00
    这根本不是问题啊,他代码垃圾你就多排时间呗.时间够就重写.跟领导要足够多的时间就行.
    2 个小时的需求就要 2 天. 1 天的要 5 天 这还是事吗
    beisilu
        24
    beisilu  
       2022-04-27 08:29:27 +08:00
    @buffzty 我就是这么干的,但是心里和能力还是不够强大。每天有效工作时间 2 小时,看着那堆东西就难以下手。
    maypu
        25
    maypu  
       2022-04-27 08:49:20 +08:00
    楼主说的不是我的同事吗,哈哈哈
    darkengine
        26
    darkengine  
       2022-04-27 08:49:25 +08:00
    还 review 个毛线啊人都走了,有时间有能力的话,你就大刀阔斧的改吧
    beisilu
        27
    beisilu  
       2022-04-27 08:58:52 +08:00
    我现在状况跟你差不多。但是比你还要惨一点:
    1.很多没有被调用的重复垃圾代码。(怀疑是凑行数)
    2.明显暴露在线上的 undefined 错误。
    关键是,因为原来的人全去做其他东西了,我是做移动端被拉进来顶前端的,现在除了多要时间也不知道该怎么办了。
    IvanLi127
        28
    IvanLi127  
       2022-04-27 08:59:28 +08:00 via Android
    变量用拼音,还不如用中文呢!哈哈哈哈
    wobuhuicode
        29
    wobuhuicode  
       2022-04-27 09:27:12 +08:00
    这有什么办法呢。等你跳多几家公司你就发现每家公司都会有这样的代码
    yuthelloworld
        30
    yuthelloworld  
       2022-04-27 09:55:49 +08:00
    摆烂吧
    txy3000
        31
    txy3000  
       2022-04-27 10:04:22 +08:00
    屎山才是常态 符合热力学第三定律🐶
    daimubai
        32
    daimubai  
       2022-04-27 10:06:48 +08:00
    大家重构完测试人员不是还需要重新测试一遍吗,如果业务逻辑复杂的话跑不了自动化,这属不属于给人家增加工作量啊
    ufan0
        33
    ufan0  
       2022-04-27 10:08:01 +08:00
    @micean #22


    为了项目进度 = 没有能力重构
    都是借口而已,写个漂亮的业务代码能花多少时间?一直写垃圾代码就是没有那个能力而已,见多了


    把我心里话用语言描述出来了,一直不知道怎么用语言表述~
    stroh
        34
    stroh  
       2022-04-27 10:08:19 +08:00   ❤️ 1
    3000 行算什么,我们公司好多项目一页 js 1 万行到 3 万行,全部不复用,相似模块从来不提出去,一个函数重复写 10 遍,就因为传值不同,命名大多是词不对意的英文或者不知名的简写,map 从来不 key index ,代码中充斥着 console.log(),控制台 100 多个警告和报错,只是代码能跑,索尼、荣耀、海尔、等大厂都在用我们这套系统,对了,代码还没做压缩...当然没人想搞那屎山一样的代码,变相的保护的源码...
    Vitta
        35
    Vitta  
       2022-04-27 10:12:13 +08:00
    不懂就问,为啥不能用拼音啊,虽然我没用,我甚至都想用汉字
    l00t
        36
    l00t  
       2022-04-27 10:34:10 +08:00
    又不是不能用…… 重构不如重写。
    root01
        37
    root01  
       2022-04-27 11:24:03 +08:00
    拼音又不是看不懂
    zw1one
        38
    zw1one  
       2022-04-27 11:33:09 +08:00
    摆烂,找领导摊牌:这是坨屎,宁看怎么办?你不管我也不管了。
    Dkngit
        39
    Dkngit  
       2022-04-27 11:33:58 +08:00
    政企有些事务意译英文也是无法理解的,还不如英译用拼音
    buffzty
        40
    buffzty  
       2022-04-27 11:37:52 +08:00
    @beisilu 如果你每天有效工作时间 2 小时,那就至少要 5 天排期.或者十天也行.让领导知难而退 还让之前的人改,或者让你重写. 我遇到屎山都是要么不改 要么重构
    liangkang1436
        41
    liangkang1436  
    OP
       2022-04-27 12:27:46 +08:00 via Android
    @stroh 想想我都头疼
    liangkang1436
        42
    liangkang1436  
    OP
       2022-04-27 12:28:09 +08:00 via Android
    @root01 拼音首字母缩写
    zxxufo008
        43
    zxxufo008  
       2022-04-27 12:29:54 +08:00
    不能润,还是建议自己精简下,因为估计以后还会有坑
    mosfet
        44
    mosfet  
       2022-04-27 13:11:10 +08:00
    你们组老大咋想的呢?
    之前留下来的能跑的代码,我看了一会儿就觉得要重构。给老大看了,也觉的是要重构
    然后我理清业务逻辑后,直接重写了,花了一个多星期吧
    hhjswf
        45
    hhjswf  
       2022-04-27 13:21:38 +08:00
    @IvanLi127 你没做 2g 的项目吧?各种专业名词真的顶不住。。
    ychost
        46
    ychost  
       2022-04-27 13:42:19 +08:00
    这种代码 CR 肯定过不了
    abenx
        47
    abenx  
       2022-04-27 13:47:13 +08:00
    我怀疑你再说我
    aimechan
        48
    aimechan  
       2022-04-27 13:51:14 +08:00
    @micean 说的对
    slmaaw
        49
    slmaaw  
       2022-04-27 13:51:30 +08:00 via Android
    接屎山都是这样的感觉 最有效的做法是找个人把屎山丢出去 实在丢不出去只能重构 降低心智负担
    seres
        50
    seres  
       2022-04-27 13:54:37 +08:00
    变量用拼音可以用英语不好解释,用首字母缩写就过分了。。。
    aimechan
        51
    aimechan  
       2022-04-27 13:58:09 +08:00
    一般这种情况,PM 也是屎,只顾当前自身利益不管产品长期发展,纵容时间紧凑就压缩质量,是在造成后患,反正出了问题也不是他 PM 的问题,把矛盾转嫁到执行层。
    dongdongdong
        52
    dongdongdong  
       2022-04-27 14:22:57 +08:00
    不会是我吧
    fyxtc
        53
    fyxtc  
       2022-04-27 14:34:19 +08:00
    非杠,其实我觉得这种情况,自己可能也需要反思一下,为什么我会碰到这样的同事
    bzsh
        54
    bzsh  
       2022-04-27 14:38:04 +08:00
    @feather12315 大实话
    ChefIsAwesome
        55
    ChefIsAwesome  
       2022-04-27 14:42:35 +08:00
    前两天刚碰上个字段名叫 childs 的。见怪不怪。
    Bongzyu
        56
    Bongzyu  
       2022-04-27 14:43:22 +08:00   ❤️ 3
    这有啥,你那是没见过在注释里撒谎的😭
    dongtingyue
        57
    dongtingyue  
       2022-04-27 15:34:05 +08:00
    - 变量命名用拼音,不用英语
    因为不知道用啥英文。。。例如代币新代币,维护列表运维列表

    - 不分离代码,一个 JS 写 3000 多行,一个类 2000 多行
    - 到处复制粘贴
    半天时间完成一个功能没时间优化,怎么简单怎么来

    - 拒绝格式化,不改
    下单子花时间优化代码
    edward1987
        58
    edward1987  
       2022-04-27 15:42:11 +08:00
    @Bongzyu #56 哈哈哈哈看来有故事
    sun1993
        59
    sun1993  
       2022-04-27 15:55:40 +08:00
    我的建议是赶紧润
    onionKnight888
        60
    onionKnight888  
       2022-04-27 16:14:16 +08:00
    润啊
    sogwsc
        61
    sogwsc  
       2022-04-27 17:00:59 +08:00
    @arthas2234 一般领导只关心怎么让项目继续平稳的运行下去
    即使上一个人天坑 也只会安排你赶紧填上
    cat9life
        62
    cat9life  
       2022-04-27 17:21:51 +08:00
    快跑.jpg
    snowlyg
        63
    snowlyg  
       2022-04-27 17:59:47 +08:00
    哪里的公司,怎么招进去的
    Shiu
        64
    Shiu  
       2022-04-27 18:12:46 +08:00
    跟我司代码如出一辙
    kyleduo
        65
    kyleduo  
       2022-04-27 18:50:30 +08:00
    有些木匠懂得榫卯,有些熟悉钉子。家具都能立起来,只有木头,知道不同滋味。
    cjban
        66
    cjban  
       2022-04-27 19:33:04 +08:00 via iPhone
    这个主题配合头像简直完美哈哈哈哈哈哈哈
    ghost024
        67
    ghost024  
       2022-04-27 19:37:39 +08:00
    那你是没见过我之前的一个跑路的同事是用随机数来表示报表的其中一个百分比的数据,我看到代码的时候人都傻了
    liangkang1436
        68
    liangkang1436  
    OP
       2022-04-27 19:41:13 +08:00 via Android
    @ghost024 你永远不知道你的同事写代码的时候脑子里在想什么
    Danswerme
        69
    Danswerme  
       2022-04-27 19:42:48 +08:00
    @Bongzyu 笑死,居然还有撒谎的,细说。
    ghost024
        70
    ghost024  
       2022-04-27 19:52:38 +08:00
    @liangkang1436 他写了注释,说因为这个统计的数不准确,所以直接 0.8+0.2*随机数,保证数据好看,但是就是对不上:)
    liangkang1436
        71
    liangkang1436  
    OP
       2022-04-27 19:56:41 +08:00 via Android
    @ghost024 66666966
    ochatokori
        72
    ochatokori  
       2022-04-27 20:01:58 +08:00 via Android
    我同事也是
    加功能就上千行的代码复制粘贴出一个新文件改一小点。
    vue 组件里的样式不写 scope 也不写组件内的父级类名直接改 ui 库组件的样式。
    处理冲突看都不看直接应用自己的。

    我想打屎她
    lsdvincent
        73
    lsdvincent  
       2022-04-29 17:32:06 +08:00 via iPhone
    又不是不能用
    haah
        74
    haah  
       2022-05-01 18:03:57 +08:00
    这不是你同事的错!只能说贵司没有建立质量保证体系。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   5294 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 44ms · UTC 09:23 · PVG 17:23 · LAX 02:23 · JFK 05:23
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.