Początki z OOP :F Opcje

[margyw]

Witam,

miałem kilku letnią przerwę w php i teraz, kiedy znowu mnie wciągnęło, za żadne skarby nie potrafię ogarnąć obiektówki (IMG:http://forum.php.pl/style_emoticons/default/worriedsmiley.gif) .

Chciałbym, żebyście sprawdzili tą klasę i w ogóle naprowadzili mnie na dobrą drogę, czy tak powinno się używać OOP. Jako że uczyłem się strukturalnie to teraz trudno mi się odzwyczaić (IMG:http://forum.php.pl/style_emoticons/default/sciana.gif) . Jest to prosty kodzik, który ma wyświetlać plik, który mu zapodam. Pisałem to pod malutki homepage, więc użyłem go do wyświetlania podstron. A więc:

[PHP] pobierz, plaintext 
<?php
class Page
{
    private $file;
    
    
    public function __construct($file)
    {
        $this->file = 'content/'.$file.'.txt';
    }
    
 
    private function loadPage()
    {    
        $content = file($this->file);
        $content = join('', $content);
        
        return $content;        
    }
    
    
    private function checkPage()
    {
        if(file_exists($this->file)) { return TRUE; } else { return FALSE; }    
    }
    
    
    public function viewPage()
    {    
        if($this->checkPage()==TRUE)
        {
            echo $this->loadPage();            
        }else{    
            echo '<h2>The page you have entered is incorrect!</h2>';    
        }    
    }
    
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
if(isset($_GET['go'])) 
    {                         
        $page = new Page($_GET['go']);
        $page->viewPage();                            
    }else{                        
        $page = new Page('main');
        $page->viewPage();                        
    }
?>
[PHP] pobierz, plaintext 

Tak więc, jest to poprawne użycie obiektówki, czy taki kod mija się z celem? Bo ja już sam nie wiem, może kiedyś to pojmę (IMG:http://forum.php.pl/style_emoticons/default/dry.gif)

Z góry dzięki, pozdrawiam.

Ten post edytował margyw 28.06.2009, 23:56:46

[erix]

jesli nie znajdziemy pliku, zwraca false wiec dalsze sprawdzanie tresci jest dyskusjne.. bo w takim razie nalezalo by wyciac biale znaki, pojednyncze znaki, etc.

I przy okazji sypiąc notice. A przecież wszystkim chyba zależy, aby dążyć do kodu sprawnie działającym nawet na raportowaniu E_STRICT? (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)