脂肪模型瘦控制器在Zend框架

Question

我有我的應用程序，它使用了太多的商業邏輯在控制器內的幾個控制器。例如看看這個動作方法

public function publishedAction() 
{ 
    if ($this->getRequest()->isXmlHttpRequest()) { 
     $dataTable = new Model_DataTables('item i', array('i.id', 'i.title', 'i.from_age', 'i.to_age', 'i.gender', 'i.created_at', 'i.ispublished')); 
     $item = new Model_Item(); 
     $output = $dataTable->aaData(); 
     $iResult = $dataTable->iResult(); 
     foreach($iResult as $k => $v) { 
      $id = $iResult[$k][0]; 
      $iResult[$k][0] = date('d-m-Y', strtotime($iResult[$k][5])); 
      $iResult[$k][2] = $item->formatAge($iResult[$k][2], $iResult[$k][3]); 
      $iResult[$k][3] = ($iResult[$k][4] == 1) ? 'Male' : 'Female'; 
      $iResult[$k][4] = "<a href=/admin/item/view/".$id.">View</a>"; 
      $iResult[$k]['DT_RowId'] = "$id"; 
     } 
     $output['aaData'] = $iResult; 
     $this->_helper->json($output); 
    } 
} 

參照上述動作方法，用於格式化數據的邏輯在控制器本身進行處理。我在同一個控制器中有幾個這樣的方法，導致我的控制器變胖。而許多有經驗的人都同意儘可能地控制控制器。我想問我應該在相應的模型中移動我的上述方法的所有業務邏輯，還是順着當前的流程走？

更新：

參照上面的代碼，我把所有的業務邏輯在我的模型

public function getPublishedItem() 
{ 
    $dataTable = new Model_DataTables('item i', array('i.id', 'i.title', 'i.from_age', 'i.to_age', 'i.gender', 'i.created_at', 'i.ispublished')); 
    $output = $dataTable->aaData(); 
    $iResult = $dataTable->iResult(); 
    foreach($iResult as $k => $v) { 
     $id = $iResult[$k][0]; 
     $iResult[$k][0] = date('d-m-Y', strtotime($iResult[$k][5])); 
     $iResult[$k][2] = $this->formatAge($iResult[$k][2], $iResult[$k][3]); 
     $iResult[$k][3] = ($iResult[$k][4] == 1) ? 'Male' : 'Female'; 
     $iResult[$k][4] = "<a href=/admin/item/view/".$id.">View</a>"; 
     $iResult[$k]['DT_RowId'] = $id; 
    } 
    $output['aaData'] = $iResult; 
    return $output; 
} 

，並在控制器

public function publishedAction() 
{ 
    if ($this->getRequest()->isXmlHttpRequest()) { 
     $item = new Model_Item(); 
     $this->_helper->json($item->getPublishedItem()); 
    } 
} 

是正確的做法。或者我做錯了什麼？

Answer 1

既然你從你的模型（Model_DataTables）檢索$iResult，豈不是更好，以簡單地做：

public function publishedAction() 
{ 
    if ($this->getRequest()->isXmlHttpRequest()) { 
     $dataTable = new Model_DataTables('item i', array('i.id', 'i.title', 'i.from_age', 'i.to_age', 'i.gender', 'i.created_at', 'i.ispublished')); 
     $output['aaData'] = $dataTable->getIResult(); 
     $this->_helper->json($output); 
    } 
} 

其中getIResult()包含您的foreach。

另一種解決方案是使用一個視圖助手，因爲它似乎要格式化HTML內容。那麼你的行動會是這樣的：

public function publishedAction() 
{ 
    if ($this->getRequest()->isXmlHttpRequest()) { 
     $dataTable = new Model_DataTables('item i', array('i.id', 'i.title', 'i.from_age', 'i.to_age', 'i.gender', 'i.created_at', 'i.ispublished')); 
     $output['aaData'] = $this->view->formatIResult($dataTable->getIResult()); 
     $this->_helper->json($output); 
    } 
} 

其中formatIResult()是您的視圖助手的名稱。