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

[alegorn]

bede sie czepiac. tyci (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[PHP] pobierz, plaintext 
<?php
class Page
{
    private $file;
    protected $content;
    
    public function __construct($file)
    {
        $this->file = 'content/'.$file.'.txt';
        if ( !( $this->content = file_get_contents($this->file)) ){
            throw new Exception('The page you have entered is incorrect!');
        }
    }
    
 
    private function getContent()
    {    
        return $this->content;        
    }
    
    
    public function viewPage()
    {    
            echo $this->getContent();
    }
    
}
?>
[PHP] pobierz, plaintext 

ja bym zrobil te klase jak wyzej....

w twoim przykladzie jest taka linijka... :

[PHP] pobierz, plaintext 
<?php
if($this->loadPage()==TRUE)
?>
[PHP] pobierz, plaintext 

wytlumacz mi jak ma to zadzialac, skoro loadPage, w twoim przykladzie zrwaca string? to pierwsza uwaga.
2. staraj sie nie uzywac @, skoro bierzesz pod uwage error - oprogramuj go.
3. skoro w konstruktorze na sztywno ustawiasz konkretny plik, to po co leniwie zaczytujesz jego tresc... ? na upartego da sie, ale mam spore wawtpliwosci, wobec takiego rozwiazania.

Ten post edytował alegorn 29.06.2009, 09:04:19