物体拒绝自己是否正确？

Question

我想解析几种文件格式。我想知道“冒险”免费创建对象是否是正确的 OOP。

class ParserFactory
{
    private fn;

    public function ParserFactory(fn)
    {
        this->fn = fn;
    }
    public function getParser()
    {
        a = new FormatAParser(this->fn);
        if ( a->isValid() )
        {
            return( a );
        }
        b = new FormatBParser(this->fn);
        // ... and so on...
    }
}

class FormatAParser
{
    /*
        The object is telling us if is able to continue to work...
        **CLEAN OR DIRTY DESIGN ?**
    */
    public function isValid()
    {            
        header = SomeConversionAndReadingStuff();
        if ( header != "formatA" )
        {
            return(false)
        }
        return(true);
    }
    public function parse()
    {
        /*
        Do the parsing, using the conversion stuff done in isValid
        */
    }
}

谢谢

编辑
我有很好的答案。因此，对象可以检查其自身的有效性。无论如何，由于我选择解析器（格式检测）的过程方式，我的代码不是 OOP。为了改善这一点，最好使用像这样的工厂模式（PHP 代码）：

class Parser
{
    protected $raw;

    public function setRaw($raw)
    {
        $this->raw = $raw;
    }
}

class ParserA extends Parser
{
    public function __construct()
    {
        echo "ParserA constructor\n";
    }

    public function isValid()
    {
        if ( $this->raw == "A" )
        {
            return( true );
        }
        return(false);
    }
}

class ParserB extends Parser
{
    public function __construct()
    {
        echo "ParserB constructor\n";
    }
    public function isValid()
    {
        if ( $this->raw == "B" )
        {
            return( true );
        }
        return(false);
    }
}

class ParserFactory
{
    static private $parserClasses = array();

    public static function registerParser($parserClassName)
    {
        self::$parserClasses[] = $parserClassName;
    }

    public static function getParser($raw)
    {
        foreach( self::$parserClasses as $parserClass )
        {
            $parser = new $parserClass();
            $parser->setRaw($raw);
            if ( $parser->isValid() )
            {
                return( $parser );
            }
        }
    }
}

ParserFactory::registerParser("ParserA");
ParserFactory::registerParser("ParserB");
ParserFactory::getParser("B");

Answer 1

更常见的设计是

public function getParser()
{
    if (FormatAParser::IsRecognizedHeader(this->fn)
    {
        return new FormatAParser(this->fn);
    }
    if (FormatBParser::IsRecognizedHeader(this->fn)
    {
        return new FormatBParser(this->fn);
    }
}

使用类而不是对象方法来检查标头。一个明显的变化是收集列表中的不同解析器，而不是像这样手动循环它们。这需要能够将类（而不是对象）添加到列表中。

Answer 2

我不认为让一个对象理解并评估其自身的有效性有什么不面向对象的地方。您始终可以将 isValid 方法设为静态并传入“条件”（在本例中为标头）。这样，您不必实例化解析器，直到您知道需要它。

Answer 3

我认为这里可能的非面向对象性来自于这样一个事实：您有多个 if 语句，每个语句显然都检查某种类型（在本例中是一种文件类型）。这与我们的观念一致，即应该有一个类来代表每种文件类型；这种切换通常是一种代码味道，表明设计中的某些东西不太正确。然而，实际上，我认为你的解决方案是好的，但 MSalters 更好，因为很难知道解析器将如何工作，因为文件本身无法为你自动生成类实例。