Какой паттерн подходит для этого?

Question

[WQP]

Необходимо как-то связать кучу классов, которые бы форматировали код и возвращали его. При этом точка входа у него должна быть одна.

Что-то типа этого:

FormatterInterface

interface FormatterInterface
{
    public function format($data, $type);
}

Formatter

class Formatter
{
    static public function execute(FormatterInterface $formatter, $type, $data)
    {
        return $formatter->format($data, $type);
    }
}

PresetFormatter (таких классов будет уже куча)

class PresetFormatter implements FormatterInterface
{
    private $data;

    private function traffic($data)
    {
        // тут код форматирования для traffic
    }

    private function conversion($data)
    {
        // тут код форматирования для conversion
    }

    public function format($data, $type)
    {
        switch ($type) {
            case 'traffic':
                $this->traffic($data);
                break;
            case 'conversion':
                $this->conversion($data);
                break;
        }

        return $this->data;
    }
}

Formatter::execute('PresetFormatter', 'traffic', [])

Все методы не получится указать в классе formater их будет очень много.

Как я понял всё классы надо связать интерфейсом

Comments

[Александр Шаповал]

Напишите пример того, чего вы пытаетесь этим добиться, какие методы хотите передать и зачем

[WQP]

Александр Шаповал: поправил вопрос

[Александр Шаповал]

WQP: передавайте метод не строкой а массивом в котором будет список методов, добавляйте их туда столько сколько нужно.

[WQP]

Александр Шаповал: обновил ещё раз. Такая реализация работает. Но меня терзают сомнения, что я не так сделал

[Александр Шаповал]

Или придумайте какие-то группы и передавайте эту группу вместе с данными.
Например группа для имени файла filename, передайте и для этой группы применяете необходимые методы.

[Александр Шаповал]

И еще вы не ответили зачем вам это нужно, обычно для форматирования текста не передают огромный список методов.

[WQP]

Александр Шаповал: я получаю по api огромный массив данных. Они в очень неудобном формате. И если я меняю запрос к api, то и структура ответа меняется, а там всё также неудобный формат. Методов запросов api много и для каждого необходимо сделать форматирование.

[Александр Шаповал]

Кажется начинаю понимать, у вас куча классов к которым нужно подключить нужные методы? Возможно вам подойдут трейты (trait) https://www.youtube.com/watch?v=nU7ieWzAPLo

[WQP]

Александр Шаповал: У меня есть класс назовём его API. И в нём есть методы


Дак вот к каждому методу надо сделать форматер. И хочу я это вынести в отдельный класс. Думал я о trait. Но если его использовать, то это будет огромный файл. Проще разбить каждый метод форматирования на классы. Хотя можно каждый метод разбить на trait. Но тогда получится, что в классе API, надо будет писать кучу use

[Александр Шаповал]

WQP: вы может не в ту сторону смотрите, для меня классы призваны добавить универсальности коду, его краткости и предотвращения дублирования, а вы хотите наплодить целую кучу классов.
Разделите ваши данные на группы как я говорил, я просто не знаю какие данные приходят и что вы с ними делаете.
Что бы вам помочь нужно знать какие конкретные данные приходят и что с ними нужно делать в тех или иных случаях.

Answer 1

[Матвей Правосудов]

Я думаю подойдет паттерн Фасад, который обеспечит единую точку входа компоненту.

Answer 2

[kpa6uu]

Стратегия решит проблему.
Каждый форматтер - отдельный класс. Все они реализуют один интерфейс.
Получать их можно через фабрику, в которой реализован кэш экземпляров объектов.

<?php

class FormatterFactory
{
	const TRAFFIC = 1;
	const CONVERSION = 2;

	protected static $cache = array();

	public static function get($formatterId)
	{
		if (static::isInCache($formatterId)) {
			return static::getFromCache($formatterId);
		}

		$formatter = static::createNewFormatter($formatterId);
		static::setToCache($formatterId, $formatter);

		return static::getFromCache($formatterId);
	}

	protected static function createNewFormatter($formatterId)
	{
		switch ($formatterId) {
			case static::TRAFFIC:
				$object = new TrafficFormatter();
				break;

			case static::CONVERSION:
				$object = new ConversionFormatter();
				break;

			default:
				throw new Exception("Unknown formatterId: {$formatterId}");
				break;
		}

		return $object;
	}

	protected static function isInCache($formatterId)
	{
		return isset(static::$cache[$formatterId]);
	}

	protected static function setToCache($formatterId, FormatterInterface $formatter)
	{
		static::$cache[$formatterId] = $formatter;
	}

	protected static function getFromCache($formatterId)
	{
		return isset(static::$cache[$formatterId]) ? static::$cache[$formatterId] : null;
	}
}

interface FormatterInterface
{
	public function format($data);
}

class TrafficFormatter implements FormatterInterface
{

	public function format($data)
	{
		return $data . 'traffic';
	}
}

class ConversionFormatter implements FormatterInterface
{
	public function format($data)
	{
		return $data . 'conversion';
	}
}

class Formatter
{
	/**
	 * @param FormatterInterface[] $formatters
	 * @param string $data
	 * @return string
	 */
	static public function format($formatters, $data)
	{
		foreach ($formatters as $formatter) {
			$data = $formatter->format($data);
		}

		return $data;
	}
}

$data = 'hello';

$data = Formatter::format(array(
	FormatterFactory::get(FormatterFactory::TRAFFIC),
	FormatterFactory::get(FormatterFactory::CONVERSION),
), $data);

echo $data; // hellotrafficconversion