4

或任何与此相关的框架。

以 Zend Framework 2 为例，我有以下表类：

<?php

namespace Contact\Model;

use Zend\Db\TableGateway\TableGateway;
use Zend\Db\TableGateway\AbstractTableGateway;
use Zend\Log\Logger;

class UserContactsTable extends AbstractTableGateway
{
    protected $tableGateway;

    /**
     *
     * @var \Zend\Log\Logger Instance
     */
    protected $logger;

    public function __construct(TableGateway $tableGateway, Logger $logger )
    {
        $this->tableGateway = $tableGateway;
        $this->logger       = $logger;
    }

    /**
     * Save a contact
     * 
     * @param \Sms\Model\UserContact $userContact
     */
    public function saveUserContact(UserContact $userContact)
    {
        $data = array(
            'user_id'       => $userContact->user_id,
            'contact_id'    => $userContact->contact_id
        );

        try {
            $this->tableGateway->insert($data);
        } catch (\Exception $e) {
                    //log
            $this->logger->crit($omeErrMsg);

        }
    }
}
?>

我应该在这里登录吗？我应该将我的记录器绑定到表类吗？如果插入失败并在控制器中捕获并在那里记录，我是否应该让 saveUserContact 函数抛出异常？

最好的做法是什么？

我最初的想法是创建一个带有一些恒定错误消息的类，例如记录器在表类中使用的插入和更新失败，但我不确定这里的正确过程是什么。

这并不仅限于 PHP 或 Zend Framework 2，而恰好是我正在使用的语言。

4

2 回答 2

6

我认为系统的各个组件应该尽可能地分离。所以在这个例子中，如果saveUserContact碰巧失败，那么它可能会导致抛出异常，因为这不是预期的行为。此类不需要知道“更进一步”会发生什么，例如错误记录。

正如您所提到的，最好抛出异常并将其捕获在您的控制器（或其他形式的侦听器）中，然后处理日志记录。

这种方法的好处是您的系统将更容易测试，因为在构建要测试的 UserContactsTable（模拟）对象时，您将有更少的对象来存根。

于 2013-01-11T17:52:48.560 回答
2

一般来说，我觉得你应该在它们发生的地方记录失败（除非它们是预期的，在这种情况下会很吵）但是将异常传播到堆栈（或包装器异常），以便调用者可以决定是否忽略/重试/失败（并记录自己的、更与业务逻辑相关的消息）。

于 2013-01-11T17:40:20.163 回答