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

哎呀,给我逗乐了,我真没忍住笑出来了

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

    微信图片_20240323173833.png

    纠缠 量子 量子纠缠 纠缠量子

    真的,看到这种代码,我真的笑出声来,啊哈哈哈哈,我是傻逼,傻逼是我

    43 条回复    2024-03-26 11:40:47 +08:00
    veture
        1
    veture  
       63 天前 via Android
    看着像我上家的代码一样 里面基本上都是这种东西 都几行几行长
    amon
        2
    amon  
       63 天前
    不懂就问,这个三条线的是啥玩意 /害怕
    Baymaxbowen
        3
    Baymaxbowen  
       63 天前 via iPhone
    @amon 三个等于 🟰
    ma836323493
        4
    ma836323493  
       63 天前
    @amon #2

    aikilan
        5
    aikilan  
    OP
       63 天前
    @veture 哥,大发慈悲的告诉我,你上家是哪家,也许我们就是一家人!
    amon
        6
    amon  
       63 天前
    js 中 === 我知道,这个三条线的是字体还是特殊符号?
    @Baymaxbowen
    @ma836323493
    xmumiffy
        7
    xmumiffy  
       63 天前 via Android
    @amon 字体效果
    hefish
        8
    hefish  
       63 天前
    知足吧,这个好歹还能看懂。 总比看不懂好。
    FanyFull
        9
    FanyFull  
       63 天前 via Android
    @amon 有的字体有 ligature ,即连字效果。
    wapzjn
        10
    wapzjn  
       63 天前
    那我不得不提这张图了
    https://imgur.com/a/AMaB9bB
    aikilan
        11
    aikilan  
    OP
       63 天前 via iPhone
    @hefish 屎马拉雅山上一片雪花中的其中一个原子罢了
    wapzjn
        12
    wapzjn  
       63 天前   ❤️ 14
    ZK4fu66uZ842Q4rq
        13
    ZK4fu66uZ842Q4rq  
       63 天前
    想看看你优化后,这段代码是什么样
    aliveyang
        14
    aliveyang  
       63 天前
    挺直观的
    wuzzispacelake
        15
    wuzzispacelake  
       63 天前   ❤️ 5
    很多人根本不知道如何让 if 变得可读,最起码

    const isSomeTypedOrder = item.orderType === 4 || item.orderType === 5
    const isSomeKindOfOrderStatus = item.orderStatus !== 2 && (...)
    const isXXX = ...

    if (!isSomeTypedOrder && isSomeKindOfOrderStaus || ...) {
    // ...
    }

    然后再在代码旁边写上注释解释一下这个业务的意思和为什么要写这个条件

    有逻辑的人会自动本能地将条件一条一条按照逻辑摆放清楚,不需要额外教育;没逻辑的人会因为在代码和需求的上下文的频繁切换中大脑过热而降频,为了完成任务只能将一坨判断表达式写在一起,本质是脑袋不清楚

    另外还有一些人用一个字母代表一个变量,问就是这个字母周围的代码很简单读读就懂了没必要写全,那么我只能祝福你早日下班
    ysb255236295
        16
    ysb255236295  
       63 天前
    https://imgur.com/a/XXPA1NW
    看看我公司其他团队写的组件吧
    skyqiao
        17
    skyqiao  
       63 天前
    像极了让干活又不肯给钱的时候
    zhuangzhuang1988
        18
    zhuangzhuang1988  
       63 天前
    export enum E_OrderType {
    /**
    * 这是 A
    */
    A = 1,
    /**
    * 这是 B
    */
    B = 2,
    }

    export enum E_OrderStatus {
    /**
    * 这是 A
    */
    A = 1,
    /**
    * 这是 B
    */
    B = 2,
    }

    export interface IOrderItem {
    orderType: E_OrderType
    orderStatus: E_OrderStatus
    paidAmount: number
    }

    function isOrderXXX(order: IOrderItem) {
    // example
    return item.orderType === E_OrderType.A || item.orderStatus === E_OrderStatus.B || ...
    }

    上 TS 辅助把
    aikilan
        19
    aikilan  
    OP
       63 天前 via iPhone
    @wuzzispacelake 是吧……我都想不通什么情况下才会写这种代码
    irisdev
        20
    irisdev  
       63 天前
    我想可能是前期没约定好后端 xjb 加状态加字段改接口...这是我转后端的原因之一....
    darkengine
        21
    darkengine  
       63 天前
    @amon 三条线这个是 IDE 搞的花里胡哨的东西
    lasuar
        22
    lasuar  
       63 天前
    @wapzjn 真的逆天
    kneo
        23
    kneo  
       63 天前 via Android
    还给你分行对齐,不错了。
    veture
        24
    veture  
       63 天前 via Android
    @aikilan 一个做知产的公司
    y1y1
        25
    y1y1  
       63 天前
    周易八卦
    bzj
        26
    bzj  
       63 天前
    @wuzzispacelake 多此一举没事找事,那个 if 比你写的这玩意可读性高多了,跟你怎么写没关系,而是系统设计的问题,有两种可能,一是后端偷懒只提供一个接口,二是项目是在迭代过程中条件不断增多
    zhonj
        27
    zhonj  
       63 天前
    第一步尝试懂了这段代码或者在这基础上加点代码能实现需求
    第二步举一反三继承并扩展这段代码的风格,
    然后你发现你就是公司核心骨干人员了,优化毕业之类的情况你永远是最后的人选。
    YsHaNg
        28
    YsHaNg  
       63 天前
    @darkengine 让我想到了 apl 代码
    dyv9
        29
    dyv9  
       63 天前 via Android
    后端菜,写的代码 没用枚举型或者枚举型没明确给 Json Serializer 和 Deserializer ,定制了 json serializer 后就有可读性了。
    aikilan
        30
    aikilan  
    OP
       62 天前 via iPhone
    @zhonj 写这段代码的人被优化了已经,以为拉屎可以防住裁员,实际老板不知道你在拉屎,该裁还是裁
    aikilan
        31
    aikilan  
    OP
       62 天前 via iPhone
    @dyv9 给了……而且我们接口自动生成,枚举值也一步到位
    zbowen66
        32
    zbowen66  
       62 天前
    四年前刚入职时,公司前端专家的代码:
    zhonj
        33
    zhonj  
       62 天前
    @aikilan #30 说明这个人还是不够核心,写边缘代码😀,所以要在核心处写高级代码,让所有人都看不懂。
    这样做的好处大大的,首先还是那个说法写的乱七八糟的东西越多让别人看不懂,写到核心处,没人敢轻易裁你。其次哪怕裁了你,说明是个垃圾公司就应该毁灭,你写的高级代码会导致公司要改会消耗大量时间,你不爽么?
    zhonj
        34
    zhonj  
       62 天前
    @aikilan #30 我朋友上家公司写的代码没一行注释,核心代码极其恶心复杂,可以理解为跑一个 springboot 买本地服务器起了一整套 k8s 的工作量。你不按照他定义的规则来写,后面公司嫌他贵把他开了,后面反复招人吃不下这一套,公司业务停滞了几个月,最终公司老板气炸了,他开心了,拿了赔偿,恶心到了公司。
    encro
        35
    encro  
       62 天前
    这代码,说实话:

    像极了需求的样子。。。。
    ewpui
        36
    ewpui  
       62 天前
    哈哈哈哈,可能一开始只有一个 if,都是后来加上去的
    ktyang
        37
    ktyang  
       62 天前
    我们算法策略那边有几万行这种代码。。。
    damila
        38
    damila  
       61 天前
    上家就是这种代码,比这还要复杂,一个 if 里面七八个条件,同时里面又有多层嵌套。没人敢重构,所以只能往屎山上继续拉屎
    zhuisui
        39
    zhuisui  
       61 天前
    @bzj 说明你不懂什么是封装。这里将代表多个细粒度业务的条件判断,分组归类为少量粗粒度业务。

    你知道什么是系统设计吗?
    一个 statement 一个 closure 也需要设计,顶层设计者不会关心这个粒度,但这是成长的必经之路。
    linch97
        40
    linch97  
       61 天前
    @wapzjn #12 JS 魅力时刻
    madao199
        41
    madao199  
       61 天前
    抽个变量也不至于是一坨
    bzj
        42
    bzj  
       61 天前
    @zhuisui

    不是,你一个前端扯什么封装?

    涉及到逻辑判断的一律放在后端处理,像上面提到的 orderType=1 ,orderStatus=1 ,已经涉及到数据库结构了,这不是前端该操心的事情,通常做法是由后端提供一个包含各类条件的接口,顶层设计者不会关心是因为这纯粹是常识,谁能想到前端给自己加戏,能写出这玩意。
    zhuisui
        43
    zhuisui  
       60 天前
    @bzj 前端没业务?谁跟你说封装只有后端能做了,封装是针对业务的、针对代码逻辑的,不是针对什么前端、后端还是什么端的。
    至于业务逻辑该放哪儿、开发者能不能决定得了,那是另一码事。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2402 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 30ms · UTC 03:26 · PVG 11:26 · LAX 20:26 · JFK 23:26
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.