我的表单有什么问题？

Question

我在尝试将表单的信息保存到数据库时遇到问题。即使在选定的网络中为每个影院手动设置剧院ID之后，我的表单似乎仍然无效。 这里是我的模块的的actions.class.php相关部分：

这里的executeCreate（）：

public function executeCreate(sfWebRequest $request) { 
    $this->form = $this->configuration->getForm(); 
    $this->showing = $this->form->getObject(); 
    $this->processCreateForm($request, $this->form); 
    $this->setTemplate('new'); 
} 

现在processCreateForm（）：

protected function processCreateForm(sfWebRequest $request, sfForm $form) { 
    $form->bind($request->getParameter($form->getName()), $request->getFiles($form->getName())); 

    $form_name = $form->getName(); 
    $parameters = $request->getParameter($form_name); 
    $network_id = $parameters['network_id']; 

    $theaters_list = Doctrine_Query::create() 
      [...] 
      ->execute(); 

    foreach ($theaters_list as $theater) { 
     $form->getObject()->setTheaterId($theater->theater_id); 
     $form->bind($request->getParameter($form->getName()), $request->getFiles($form->getName())); 

     if ($form->isValid()) { 
      $showing = $form->save(); 
     } else { 
      foreach ($form->getErrorSchema()->getErrors() as $key => $error) { 
       echo '<p>' . $key . ': ' . $error . '</p>'; 
      } 
     } 
    } 
    $this->getUser()->setFlash('update_success', true); 
    $this->setTemplate('new'); 
} 

这里的输出：

谢谢您的帮助

Answer 1

有两个奇怪的事情会在这里，我想破坏你的代码。

1. 您运行bind()方法两次，这可能会重置对象上的值。

2. 我不认为getObject()方法通过引用返回对象。

所以，当你运行：

$form->getObject()->setX($val); 
    $form->save(); 

那么你更新由形式返回的对象的字段，但是然后保存它仍然是绑定到形式的原始对象。

尝试做这样的事情：

$myObject = $form->updateObject()->getObject(); 
    $myObject->setX($value); 
    $myObject->save(); 

如果您使用表单编辑现有的对象，而不是创建一个新的updateObject()是很重要的。没有这个，你会得到对象的旧值。

如果你想在循环中运行它，你只能循环设置和保存部分。所以，你会有这样的事情在你的processCreateForm：

protected function processCreateForm(sfWebRequest $request, sfForm $form) 
{ 
    $form->bind($request->getParameter($form->getName()), $request->getFiles($form->getName())); 

    if ($form->isValid()) { //You can check the validity of your form at this point. 

     //Create $theatersList 
     ... 

     $myObject = $form->updateObject(); 

     foreach ($theatersList as $theater) { 
      $myObject->setTheaterId($theater->theater_id); 
      $showing = $myObject->save(); 

      //Do something with $showing 

     } 
    } else { 
     //Print the errors. 
    } 
} 

使用此代码，你可以取消设置在您的形式theatre_id小部件，因为它不应该由用户设置，并且不必是表单的一部分验证。

编辑

一些修改代码：

protected function processCreateForm(sfWebRequest $request, sfForm $form) 
{ 
    $form->bind($request->getParameter($form->getName()), $request->getFiles($form->getName())); 

    if ($form->isValid()) { //You can check the validity of your form at this point. 

     //Create $theatersList 
     ... 

     $myObject = $form->updateObject(); 
     $myObjectVars = $myObject->toArray(); 

     foreach ($theatersList as $theater) { 

      $myNewObject = new SomeClass(); 
      $myNewObject->fromArray($myObjectVars); 
      $myNewObject->setTheaterId($theater->theater_id); 
      $showing = $myNewObject->save(); 

      //Do something with $showing 

      $myNewObject->free(); 
      unset($myNewObject); 
     } 
    } else { 
     //Print the errors. 
    } 
}