0

你怎么看这段代码？最好只编写一个方法 getByPoint() 并传递一个进一步的字符串参数'open'，'close'？

否则包装函数 getOpenedByPoint() 和 getClosedByPoint() 是否合理？

谢谢你的回答；）

/**
 * Get all opened verbals, not cancelled, for a point
 * @param type $point_id
 * @return array
 */
    public function getOpenedByPoint($point_id = null)
    {
        $conditions = array('Verbal.cancelled' => 0, 'Verbal.close' => 0);
        return $this->getByPoint($point_id, $conditions);
    }

/**
 * Get all closed verbals, not cancelled, for a point
 * @param type $point_id
 * @return array
 */
    public function getClosedByPoint($point_id = null)
    {
        $conditions = array('Verbal.cancelled' => 0, 'Verbal.close' => 1);
        return $this->getByPoint($point_id, $conditions);
    }

/**
 * Get all verbals for a point
 * @param type $point_id
 * @return array
 */
    public function getByPoint($point_id = null, $conditions = array())
    {
        if($point_id) {
            $conditions = Hash::merge(array('Verbal.point_id' => $point_id), $conditions);
            return $this->find('all', array(
                'contain' => array('Driver','Car'),
                'fields' => array('Verbal.number','Verbal.year','Verbal.data_out','Driver.cognome','Driver.nome','Car.model','Car.plate'),
                'conditions' => $conditions,
                'order' => array('Verbal.data_out' => 'ASC')
            ));
        }
        return array();
    }
4

3 回答 3

1
  • 这比传入“关闭”或“打开”参数更具可读性。
  • 如果 $point_id 对 getByPoint 函数很重要，为什么在所有函数中都将其默认为 null？这里的味道是，当您从 getByPoint 函数返回一个空数组时，您将不知道是否没有找到任何结果，或者传入的 $point_id 无效。因此，最好设置一个检查点，这样，如果出现问题，您就会知道它是无效的 point_id 而不是数据库中没有结果。
于 2013-11-11T14:48:16.490 回答
0

另一个想法是将不同的状态定义为参数，例如：

public function getByPoint($point_id = null, $open = true, $cancelled = false) {
    ...
}

为了提高调用该getByPoint()方法的代码的可读性，您可以定义具有不同状态的类常量。方法调用将如下所示：

$this->Verbal->getByPoint(123, Verbal::OPEN, Verbal::NOT_CANCELLED);
于 2013-11-12T15:50:44.790 回答
0

如果您维护当前代码，我会将 getByPoint 的可见性更改为私有，以便您从控制器强制使用 getClosedByPoint() 或 getOpenedByPoint()。

如果我是你，我只会使用一个功能。

于 2013-11-11T14:49:16.103 回答