加新支付方式只需新增一个类文件:定义统一PaymentStrategy接口规范方法签名,工厂校验类存在并规范命名,控制器仅传参不写逻辑且注入依赖,策略内只处理支付核心流程、细化异常类型。

定义 PaymentStrategy 接口必须统一方法签名
所有支付策略类必须实现同一接口,否则运行时调用会出错。比如 pay() 方法参数类型、返回值、是否抛异常,都得一致。常见翻车点是微信回调要 $data 数组,支付宝要 $notify_id 字符串,结果策略类各自为政,上下文一调就 fatal error。
正确做法是接口强制收窄输入输出:
-
pay(array $params): array—— 统一接收关联数组(含amount、order_no、return_url等) -
notify(): array—— 统一返回标准化结构:['status' => 'success', 'order_no' => 'xxx'] -
refund(array $params): bool—— 所有退款操作返回布尔值,不混用 string 或 exception
PaymentFactory::make() 必须校验类存在且命名规范
ThinkPHP 6+ 自动加载靠 PSR-4,路径和类名必须严格匹配。硬拼字符串 "AlipayStrategy" 而不检查类是否存在,线上一传 channel=bitcoin 就直接 Class not found。
工厂方法里这几行不能省:
立即学习“PHP免费学习笔记(深入)”;
- 用
ucfirst($channel)转 PascalCase,再拼完整命名空间:app\common\strategy\payment\AlipayStrategy - 调用前必须
class_exists('app\common\strategy\payment\AlipayStrategy'),不存在就 throwInvalidArgumentException("Unsupported payment channel: {$channel}") - 禁止用
new $className(),要用App::make($className, [$config])或手动 new 并传入依赖(如配置数组)
控制器里别在策略上调用 Db:: 或 Cache::
策略类不是服务类,它只负责“怎么付”,不负责“存哪”或“查谁”。有人在 AlipayStrategy::pay() 里直接写 Db::name('pay_log')->insert(...),结果单元测试没法 mock 数据库,一跑就连真实库。
正确姿势是把依赖通过构造函数注入:
- 策略类构造函数接收
array $config和可选的LogInterface $logger - 日志、缓存、数据库操作全交给控制器或专门的服务层处理
- 策略内部只做签名、组装请求、解析响应三件事,保持无状态
异常粒度要细,别用一个 try-catch 包全场
支付失败分很多种:参数错(400)、签名失败(401)、余额不足(402)、网络超时(504)。如果控制器里只写 try { $strategy->pay($params); } catch (\Exception $e) { return json(['msg' => '支付失败']); },前端根本没法区分该重试还是该提示用户改参数。
建议策略内部按场景 throw 不同异常:
-
InvalidParameterException→ 返回 400 -
SignatureVerifyException→ 返回 401 -
NetworkTimeoutException→ 返回 504,前端自动重试 - 控制器 catch 后映射成对应 HTTP 状态码和 message
最易被忽略的是 notify 回调里的异常处理——这里没用户界面,但错误日志必须带原始 POST 内容和时间戳,否则排查三方平台回调失败时两眼一抹黑。



















