Witam, zgodnie z przykladem w ksiazce zrobilem cos podobnego. Sa jednostki mozna je laczyc w armie i na nich operowac. Przyklad zastosowania kompozycji:

[PHP] pobierz, plaintext 
 <?php
 abstract class Unit
 {
    abstract public function bombardStrength ();
 }
 
 abstract class CompositeUnit
 {
    protected $units = array();
 
    public function addUnit (Unit $unit)
    {
        if (!in_array($unit, $this->units, true))
        {
            $this->units[] = $unit; 
        }
        return $this;
    }
 
    public function getArmy ()
    {
        $units = array();
        $list;
        foreach ($this->units as $unit)
        {
            $class = get_class($unit);
            if (!array_key_exists($class, $units))
            {
                $units[$class] = 1;
            } else
            {
                $units[$class]++;
            }
        }
        foreach ($this->units as $unit => $number)
        {
            $list .= $unit . ' x ' . $number . '<br />';
        }
        return $list;
    }
 }
 
 class Archer extends Unit
 {
    public function bombardStrength ()
    {
        return 2;
    }
 }
 
 class Warrior extends Unit
 {
    public function bombardStrength ()
    {
        return 4;
    }
 }
 
 class Calvary extends Unit
 {
    public function bombardStrength ()
    {
        return 6;
    }
 }
 
 class Army extends CompositeUnit
 {
    public function bombardStrength ()
    {
        $ret = 0;
        foreach ($this->units as $unit)
        {
            $ret += $unit->bombardStrength();
        }
        return $ret;
    }
 }
 
 $army = new Army;
 $army->addUnit(new Archer)->addUnit(new Warrior)->addUnit(new Warrior)->addUnit(new Calvary);
 echo $army->getArmy();
[PHP] pobierz, plaintext 

Przy skomplikowaniu wywala mi blad z tytułu. Jak to rozwiazac? Myslalem nad uzyciem __toString ale to chyba nie tego typu problem. Bo tutaj przeciez jest to zwykly tekstowy klucz nie rozumiem gdzie jest blad..

ech, niepotrzebne $this przed tablica mozna zamknac

To dodaj jeszcze o która linie chodzi, bo nikt nie będzie tego przeczesywał na ślepo

W CompositeUnit->getArmy() rzutujesz obiekty dziedziczące z Unit na string (linia 37).

Błąd tkwi jednak zapewne dwie linijki wyżej, tj. w linii 35, gdzie posługujesz się zmienną $this->units, a chciałeś/powinieneś posłużyć się zmienną $units.

już rozwiązane wiec -> odpowiedz do kosza