Zend_Db_Adapter::beginTransaction() 是否堆栈？

Question

考虑以下几点：

/** (Cas_Template_Tree::DeleteNode)
 * Deletes the given node from the tree, and all of it's children.
 *
 * @static
 * @throws Exception
 * @param Cas_Template_Node $node
 * @return void
 */
public static function DeleteNode(Cas_Template_Node $node)
{
    $table = new Cas_Table_Templates();
    $adapter = $table->getAdapter();
    $leftStr = $adapter->quoteIdentifier('Left');
    $rightStr = $adapter->quoteIdentifier('Right');
    try
    {
        $adapter->beginTransaction();
        $row = $table->find($node->GetId())->current();
        $dependantRowSelector = array(
            "$leftStr >= ?" => $row->Left,
            "$rightStr <= ?" => $row->Right
        );
        //Get the rows removed so that we can nuke the ACLs later.
        $rowsToDelete = $table->fetchAll($dependantRowSelector)->toArray();
        //Delete the rows.
        $table->delete($dependantRowSelector);
        //Delete access control lists on those rows.
        foreach ($rowsToDelete as $rowToDelete)
        {
            Cas_Acl::CreateExisting($rowToDelete['Acl'])->Delete();
        }
        $left = (int)$row->Left;
        $right = (int)$row->Right;
        $difference = $right - $left + 1;
        $table->update(array('Left' => new Zend_Db_Expr("$leftStr - $difference")),
                       array("$leftStr > ?" => $right));
        $table->update(array('Right' => new Zend_Db_Expr("$rightStr - $difference")),
                       array("$rightStr > ?" => $right));
        $adapter->commit();
    }
    catch (Exception $ex)
    {
        $adapter->rollBack();
        throw $ex;
    }
}

/** (Cas_Acl::Delete)
 * Removes this ACL (and all of its dependent access control entries) from the database.
 * @return void
 */
public function Delete()
{
    $aclTable = new Cas_Table_AccessControlLists();
    $aceTable = new Cas_Table_AccessControlEntries();
    $adapter = Zend_Db_Table_Abstract::getDefaultAdapter();
    $identifierName = $adapter->quoteIdentifier('Identifier');
    $aclName = $adapter->quoteIdentifier('Acl');
    try
    {
        $adapter->beginTransaction();
        $aceTable->delete(array("$aclName = ?" => $this->GetId()));
        $aclTable->delete(array("$identifierName = ?" => $this->GetId()));
    }
    catch (Exception $ex)
    {
        $adapter->rollBack();
        throw $ex;
    }
}

注意这两个都要求事务如何工作，否则操作将不是原子的（这会很糟糕；））但是，这里有两个事务块。原始的 DeleteNode 方法调用Cas_Acl::Delete()，它也尝试在事务块内执行自身。理想情况下，Zend_Db 会足够聪明地识别这种情况，并且对于这个特定的调用忽略 Cas_Acl::Delete 中的开始事务和提交/回滚调用。

上面的代码安全吗？能否以任何方式显着改善？

Answer 1

AFAIK Zend_Db 无法识别嵌套事务。看代码。

public function beginTransaction()
{
    $this->_connect();
    $q = $this->_profiler->queryStart('begin', Zend_Db_Profiler::TRANSACTION);
    $this->_beginTransaction();
    $this->_profiler->queryEnd($q);
    return $this;
}

这里没有代码可以识别另一个事务（但也许分析器可以用来做这样的事情）并且_beginTransaction 依赖于 PDO 的beginTransaction。

您可能要做的是将第二个参数添加到Delete() 方法，该方法确定是否使用事务并在DeleteNode() 中使用false 参数调用它：

//RB called as Cas_Acl::CreateExisting($rowToDelete['Acl'])->Delete(false);
public function Delete($useTransaction = true)
{
    $aclTable = new Cas_Table_AccessControlLists();
    $aceTable = new Cas_Table_AccessControlEntries();
    $adapter = Zend_Db_Table_Abstract::getDefaultAdapter();
    $identifierName = $adapter->quoteIdentifier('Identifier');
    $aclName = $adapter->quoteIdentifier('Acl');
    try
    {
        if ($useTransaction === true) {
            $adapter->beginTransaction();
        }
        $aceTable->delete(array("$aclName = ?" => $this->GetId()));
        $aclTable->delete(array("$identifierName = ?" => $this->GetId()));
        //BTW isn't commit() should be called here? 
    }
    catch (Exception $ex)
    {   
        if ($useTransaction === true) {
            $adapter->rollBack();
        }
        throw $ex;
    }
}

Answer 2

我使用自定义 transactionManager 对象并在此处添加此逻辑（如果尚未开始，则开始事务）。

我发现这种类型的对象（委托？）非常有用。我也使用它来使用具有正确访问权限的特定数据库连接。事务中的所有写入和读取查询都使用此 TransactionManager 对象。

对 roolback 的调用是在控制器中完成的，但也通过 TransactionManager 来检测它是否真的应该这样做，这也非常有用，因为调用 Rollback 两次会使大多数现有数据库崩溃。

最后我们尝试使用通用规则，在代码中使事务控制代码非常“高级”（仅在控制器上）而不是在更抽象的子级别对象中。因此，通常我们可以对大多数对象调用任何 ->Delete() 操作，并且我们知道该对象不会尝试处理事务，我们应该这样做。但是，实际上，这只是一般规则，有时会发生封闭事务，TransactionManager 帮助我们隐藏问题（并且可以使用堆栈跟踪记录它们）。