Улучшения PHP класса - PullRequest
Новая
       8

Улучшения PHP класса

0 голосов
/ 05 февраля 2011

Я сделал свой первый урок, используя TDD (SimpleTest).Это работает довольно хорошо.Этот класс анализирует XML-файл конфигурации и возвращает его в виде массива.Как я могу улучшить его (производительность, любые советы)?А как насчет ответственности класса?Может быть, XMLtoArray следует переместить в другой класс, я не знаю ... 

<?php
class Configuration
{
    private $domdocument_object;
    private $domxpath_object;

    public function __construct($filename) {
        $this->loadXML($filename);
        $this->domxpath_object = new DOMXPath($this->domdocument_object);
    }

    private function loadXML($filename)
    {
        if (!file_exists($filename))
        {
            throw new ConfigurationException('Configuration file not found');
        }

        $this->domdocument_object = $domdocument_object = new DOMDocument();
        $this->domdocument_object->preserveWhiteSpace = false;

        if (!$this->domdocument_object->load($filename))
        {
            throw new ConfigurationException('Malformed configuration file');
        }
    }

    public function get($path = '/*') {
        $configuration = array();

        $domnodelist_object = $this->domxpath_object->query($path);
        $configuration = $this->XMLToArray($domnodelist_object);

        /**
         * Get a configuration entry as string or array
         *
         * For example:
         * $xml = '<foo><bar>baz</bar></foo>'
         * $path = '/foo/bar/'
         * return just baz, as string instead of an array('baz');
         *
         * Another example:
         * $xml = '<foo><bar>baz</bar><lorem>ipsum</lorem></foo>';
         * $path = '/foo'
         * return just array('bar' => 'baz', 'lorem' => ipsum);
         * instead of array('foo' => array('bar' => 'baz', 'lorem' => ipsum));
         */
        while (!is_string($configuration) && count($configuration) == 1)
        {
            $configuration_values = array_values($configuration);
            $configuration = $configuration_values[0];
        }

        if (empty($configuration))
        {
            $configuration = null;
        }

        return $configuration;
    }

    public function XMLToArray(DOMNodeList $domnodelist_object) {
        $configuration = array();

        foreach ($domnodelist_object as $element)
        {
            if ($element->nodeType == XML_DOCUMENT_NODE)
            {
                if ($element->hasChildNodes())
                {
                    $configuration = $this->XMLToArray($element->childNodes);
                }
            }
            else if ($element->nodeType == XML_ELEMENT_NODE)
            {
                if (!$element->hasChildNodes())
                {
                    $configuration[$element->nodeName] = null;
                }
                else if (
                    $element->firstChild->nodeType == XML_TEXT_NODE ||
                    $element->firstChild->nodeType == XML_CDATA_SECTION_NODE
                )
                {
                    $configuration[$element->nodeName] = $element->nodeValue;
                }
                else if ($element->firstChild->nodeType == XML_ELEMENT_NODE)
                {
                    $configuration[$element->nodeName] = $this->XMLToArray($element->childNodes);
                }
            }
        }

        return $configuration;
    }
}
?>

Этот класс игнорирует атрибуты XML.Спасибо.

Ответы [ 2 ]

1 голос
/ 05 февраля 2011

Что-то, что выделялось для меня, это то, что вы создаете новый объект каждый раз, когда объект выполняется, вы должны хранить объект локально (в объекте), а затем он использует только 1 часть вашей памяти.

Вот изменения, которые я бы сделал: 

class Configuration
{
    private $domdocument_object;
    private $domxpath_object; //+

    public function __construct($filename)
    {
        $this->loadXML($filename);
        $this->domxpath_object = new DOMXPath($this->domdocument_object); //+
    }

    public function get($path = '/*')
    {
        //Remove the following
        $domxpath_object = new DOMXPath($this->domdocument_object);
    }
}

, а затем измените $domxpath_object на $this->domxpath_object, где это необходимо.

Но это действительно нужно перенести на CoderReview , поскольку это не по теме.

0 голосов
/ 05 февраля 2011

Это отрицательно сказывается на производительности: 

$xml = preg_replace("/>\s+</", "><", $xml);

Кроме того, надежность не гарантируется (это может привести к нежелательному изменению комментариев и разделов CDATA).Хотя найти лучшее решение нелегко.Итерирование по всем текстовым узлам, обрезка их будет более надежной, но не быстрее.

Если вы просто хотите сделать это, то SAX-парсер массива php или SimpleXML могут быть более подходящими.Оба варианта могут быть быстрее (я не тестировал).

...