Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> OOP -> czy idę w dobrym kierunku
PawelC
post
Post #1





Grupa: Zarejestrowani
Postów: 1 173
Pomógł: 121
Dołączył: 24.09.2007
Skąd: Toruń

Ostrzeżenie: (0%)
-----


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:
  1. <?php
  2. class content
  3. {
  4.    
  5.    public function getContent()
  6.    {
  7.        $tablica=file('tekst.txt');
  8.        foreach($tablica as $key)
  9.        {
  10.            $zdania[]=$key;
  11.        }
  12.        return $this->zdania=$zdania;
  13.    }
  14.    
  15.    
  16.    public function shuffle()
  17.    {
  18.        shuffle($this->zdania);
  19.        $wylosowane=$this->zdania;
  20.        return $this->wylosowane=$wylosowane[0];
  21.    }
  22.    
  23.    public function view()
  24.    {
  25.        return '<h1>'.$this->wylosowane.'<h1>';
  26.    }
  27. }
  28. $new=new content;
  29. $new->getContent();
  30. $new->shuffle();
  31. echo $new->view();
  32. ?>

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
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
Crozin
post
Post #2





Grupa: Zarejestrowani
Postów: 6 476
Pomógł: 1306
Dołączył: 6.08.2006
Skąd: Kraków

Ostrzeżenie: (0%)
-----


A ja polecam rozpoczęcie zabawy z czymkolwiek od włączenia raportowania wszystkich błędów - masz trochę złych praktyk.

content::getContent - zmień nazwę tej metody. Tak się już przyjęło, że metody na get/set pełnią rolę getterów i setterów i wyłącznie dla czytelności, przejrzystości, a przede wszystkim łopato-logiczności* kodu dobrze jest się trzymać takich zasad.

Zero jakiekolwiek możliwości konfiguracji. Dane muszą być ładowane z jednego określonego pliku, wyświetlane są w jeden sposób.

Pozatym... tutaj to takie OOP na siłe:
  1. <?php
  2. $data = file('plik'); $result = $data[array_rand($data)];
  3. echo sprintf('<h1>%s</h1>', $data);
  4. ?>
Tak więc, proponowałbym jakieś nieco bardziej zaawansowane rzeczy robić.

* - ponoć to się z "-" pisze wg Opery (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post

Posty w temacie


Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 12.10.2025 - 04:13