liangkang1436

被同事气死了

  •  1
     
  •   liangkang1436 · Apr 26, 2022 via Android · 11149 views
    This topic created in 1651 days ago, the information mentioned may be changed or developed.
    - 变量命名用拼音,不用英语

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

    - 拒绝格式化,不改

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



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

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

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

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

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

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

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

    我想打屎她
    lsdvincent
    lsdvincent
    Apr 29, 2022 via iPhone
    73
    又不是不能用
    haah
    haah
    May 1, 2022
    74
    这不是你同事的错!只能说贵司没有建立质量保证体系。
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Privacy   ·   Solana   ·   3886 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 150ms · UTC 00:50 · PVG 08:50 · LAX 16:50 · JFK 19:50
    ♥ Do have faith in what you're doing.