一个简单低级的代码逻辑问题

[kayseen]

假设存在多个判断条件，以下以修改用户信息为例，哪种代码结构会更清晰而且性能会好一点？ 以下 python 伪代码：

一：

if 用户存在：
	if 密码符合规范：
		if 手机号符合规范：
			if 邮箱符合规范：
				do 修改用户信息
			else：
				return 邮箱不符合规范
		else：
			return 手机号不符合规范
	else：
		return 密码不符合规范
else：
	return 用户不存在

二：

if 用户不存在：
	return 用户不存在
if 密码不符合规范：
	return 密码不符合规范
if 手机号不符合规范：
	return 手机号不符合规范
if 邮箱不符合规范：
	return 邮箱不符合规范
do 修改用户信息

[xiao109]

那肯定是第二种啊，起码不用数空格

[WoWTxT]

性能不用考虑，清晰肯定第二种

[Vinceeeent]

第一种看着都累啊

[wangtian2020]

第二种

从代码的“面相”来看，第一种长的像“回调地狱”，第二种就长得像 async 方法了（ JavaScript ）

[xingshu1990]

原先在 Python 高级编程里看到过第二种 这种优化：

Python 下，第二种属于短路原则。

Python 里的“短路原则”：在条件语句中，如果存在多个条件需要判断时，位置靠前的条件导致整个判断语句为 True （ or 语句）或者为 False （ and 语句），就不再进行后续条件的判断。

[looyer]

第二种 另外表单输入判断前置 涉及到数据库查询结果的判断后置 只有表单数据都确认规范了 再去请求数据库做进一步判断

[Jooooooooo]

第二种清晰.

[66beta]

第二种是几乎适合所有语言的短路原则

[shyrock]

必须是二啊。
第一种我是没想出来哪种场景有用。

[xingshu1990]

@shyrock 第一种是无任何编程基础的新人学着写的。

[FutherAll]

提前返回

[ayase252]

无疑是第二种，early return

第一种可读性太差

[Droi]

可以试试设计模式了。不知道职责链模式能不能解决你的这个问题。

[adoal]

Monad 了解一下

[EchoDeveloper]

第二种就是卫语句,显然更好. 把不该出现的情况提前返回.

[ianEros]

Fail-fast

[qiaobeier]

肯定第二种啊，连你这种伪代码第二种也是漂亮太多了。

[ccraohng]

if 密码不符合规范：
return 密码不符合规范
if 手机号不符合规范：
return 手机号不符合规范
if 邮箱不符合规范：
return 邮箱不符合规范


都是校验，先走规则校验

[dengji85]

@Droi 借楼问一下，这种业务判断多少人会真的去用设计模式，用了会不会增加复杂度

[PiersSoCool]

肯定第二种