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 

Witam!

Taka jest specyfika tego działu, że jestem zmuszony sobie pomarudzić 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

Pozdrawiam!

dzięki

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

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!