Napisałem taki kod:

[PHP] pobierz, plaintext 
<?php defined('SYSPATH') OR die('No direct access allowed.');
 
/*  
 *  @author     Administrator
 *  @filename   Menu.php
 *  @date       2009-11-21
 */
 
class Side_Menu_Core
{
    protected $type; //guest, user, admin
    protected $items;
    static $instance = NULL;
 
    private function  __construct($type = null) {
        $this->type = $type;
 
        /*
         * pobranie z bazy stałych elementów to chyba dobre miejsce
         */
    }
 
    public static function instance($type = null){
        if (self::$instance === NULL)
                        self::$instance = new Side_Menu($type);
                self::$instance->type = $type;
                return self::$instance;
    }
 
    public function add(Menu_item $item){
        $this->items[$item->id] = $item;
        if($item->parent > 0){
            $this->items[$item->parent]->append($item->id);
        }
        print Kohana::debug($this->items);
    }
 
    public function get($id){
        return $this->items[$id];
    }
 
    public function set_active($id){
        if($id == 0){
            return;
        }
        $item = $this->get($id);
        $item->active = TRUE;
        $this->set_active($item->parent);
    }
 
    public function  __toString() {
        return $this->render();
    }
 
    public function render(){
        $output = '<ul>';
        foreach($this->items as $item){
                $output .= $item->render();
        }
        $output .= '</ul>';
 
        return $output;
    }
 
    public function get_parents(){
 
    }
 
    public function get_children($id_parent){
 
    }
}
 
class Menu_item
{
    public $id = 0;
    public $parent = 0;
    public $children;
    public $title;
    public $link;
    public $active;
 
    public function  __construct($setting) {
        if(!empty($setting['parent'])){
            $this->parent = $setting['parent'];
        }
        $this->title = $setting['title'];
        $this->link = $setting['link'];
        $this->id = $setting['id'];
    }
 
    public function append($id){
        $this->children[] = $id;
    }
 
    public function render(){
        $output = '';
             if (empty($this->children) && $this->parent != 0){
                 return '<li>'.$this->title.'</li>';
             }
 
            if(!empty($this->children)){
                $output .= '<li>'.$this->title.'</li>';
                $output .= '<ul>';
                foreach($this->children as $child){
                    $output .= Side_Menu_Core::instance()->get($child)->render();
                }
                $output .= '</ul>';
            }
        return $output;
    }
 
    public function render_without_children(){
        return $output = '<li>'.$this->title.'</li>';
    }
}
?>
[PHP] pobierz, plaintext 

Mam problem, gdy oznaczę jeden wpis jako child, on dalej wyświetla się jako parent. Probowałem różnymi ifami, że parent = 0 empty(child), ale dalej wyświetla się drugi raz w głównej linii.

Przy okazji prosiłbym o uwagi to tej klasy.

Jak poddajesz coś pod ocenę, to przynajmniej udostępnij samodzielny kod, dołącz klasę Side_Menu, a nie

Fatal error: Class 'Side_Menu' not found in /var/www/robocze/abcdefghijkl.php on line 25 :

[PHP] pobierz, plaintext 
<?php // defined('SYSPATH') OR die('No direct access allowed.');
 
/*  
 *  @author     Administrator
 *  @filename   Menu.php
 *  @date       2009-11-21
 *  @TODO brak opisu klasy, za co odpowiada
 */
 
class Side_Menu_Core
{
    protected $type; //guest, user, admin
    protected $items;
    static $instance = NULL;
 
    private function  __construct($type = null) {
        $this->type = $type;
 
        /*
         * pobranie z bazy stałych elementów to chyba dobre miejsce
         * 
         * Jak najbardziej
         */
    }
 
    public static function instance($type = null) {
        if (self::$instance === NULL)
            self::$instance = new Side_Menu($type);
            self::$instance->type = $type; 
            return self::$instance;
    }
 
    public function add(Menu_item $item){
        $this->items[$item->id] = $item;
        if($item->parent > 0){
            $this->items[$item->parent]->append($item->id);
        }
        // no Kochana specific methods!
        //print Kohana::debug($this->items);
    }
 
    public function get($id){
        return $this->items[$id];
    }
 
    public function set_active($id){
        if($id == 0){
            return;
        }
        $item = $this->get($id);
        $item->active = TRUE;
        $this->set_active($item->parent);
    }
 
    public function  __toString() {
        return $this->render();
    }
 
    public function render(){
        $output = '<ul>';
        foreach($this->items as $item){
                $output .= $item->render();
        }
        $output .= '</ul>';
 
        return $output;
    }
 
    public function get_parents(){
 
    }
 
    public function get_children($id_parent){
 
    }
}
 
class Menu_item
{
    public $id = 0;
    public $parent = 0;
    public $children = array();
    public $title;
    public $link;
    public $active;
 
    public function  __construct($setting) {
        if(!empty($setting['parent'])) {
            $this->parent = $setting['parent'];
        }
        $this->title = $setting['title'];
        $this->link = $setting['link'];
        $this->id = $setting['id'];
    }
 
    public function append($id) {
        $this->children[] = $id;
    }
 
    public function render() {
        $output = '';
             if (empty($this->children) && $this->parent != 0) {
                 return '<li>'.$this->title.'</li>';
             }
 
            if(!empty($this->children)) {
                $output .= '<li>'.$this->title.'</li>';
                $output .= '<ul>';
                foreach($this->children as $child) {
                    $output .= Side_Menu_Core::instance()->get($child)->render();
                }
                $output .= '</ul>';
            }
        return $output;
    }
 
    public function render_without_children() {
        return $output = '<li>'.$this->title.'</li>';
    }
}
// to już próba utworzenia Twoich obiektów, niestety jw.
$side_menu_core = Side_Menu_Core::instance();
$menu_item = new Menu_item();
 
?>
[PHP] pobierz, plaintext 

ps. nie lepiej użyć hierarchii obiektów, żeby sprawdzić, czy dany element jest rodzicem czy dzieckiem elementu wejściowego

Dzięki za odpowiedź ale już sobie poradziłem.
Klasa Menu i Menu_Core to jest ta sama klasa. Jest to moja biblioteka z frameworka Kohana on sobie tak nazywa klasy. Więc kod działa. Jakbyś zmienił nazwę klasy na Side_Menu to powinno zadziałać samodzielnie.

Zero problematyki OOP. ->PHP