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

被已离职同事遗留的项目气晕

  •  
  •   LandCruiser · 230 天前 · 17081 次点击
    这是一个创建于 230 天前的主题,其中的信息可能已经有所发展或是发生改变。

    我打工的小作坊主打一个短平快,项目制。需求,代码一直在变化。主要服务银行,政府等老式客户。 这大哥遗留的代码,vue2.7setup,TS+自己封装的一塌糊涂的 element-ui,不知道当初怎么跟服务端协商的,请求参数加密,登录前端校验 token 过期时间,各种离谱操作。工期又紧,晕了

    109 条回复
    1  2  
    wsseo
        1
    wsseo  
       230 天前
    是这样的
    a4854857
        2
    a4854857  
       230 天前
    都不知道几手代码了.是这样的
    laogui
        3
    laogui  
       230 天前 via Android   ❤️ 13
    是这样的,你后面的人也会被你的代码气晕😂
    guanzhangzhang
        4
    guanzhangzhang  
       230 天前
    世界都是草台班子组成,我搞东西都留 readme 和折腾博客还有代码注释
    Lirika
        5
    Lirika  
       230 天前
    传说中的屎上雕花,你让他回来看他都不一定看得懂
    qazwsxkevin
        6
    qazwsxkevin  
       230 天前   ❤️ 58
    你好,我就是离职的那个,你也会说“跟服务端协商”,“工期又紧”,“各种离谱操作“了,我很欣慰。
    C4D4zRNpq9vFSlJW
        7
    C4D4zRNpq9vFSlJW  
       230 天前   ❤️ 1
    @guanzhangzhang 那也没用的,经过若干年后,经手的人多了,后来的人不更新 readme 和注释,再后来的人也是没办法,甚至会因为老的 readme 被误导。目前这种职场环境,这种情况无解。
    cybort
        8
    cybort  
       230 天前 via Android
    要不为啥离职 O⁠_⁠o
    doommm
        9
    doommm  
       230 天前 via Android
    短平快,需求代码又一直变,应该也缺乏文档,只能靠口口相传,变成什么样都不奇怪。不过这个技术栈我觉得没什么问题
    guanzhangzhang
        10
    guanzhangzhang  
       230 天前
    @imboring 我自己做事是这样,其他人就不管了,尽量让以后接我手的人不难受😎
    C4D4zRNpq9vFSlJW
        11
    C4D4zRNpq9vFSlJW  
       230 天前
    @guanzhangzhang 是的,我也是这样,不然浑身难受。
    Jame00001
        12
    Jame00001  
       230 天前
    只要接手的所有人水平还行,bug 都是有迹可循的。怕就怕某一个是个半吊子,直接把 bug 当基座往上堆业务。还有一堆破釜沉舟式代码,完全不可扩展。
    Rrrrrr
        13
    Rrrrrr  
       230 天前
    别人离职了,你还没放过
    ivvei
        14
    ivvei  
       230 天前 via Android   ❤️ 18
    你碰过了,以后这个代码就属于是你留下的了
    kneo
        15
    kneo  
       230 天前 via Android
    我看不出来“请求参数加密”有什么离谱的。
    chenshaxiong
        16
    chenshaxiong  
       230 天前 via iPhone
    @imboring 老板难受,老板痒呗
    murmur
        17
    murmur  
       230 天前
    @kneo 今天刚处理一个类似的需求,有个客户没事就找人漏扫,昨天要求我们 api 地址和 webpack 加载的模块名在生成的 bundle 不能用明文
    Baymaxbowen
        18
    Baymaxbowen  
       230 天前
    各种客户急着要,今天就要上,A 客户要这样,B 客户要那样,C 客户都不要
    zhumengyang
        19
    zhumengyang  
       230 天前
    (又不是不能用
    LandCruiser
        20
    LandCruiser  
    OP
       230 天前
    @qazwsxkevin 工期紧还搞 ts 各种封装?完全是反向操作啊
    LandCruiser
        21
    LandCruiser  
    OP
       230 天前
    @cybort 全部代码他自己写的,不用 ts ,不过度封装工期就不紧,我基本都是 6 点下班
    yogogo
        22
    yogogo  
       230 天前 via Android
    这叫不可替代
    estk
        23
    estk  
       230 天前 via iPhone
    @LandCruiser #21
    什么年代了还有不用 ts 的纯 js 项目?
    LandCruiser
        24
    LandCruiser  
    OP
       230 天前
    @estk 注意看我交代的背景,小作坊,需求经常变化。不用 TS 到点就下班,工作很轻松。这个大哥遗留的项目是 TS 的,原本 5 秒写完的发送一个请求代码,在他这里边直接变成一个 service ,复杂谈不上,就是一堆看起来很严谨,但是没用的代码。。。
    kneo
        25
    kneo  
       230 天前 via Android   ❤️ 1
    @LandCruiser 变成 service 和用 ts 有啥关系?你说的 ts 是 typescript ?
    yidinghe
        26
    yidinghe  
       230 天前
    不一定是他一个人留下来的,我这边也有一个前端项目,历经多个版本最后形成了结合 jQuery 和 VUE 的形态,也就是部分页面是纯静态的,部分页面是要编译的……
    qloog
        27
    qloog  
       230 天前
    那机会不就来了😝,给它弄的优雅点
    kneo
        28
    kneo  
       230 天前 via Android   ❤️ 19
    再多说一句,从你的描述上来看,你的前同事看起来是一位非常优秀的同事。看这架势,他辛辛苦苦盖了间房子,他的后辈找不着钥匙恐怕要直接把房子给拆了。
    ColdBird
        29
    ColdBird  
       230 天前
    人员交替快的地方都 n 手代码了,有的 vue 文件动不动 3 、5K 行,然后到处都是复制粘贴的代码,习以为常
    43n5Z6GyW39943pj
        30
    43n5Z6GyW39943pj  
       230 天前   ❤️ 7
    看描述前面大哥技术确实没有问题

    倒是你的想法 "原本 5 秒写完的发送一个请求代码", 有点⚠️
    lyb11232345688
        31
    lyb11232345688  
       230 天前
    人家自己都干不下去了 你还接着干
    chenalex
        32
    chenalex  
       230 天前
    除了"登录前端校验 token 过期时间", 其他没看出有什么问题
    92Developer
        33
    92Developer  
       230 天前
    多拉些你的屎,到时辞了你还得把你招回来。
    zzNucker
        34
    zzNucker  
       230 天前   ❤️ 2
    你把你写的代码和他写的代码都发给 ChatGPT ,让公正的 GPT 评价一下
    angrylid
        35
    angrylid  
       230 天前 via Android
    你是想说,很多请求本来可以直接在 methos 里面 fetch 短快平,但是他非要抽离出去自己搓了一套类似 rtk query 的 endpoints 或者 spring controller 的玩意?

    不过这跟 ts 有什么关系,你不想标注类型,tsconfig 调宽松到处 any 不就退化成 js 了吗?
    chinaguaiu
        36
    chinaguaiu  
       230 天前
    @guanzhangzhang #4 写注释还可以,不会被轻易删除。但是 readme 和博客真的没人看,也容易弄丢,工期紧的时候也没有太多时间另外去整理思路写 readme
    legendBro
        37
    legendBro  
       230 天前
    你猜为什么他离职
    zephyru
        38
    zephyru  
       230 天前   ❤️ 1
    加 ts 封装(或者任何抽象吧)基本考虑的是维护方便,不想管直接 any 走天下也没什么区别...我觉得不至于套一层 ts 封装把工期拉长多少...
    guanzhangzhang
        39
    guanzhangzhang  
       230 天前
    @chinaguaiu gitlab 上的 README 还不看吗,那看啥,oa 上留的聊天记录吗🙄
    darson
        40
    darson  
       230 天前
    大作坊其实都差不多的
    JefferyWang
        41
    JefferyWang  
       230 天前
    你后面的人也会被你的代码气晕
    wdg8106
        42
    wdg8106  
       230 天前
    一句工期紧就能说明所有的问题了,后面的人接手你的代码,估计也是一样的吐槽
    这不是你或他的问题,是公司的问题,要抓住主要矛盾呀
    再就是工期紧跟你有啥关系,正确评估出来工期,反馈给 leader 就行,该延期就延期呗(有加班费另说)
    Yasuo666
        43
    Yasuo666  
       230 天前
    他当时的处境跟你一样,当然你的下一个接班人也会跟你有同样的遭遇。
    DAMNYOU
        44
    DAMNYOU  
       230 天前
    纠正一下。这个现在是你的烂摊子。不是他 哈哈
    godwinma
        45
    godwinma  
       230 天前
    @qazwsxkevin #6 哈哈,这。。。是本尊来回复了吗?世界这么小。
    propc
        46
    propc  
       230 天前
    @qazwsxkevin +10086 优秀,应该如此~~~
    wanqiangcrack
        47
    wanqiangcrack  
       230 天前
    看来喜欢套壳这点,在哪都是一样的。 将来别人看你的代码也会是一样的感受,什么 chatgpt 替代开发,做梦哟。chatgpt 恐怕自己的屎山也不小。
    otakustay
        48
    otakustay  
       230 天前
    别在意,你的下一个接手者也会被你气晕的,哪怕这项目是你从 0 开始搞起来的
    ZGame
        49
    ZGame  
       230 天前
    @LandCruiser #24 我怎么感觉是你菜...
    an0nym0u5u5er
        50
    an0nym0u5u5er  
       230 天前
    如果你接手的代码就一两个人开发的,那么其实还好,只要掌握一下他的思路,跟跟代码,也就 OK 了
    几年前我修过一个考试系统,这个系统的基础框架也是作者自己开发的,没用市面上的框架,但是其实大同小异。
    最恶心的是在 IDE 里面一个 git blame 就能看到 10 个人改过的那种,5 年以上的项目,我也经历过,还被 TMD 老人 diss 能力不行,几乎没文档,几年一直迭代,shit in shit !
    yuntun
        51
    yuntun  
       230 天前
    这才是核心技术人员(赞赏
    RogerL
        52
    RogerL  
       230 天前
    听着咋像是我写的?不过我没干过校验 token 过期的事情
    blackmirror
        53
    blackmirror  
       230 天前
    你就说能不能用就是了,能用还要什么自行车
    daodao
        54
    daodao  
       230 天前
    别人都离职了,你还不放过自己,还要在屎上雕花。赶紧甩膀子别干了~
    taoey
        55
    taoey  
       230 天前
    是这样的,建议重构(估计老板不会给你时间)
    yifangtongxing28
        56
    yifangtongxing28  
       230 天前
    码农这份钱难挣就体现在,老板只管加需求,从不考虑你的代码是不是有问题

    当然,人和代码,总有一个能跑就行了
    bojackhorseman
        57
    bojackhorseman  
       230 天前
    啊,怎么感觉是在说我哈哈
    joyhub2140
        58
    joyhub2140  
       230 天前
    这真是祖传代码了,严重影响生活质量的玩意儿。

    接了这个真是倒大霉,能力挽狂澜的都不是普通人,要么甩手开摆,要么继续往上堆屎,直到屎山摇摇欲坠。
    chinaguaiu
        59
    chinaguaiu  
       230 天前
    @guanzhangzhang #39 🤣微信聊天记录+口口相传
    991547436
        60
    991547436  
       230 天前
    再拉一坨呗,还能咋办
    wupher
        61
    wupher  
       230 天前
    我也碰到过类似情况。

    一开始是沉迷封装,啥都要搞自己的一套,但本质上还是 Spring ,可能显示我为公司封装了框架??

    后面就碰到分项目,其它人直接 copy / plaster ,碰上不支持开始各种魔改,或者要求他更新。

    最后,这东西变成了防御性编程 :-)

    屎上雕花这个算最准确的了。
    yusheng88
        62
    yusheng88  
       230 天前   ❤️ 4
    只听你的描述:
    1 、 你没了解前辈的需求背景,业务逻辑 [看不懂代码||看不惯代码风格] ,就吐槽代码为“屎山”
    2 、 你的前辈使用了 TS 和封装,你觉得繁琐,评价为“屎”

    给我的感觉,你的前辈在做工程化的项目,你在“短平快”地糊屎。
    你觉得你比你前辈优秀,可以贴代码||方案出来,看看别人是否赞同你
    luckyjack
        63
    luckyjack  
       230 天前   ❤️ 4
    未经他人苦,莫喷他人屎(狗头
    chill777
        64
    chill777  
       230 天前
    小公司,就自己重构,反正参与项目的人没几个。
    jefferyJQ
        65
    jefferyJQ  
       230 天前
    @LandCruiser #24 你的意思是他前端写的代码分层了,你觉得分层没用?🤣
    justsosososo
        66
    justsosososo  
       230 天前
    是你 ts 不熟吧,哈哈哈哈
    nikola11
        67
    nikola11  
       230 天前
    没有完美的代码 能跑比啥都强
    aikilan
        68
    aikilan  
       230 天前
    我上次喷菜鸡前同事写的屎山,直到我放出代码,才开始有人同情我,你不如也放出来糊糊大家的眼睛。
    alexwrong
        69
    alexwrong  
       230 天前
    你离职后你写的代码 也是屎山
    needpp
        70
    needpp  
       230 天前
    @qazwsxkevin 哈哈哈,终于知道你为何离职了
    salmon5
        71
    salmon5  
       230 天前
    ta 只是技术不行,不是人品不行(没在你工位上拉屎),所以可以理解。
    isnullstring
        72
    isnullstring  
       230 天前
    短平快,那就没什么好说的,不爽自己重新整
    migu
        73
    migu  
       230 天前
    不是,OP 哪里贴了代码,咋不行啊,一行代码都没看到,你们咋就觉得是那位同事的问题了(😵
    dz5362
        74
    dz5362  
       230 天前
    @qazwsxkevin #6 哈哈哈,有趣
    ThinkCat
        75
    ThinkCat  
       230 天前
    代码和人,有一个能跑就行
    sleepybear1113
        76
    sleepybear1113  
       230 天前
    我很欣慰的,离职 2 年路上遇到了前同事,说只有我的代码是注释完整的,可读性好很多,至少不算是特别屎山
    lwldcr
        77
    lwldcr  
       230 天前
    看我上个帖子
    flyff1991
        78
    flyff1991  
       230 天前
    @laogui 后面的人是被他前任
    flyff1991
        79
    flyff1991  
       230 天前
    @ivvei 不存在的 提交记录在那,就算屎上拉屎,也不能说前面的屎是他拉的
    wusheng0
        80
    wusheng0  
       230 天前 via Android   ❤️ 2
    吐槽 TS 没看懂,配置一下全部 any ,不就和 js 一样了。

    后来的人:被已离职同事遗留的项目气晕,TS 都用 any ,什么水平
    leegradyllljjjj
        81
    leegradyllljjjj  
       230 天前
    堆屎者恒被人堆之,此为无间堆屎地狱
    FireKey
        82
    FireKey  
       230 天前
    这种情况让 ai 去读代码真的很好用
    ruoxie
        83
    ruoxie  
       230 天前 via Android
    其它不知道,但是吐槽 ts 就有点。。。19 年开始用 ts ,不用 ts 影响我下班,特别是接手不是 ts 的项目
    Daming
        84
    Daming  
       230 天前
    1. OP 大概率不熟悉 setup
    2. OP 大概率不熟悉 TS
    3. 参数加密没毛病
    TimPeake
        85
    TimPeake  
       230 天前
    看到项目里有人用 jsonSchem 封装 element ui 表格 ,我就想上去给个大逼兜子😡😡😡
    Jinnn
        86
    Jinnn  
       230 天前
    都短平快了,还会有什么项目质量
    xiangyuecn
        87
    xiangyuecn  
       230 天前
    手艺人😜
    behindeye
        88
    behindeye  
       230 天前   ❤️ 1
    我也接触过屎山,但我知道及时我看这些代码很不顺眼,很不爽,我也不会随便去重构,虽然它烂,但是它能运行就说明此时此刻的状态是稳定的,也别手贱跟领导说要重构,就在这基础上修修改改,我曾经吃过亏,觉得要大刀阔斧去优化代码,殊不知任何一个魔法值的变量都是有意义的,包括随处散落的硬编码字符串,你理解不了这块逻辑,不代表这块就是没有用
    ShuWei
        89
    ShuWei  
       230 天前
    我经常被自己几个月前的代码气到不行
    BraveChi
        90
    BraveChi  
       229 天前 via Android
    💩山代码的主观原因是:缺少设计模式和缺少代码规范和审查。客观原因是需求变动混乱(不是快,而是乱或者开发人员不理解需求从而修改多次),设计分析不足出现问题重复改动。后人几乎不要想着纠正过来,除非拿出两三倍的工作量,但是现实不实际。
    murmur
        91
    murmur  
       229 天前
    @estk 太有了,我们的新项目第一件事就是把 tslint 删掉,然后告诉大家全按 js 语法写,不要跳类型体操
    kneo
        92
    kneo  
       229 天前 via Android
    @chenalex 前段校验 token 过期时间单独拿出来说也不是什么离谱的事情。很多系统都会有 token 过期自动重新申请的功能。前段以前校验可以增加平滑度。前段校验也不等于后端就不校验。
    haoswil
        93
    haoswil  
       229 天前
    @guanzhangzhang 馆长好,没想到看到熟人,在这里逛都是职业焦虑的
    jjwjiang
        94
    jjwjiang  
       229 天前
    @Daming 4. 前端校验 token 时限也没问题
    总感觉还是 OP 自己的水平问题
    LandCruiser
        95
    LandCruiser  
    OP
       229 天前
    @kneo 你的描述不准确,我这个前同事是技术优秀,但不懂得因地制宜,把简单问题复杂化了。也不存在找不着钥匙的问题,活儿我都干完了。拆房子是真的,交不起物业费没必要住别墅。随便搭个草房子就可以了。早点下班回家才是真的,其他都是扯淡。
    LandCruiser
        96
    LandCruiser  
    OP
       229 天前
    @jjwjiang 有问题,前端校验 token 过期与否还是需要调一个接口获取服务器时间(或者什么标识)。这不是脱了裤子放屁吗?
    mHKy6969
        97
    mHKy6969  
       229 天前
    政府有什么,街道退休人员登记系统都是发包。12345 政务 2023 年全年投诉登记表 10M ,12345 政务除微信小程序,网页版没法登录,广东省建委投诉平台,20 年前可直接投诉今天关闭了。政府只有一地鸡毛。
    LandCruiser
        98
    LandCruiser  
    OP
       229 天前
    @Daming 运行在银行内网的项目,请传授传授参数加密的作用,学习下。
    LandCruiser
        99
    LandCruiser  
    OP
       229 天前
    下沉了,争论也没啥意义
    sinx003
        100
    sinx003  
       229 天前
    也可能这哥们接手的时候 也是这样.受不了了就走了
    1  2  
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   1069 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 29ms · UTC 19:31 · PVG 03:31 · LAX 11:31 · JFK 14:31
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.