Question

在Symfony 4项目中，我有两个具有相同依赖项的命令（可能还会有更多命令）。目前，对于这两个命令，我在代码中重复执行一些代码，我知道我应该能够使它更加干燥和高效，但是不确定如何组织它。

到目前为止，这是我的情况：

# config/services.yaml

parameters:
    api_client_id: '%env(API_CLIENT_ID)%'
    api_client_secret: '%env(API_CLIENT_SECRET)%'
    api_client_id_sandbox: '%env(API_CLIENT_ID_SANDBOX)%'
    api_client_secret_sandbox: '%env(API_CLIENT_SECRET_SANDBOX)%'
    api_env: '%env(API_ENV)%'

services:
    # default configuration for services in *this* file
    _defaults:
        autowire: true      # Automatically injects dependencies in your services.
        autoconfigure: true # Automatically registers your services as commands, event subscribers, etc.
        public: false       # Allows optimizing the container by removing unused services; this also means
                            # fetching services directly from the container via $container->get() won't work.
                            # The best practice is to be explicit about your dependencies anyway.

    # makes classes in src/ available to be used as services
    # this creates a service per class whose id is the fully-qualified class name
    App\:
        resource: '../src/*'
        exclude: '../src/{Entity,Migrations,Tests,Kernel.php}'

    # controllers are imported separately to make sure services can be injected
    # as action arguments even if you don't extend any base controller class
    App\Controller\:
        resource: '../src/Controller'
        tags: ['controller.service_arguments']

    # add more service definitions when explicit configuration is needed
    # please note that last definitions always *replace* previous ones


    # Registering 3rd Party API Manager.
    app.api_service_factory:
        class: App\Service\APIServiceFactory
        arguments:
            - "%api_client_id%"
            - "%api_client_secret%"
            - "%api_client_id_sandbox%"
            - "%api_client_secret_sandbox%"
            - "%api_env%"  


    # Collect promotions command.
    App\Command\CollectPromotionsCommand:
        arguments:
            - "@app.api_service_factory"
        tags:
            - { name: 'console.command', command: 'app:collect-promotions' }


    # Processes the reserved topups orders.
    App\Command\ProcessReservedTopupsCommand:
        arguments:
            - "@app.api_service_factory"
        tags:
            - { name: 'console.command', command: 'app:process-reserved-topups' }

<?php
// src/Command/CollectPromotionsCommand.php

namespace App\Command;

use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Style\SymfonyStyle;
use App\Service\APIServiceFactory;
use Doctrine\ORM\EntityManagerInterface;

class CollectPromotionsCommand extends Command
{

  /**
   * @var APIServiceFactory
   */
  protected $apiServiceFactory;

  /**
   * @var EntityManagerInterface
   */
  protected $em;

  protected static $defaultName = 'app:collect-promotions';

  public function __construct(APIServiceFactory $apiServiceFactory, EntityManagerInterface $em) {
    $this->apiServiceFactory  = $apiServiceFactory;
    $this->em                         = $em;

    parent::__construct();
  }

// ...


}

<?php
// src/Command/ProcessReservedTopupsCommand.php

namespace App\Command;

use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Style\SymfonyStyle;
use App\Service\APIServiceFactory;
use Doctrine\ORM\EntityManagerInterface;

class ProcessReservedTopupsCommand extends Command
{

  /**
   * @var APIServiceFactory
   */
  protected $apiServiceFactory;

  /**
   * @var EntityManagerInterface
   */
  protected $em;

  protected static $defaultName = 'app:collect-promotions';

  public function __construct(APIServiceFactory $apiServiceFactory, EntityManagerInterface $em) {
    $this->apiServiceFactory  = $apiServiceFactory;
    $this->em                         = $em;

    parent::__construct();
  }

// ...


}

任何帮助使此代码看起来更体面的方法都非常感谢。

Answer 1

Personnaly I wouldn't do it because I prefer clarity even if it implies a bit of duplication.

But if you are wondering how to avoid code duplication in your serivces' constructiors and configuration here is the answer: https://symfony.com/doc/current/service_container/parent_services.html. It is about introdusing a base class for your commands. Symfony let you avoid duplication in configuration as well.

Your example will look like :

class BaseCommand extends Command
{
  /**
   * @var APIServiceFactory
   */
  protected $apiServiceFactory;

  /**
   * @var EntityManagerInterface
   */
  protected $em;

  public function __construct(APIServiceFactory $apiServiceFactory, EntityManagerInterface $em) {
    $this->apiServiceFactory  = $apiServiceFactory;
    $this->em = $em;

    parent::__construct();
  }

  // ...
}

and

class CollectPromotionsCommand extends BaseCommand
{
    // ...
}

and

class ProcessReservedTopupsCommand extends BaseCommand
{
    // ...
}

Configuraiton:

# Base API command depending on API.
App\Command\BaseCommand:
    abstract: true
    arguments:
        - "@app.api_service_factory"

# Collect promotions command.
App\Command\CollectPromotionsCommand:
    parent: App\Command\BaseCommand
    tags:
        - { name: 'console.command', command: 'app:collect-promotions' }


# Processes the reserved topups orders.
App\Command\ProcessReservedTopupsCommand:
    parent: App\Command\BaseCommand
    tags:
        - { name: 'console.command', command: 'app:process-reserved-topups' }

Symfony具有相同依赖项的多个命令类：避免代码重复

1 个答案: