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]

mniej wiecej.

w zasadzie, programowanie obiektowe daje rezultaty przy wiekszych projektach, przy malych skryptach, tak naprawde jest to zabawa pro forma.
choc jest oczywiscie dobra wprawka.. zabawa zaczyna sie dopiero jak robisz interfejsy, abstrakty, itp (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
ale lekko korygujac :

[PHP] pobierz, plaintext 
<?php
class Admin()
{
  private static $_instance;
  //robimy singletona:)
  public function instance(){...........}
  public function add(){...........}
  public function del(){ ............}
 
}
?>
[PHP] pobierz, plaintext 

i teraz uzycie

[PHP] pobierz, plaintext 
<?php
// nie zostawiaj nic przypadkowi
$co = (isset($_GET['co'])&&!empty($_GET['co']))?$_GET['co']:null;
 
if(!empty($co))
   if($co=='add')
           $art->instance()->add();
       elseif($co=='del')
           $art->instance()->del();
       
       ....................
?>
[PHP] pobierz, plaintext 

aa, switch jest rzeczywiscie lepszy, w sensie znacznie wydajniejszy, pozwala sporo czasu zaoszczedzic.