Question

I am currently writing a text file parser for several formats. While the text file is parsed, there are different kinds of operations to do. I am trying to do something clean using OOP. Here is where I am stuck :

abstract class Parser
{
    abstract protected function DoSomeStuff($data);

    public function Parse($src)
    {
        if ( $this->GetFormat($src) == 1 )
        {                       
            $data =$this->GetSomeDataFromFormat1($src);
            DoSomeStuff($data);
        }
        if ( $this->GetFormat($src) == 2 )
        {
            $data = $this->GetSomeDataFromFormat2($src);
            DoSomeStuff($data);
        }
    }   
}

class DoSomething extends Parser
{
    protected function DoSomeStuff($data)
    {
        // Doing some stuff with data
    }
}

class DoSomethingElse extends Parser
{
    protected function DoSomeStuff($data)
    {
        // Doing some other stuff with data
    }
}

$ds = new DoSomething();
$ds->Parse(...);

$dse = new DoSomethingElse();
$dse->Parse(...);

As you can see : all the code for all files formats is in class Parser. What can I do to make this cleaner ?

Thanks Antoine

Answer 1

诸如Strategy patterns可能有助于清理。

另见:StackOverflow: [java] Strategy patterns search。

Answer 2

遵循每一级档案规则通常是一个好主意。

除此以外,在清理工作方面,还有许多工作要做。

唯一要决定的另一点是,你如何装上每一类档案,可以采取几种方式。 I d 建议在工厂类别中登记。

Answer 3

如果您返回一些诸如“CSV”或“XML”之类的物品,那么就不易了解什么是最佳的。

另一种情况是:

if ( $this->GetFormat($src) == 1 )
{
}
if ( $this->GetFormat($src) == 2 )
{
}

//This should be a switch
$src = $this->getFormat($src);

switch($src){
    case 1:

    break;
    case 2:

    break;
}

这样一来,《刑法》便无法在一劳永逸时装上两倍的方法,而开关的记忆却少于一种情况。

并且最后一点可以找到

GetSomeDataFromFormat1 and GetSomeDataFromFormat2

这可能是你给我们的代码,但摘要类别只应指在其类别内提供的其他摘要方法或方法,这些方法或方法应当要求使用者知道他们必须增加两种方法,而不是像你的代码一样容易阅读,而后者是编码的主要方面。

友情链接