收藏
回答

话题 全部话题

有奖话题 | 这行代码当年是怎么通过Review的?

"虽然它现在跑得挺好,但每次看到都怕它半夜打我"

"这段代码的作者至今是个谜..."

"如果删了这段代码会崩溃,不删我会崩溃"

......


在漫长的开发旅程中,不知你是否听过这些对于代码的吐槽?

总有一些代码让开发者们又爱又恨,或许是多年前遗留的“祖传逻辑”,又或许是紧急上线时不得已的“临时方案”。


今天,就让我们一起来挖掘那些藏在代码深处的“宝藏”,开启一场别开生面的代码吐槽大会吧!

这不仅仅是一次轻松的吐槽大会,更是一次技术人之间的深度交流。通过分享和倾听,我们可以更好地理解彼此的开发经历,从中汲取经验,提升自己的编码实践。

无论是自嘲还是调侃,让我们用一场轻松的技术吐槽打破日常的严谨,让代码世界多一些欢笑和理解。


快来参与吧!让我们一起在代码的海洋里,寻找那些让人又爱又恨的瞬间,用幽默和智慧为我们的开发旅程中增添一抹亮色!


即日起至 2025 年 04 月 20 日,在下方评论区晒出最想吐槽的代码片段(关键信息可打码处理),精选评论的用户将获得官方精美礼品一份。

写回答关注话题邀请回答

26 个回答

  • 暧迷路
    暧迷路
    04-09

    如果真要吐槽的话,隐私协议的功能无疑是最值得一提的。从通知到实施,团队连夜修改了数十个小程序,还需根据每个小程序的特点设计不同的风格,最终还是由官方进行了整合。

    04-09
    赞同 8
    回复 3
    查看更多(2)
  • FlashFact
    FlashFact
    2天前

    笑了,要论BUG,谁比得上微信小程序官方啊。别的不说,miniprogram-ci 下载量暴跌你们知道吗,因为去年以来打包的代码已经和开发者工具不一样了,一堆BUG,而且难用死了

    2天前
    赞同 7
    回复 2
    • 清蒸鱼
      清蒸鱼
      2天前
      大部分人用工具打包,或许就因此不维护了?我猜
      2天前
      回复
    查看更多(1)
  • 小卒过河〰
    小卒过河〰
    04-11

    // 千万不要动!动了你会后悔的。

    if (userId == 123456789) {

        return; // 特殊用户,不处理,问就是历史原因。

    }

    👀 背景故事是这样的:

    几年前系统上线前一天,老板突然说他自己用的账号老是出问题,让“先跳过逻辑别拦着我操作”。

    当时大家都在救火,也没细问,硬生生加了这个判断。

    我们本来打算上线后再处理掉,结果——这个 userId 成为了代码里的“钉子户”。每次代码重构时都有人问:“这个还能删吗?”

    答曰:“别问,问就是留着命要紧。”😵‍💫


    👨‍🔬 现在的状态是:

    代码还在,老板早就不用这个账号了,甚至系统已经换了一套登录逻辑……

    但这行代码依然在,像个守护神一样沉默地坐在那,谁都不敢动它,因为删它的那个人已经不在了(指离职)。


    🧠 我的心声:

    不是这行代码跑得好,而是我们习惯了它的存在。


    04-11
    赞同 5
    回复 1
    • 清蒸鱼
      清蒸鱼
      星期二 16:00
      经验总结:该怼老板还是要怼,“老板,你不熟悉业务逻辑就好好学啊!”
      星期二 16:00
      回复
  • 咕嘟咕嘟扑通扑通
    咕嘟咕嘟扑通扑通
    04-09

    应该是这一段吧🥲他只是一个按钮啊,本想默默无闻,却注定背负点击的宿命。每一次被按下,都是一次被理解或误解的开始。😬

    04-09
    赞同 3
    回复
  • Memory (私信不回复)
    Memory (私信不回复)
    04-09

    真的好想知道微信支付、微信小店这些代码都是什么人写的,怎么通过Review的!?!?!?

    (不敢写太多,怕挨打)

    1、同一参数不同参数类型一个number一个stringhttps://developers.weixin.qq.com/doc/store/shop/API/order/callback/channels_ec_order_new.htmlhttps://developers.weixin.qq.com/doc/store/shop/API/order/get.html

    2、https://pay.weixin.qq.com/doc/v3/merchant/4012289679 分账动账通知

    event_type :

    TRANSACTION.SUCCESS:分账

    TRANSACTION.SUCCESS:分账回退

    04-09
    赞同 3
    回复 4
    • 立十
      立十
      04-09
      慕姨还有怕的时候呀。🤣
      04-09
      回复
    查看更多(3)
  • 小黎
    小黎
    04-09

    公司大佬写代码自带混淆,命名全是(e、_e、__e、e_ 、e__、_e_ )这种,老板叫我改他的东西,我调试半天眼睛花了也没看明白,最后还是向他请教才解决了问题(他是这么跟我说的:”这里有个机关,要这样弄一下“,我感觉除了他自己没人能看懂)

    最离谱的一次是我同事也接收到老板的指派,叫他去改这个大佬封装的一个日期input框的功能,周一老板给他说的,周三我同事很高兴的告诉我说,他调试了两天,终于调试到需要他改动下手的地方了,只见他浏览器控制器十几个js文件调试跳来跳去的,他跟我说他debugger都不知道打那里,以为是打那儿,结果它就是不走哪儿,没一句注释,全是一样的变量字母,他说他那个input框就封装了10多种事件,还和你输入日期格式有有关,会触发相应的事件。

    我只想说,就不能加个注释么。


    04-09
    赞同 2
    回复 4
    • 从君华
      从君华
      04-09
      这就是传说中的防御型编程吗
      04-09
      回复
    查看更多(3)
  • 一笑皆春
    一笑皆春
    04-09

    真想找个代码吐槽一下,发现没有啥可吐槽的


    看过别人的代码,最想吐槽的就是排版了


    写完代码从来不知道格式化,上眼一看上下参差不齐,中间突然又多了好几个空行,你说头疼不头疼。


    翻到最顶部,好几个无用的依赖引用,孤独的在那翘首望删就是不知道删,留在那等过年呢。


    再看参数定义,这定义一个,那定义一个,中间再来一个方法,鱼龙混杂在一起,简直没有更乱的了。


    祝愿所有的程序猿们,都能写出一手规范整洁耐看的代码~


    附上一张同事写的代码,看看这引用有多无用:

    04-09
    赞同 2
    回复 3
    • 慢慢。
      慢慢。
      04-11
      这不是设个 eslint 就能解决的事么
      04-11
      回复
    查看更多(2)
  • 阿飞
    阿飞
    11小时前

    咱就是说,能不能把文档写得够明白,别影响测试、线上使用。如果是以前的参数,那么在申请模板的时候是不是应该不可选?

    11小时前
    赞同 1
    回复
  • 網友小林
    網友小林
    1天前

    请具体描述问题出现的流程,并提供能复现问题的简单代码片段(https://developers.weixin.qq.com/miniprogram/dev/devtools/minicode.html)。

    1天前
    赞同 1
    回复
  • 醉云烟酒姜盼15131400055
    醉云烟酒姜盼15131400055
    1天前
    # 老板娘需要密码?
    def login(username, password):
        if username == "boss_li" and sys.argv[0].endswith("boss_pc.exe"):  # 人家的电脑
            print("⚠️  特权跳验证")
            return True  # 放行
        else:
            return check_password(username, password)
    


    店里的进销存系统源码

    1天前
    赞同 1
    回复

正在加载...

登录 后发表内容