klasa do oceny Opcje

[karis]

jestem początkujący w OOP, ale udało mi się napisać własną klasę
proszę o ocenę, czy dobrze to zrobiłem, co mogę poprawić, co dodać etc.

[PHP] pobierz, plaintext 
<?php
class File
{
     private $plik;
     private $open;
     
     function __construct()
     {
        return 0; 
     }
     public function get($plik)
     {
         $this->plik=$plik;
         $openFile = file_get_contents($plik);
         $this->open = $openFile;
         if(!$openFile)
         {
             return false;
         }
         else
         {
             return true;
         }
     }
     public function save($dane)
     {
         $op = $this->get($this->plik);
         if(!$op)
         {
             return false;
         }
         else
         {
             $dane = $dane."\n".$this->open;
             $save = file_put_contents($this->plik, $dane);
             if($save)
             {
                
                 return true;
             }
             else
             {
                
                 return false;
             }
         }
     }
     public function show()
     {
         echo $this->open;
         return true;
     }
}
$plik = new File();
$plik->get('123.php');
$plik->save('kupsko hahahahha');
$plik->show();
 
 
 
 
?>
[PHP] pobierz, plaintext 

[ayeo]

Witam!

Taka jest specyfika tego działu, że jestem zmuszony sobie pomarudzić (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Mieszasz nazwy polskie z angielskimi co jest niby detalem, ale to bardzo zły nawyk. Użycie konstruktora w taki sposób jest całkowicie pozbawione sensu.

[PHP] pobierz, plaintext 
<?php
$save = file_put_contents($this->plik, $dane);
             if($save)
             {
                
                 return true;
             }
             else
             {
                
                 return false;
             }
?>
[PHP] pobierz, plaintext 

Lepiej po prostu:

[PHP] pobierz, plaintext 
<?php
return file_put_contents($this->plik, $dane);
?>
[PHP] pobierz, plaintext 

Nazwy metod są mylące. Reasumując nie jest tragicznie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Pozdrawiam!

[karis]

dzięki (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

skoro już zacząłem, to może powiecie mi jak udoskonalić tą klasę

[PHP] pobierz, plaintext 
<?php
class Tagi
{
     
    private $nazwaTagu;
    private $zawartosc;
    
    public function dane($zawartosc)
    {
         if(eregi('http://', $zawartosc))
         {
             //pobieranie adresu
             $zdalny = file_get_contents($zawartosc);
             $this->zawartosc = $zdalny;
         }
         else
         {
             $this->zawartosc = $zawartosc;
         }
    }
    
    public function pobierzTag($nazwaTagu)
    {
         $this->nazwaTagu = $nazwaTagu;
         preg_match_all('%<'.$nazwaTagu.' (.*)>%', $this->zawartosc, $wynik);
         
         $wynik = $wynik[0];
         
         return $wynik;
         
    }
    
    function __destruct()
    {
         unset($wynik);
         unset($zdalny);
    }
     
}
 
$tag = new Tagi();
$tag->dane('http://farby.info.pl');
$wynik = $tag->pobierzTag('img');
 
print_r($wynik);
?>
[PHP] pobierz, plaintext 

żeby pobierała tylko te tagi, które zostały zdefinioowane, oraz żeby brało pod uwage taki otwarte lub zamknięte

[ayeo]

Witam!

Destruktor niszczy zmienne których nie ma. Eregi to zło, korzystaj preg. Nie rozumiem o co chodzi ze zdefiniowanymi tagami, ale wydaje mi się, że to kwestia regexpa, a nie OOP.

Pozdrawiam!