我先来一张,在 Services 层里捕获异常弹出 MessageBox 。
现在软件一运行,先弹 3 个对话框。
此人的逆天程度远不止此,这只是一部分,今天更新完代码看到后我实在受不了了。
![]() | 1 AllenZ0 71 天前 我蹲个位,花生瓜子饮料。 |
![]() | 2 akakidz 71 天前 ![]() |
![]() | 3 FreeEx 71 天前 估计是 debug 之后忘记删除了 :) |
5 huihushijie1996 71 天前 冷静你只是一个打工的 上下级反馈丢锅就行 |
![]() | 6 lujiaxing 71 天前 吃异常可还行. 回头发生了什么鬼晓得. |
7 MangoCloud 71 天前 via Android 还有 c#的,只有有代码就好,不算差 |
![]() | 8 HeHeDaGe 71 天前 |
![]() | 10 lujiaxing 71 天前 |
12 SwaggyMacro OP @FreeEx 按照我对此人的观察,他应该是认真的。 因为他在 Model 里面也写一堆功能,然后捕获异常用 Messagebox 弹窗。 ![]() 然后一个 写数据的通讯类,不管是写 Bool 还是 String 还是 Int ,他所有的形参都是 string 类型,然后他再自己转换,写完也不返回 Bool 值,返回一段字符串说写入失败或者写入成功之类的提示。 |
![]() | 13 sks4728 71 天前 有备注, 超过 80%的代码 |
14 SwaggyMacro OP @huihushijie1996 唉,是的。上次领导都当面笑他是不是不太熟面向对象了 |
15 SwaggyMacro OP @HeHeDaGe 哈哈哈哈哈哈,我上一家公司的代码也是这样的,用数字当 Key ,然后这个项目的 MainActivity.java 有 7000+ 行(安卓项目) |
![]() | 16 guanzhangzhang 71 天前 ![]() |
18 SwaggyMacro OP @akakidz 逆天,逻辑鬼才 |
![]() | 20 lujiaxing 71 天前 |
![]() | 21 weixind 71 天前 我看到过的骨骼最惊奇的 js 代码如下: switch (true) { case case1: xxx; break; case case2: xxx; break; case case3: xxx; break; } 我过于才疏学浅,完全想不到是从哪里学来的写法。 |
![]() | 22 AokiNet 71 天前 |
![]() | 23 XiaoCilang 71 天前 |
25 K332 71 天前 确实有,但是也懒得改,索性无视了 |
![]() | 26 yuwangG 71 天前 ![]() 就说能不能跑? |
28 SwaggyMacro OP @AokiNet V 站能骂脏话吗 |
![]() | 29 ldyisbest 71 天前 还行,不如我同事一个函数 4000 行,一个函数 8000 行来的给力 |
![]() | 30 minami 71 天前 @SwaggyMacro #12 这就是传说中的 UNIX 哲学吗,建议改行去写 shell ![]() |
![]() | 31 quanjw 71 天前 问题也不是很大,要改也简单的,弹窗改为写日志。 |
32 flytsuki 71 天前 |
![]() | 33 492 71 天前 还行,我这边有一个项目,之前的外包做的,百来个文件吧。90%的文件基于某一个母文件的基础上,复制粘贴,再改其中某部分的代码,用的 js ,也没有 eslint 。随便一个页面的就有大概 300 行的 state ,不敢动也不敢改。不知道哪个 state 有用,也不知道哪个 func 没用。一个页面 5000 行很正常。 |
34 pakholeung372 71 天前 @lujiaxing 这种 123 命名没什么问题吧,我也是这样子的,因为我看 vscode 源码也是这样的 |
![]() | 35 zhumengyang 71 天前 又不是不能跑 |
36 yvyvyv 71 天前 ![]() 看完上边的代码我觉得我又行了 |
![]() | 37 54xavier 71 天前 ![]() |
38 wysnxzm 71 天前 ![]() |
![]() | 39 ytmsdy 71 天前 @lujiaxing #9 艹!我看第一张图的时候,还仔仔细细看了,觉得好像也没有可以吐槽的,无非就是代码行数多了点,直到我看到下面的其他图片,作为一个十几年的老码农,见过各种各样的屎山代码的我也绷不住了! |
40 HAZhan 71 天前 一看是工控行业,突然就感觉正常了 |
41 xz410236056 71 天前 楼上各位代码还算好,大家要是见过 objective-c 代码,再想象一下屎山 OC 的话,更恶心(满屏幕字,逻辑都不想看) |
![]() | 42 fkdtz 71 天前 |
![]() | 43 Siriusie 71 天前 |
45 zhuynfox 71 天前 上家公司的一个同事,字段名全都是 a b a1 b1 你能想象一两千行的代码都是这种命名嘛 |
![]() | 46 sq955 71 天前 services 层的 catch 不能弹窗,是为了保持 service 层单一职责吗? @SwaggyMacro #0 |
48 MangK 71 天前 ![]() 异常里谈出提示框算什么。我的神仙前辈直接抛出异常,然后在 catch 里写业务逻辑 https://imgur.com/a/Ou1S4lm |
![]() | 49 duan602728596 71 天前 ![]() 虽然我看不到后端的代码,当我请求用户信息时,后端接口返回: const userInfoRes: { data: { ...... password: string; salt: string; ...... }; } = await fetch('http://xxxxxx') |
![]() | 51 z1645444 71 天前 收藏了,这是 2025 best code review post XD |
52 MangK 71 天前  |
57 ynxh 71 天前 countInfo.value.todo.children.one.count = todoReviewStatistic?.catalogingReviewCount ll 0 countInfo.value.todo.children.two.count = todoReviewStatistic?.useDataReviewCount ll 0 countInfo.value.todo.children.six.count = todoReviewStatistic?.supplyDemandNewReviewCount ll 0 countInfo.value.todo,children.five.count = todoReviewStatistic?.userAppPermission ll 0 countInfo.value.todo.children.seven.count = todoReviewstatistic?.resourceCount ll 0 countInfo.value.todo.children.eight.count = todoReviewStatistic?.frontendNodecount ll 0 品一下 |
59 MangK 71 天前 ![]() |
![]() | 60 54xavier 71 天前 ![]() |
63 Cruzz 71 天前 我这有个小老弟,跟我说,哥我发现 JSONObject 真好用,然后他所有的接口入参出参全是 JSONObject ,绝望 |
64 stdout 71 天前 你是没有见过超过 1w 行的函数,超过 1w 行的存储过程。你这些都是小儿科 |
![]() | 67 villivateur 71 天前 建议你看看我们传统工业行业的垃圾 C89 代码,你这至少缩进很好看了,那些搞硬件的人写的代码才是字面意义上的屎。 |
![]() | 68 SuperSuperCoder 71 天前 挂我是吧 |
![]() | 71 fionasit007 71 天前 @lujiaxing #58 意思是从数字变成 new 了?说实话这种我也干过哈哈,一般情况 123 这种是改的区别不大,要是后面重写,原方法又要用,就起个 new ,这样方便看那一版才是差别最大的一版 |
![]() | 72 Aaron01 71 天前 via iPhone 看得我都有自信了,看来代码是能跑就行 |
73 sthwrong 71 天前 @sthwrong #69 额,好像在另外一个帖子回复的,写 java 的,map ,jsonObject, 代码主打一个正常场景能跑就行,异常场景等他被开了给他填坑的人处理。 |
![]() | 74 fionasit007 71 天前 @MangK #59 catch 里面又异常了咋办 ![]() |
![]() | 75 irisdev 71 天前 ![]() 看不懂,这到底是 api 项目还是客户端项目,客户端项目 messagebox 提示下有问题吗 |
![]() | 76 hellodigua 71 天前 见过一万行的表单文件吗,一模一样的还有 6 份 |
![]() | 77 darksword21 PRO 如果他们都能有工作,那我不应该担心不好找工作 |
79 Greendays 71 天前 ![]() |
80 hwdq0012 71 天前 ![]() |
![]() | 81 mccoymir 71 天前 |
![]() | 82 olaloong 71 天前 ![]() |
![]() | 84 yhxx 71 天前 if userName = '张三', return score = 100 if userName = '李四', return score = 110 if userName = '王五', return score = 120 ..... 一共几千个 某政府部门的查分网站 js 代码 |
86 BestPix 71 天前 最近在用 ai 补注释哈哈 |
90 pandoudousteve 71 天前 昨天我边上老哥还在嘲笑另外一个开发写的 for ( int i=0;i<xxx:i++),说他怎么这个耿直,看了这个帖子,释怀了 |
92 KikuCN 71 天前 @weixind 这种写法还挺常见的吧,避免多层 if 嵌套 ```js switch (true) { case score >= 90: grade = "A"; break; case score >= 80: grade = "B"; break; case score >= 70: grade = "C"; break; default: grade = "D"; } ``` |
![]() | 94 nekoneko 71 天前 |
![]() | 97 nzynzynzy 71 天前 |
![]() | 99 satoru 71 天前 为什么现在的程序员这么喜欢 try catch 整个方法?是觉得自己在写框架核心吗 |
100 assiadamo 71 天前 ![]() 有些人自己写写错的代码就算了,受不了的是绝对自信自己是对的,说别人不对逼别人改,给他一堆证据还不承认,也不自己去求证,逼逼叨叨的 |