创建对象时如何应用开闭原则

Question

我正忙于解析 xml 文档（google docs api）并将各个文档放入对象中。

有不同类型的文档（文档、电子表格、演示文稿）。这些文档的大多数信息是相同的，但有些信息有所不同。

这个想法是创建一个基文档类来保存所有共享信息，同时为每个特定文档类型使用子类。

问题是为不同类型创建正确的类。有两种方法可以区分文档的类型。每个条目都有一个类别元素，我可以在其中找到类型。将使用的另一种方法是通过resourceId，采用type:id 的形式。

最简单的选择是创建一个 if 语句（或 switch 语句）检查条目的类型，并为其创建相应的对象。但如果要添加新类型，则需要编辑代码。

现在我不确定是否有其他方法可以解决这个问题，所以这就是我在这里问的原因。我可以将正确类型的对象的创建封装在工厂方法中，因此所需的更改量很小。

现在，我有这样的事情：

public static function factory(SimpleXMLElement $element)
{
    $element->registerXPathNamespace("d", "http://www.w3.org/2005/Atom");
    $category = $element->xpath("d:category[@scheme='http://schemas.google.com/g/2005#kind']");

    if($category[0]['label'] == "spreadsheet")
    {
        return new Model_Google_Spreadsheet($element);
    }
    else
    {
        return new Model_Google_Base($element);
    }
}

所以我的问题是，是否有另一种我没有看到的方法来处理这种情况？

编辑： 添加了示例代码

Answer 1

用您的代码示例更新了答案

这是您的新工厂：

public static function factory(SimpleXMLElement $element)
{
    $element->registerXPathNamespace("d", "http://www.w3.org/2005/Atom");
    $category = $element->xpath("d:category[@scheme='http://schemas.google.com/g/2005#kind']");
    $className = 'Model_Google_ '.$category[0]['label'];
    if (class_exists($className)){
       return new $className($element);
    } else {
        throw new Exception('Cannot handle '.$category[0]['label']);
    }
}

我不确定我是否完全理解您的观点...为了改写问题，我理解“如何创建正确的对象而不在客户端代码中对选择进行硬编码”

使用自动加载

所以让我们从基本客户端代码开始

class BaseFactory
{
    public function createForType($pInformations)
    {
       switch ($pInformations['TypeOrWhatsoEver']) {
           case 'Type1': return $this->_createType1($pInformations);
           case 'Type2': return $this->_createType2($pInformations);
           default : throw new Exception('Cannot handle this !');
       }
    }
}

现在，让我们看看是否可以更改它以避免 if / switch 语句（并不总是必需的，但可以）

我们在这里将使用 PHP 自动加载功能。

首先，考虑自动加载已经就位，这是我们的新工厂

class BaseFactory
{
    public function createForType($pInformations)
    {
       $handlerClassName = 'GoogleDocHandler'.$pInformations['TypeOrWhatsoEver'];
       if (class_exists($handlerClassName)){
           //class_exists will trigger the _autoload
           $handler = new $handlerClassName();
           if ($handler instanceof InterfaceForHandlers){
               $handler->configure($pInformations);
               return $handler;
           } else {
               throw new Exception('Handlers should implements InterfaceForHandlers');
           }
       }  else {
           throw new Exception('No Handlers for '.$pInformations['TypeOrWhatsoEver']);
       }
   }
}

现在我们必须添加自动加载功能

class BaseFactory
{
    public static function autoload($className)
    {
        $path = self::BASEPATH.
                $className.'.php'; 

        if (file_exists($path){
            include($path); 
        }
    }
}

，您只需像

spl_autoload_register(array('BaseFactory', 'autoload'));

现在一样注册您的自动加载器，每次您必须为类型编写新的处理程序时，它都会自动额外。

使用责任链

您可能不想在工厂中编写更“动态”的东西，使用处理多个类型的子类。

例如

class BaseClass
{
    public function handles($type);
}
class TypeAClass extends BaseClass
{
    public function handles($type){
        return $type === 'Type1';
    }
}
//....

，在 BaseFactory 代码中，您可以加载所有处理程序并执行类似的操作

class BaseFactory
{ 
    public function create($pInformations)
    {
        $directories = new \RegexIterator(
            new \RecursiveIteratorIterator(
                new \RecursiveDirectoryIterator(self::BasePath)
            ), '/^.*\.php$/i'
        );

        foreach ($directories as $file){
            require_once($fileName->getPathName());
            $handler = $this->_createHandler($file);//gets the classname and create it
            if ($handler->handles($pInformations['type'])){
                return $handler;
            }
        }
        throw new Exception('No Handlers for '.$pInformations['TypeOrWhatsoEver']);
    }
}

Answer 2

我同意Oktopus的观点，一般有两种无需硬编码的方法；第一种方法是通过动态附加字符串来查找类名，并确保您的类被正确命名，第二种方法是加载所有处理程序类，并使用表示可以处理该类型的类。我会选择第一个。对于您的示例代码，这将类似于以下内容：

<?php
public static function factory(SimpleXMLElement $element)
{
    $element->registerXPathNamespace("d", "http://www.w3.org/2005/Atom");
    $category = $element->xpath("d:category[@scheme='http://schemas.google.com/g/2005#kind']");

    $classname = sprintf( 'Model_Google_%s', ucfirst( strtolower( (string) $category[0]['label'] ) ) );

    if( class_exists( $classname, true /* true means, do autoloading here */ ) ) {
        return new $classname( $element );
    }
    return new Model_Google_Base($element);
}

在责任链上：虽然这是一个关于如何编写它的漂亮示例，但我发现责任链意味着所有可能的处理程序必须加载并询问它们是否可以处理该类型。从我的角度来看，生成的代码更干净、更明确，但性能受到影响，复杂性也增加。这就是为什么我会选择动态类名解析。

Answer 3

也许您可以对输入文件应用 xsl 转换（在工厂方法中）。
转换的结果应该是一个统一的 xml 文件，提供有关要使用哪个类的输入？

我知道这并不比许多 if 更好，但这样，您至少可以避免重写代码。