好多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;
}
}
代码风格问题而已，完全可以使用do...while(0)在中间出错时候设置标志位和errstr然后break,然后在最后根据返回值，判断是commit还是rollback
我觉得是你自己的问题...你想想看M('Shop')->find($final_shop_id))这种语句有什么需要rollback的吗？查询又不是修改数据对不对，你不执行rollback会产生脏数据吗？
所以正确方式是在你$this->create || !$this->save之前开启事务，然后验证是否成功从而考虑是否触发rollback，是不是瞬间简化了？
另外，从核心逻辑上来说，只要你不执行commit，那么当这个php进程结束之后什么都不会改变，会自动rollback的，但是明文的rollback是好习惯，可以提高代码的可读性，应改坚持。
补充回答：
查询语句和数据库结构修改语言是不接受事务管理的，同时事务开启会锁定当前的mysql进程，因此应该尽量晚的开启并尽快的结束一个事务，尤其是使用了长连接或者连接池的时候。
针对评论的补充：
我一般会这样做：
class Model {
/**
* 最后发生的错误的提示信息
*/
protected $_error = false;
protected function _setError($error) {
$this->_error = $error;
return false;
}
public function getError($emptyError = true) {
$error = $this->_error;
if ($emptyError) $this->_error = false;
return $error;
}
public function example(param1, param2...) {
if (false) {
return $this->_setError('Your message or translate key');
}
...
//Core logic if all parameters is valid
$this->beginTrans();
try {
//update, insert, delete data
...
} catch (\Exception $e) {
$this->rollback();
return $this->_setError($e->getError());
}
return $this->commit();
}
}