OOP -> czy idę w dobrym kierunku Opcje

[PawelC]

Bawię się cały czas OOP, ponieważ nudzi mnie używanie gotowych frameworków, a tak to przynajmniej się człowiek czegoś nauczy. I mam pytanie, czy idę w dobrym kierunku w OOP, dla przykładu podaje próbkę działającego kodu:

[PHP] pobierz, plaintext 
<?php
class content
{
    
    public function getContent()
    {
        $tablica=file('tekst.txt');
        foreach($tablica as $key)
        {
            $zdania[]=$key;
        }
        return $this->zdania=$zdania;
    }
    
    
    public function shuffle()
    {
        shuffle($this->zdania);
        $wylosowane=$this->zdania;
        return $this->wylosowane=$wylosowane[0];
    }
    
    public function view()
    {
        return '<h1>'.$this->wylosowane.'<h1>';
    }
}
$new=new content;
$new->getContent();
$new->shuffle();
echo $new->view();
?>
[PHP] pobierz, plaintext 

Ta klasa ma za zadanie wylosowanie jednego zdania, spośród 3tys zdań i wywalenie go na ekran w tagu h1

Ten post edytował ExPlOiT 21.06.2009, 02:23:47

[dr_bonzo]

[PHP] pobierz, plaintext 
<?php
public function view(){
                   $this->explode();
                      $this->shuffle();
                   return '<h1>'.$this->wylosowane.'<h1>';
               }
[/php
Bez sensu co chwile robic explode przy kazdym wyciaganiu zdania, szczegolnie jestli zdania sa w pliku wrzucone tak ze kazde zdanie w osobnej linijce.
 
Po co metoda view()? nie prosciej pobrac wylosowane zdanie i DOWOLNIE potem je wyswietlic a nie tylko w <h1>?
 
[php]
public function shuffle(){
                   shuffle($this->zdania);
                   $wylosowane = $this->zdania;
                   return $this->wylosowane = $wylosowane[0];
               }
               
               
               public function view(){
                   $this->explode();
                      $this->shuffle();
                   return '<h1>'.$this->wylosowane.'<h1>';
               }
?>
[PHP] pobierz, plaintext 

Kod tutaj jest conajmniej zagmatwany i nieczytelny.
Nie prosciej zrobic mieszanie zdan zaraz po zaladowaniu, a potem pobierac nastepne z tablicy?