方法调用的前置条件，应该在调用前判断还是在方法里判断

方法外部判断

if ($user->is_human) {
    ActionService::sleep($user);
}

or

内部判断

class ActionService {
    public static sleep($user) {
        if (!$user->is_human) {
            return;
        }
        // 执行……
    }
}

有规范吗，感觉写在外部更清晰，知道什么条件下会走这个方法。
感觉方法内部一定要做过滤，这样用起来没有心智负担，那如果外部也判断是不是就重复了；但是外部不判断代码简洁了。
求助大佬们，先谢谢啦

我未曾珍惜的，我不再拥有?‍?️

haodudecao

版主 1.5k 声望

这里没有陌路，你从不曾孤独

1 人点赞

从小程序个人账户申请开始，带你一步步进行开发一个微信小程序，直到提交微信控制台上线发布。

从零开始带你一步步开发一个 Go 博客项目，让你在最短的时间内学会使用 Go 进行编码。项目结构很大程度上参考了 Laravel。

推荐文章：

更多推荐...

工作

[求职]在郑州工作4年的程序媛的简历优化 13 / 172 |

博客

冯老师的困惑——测试和正式环境掐架篇（一） 15 / 23 |

博客

简单对比一下DcatAdmin和filament的各自优缺点 28 / 33 |

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

博客

Git在项目中的那些实操(持续更新...) 22 / 3 |

博客

[开源项目] 基于 laravel 开发的一个社区/社交小程序 23 / 47 |

thebestxt

487 声望 / 开发者 @ 大连市大白猫啥都干互联网服务工作室

最佳答案

看这个判断条件属于哪个业务。举个例子，如果 sleep 这个方法要执行就一定要判断 $user->is_human，那就把判断写在方法里（后者）。如果 sleep 之前要判断的东西不一定， $user->is_human 只是要判断的可能条件其中之一，这样就写在外边（前者）。

感觉自己说了一堆废话。。

2年前评论

haodudecao （楼主）

😂😂😂

讨论数量: 17

thebestxt

487 声望 / 开发者 @ 大连市大白猫啥都干互联网服务工作室

看这个判断条件属于哪个业务。举个例子，如果 sleep 这个方法要执行就一定要判断 $user->is_human，那就把判断写在方法里（后者）。如果 sleep 之前要判断的东西不一定， $user->is_human 只是要判断的可能条件其中之一，这样就写在外边（前者）。

感觉自己说了一堆废话。。

2年前评论

haodudecao （楼主）

😂😂😂

风吹过有夏天的味道

课程读者 334 声望 / php小菜鸟 @ myself

外部判断的是流程，内部判断的是参数

2年前评论

haodudecao （楼主）

照这个说法，这里应该放到方法内部判断了，因为这里是判断这个参数是否符合要求。

外部判断的是流程，内部判断的是参数

另外，这个说法是哪里看到的，有啥关键词可查吗，谢谢。

风吹过有夏天的味道（作者）

@haodudecao 这是我个人观点😂

haodudecao （楼主）

@风吹过有夏天的味道好的谢谢

php_yt

1.1k 声望

其实都是让人看明白，使用第二种的话，可以起个sleepIfNotHuman

2年前评论

haodudecao （楼主）

感谢，是一种思路，，但是如果前置条件过多，这么写也不行

陈先生

课程读者 592 声望 / 慢就业指挥官 @ 灵活就业规划中心

你这个场景更倾向于外部，因为在执行体的内部一般做的判断只有参数类判断和内部计算结果类判断

2年前评论

haodudecao （楼主）

那岂不是每次调用，都要在外部做一层判断，既麻烦代码又重复

陈先生（作者）

@haodudecao 执行体仅仅是作为执行体。逻辑体不在这执行体里面，各司其职罢了

haodudecao （楼主）

@陈先生大概明白您说的意思，但是这个“逻辑体” “参数类” 的说法，google 也搜不到，有啥关键词吗，我去了解一下，谢谢

陈先生（作者）

@haodudecao 简单说，逻辑体= 你函数的执行内容，参数类=参数类型判断，直白说函数就做函数的事情，你用或者不用是你的事情，你自己调度，我函数就是做函数的。例如我一个函数要调用一个RPC 服务，那么这个函数就是负责调用 RPC 的，谁调用谁负责

haodudecao （楼主）

@陈先生好的，以上只是经验之谈吗，有无相关可搜索的理论支撑，我想了解一下，谢谢

陈先生（作者）

@haodudecao 没，这是我的leader 教我的，我也是个弟中之弟。。。

haodudecao （楼主）

@陈先生感谢

MArtian

版主 3.1k 声望

这种应该是在中间件里面去判断，这个请求就不应该进到控制器

2年前评论

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

帮助

方法调用的前置条件，应该在调用前判断还是在方法里判断

推荐文章：

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

方法调用的前置条件，应该在调用前判断还是在方法里判断

推荐文章：

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

请登录