Question

让我们说Human可以在口袋里放Item个。每个Item对Human都有不同的影响。

当他使用某个项目时，会转到ItemController：

class ItemController extends Controller
{
    public function useAction() {
       // Human
        $human = new Human();

       // Item
        $request = Request::createFromGlobals();
        $item_id = $request->request->get('item_id', 0);
        $item = new Item($item_id);

        // Different effects depending on the Item used
        switch($item->getArticleId()) {

            case 1: $this->heal($human, $item, 5); break; // small potion
            case 2: $this->heal($human, $item, 10); break; // medium potion
            case 3: $this->heal($human, $item, 15); break; // big potion

        }

    }

    // The following Heal Function can be placed here ?
    private function heal($human, $item, $life_points) {
        $human->addLife($life_points);
        $item->remove();
        return new Response("You have been healed of $life_points");
    }
}

heal function可以放在这里吗？我相信它不应该在控制器中。但我也相信它不应放在Item Entity内（因为响应，因为它使用$ Human）

Answer 1

这取决于。我对这类问题的推理是这样的：如果我只使用控制器中的函数，它就可以保留在那里。但如果它可能是一个共享功能，我会为它创建一个服务。也许你希望能够通过命令或不同的控制器或其他东西来治愈人类。在这种情况下，为此使用共享代码是有意义的。

Creating a service非常简单，可让您将逻辑保存在共享位置。在我看来，控制器对处理请求流更有用。

Answer 2

我认为你可以这样做：

1：继承

  class BaseHumanController extend Controller{

  public function heal($param1, $param2, $param3){

  //put logic here

  }

} 

//Extend from BaseHumanController in any controller for call heal() method
class ItemController extend BaseHumanController{

//....
 $this->heal(...)

}

2：使用heal（）方法创建一个类，并将其配置为@Peter Kruithof服务

Answer 3

我绝对认为它会进入控制器。来自维基百科：

控制器调解输入，将其转换为模型的命令或查看。

如果你看一下symfony2 crud生成器生成的delete函数，它会在实体上调用remove：

/**
 * Deletes a Bar entity.
 *
 * @Route("/{id}/delete", name="bar_delete")
 * @Method("post")
 */
public function deleteAction($id)
{
    $form = $this->createDeleteForm($id);
    $request = $this->getRequest();

    $form->bindRequest($request);

    if ($form->isValid()) {
        $em = $this->getDoctrine()->getEntityManager();
        $entity = $em->getRepository('FooBundle:Bar')->find($id);

        if (!$entity) {
            throw $this->createNotFoundException('Unable to find Bar entity.');
        }

        $em->remove($entity);
        $em->flush();
    }

    return $this->redirect($this->generateUrl('bar_index'));
}

Answer 4

我认为没有理由不应该在控制器中。如果你打算只在那里使用它，那就把它作为私有方法。

在Symfony2中，我应该在哪里放置我的控制器使用的功能？

4 个答案: