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]

A no tak, że nie czytałeś dokumentacji

[PHP] pobierz, plaintext 
<?php
(bool)'Twoja stara';
?>
[PHP] pobierz, plaintext 

i odrób pracę domową. (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

praca domowa odrobiona... bardziej chodzi o zle doswiadczenia z takim slabym porownaniem...

po pierwsze nie kontrolujesz gdzie wystepuje blad. a i o takiej konwersji mozna by wiele gadac,
bo o ile:

var_dump((bool) "false"); // bool(true)
tak już:
var_dump((bool) "0"); // bool(false)


o ile mozna by sie zastanawiac nad przydatnoscia ponizszego dla tej klasy::
var_dump((bool) ""); // bool(false)
tak var_dump((bool) "0"); // bool(false) moze wygenerowac blizej nie zidentyfikowany blad..

przy wczesniejszych zalozeniach kazde prawidlowe zrodlo tresci, (w tym przypadku plik, ale to nie istotne) nie moze byc pusty lub... nie moze zawierac tylko '0', kiepskie, jak dla mnie przynajmniej..