好多ORM可以捕获异常,然后回滚这样就非常的方便,但其实我觉得还没有解决根本问题,每一条sql,orm操作不还得自己判断是否成功吗,并且ORM也不规范,有时执行失败抛错,有时又返回false,真是搞不懂,每一步都要自己判断回滚,真是崩溃了,每一步操作都要判断结果,回滚,真是崩溃了,真是崩溃了,真是崩溃了,真是崩溃了,真是崩溃了,真是崩溃了!
好多ORM可以捕获异常,然后回滚这样就非常的方便,但其实我觉得还没有解决根本问题,每一条sql,orm操作不还得自己判断是否成功吗,并且ORM也不规范,有时执行失败抛错,有时又返回false,真是搞不懂,每一步都要自己判断回滚,真是崩溃了,每一步操作都要判断结果,回滚,真是崩溃了,真是崩溃了,真是崩溃了,真是崩溃了,真是崩溃了,真是崩溃了!
其实这里你完全可使用 try
catch
,代码如下:
public function final_shop_order($final_shop_id, $order_id)
{
try {
$this->startTrans();
if (!$final_shop_info = M('Shop')->find($final_shop_id)) {
throw new \Exception('Your are message');
}
if (.....) {
throw new \Exception('Your are message');
}
......
$this->commit();
return true;
} catch (\Exception $e) {
$this->rollback();
return false;
}
}
5 回答3.2k 阅读✓ 已解决
3 回答3.6k 阅读✓ 已解决
1 回答4k 阅读✓ 已解决
3 回答1.8k 阅读✓ 已解决
2 回答2.2k 阅读✓ 已解决
2 回答2.8k 阅读✓ 已解决
5 回答1.4k 阅读
我觉得是你自己的问题...你想想看
M('Shop')->find($final_shop_id))
这种语句有什么需要rollback的吗?查询又不是修改数据对不对,你不执行rollback会产生脏数据吗?所以正确方式是在你
$this->create || !$this->save
之前开启事务,然后验证是否成功从而考虑是否触发rollback,是不是瞬间简化了?另外,从核心逻辑上来说,只要你不执行commit,那么当这个php进程结束之后什么都不会改变,会自动rollback的,但是明文的rollback是好习惯,可以提高代码的可读性,应改坚持。
补充回答:
查询语句和数据库结构修改语言是不接受事务管理的,同时事务开启会锁定当前的mysql进程,因此应该尽量晚的开启并尽快的结束一个事务,尤其是使用了长连接或者连接池的时候。
针对评论的补充:
我一般会这样做: