Symfony2サービス

私はDBに2つのテーブルを持っている（questionとanswer）。1つの質問には多くの答えがあります。

私はいくつかを取得し、結果の配列を準備することにAnswers依存しquestion.typeます。

フレームワークのないアプリでは、Factory特定のオブジェクト（、、）を返すクラスがDBに依存しています。すべては抽象クラス拡張を宣言した方法を。すべてのタイプには、結果を準備するための独自のビジネスロジックがあります。SingleChoiceQuestionOpenQuestionMultipleChoiceQuestionquestion.typeQuestionsQuestionabstractgetResults

したがって、この状況では、ファクトリでオブジェクトを作成したときにメソッドgetResultsを使用しており、すべてが正常に機能しています。

私はsymfonyでそれを作成したいと私は、ドキュメントを読んでください。私の意見では、私はすべての私のためのサービスを作成する必要がありQuestionタイプを。

結果の配列を返すAggregatedResultsManagerメソッドgenerateで作成しました。特定のからメソッドをquestion.type呼び出すことに依存します。getResultsservice

DB構造を変更できないことを付け加えたいと思います。

私の質問：

1. 私が作成し、使用していますservicesよね？間違った場合は、理解を助け、正しい方法を教えてください。
2. 私はAggregatedResultsManager約18の質問タイプのようないくつかのサービスを持っています。

各サービスswitchで18の選択肢を作成する必要がありますが、それを防ぐにはどうすればよいですか？

switch ($this->question->getType()) {
    case Question::SINGLE:
        $results = $this->container->get('app.single_choice_question')->getResults($answers);
        break;
    // other types
}

タイプとサービス名を使用して配列を作成するアイデアがあります。

$services = [
    Question::SINGLE => 'app.single_choice_question',
    Question::MULTIPLE => 'app.multiple_choice_question',
    Question::OPEN => 'app.open_question',
];

次に、次のように各サービスで使用します。

$results = $this->container->get($services[$this->question->getType()])->getResults($answers);

18の選択肢があるスイッチを使わないのが最善の方法だと思います。ただし、サービス名を配列にハードコーディングする必要があります。

私のコード：

services.yml

app.question:
    class: AppBundle\Questions\Question
    abstract: true
    arguments: ['@doctrine.orm.entity_manager']

app.single_choice_question:
    class: AppBundle\Questions\SingleChoice
    parent: app.question

app.agreggated_results_manager:
    class:  AppBundle\Results\AggregatedResultsManager
    arguments: ['@doctrine.orm.entity_manager', '@service_container']

抽象的な質問

abstract class Question
{
    /**
     * @var EntityManager
     */
    protected $em;

    public function __construct(EntityManager $em)
    {
        $this->em = $em;
    }

    abstract public function getResults($answers);
}

SingleChoice

class SingleChoice extends Question
{
    public function getResults($answers)
    {
        $results = [];

        // business logic

        return $results;
    }
}

結果

class AggregatedResultsManager
{
    /**
     * @var EntityManager
     */
    private $em;

    /**
     * @var Question
     */
    private $question;

    /**
     * @var ContainerInterface
     */
    private $container;

    public function __construct(EntityManager $em, ContainerInterface     $container)
    {
        $this->em = $em;
        $this->container = $container;
    }

    public function generate()
    {
        if (!$this->question) {
            throw new \LogicException('Question is not set');
        }

        $answers = $this->em
            ->getRepository('AppBundle:Answer')
            ->findBy(['question' => $this->question]);

        $results = [];

        if (empty($answers)) {
            return $results;
        }

        switch ($this->question->getType()) {
            case Question::SINGLE:
                $results = $this->container->get('app.single_choice_question')->getResults($answers);
                break;
            // other types
        }

        return $results;
    }


    public function setQuestion(Question $question)
    {
        $this->question = $question;
    }
}

コントローラ

public function questionIdsAction(Question $question)
{
    $resultsManager = $this->get('app.agreggated_results_manager');
    $resultsManager->setQuestion($question);
    $results = $resultsManager->generate();

    return new JsonResponse($results);
}