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

[margyw]

Jeszcze mnie nurtuje pewna rzecz, a mianowicie podstrony.

Czy sposób, który pokazałem wyżej, tzn.

[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 

jest poprawny, czy istnieje jakaś inna forma, którą wykorzystujecie pisząc obiektywnie?

Przypuśćmy, że robiłbym PA, i np. miałbym klasę z funkcjami:

[PHP] pobierz, plaintext 
<?php
class Admin()
{
 
   public function add()
      {
           ...........
      }
 
   public function del()
      {
           ............
      }
 
}
?>
[PHP] pobierz, plaintext 

to wtedy poprawne, według zasad obiektowości, będzie wykorzystanie jej jak w w/w przykładzie?:

[PHP] pobierz, plaintext 
<?php
if(isset($_GET['co']))
    if($_GET['co']=='add')
        {
            $art = new Admin();
            $art->add();
        }elseif($_GET['co']=='del'){
            $art = new Admin();
            $art->del();
        }
        ....................
}
?>
[PHP] pobierz, plaintext 

Nie zwracajcie zbytnio uwagi na kod, można by również wykorzystać switch'a, ale nie o to chodzi, tylko o sam fakt, czy tak jest poprawnie wg, zasad OOP.

W ogóle przepraszam za tak banalne pytania, ale jakoś zagadnienie OOP nie może do mnie trafić i nie potrafię tego pogodzić z tym co już umiem.

Wydaje mi się, że za dużo strukturalności wtrącam w kod i dlatego przestaje to być stricte obiektowym kodem. Nie wiem już sam (IMG:http://forum.php.pl/style_emoticons/default/sciana.gif) .

Ten post edytował margyw 30.06.2009, 00:01:07