Возвращаясь к рекурсивной функции в php - PullRequest
1 голос
/ 10 декабря 2010

Я разрабатываю функцию для анализа 2 xml файлов.Он сравнивает их узел за узлом, а затем, если узлы разные, функция должна вернуть один из них.Но ничего не возвращается.

$xml = simplexml_load_file("file1.xml");
$xml2 = simplexml_load_file("file2.xml");

$result = parseNode($xml, $xml2);
print_r($result);
echo $result;

function parseNode($node1, $node2) {

    for ($i = 0; $i < count($node1->children()); $i++) {

        $child1 = $node1->children();
        $child2 = $node2->children();

        if ($child1[$i]->getName() != $child2[$i]->getName()) {
            return $child1[$i];
        } else {
            parseNode($child1[$i], $child2[$i]);
        }

    }
}

Ответы [ 4 ]

3 голосов
/ 10 декабря 2010
       return parseNode($child1[$i], $child2[$i]);
1 голос
/ 10 декабря 2010

Вы также можете перебирать структуры XML с помощью рекурсивных итераторов, чтобы упростить вашу функцию parseNodes().

$xml  = simplexml_load_string("<root><foo/><bar><baz/></bar></root>", "SimpleXMLIterator");
$xml2 = simplexml_load_string("<root><foo/><bar><baz/></bar><bat/></root>", "SimpleXMLIterator");

$result = parseNode($xml, $xml2);
echo $result;

function parseNode($a, $b) {
    $mit = new MultipleIterator(MultipleIterator::MIT_NEED_ANY|MultipleIterator::MIT_KEYS_NUMERIC);
    $mit->attachIterator(new RecursiveIteratorIterator($a, RecursiveIteratorIterator::SELF_FIRST));
    $mit->attachIterator(new RecursiveIteratorIterator($b, RecursiveIteratorIterator::SELF_FIRST));

    foreach ($mit as $node) {
        // One has more nodes than another!
        if (  ! isset($node[0], $node[1])) {
            return 'Curse your sudden but inevitable betrayal!';
        }
        // Nodes have different names
        if ($node[0]->getName() !== $node[1]->getName()) {
            return $node[0];
        }
    }

    // No differences in names and order
    return FALSE;
}

Настройка MultipleIterator довольно многословна (в основном из-за длинных имен классов über) но логика чертовски проста.

1 голос
/ 10 декабря 2010

Ну, вы можете сделать это с помощью простого условного утверждения ...

$xml = simplexml_load_file("file1.xml");
$xml2 = simplexml_load_file("file2.xml");

$result = parseNode($xml, $xml2);
print_r($result);
echo $result;

function parseNode($node1, $node2) {
    $child1 = $node1->children();
    $child2 = $node2->children();
    $numChildren = count($child1);
    for ($i = 0; $i < $numChildren; $i++) {
        if ($child1[$i]->getName() != $child2[$i]->getName()) {
            return $child1[$i];
        } else {
            $test = parseNode($child1[$i], $child2[$i]);
            if ($test) return $test;
        }
    }
    return false;
}
0 голосов
/ 10 декабря 2010

В рекурсивном вызове нет return.Следовательно, никакого результата.

РЕДАКТИРОВАТЬ Не голосовать за это. ircmaxell верно.Я удалил исключительную часть моего ответа.

...