一种错误的 if 写法

Junwind 的个人博客 / 741 / 32 / 创建于 1年前 / 更新于 1年前

项目中，看了一段代码，脑壳疼。
非常不建议这种 n 层的 if 嵌套写法，不满足的可以提前 return，break，甚至是 goto，减少 if 层级。
下面这种写法，及其难维护。

还有一个难点是，没有一行注释。

一种错误的 if 写法

编程好习惯

本作品采用《CC 协议》，转载必须注明作者和本文链接

六月的风

课程读者 518 声望

躺平大叔 @ 躺平社区

实践出真知

《L01 基础入门》

我们将带你从零开发一个项目并部署到线上，本课程教授 Web 开发中专业、实用的技能，如 Git 工作流、Laravel Mix 前端工作流等。

《L05 电商实战》

从零开发一个电商项目，功能包括电商后台、商品 & SKU 管理、购物车、订单管理、支付宝支付、微信支付、订单退款流程、优惠券等

推荐文章：

更多推荐...

面试 POPER 的后端开发工程师的离奇经历 34 / 104 |

[求职] 陈大剩的个人简历 15 / 22 |

Tailwind CSS 在小程序中的应用 22 / 11 |

使用laravel+colorui开发了一个关于朋友借钱的《好友欠条》小程序 12 / 29 |

[铁柱周记]第一期：一些不错的开源文本编辑器推荐 20 / 1 |

如何自学编程？如何少走弯路？ 21 / 7 |

讨论数量: 32

不负岁月

42 声望

就问能不能跑起来

1年前评论

Junwind （楼主）

肯定能跑的

xiaotuantuan

11 声望

不好维护才是正解，好维护就不需要你了

1年前评论

xiaotuantuan （作者）

@junwind 克服一下熟悉之后就好了，顺便再增加点维护难度，最好做到不容易替代😄

Junwind （楼主）

是这么个道理，但是我维护时，很困难啊

Junwind （楼主）

@xiaotuantuan 维护是肯定要的，我自己肯定不会写这种代码的

kolin

大家都很懂怎么在这个行业生存下去了哈哈哈哈

哪吒的狗腿子

@xiaotuantuan 可真有你的哈哈哈哈

王小大

Laravel 9.x 译者 150 声望

goto 是啥

1年前评论

helloStar

go有goto, PHP没有

Junwind （楼主）

goto 是一种语法结构，可以跳转到指定地方执行，一般不建议用，但是某些情况下使用还是很方便

largezhou

@helloStar ，，，有的

Silly-dog

@helloStar .... 有的

helloStar

@largezhou 受教了，以前都没用过

helloStar

@Silly-dog 受教了，以前都没用过

小李世界

Laravel 8.x 译者 1.9k 声望 / Doge 先锋 @ dogeow.com

小巫见大巫了，我这比你大两倍的 if else

1年前评论

我爱大可乐

我不信，你贴出来

Junwind （楼主）

更难的点是，没有一行注释，只能硬着头皮看代码，调试。

sanders

课程读者 386 声望 / 程序员鼓励师 @ KDD

锯齿形代码，期待楼主修改后的版本。

1年前评论

Aroad

课程读者 107 声望

然而我公司并不推荐提前return，会用多层if或者其他方式保证只有一个return出口 :joy:

1年前评论

Junwind （楼主）

多层嵌套，必然不好维护的，提前 return 跳出，明显更好维护。

LXK

要看什么时候了，多个return做钩子的时候很烦，统一入口出口还是有必要的。

代码笔记

我喜欢提前返回，但是有的插件，sonarLint插件，好像一个函数有三个return，会认为是坏味道。。像有的判断多的函数，肯定会有多个return，除非用goto

GcHoung

课程读者 1 声望 / PHP开发 @ 。。。

我见过比这更离谱的，而且还是python代码

1年前评论

mofung

16 声望

贴图里的代码foreach里都有多个if，其实可以判反不满足条件的提前continue的。业务实在复杂其实也可以拆分开几块小逻辑的。

1年前评论

tu6ge-php

Laravel 8.x 译者 503 声望

goto，够吐

1年前评论

Lu_Shangyu

20 声望

这if套的，一看就是老资格程序员了 :joy:

1年前评论

小李世界

if else 看完，发现已经到 php 文件底部了，1000多行。:joy:

zjason

26 声望

这算啥，我认识一个，不用递归，直接方法内 while ( 1 )，然后静态搞，拿到自己要的东西后brake

批量操作数据不用批量语句，直接方法内 while ( 1 )，每次根据条件取一条处理，取不到brake

1年前评论

Junwind （楼主）

所以说代码的 review 非常重要，主程如果不做这一步，项目必然不稳定，代码必然杂乱

porygonCN

Laravel 8.x 译者 177 声望 / 后端开发 @ 山重建机

如果只是多层if,可以用&&减少就用&& 如果是每层if都带着else，那这样也没有什么不可以。难以维护的根本原因是代码混乱+没有合适的注释标注，GTA5还有20亿次if呢 :joy:

1年前评论

Junwind （楼主）

是明明能力很好，代码可以写的简洁明了的，非要写成 shi 山，好像是故意怕别人看懂一样，可能就是所谓的不失业写法吧

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容，与人为善，比聪明更重要！

帮助

躺平大叔 @ 躺平社区

私信

文章归档

4 篇 2024 年 3 月 2 篇 2024 年 2 月 1 篇 2024 年 1 月 3 篇 2023 年 12 月 3 篇 2023 年 11 月 1 篇 2023 年 10 月 4 篇 2023 年 9 月 2 篇 2023 年 8 月 3 篇 2023 年 7 月 4 篇 2023 年 6 月 7 篇 2023 年 5 月 17 篇 2023 年 4 月 1 篇 2023 年 3 月 2 篇 2023 年 2 月 5 篇 2023 年 1 月 2 篇 2022 年 12 月 2 篇 2022 年 11 月 1 篇 2022 年 8 月 1 篇 2022 年 7 月 1 篇 2022 年 6 月 2 篇 2022 年 4 月 6 篇 2022 年 3 月 1 篇 2022 年 2 月 2 篇 2022 年 1 月 2 篇 2021 年 12 月 5 篇 2021 年 11 月 4 篇 2021 年 10 月 2 篇 2021 年 9 月 9 篇 2021 年 5 月 4 篇 2021 年 4 月 2 篇 2021 年 3 月 2 篇 2021 年 2 月 5 篇 2021 年 1 月 3 篇 2020 年 12 月 1 篇 2020 年 10 月 5 篇 2020 年 9 月

1个月前 update 更新小问题 1个月前使用 PhpSpreadsheet 读取时间问题 1个月前使用 PhpSpreadsheet 读取 excel 文件内容 1个月前大家听过MooPHP这款框架吗？ 2个月前 insert 失败也会递增 AUTO_INCREMENT

22 分享一段牛逼代码 19 举例理解什么是进程，线程 4 swoole 的 close() 与 disconnect() 区别 3 学一下这种传参 3 踩坑：数组转 json 后的结果是数组还是对象？

博客标签

编程好习惯

成为赞助商