Odwołanie do methody rodzica Opcje

[gcdreak]

Witam!
Mam taką klase:

[PHP] pobierz, plaintext 
<?php
class MySPLFileObject extends SplFileObject
    {
        public $fp; 
        private $stats = array();
        
        function __construct($file){
            $this->fp = new SplFileObject($file);
            $this->stats = $this->fp->fstat();
        }
        
        public function getFilename(){
            return $this->fp->getFilename();
        }
        
        public function getFileSize(){
            return $this->stats['size'];
        }
        
        public function getCreateTime(){
            return date('Y-m-d G:i', $this->stats['ctime']);
        }
        
        public function getAccessTime(){
            return date('Y-m-d G:i', $this->stats['atime']);
        }
        
        public function getModifyTime(){
            return date('Y-m-d G:i', $this->stats['mtime']);
        }
    }
    
    
    
    $info = new MySPLFileObject('arrayObject.php');
    
    printf(
        "|=== Plik <strong> %s </strong> =============| <br />".        
        "||  <br />".
        "|| ---> Rozmiar [B]: %s <br />".
        "|| ---> Utworzony: %s <br />".
        "|| ---> Urzywany: %s <br />".
        "|| ---> Zmodyfikowany: %s <br />".
        "||  <br />".
        "|=====================================|",    
        $info->getFilename(),
        $info->getFileSize(),
        $info->getCreateTime(),
        $info->getAccessTime(),
        $info->getModifyTime()
    );
    
    echo $info->current();
?>
[PHP] pobierz, plaintext 

Kiedy wywołuje metody klasy SPLFileObject to pojawia mi się informacja o pobraniu pliku?
Mam w kodzie jakiś błąd?

Powód edycji: [Spawnm] poprawiłem bbcode.

[batman]

Oj namieszałeś, namieszałeś.
Jeśli rozszerzasz jakąś klasę, to nie przesłaniaj jej konstruktora. A jeśli to robisz, to nie twórz kolejnego obiektu, tylko wywołaj konstruktor rodzica:

[PHP] pobierz, plaintext 
<?php
parent::__construct($atrybuty_przyjmowane_przez_konstruktor_rodzica)
?>
[PHP] pobierz, plaintext 

Jak już to zrobisz to możesz wywalić s kodu metodę getFilename, ponieważ jest ona wstawiona w tym miejscu bez potrzeby - przecież występuj ona w klasie rodzica.
Resztę będziesz sam w stanie poprawić.

[Crozin]

Po co dziedziczysz po SplFileObject skoro w konstruktorze MySPL... tworzysz nową instancję tego obiektu?

[Pr0100]

[PHP] pobierz, plaintext 
<?php
/* ... */ extends SplFileObject /* ... */ new SplFileObject($file); /* ... */
?>
[PHP] pobierz, plaintext 

zdecyduj się na coś albo dziedziczysz (extends) albo tworzysz instancje (new)

Rozwiązanie #1 - dziedziczenie

[PHP] pobierz, plaintext 
<?php
function __construct($file){
  parent::__construct($file);
  $this->stats = $this->fstat();
}
?>
[PHP] pobierz, plaintext 

i w całym skrypcie zamień $this->fp-> na $this->

Rozwiązanie #2 - new SplFileObject();

zamień
class MySPLFileObject extends SplFileObject
na
class MySPLFileObject

a zamiast

[PHP] pobierz, plaintext 
<?php
$info->current();
?>
[PHP] pobierz, plaintext 

daj

[PHP] pobierz, plaintext 
<?php
$info->fp->current();
?>
[PHP] pobierz, plaintext 

oczywiście możesz zrobić metode getCurrent() w klasie MySPLFileObject ale z tym chyba już sobie poradzisz (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował Pr0100 1.07.2009, 20:00:03

[gcdreak]

Zrobiłem tak:

Kod

function __construct($file){
            parent::__construct($file);
            $this->stats = $this->fstat();
        }
        
        public function getFileSize(){
            return $this->stats['size'];
        }

I mam prośbę, aby ktoś mi wytłumaczył jedną istotną kwestie.
Chodzi o $this.
Kazaliście mi zmienić

Kod

return $this->fp->stats(...)

na

Kod

return $this->stats(...)

i w zwiażku z tym moje pytanie:
kiedy należy urzywać samego $this, a kiedy $this->parametr ?

[Crozin]

Podstawowe pytanie: czy rozumiesz o co chodzi w dziedziczeniu?

[cojack]

Tu nawet nie chodzi o pytanie, czy rozumie o co chodzi w dziedziczeniu, o nie rozumie o co chodzi w programowaniu obiektowym. (Nie twierdze że wszystkie rozumy pozjadałem).

[gcdreak]

Ciągle się ucze, ale zawsze mnie ciekawiła ta różnica.