2013-10-14 94 views
2

可以說我有發送電子郵件的一個基本的HTML表單:PHP MVC脂肪控制器VS高脂模型

<form action="contactSubmit" method="POST"> 
    <label for="name" class="italic">Name:</label> 
    <input type="text" name="name" value="" maxlength="20" required="required" autofocus="autofocus" /> 
    <label for="email" class="italic">E-mail:</label> 
    <input type="email" name="reply_to" value="" maxlength="255" required="required" /> 
    <label for="comments" class="italic">Comments:</label> 
    <textarea name="message" rows="10" cols="50" required="required"></textarea> 
    <br /> 
    <input type="submit" class="submit" value="Send" /> 
</form> 

目前所有的驗證是在控制器完成:

// submit contact request 
public function contactSubmit() { 
    // process form if submitted 
    if ($this->formSubmit()) { 
     // validate input 
     $name = isset($_POST['name']) && $this->validate($_POST['name'], null, 20) ? $_POST['name'] : null; 
     $reply_to = isset($_POST['reply_to']) && $this->validate($_POST['reply_to'], 'email', 255) ? $_POST['reply_to'] : null; 
     $message = isset($_POST['message']) && $this->validate($_POST['message']) ? $_POST['message'] : null; 

     // proceed if required fields were validated 
     if (isset($name, $reply_to, $message)) { 
      $to = WEBMASTER; 
      $from = '[email protected]' . $_SERVER['SERVER_NAME']; 
      $reply_to = $name . ' <' . $reply_to . '>'; 
      $subject = $_SERVER['SERVER_NAME'] . ' - Contact Form'; 

      // send message 
      $mail = $this->model->build('mail'); 
      if ($mail->send($to, $from, $reply_to, $subject, $message)) { 
       $_SESSION['success'] = 'Your message was sent successfully.'; 
      } else { 
       // preserve input 
       $_SESSION['preserve'] = $_POST; 

       // highlight errors 
       $_SESSION['failed'] = 'The mail() function failed.'; 
      } 
     } else { 
      // preserve input 
      $_SESSION['preserve'] = $_POST; 

      // highlight errors 
      if (!isset($name)) { 
       $_SESSION['failed']['name'] = 'Please enter your name.'; 
      } 
      if (!isset($reply_to)) { 
       $_SESSION['failed']['reply_to'] = 'Please enter a valid e-mail.'; 
      } 
      if (!isset($message)) { 
       $_SESSION['failed']['message'] = 'Please enter your comments.'; 
      } 
     } 
    } 

    $this->view->redirect('contact'); 
} 

我想從「胖控制器」轉向「胖控制器」,但是我不能爲我的生活弄清楚如何將驗證從前一個控制器完全移植到前面的模型中:

public function send($to, $from, $reply_to, $subject, $message) { 
    // generic headers 
    $headers = 'MIME-Version: 1.0' . PHP_EOL; 
    $headers .= 'From: ' . $from . PHP_EOL; // should belong to a domain on the server 
    $headers .= 'Reply-to: ' . $reply_to . PHP_EOL; 

    // send message 
    return mail($to, $subject, $message, $headers); 
} 

形式只有3個所需的字段,而模型的方法接受5.表單字段的描述是從輸入的名稱,這使得難以以自定義的錯誤消息,同時保持模型便攜式用於其它應用使用不同。看起來,我所做的每一次嘗試都變得非常胖,仍然沒有達到與初始方法相同的靈活性。

可能有人請告訴我一個乾淨的方式從控制器移動驗證的模型,同時仍保持自定義錯誤消息的靈活性,並保持模型的便攜性在其他應用程序使用?

+0

[MVC Validation Advice]的可能重複(http://stackoverflow.com/questions/19351502/mvc-validation-advice) –

+0

驗證的部分與業務邏輯相關,應在[域對象] (http://c2.com/cgi/wiki?DomainObject)。數據完整性檢查應該通過存儲抽象來處理。類似MVC體系結構中的控制器具有**沒有任何**用於驗證。此外,沒有「胖模特」。模型不是一個類。 –

+1

@tereško當他說模型(實體,數據對象,命令對象,你在域層中擁有的任何對象)時,他指的是域對象。但是沒有任何mvc框架真正教會程序員如何構建域圖層。例如,Laravel在文檔中爲模型層提供了一種數據驅動的方式,而不是DDD。 –

回答

1

首先,你真的使用純粹的面向對象?爲什麼要使用帶外部數據的模型方法?該模型應該與它自己的屬性一起工作。如果創建對象所需的數據無效,則不應該實例化模型對象。

您需要創建更多layeres將與模型通信。例如,您可以擁有一個可以處理此驗證的modelFactories圖層。比方說,你有一個用戶模型,在你有一個UserFactory的modelFactories層。你可以在這裏放置驗證邏輯。如果創建用戶模型所需的數據無效,則不要創建該模型。

更抽象的方式是添加更多layeres像dataTransferObjects層。在這裏,您可以擁有將數據傳輸到模型或模型工廠的對象。再次,您可以將驗證邏輯放在此處,然後可以使用modelFactory從UserDTO對象(用戶數據傳輸對象)創建用戶模型。

事情是,你不應該在沒有準備好數據之前創建模型,就像驗證它。這就是爲什麼一個模型應該與它自己的屬性一起工作,並且不會在外部數據中正確傳遞(只傳遞數據來初始化對象)。

您應該閱讀更多關於領域驅動設計和設計模式。

所以這樣你的模型逗留分離,並且可以在其他應用程序重用。

+0

所以你說脂肪控制器方法沒有錯?因爲這正是我正在做的,所以在實例化模型之前驗證數據。 –

+1

是的我在說控制器側的驗證比模型更好。但最好的辦法是通過添加更多的layeres來重構您的應用程序,如上所述。當然這也取決於:)如果它是一個小應用程序(並且不需要擴展)不要過度構建它。 –

1

我說不應該胖。處理HTML表單時,我認爲擁有一個Form對象(如CommentForm)具有自己的字段級別驗證的字段對象(例如,EmailField extends Field)是最乾淨的。

如果表單驗證,那麼你肯定有乾淨的數據。該表單還可以創建並返回各種模型,這些模型本身也可以在成員級別進行驗證。但沒有什麼是脂肪......每一點只需要知道立即關心他們的東西。

控制器看起來是這樣的:

if ($form->validate($request->post())) { 
    // grab the email model from the form 
    $email = $form->getEmail(); 

    // assume $mailTransport implements some mailer interface 
    if ($mailTransport->send($email) == true) { 
    // sent email 
    return $response->redirect('success'); 
    } 
    else { 
    // something unexpected happened 
    $view->flashError('Unable to send email'); 
    } 
} 

$view->form = $form; 

每個控制器處理最終看起來非常相似,很乾淨。這些模型對HTML表單一無所知。這是一個反覆重複的簡單模式。

有很多方法可以做到這一點;以上只是我最喜歡的通用解決方案。