PHP对接Stripe时starting_after条件传参的代码优化

你当前的实现存在三个明显问题：

• 直接读取$_POST['startingAfter']未做存在性校验，当POST请求未携带该字段时会触发PHP「Undefined array key」警告
• switch(true/false)的判断逻辑完全不符合业务场景：startingAfter是Stripe的资源ID字符串，并非布尔值，原逻辑无法正确匹配分支
• 两次调用$stripe->charges->all的代码重复度极高，后续调整公共请求参数时需要同步修改两处，维护成本高

完全不需要分支重复写接口调用逻辑，更简洁的写法是先构造基础参数数组，仅在startingAfter值合法非空时，再向参数数组追加starting_after键，最后统一发起请求即可，从根源上避免向Stripe接口传入空值的问题。

优化后的代码如下：

<?php
// 校验POST参数是否存在，不存在则赋值为空字符串
$startingAfter = isset($_POST['startingAfter']) ? trim($_POST['startingAfter']) : '';
$itemsToDisplay = 10;

require_once('../modules/vendor/stripe/stripe-php/init.php');
$stripe = new \Stripe\StripeClient('sk_test_xxx');

// 构造基础请求参数
$chargeParams = [
    'limit' => $itemsToDisplay,
];
// 仅当分页游标非空时，才追加对应参数
if ($startingAfter !== '') {
    $chargeParams['starting_after'] = $startingAfter;
}
// 统一发起接口请求
$customers = $stripe->charges->all($chargeParams);
?>

如果后续需要新增其他可选请求参数（比如反向分页的ending_before、创建时间范围过滤等），都可以沿用这个逻辑：判断参数值合法后再追加到$chargeParams数组中即可，代码结构清晰，也不会产生冗余的重复代码。

内容的提问来源于stack exchange，提问作者mfragale