php 一個動作在兩個以上的地方用到，怎麼提出？

Question

給個假設

Ａ

$do->query(        "INSERT INTO `abc` (
          `public_order_id`,
          `add_time`,
          `status`
        )
        VALUES (
          '{$public_order_id}',
          '{$add_time}',
          '已取消'
        ) "
      );

Ｂ

$do->query(        "INSERT INTO `abc` (
          `public_order_id`,
          `add_time`,
          `status`
        )
        VALUES (
          '{$public_order_id}',
          '{$add_time}',
          '已完成'
        ) "
      );

假設我這個動作會用到的地方兩個以上
但是唯一會變得就是 status
Ａ是「已取消」
Ｂ會變成「已完成」
到其他地方又會有另外的狀態
其他都一樣，
那這樣的 mysql query 怎麼提出？
原本想說放在 function 但是不行～

---

想到一個問題

假設如果要讓function 可以用外面的值
我可以用 global 來讓外面的值進來 function
但是如果他不是長這樣 $abc 而是這樣 $_SESSION['id'] 和 $oder['id'] 呢？

$abc = 1;
function XXX() {
global $abc;
echo $abc; // 可顯示
}

Answer 1

如果你这全部都是手写 SQL 那就封装成对象，依赖一下 $do 就行了，即传参的时候把 $do 也传过去function changeOrderStatus($do,$status,$public_order_id,$add_time){   return $do->query(        "INSERT INTO `abc` (           `public_order_id`,           `add_time`,           `status`         )         VALUES (           '{$public_order_id}',           '{$add_time}',           '已完成'         ) "       ); }一般来说 上面这样就够了 ，但是从良好的实践出发，原则上一个方法的参数不超过两个，两个以上就应该当成对象处理，所以这里简单处理一下。假定你的 add_time 就是取得当前时间，$status有默认值。function changeOrderStatus($db,$params){    // 这里的 $db 最好做好 类型判断 判断     if(!$db instanceof \PDO){        throw new \Exception('$db not a PDO object instance.');     }     $params = array_merge([        'status'=>'已完成',//status 默认值         'public_order_id'=>null,        'add_time'=>date('Y-m-d H:i:s'),     ],$params);    if(empty($params['public_order_id'])){        throw new \Exception('public_order_id is must.');     }    return $db->query(         vprintf('INSERT INTO `abc` (`public_order_id`,`add_time`,`status`)              VALUES ("%s","%s","%s");',                 $params['public_order_id']                 ,$params['add_time']                 ,$params['status']         )     ); }// CallchangeOrderStatus($do,[    'status'=>'已取消',    'public_order_id'=>1, ]);