vererbte Liste

[r*bot]

Ich möchte eine verschachtelte Liste erstellen. Die Liste enthält ein Array aus Items. Diese Items haben einen Namen und können ihrerseits wiederum ein Array von (abhängigen) Items enthalten. Und genau da hapert es.

Anstatt

Setze A.
Setze B.
Setze C.
Ich bin A.
- A hat C mit drin.
Ich bin B.

auszugeben, gibt mir dieser Fetzen Code

<?php

class Liste{

    var $myItems = array();

    function addItem( $item ){
        $this->myItems[] = $item;
        echo "Setze $item->name.<br>";
    }
    
}

class Item extends Liste{
    
    var $name;

    function Item( $name ){
        $this->name = $name;
    }

    function showItems(){
        echo "Ich bin $this->name.<br>";
        while( list( $key, $value ) = each ( $this->myItems )){
            echo "- $this->name hat $value->name mit drin.<br>";
            $value->showItems();
        } 
    }

}

$list = new Liste();
$a = new Item( "A" );
$list->addItem( $a );

$b = new Item( "B" );
$list->addItem( $b );

$c = new Item( "C" );
$a->addItem( $c );

while( list( $key, $value ) = each( $list->myItems )){
    $value->showItems();
}

?>

nur ein

Setze A.
Setze B.
Setze C.
Ich bin A.
Ich bin B.

aus.

Sieht vielleicht jemand von Euch meinen Fehler?
Danke für Eure Mühe, auch wenn Ihr nix finden solltet.

[oms]

Hi!
Das kann nicht klappen, weil du keine Referenz auf das $a Objekt übergibst. Somit legt er eine Kopie von $a und hängt diese an deine Liste ran. Wenn du dann $a ein Item hinzufügst, hat das dann keine Auswirkungen auf das kopierte $a Objekt in der Liste.
Mit

$a = new Item( "A" );
$list->addItem( &$a ); 

sollte es klappen.
Gruß oms

[pauino]

hi

oms hat recht, du musst das aber in der methode vereinbaren (glaube ich) also :

    function addItem( &$item ){
        $this->myItems[] = &$item;
        echo "Setze $item->name.<br>";
    } 

so funktioniert es bei mir
gruesse von pau

[r*bot]

Herrlich. Tausend Dank.

Natürlich muss es ne Referenz sein... *andenkoppklatsch*

Allerdings ist "Call by Reference" wie oms es vorgeschlagen hat depreciated. Die Vereinbarung, dass es eine Referenz ist, muss in der Methode stehen.

[oms]

Aber doch _nur_ in der Parameterliste, oder?!

[pauino]

--> oms

hatte ich auch erst gedacht geht aber nicht
(sehr ungewoehlich)

gruesse von pau

[oms]

Ja gut, es erscheint mir logisch!
Sonst würde man das Item von &$item nach $this->myItems[] ja wieder kopieren.

Grüße oms