Шаблон фасада, это нормально? - PullRequest
Новая
       5

Шаблон фасада, это нормально?

0 голосов
/ 30 сентября 2011

У меня есть два сервера, к которым я буду подключаться с одного клиента.Для каждого сервера я буду использовать ftp "put" и "rm".

Должен ли я построить один фасад и иметь такой интерфейс: 

void putFileOnServer1(String file)
void putFileOnServer2(String file)
void removeFromServer1(String file)
void removeFromServer2(String file)

И, еслиФасад обрабатывать все налаживания соединений и отсоединения?Если да, то должен ли он использовать фабрику для этого?

Ответы [ 3 ]

1 голос
/ 30 сентября 2011

У ftp-серверов разные интерфейсы?Или все они понимают один и тот же набор команд, которые вы хотите использовать уже?

  1. Если это так, то просто создайте один класс FtpServer, который принимает информацию о соединении.И создайте класс FtpClient, который принимает несколько серверов, которые вы можете выбрать, например, по некоторому ключу.(По крайней мере, это, в некоторой степени, вероятно, было бы то, что я бы сделал). 

    class FtpClient
{
    public function addServer( FtpServer $server, $key );

    public function selectServer( $key );

    public function putFileOnServer( $file );

    public function removeFileFromServer( $file );
}

  2. Если нет, и у вас уже есть класс для каждой отдельной реализации, которые отличаются наих интерфейсы, например: 

    class FtpServerFoo
{
    public function selectFile( $file );
    public function removeSelectedFile();
}

class FtpServerBar
{
    public function removeFile( $file );
}

    ... вы должны взглянуть на Pattern Adapter : 

    abstract class FtpServer
{
    abstract public function putFile( $file );
    abstract public function removeFile( $file );
}

class FtpServerAdapterFoo
    extends FtpServer
{
    public function __construct( FtpServerFoo $server )
    {
    }

    public function removeFile( $file )
    {
        $this->server->selectFile( $file );
        $this->server->removeSelectedFile();
    }
}

class FtpServerAdapterBar
    extends FtpServer
{
    public function __construct( FtpServerBar $server )
    {
    }

    public function removeFile( $file )
    {
        $this->server->removeFile( $file );
    }
}

$cilent = new FtpClient();
$client->addServer( new FtpServerAdapterFoo( new FtpServerFoo() ), 0 );
$client->addServer( new FtpServerAdapterBar( new FtpServerBar() ), 1 );

$client->selectServer( 0 );
$client->putFileOnServer( $file );

$client->selectServer( 1 );
$client->removeFileFromServer( $someOtherfile );

  3. Если выпока нет отдельных классов для разных FTP-серверов, тогда вы можете просто реализовать один и тот же интерфейс (или наследовать абстрактный класс) для каждой реализации ftp-сервера и использовать тот же тип класса FtpClient, что и выше.

    Хотя на самом деле здесь не задействован рисунок фасада.

1 голос
/ 30 сентября 2011

У вас есть два метода: PutFileOnServer и RemoveFromServer. Какой сервер вы устанавливаете или удаляете, должен быть частью абстракции.

0 голосов
/ 30 сентября 2011

Обычно вы используете фасады для уменьшения сложности объектов разных типов. В этом случае, однако, кажется, что вы хотите использовать только один функциональный «тип», «FTPServer». Таким образом, по большей части у вас должно быть два экземпляра этого типа, и этот тип будет иметь методы «put» и «remove».

Когда вы добавляете ненужные функциональные точки, вы фактически увеличиваете сложность обслуживания. Например, если вам нужно добавить новый параметр в ваши функции (может быть, ограничения доступа или что-то еще), вам нужно не только изменить для каждого места использования, но теперь необходимо добавить в каждый метод фасада. Абстракция должна уменьшать этот тип связи, а не увеличивать ее.

...