CakePHP2.x 系の トランザクション処理はどこに書くのが正しいのか？

Question

タイトルのとおりなのですが、
Save メソッドを実行する際に Controller 側から Modelに書いた Save 用のメソッドを呼び出して保存をしています。

その時に transaction 処理は controller 側に書くのが正しいのか、
それとも Model 側で処理を書くのが正しいのか。

どっちでも良いのかもしれないですが、よく分かってないです。
以下がスクリプトの例になります。（間違ってたらどうしよう。。。）

---

パターン１

▼Controller

if ($this->request->is('post')) {

    try {
        $this->Hoge->begin();
        if ($this->Hoge->insert($this->request->data)) {
            if ($this->Fuga->insert($this->request->data)) {
                $this->Hoge->commit();
                $this->redirect('/');
            } else {
                throw new Exception('error');
            }
        } else {
            throw new Exception('error');
        }
    }
} catch (Exception $e) {
    $this->Hoge->rollback();
}

▼Model

public function insert($data) {
    $this->create();
    if ($this->save($data)) {
        $result = true;
    }
        $result = false;
    }
    return $result;
}

---

パターン2

▼Controller

if ($this->request->is('post')) {

    if ($this->Hoge->insert($this->request->data)) {
        if ($this->Fuga->insert($this->request->data)) {
            $this->redirect('/');
        }
    }

▼Model

public function insert($data) {

    try {
        $model->begin(); 
        $model->create();
        if ($model->save($data)) {
            $result = true;
            $model->commit();
        } else {            
            throw new Exception('Error');
        }
    } catch (Exception $e) {
        $model->rollback();
    }
    return $result;    
}

※パターン２の場合、２回目のsave時のデータが一回目のデータと紐づく場合
２回目がこけてRollbackされたら整合性が合わなくなるので、
本来のソースでは、bigin と commit のタイミングを調整しています。
ただ、そもそもそれも正しいのか分からないのですが。。。

どなた様か、こういう場合のデザインパターンはこれやー！っていうのがあればごご教示お願いいたします。

Answer 1

複数のモデルそれぞれに保存するならパターン1のControllerでいいと思いますよ。
なるべくモデル内でやりたいなら、Hogeモデル内のinsert関数内で、Fugaモデルのsaveも一緒にやってもいいと思います。

処理的にもパターン1は、戻り値とExceptionの両方を見ていて、問題ないと思います。
モデルのinsert関数内でsave()の戻り値がfalseならExceptionを投げるほうがよりスマートかもしれませんけど

Controller

if ($this->request->is('post')) {
    $this->Hoge->begin();
    try {
        $this->Hoge->insert($this->request->data);
        $this->Fuga->insert($this->request->data);
        $this->Hoge->commit();
        $this->redirect('/');
    } catch (Exception $e) {
        $this->Hoge->rollback();
    }
}

Model

public function insert($data) {
    $this->create();
    if(!$this->save($data)) {
        throw new InternalErrorException('error');
    }
}

Answer 2

Controllerがベストだと思います。
複雑なビジネスロジックになるとモデルのアソシエーションだけでは整合性が保てないケースがあると思います。例えば、受注データを元に在庫を引き当てて出荷データを作るとか。受注データの更新、在庫データの更新、出荷データの作成などこれ一連で1トランザクションとかざらにあると思います。
こういう場合、DBテーブルベースのモデル内でトランザクション処理をするのは不可能ではないですが、かなり厄介です。
しかし、詰め込み過ぎるとControllerの可読性が悪くなる場合もあるので、ビジネスロジックだけを持ったモデルを作成して、その中でトランザクション処理を書くという方法もあると思います。

Answer 3

コントローラー内でやるのが確実かと思います。

複数の場合面倒なので、アソシエーションが組まれたModelをまとめてbegin()/commit()/rollback()してくれるメソッドをmodel.phpあたりに作ってしまえば楽かもしれません。
そのうえで、保存処理についても、一度に複数のアソシエーションモデルのデータを保存できるsaveAssociated()メソッドを使ったほうがよりスマートな気がします。