发布到同一页面的好习惯？

Question

我最近一直在阅读许多不同的内容，以提高我的编码质量。

我想知道将表单发布到同一页面是否是一个好习惯，我经常在当前的应用程序中使用这种类型的代码片段。

public function addAction(){
    $form = new Application_Form_Add();

    $this->view->form = $form;

    if(!$this->_request->isPost() && $form->isValid($this->_request->getParams())){
        $mapper = new Application_Model_ModelMapper();
        $model = new Application_Model_Model($form->getValues());

        if(!$mapper->save($model)){
            $this->view->messages('an error occurred etc ... ');
            return;     
        }

        $this->_helper->redirector->gotoRoute(array('id' => $model->id), 'model_view');
    }
}

所以我的添加操作在 GET 请求上显示表单并在 POST 请求上处理它。

我看到了 Matthew Weier 的这篇 文章 O'Phinney，我想每个人都同意他是当今的 PHP 大师之一。在他的例子中，他做了两种不同的动作，一种是显示形式，一种是处理形式。因此，如果表单未验证，他会使用 $this->render 重新渲染表单视图。

将表单提交到同一页面是一种不好的做法吗？如果是，为什么？

Answer 1

出于代码可读性和可维护性的目的，两个不同的控制器操作（尽管它们与相同的表单相关联）最好分离为两个函数/脚本。

想象一下，如果所有 CRUD 操作都组合在一个函数/脚本中会是什么样子。那会很混乱。

Answer 2

我不认为这有什么坏处。

无论我们对同一页面和不同页面执行此操作。

两种方法在各自的位置上都是正确的。

使用单页是一种排序和甜蜜的方式，使用 2 页为我们的代码提供了一种托管方式。

在我看来，两者是平等的。

Answer 3

在我看来，我更喜欢使用一个动作来实现它。这将所有的东西放在一起。 O'Phinney 的控制器可以像这样：

• addAction （使用 saveAction 来保存）
• editAction （使用 saveAction 来保存）
• saveAction （不呈现自身）

为所有控制器保存/编辑逻辑提供一种方法是唯一的原因（假设！）它和额外的行动。很酷，将所有东西都放在一处。但是：这需要编辑和添加两者具有完全相同的逻辑。一旦有任何if/else喜欢

public function saveAction()
{
    if($action == 'edit)
    {
        // edit logic 
    } else if($action == 'add')
    {
        // add logic
    }
}

根据操作有不同的逻辑，这完全利用了一个操作的想法。因此，如果您知道逻辑始终相同并且不太可能更改，那么这就是节省大量编码工作的方法:)。

另一方面，如果您的添加/编辑逻辑有点不同，我会将所有逻辑放在一起：

• addAction （打印表单并在 POST 上保存）
• editAction （打印表单并在 POST 上保存）

这可能看起来像代码-重复（也许是），但是，我的编辑/添加操作大约有 3-6 行代码。但一旦其中任何一个的逻辑发生变化，你就真正了解这个地方了。

所以这是个人意见，一旦事情变得更加复杂，我就会坚持采取单独的行动。

（请注意：对于逻辑，我指的是控制器逻辑，所有数据逻辑无论如何都应该在模型中）