0

これが私が試したものですが、数字が間違っていて、理由がわかりません

ここに画像の説明を入力してください

1、2、3、4、5などである必要があります。

これが私のPHPコードです：

function GetNavigations(SimpleXMLElement $element, $level = 0, $mrg = 0)
{   
    $value = trim((string) $element); 
    $children = $element->children(); 
    $attributes = $element->attributes();
    //echo '<ul>';  
    if(count($children) == 0 && !empty($value))
    {   
    if($element->getName() == 'GroupName')
        {
            if($attributes['ParentId'] != '')
            {
                //$mrg = $level/2 * 10;
                echo '<li>'.$mrg.'<a class="btngroup" href="load.php?active=menu&group_name_id='.$attributes['GroupNameId'].'">'.$element.'</a></li>';
            }   
        }
    }

    if(count($children))
    {
        foreach($children as $child)
        {
            GetNavigations($child, $level+1, $mrg+1);
        } 
    }
    //echo '</ul>';
}
4

1 に答える 1

1

GetNavigations($child, $level+1, $mrg+1)$mrgノードのすべての子に同じ値を渡しますが、$mrgそのループ内の他の場所では変更されていないため、多くの子があります。の代わりに$mrg+1、渡すことができます++$mrg-またはもっと読みやすく、$mrg++;前の行として追加して、単に渡すことができます$mrg。

ただし、関数が表示された直接の子の数だけを認識し、子孫の数は認識しないという問題があります。値をで呼び出しGetNavigations、ネストされたアイテムを20個表示すると、次の値は！ではなくになります。それらはすべて同じ名前ですが、関数を実行するたびに、新しい変数があります。$mrg2$mrg323$mrg

これを回避するには、次のいずれかを実行できます。

  • $mrg参照によって渡します（関数宣言をfunction GetNavigations(SimpleXMLElement $element, $level = 0, &$mrg)追加されたものに変更して、関数&のすべてのコピーが同じ変数に書き込めるようにします。
  • $mrgoutの新しい値をreturn関数の値として渡します。

何が起こっているのかコードを読んでいる人には少しわかりやすいので、私はおそらく2番目のアプローチを好むでしょう：

function GetNavigations(SimpleXMLElement $element, $level = 0, $mrg = 0)
{   
    /* [snip] */

    if($element->getName() == 'GroupName')
    {
            // Increment counter, because we're displaying something
            $mrg++;

            /* [snip] */
    }

    /* [snip] */

    if(count($children))
    {
        foreach($children as $child)
        {
            // Recurse, and get incremented value of counter
            $mrg = GetNavigations($child, $level+1, $mrg);
        } 
    }

    /* [snip] */

    // Let caller know where the counter has got to
    return $mrg;
}
于 2013-02-16T21:47:19.127 に答える