Ostatnio zacząłem czytam o programowaniu obiektowym trochę w necie. Napisałem sobie pierwszą klasę, która ma być odpowiedzialna za obsługę artykułów (tworzenie newsów, kategorii, administrację).

[PHP] pobierz, plaintext 
<?
class Articles{
var $id; //nr artukułu
 
 
function remove_news($id){
      $sql = "DELETE FROM rada_news WHERE id = $id";
      $result = mysql_query($sql);
      echo '<p id="succes">Usunięto aktualość o id='.$id.'</p>';
}
 
function add_news($tytul,$img,$kategoria,$tekst){
      $data=Date("Y-m-d");
      $sql = "insert into rada_news  ( tytul, img, kategoria, tekst, data, akcept ) values ( '$tytul','$img', '$kategoria', '$tekst', '$data', '0');";
      $result = mysql_query($sql);
      echo '<p id="succes"><b>Dodano aktualość '.$tytul.'</b></p>';
}
 
function change_st($id,$stan){
      $sql = "UPDATE rada_news SET akcept='$stan' WHERE id=$id";      
      $result = mysql_query($sql);
      if($stan==1) $statut='aktywnty';
      if($stan==0) $statut='niaktywnty';
      echo '<p id="succes">Zmieniono status wiadomości o id='.$id.' na <b>'.$statut.'</b></p>';
}
 
function edit($id,$tytul,$img,$kategoria,$tekst){
      $sql = "UPDATE rada_news SET tytul='$tytul', img='$img', tekst='$tekst', kategoria='$kategoria'  WHERE id=$id";       
      $result = mysql_query($sql);
      echo '<p id="succes">Zapisano poprawnie aktualość: <b>'.$tytul.'</b></p>';
}
 
function pobierz_dane($opcja,$id){
    $sql = "SELECT $opcja as pole FROM `rada_news` WHERE id='$id'";
    //echo $sql.'<br>';
    $res = mysql_query($sql);
    $row = mysql_fetch_array($res);
    return $row['pole'];
}
 
function show_news($id){
      $sql = "SELECT * FROM rada_news WHERE akcept=1 AND id=$id";
      $result = mysql_query($sql);
      $ile = mysql_num_rows($result);
      $row = mysql_fetch_array($result);
      //echo $ile;
      if($ile==0){
          echo '<p id="error">Brak aktualości w bazie</p>';
      }
      elseif($ile>1){
          echo '<p id="error">Błąd w bazie. Gdy błąd będzie się powtarzał skontaktuj się z administratorem</p>';
      }
      else{
          $tytul=$this->pobierz_dane('tytul',$id);
          $tekst=$this->pobierz_dane('tekst',$id);
          $data=$this->pobierz_dane('data',$id);
          $img=$this->pobierz_dane('img',$id);
          echo'<div id="main_text" style=";margin-top:10px;background-color: white; padding:0px 0px;">';
          echo'<div style="border-bottom:1px solid #6600ff; padding:5px 5px;background:#f4f9fb"><b>'.$tytul.'</b>';
          echo'<div style="font-size:10px; color:silver">Dodano '.$data.'</div></div>';
          echo'<P style="margin:0px;padding:8px 5px">';
          if($img) echo '<img src="'.$img.'" align="left" style="margin-right:5px">';
          echo $tekst.'</p>';
          echo '</div>';
      } 
}
 
function skroc($tekst,$ile)
{
        $znaki = strlen($tekst);
        if ($znaki>=$ile)
        {
                $tnij = substr($tekst,0,strpos($tekst, ".", $ile));
                $skrocone = $tnij.".";
        }
        else
        {
                $skrocone = $tekst;
        }
        return $skrocone;
}
 
function html2txt($document){
  $search = array('@<[\/\!]*?[^<>]*?'.'>@si'
                  );
  $text = preg_replace($search, '', $document);
 
  return $text;
}
 
function show_news_zaj($id){
      $sql = "SELECT * FROM rada_news WHERE akcept=1 AND id=$id";
      $result = mysql_query($sql);
      $ile = mysql_num_rows($result);
      $row = mysql_fetch_array($result);
      //echo $ile;
      if($ile==0){
          echo '<p id="error">Brak aktualości w bazie</p>';
      }
      elseif($ile>1){
          echo '<p id="error">Błąd w bazie. Gdy błąd będzie się powtarzał skontaktuj się z administratorem</p>';
      }
      else{
          $tytul=$this->pobierz_dane('tytul',$id);
          $tekst=$this->pobierz_dane('tekst',$id);
          $data=$this->pobierz_dane('data',$id);
          $img=$this->pobierz_dane('img',$id);
          echo'<table><tr><td><div id="main_text" style=";margin-top:10px;background-color: white; padding:0px 0px;">';
          echo'<div style="border-bottom:1px solid #3b8cdd; padding:5px 5px;background:#f4f9fb"><b><a href="'.WWW.'aktualnosc/'.$row['id'].',_">'.$tytul.'</a></b>';
          echo'<div style="font-size:10px; color:silver">Dodano '.$data.'</div></div>';
          echo'<P style="margin:0px;padding:8px 5px">';
          if($img) echo '<img src="'.$img.'" align="left" style="margin-right:5px">';
          $tekst = $this->html2txt($tekst);
          $tekst = $this->skroc($tekst,400);
          echo $tekst.'</p>';
          echo '</div></td></tr></table>';
      } 
}
 
}
?>
 
[PHP] pobierz, plaintext 

Czy sądzicie że dobrze zaczynam? Czy to jest programowanie obiektowe, czy tylko pseudo-obiektowe? Co muszę poprawić, na co zwrócić uwagę

Zwróć uwagę na to żeby oddzielić warstwe prezentacji od pobierania i przetwarzania danych....
Czyli do klasy ładujesz metody które służą do pobierania danych, a nie do ich wyświetlania, np:

[PHP] pobierz, plaintext 
funtion get_news ($id) {
      $sql = "SELECT * FROM rada_news WHERE akcept=1 AND id=".(int)$id;
      $result = mysql_query($sql);
      $row = mysql_fetch_array($result);
      return $row; 
}
[PHP] pobierz, plaintext 

No niestety jest to do bani.
- to co próbowałeś napisać to php4, zainteresuj się php5
- jak kolega napisał wyżej wzorzec MVC

... a można powiedzieć chociaż że to co napisałem jest zalążkiem programowania obiektowego??

Akurat wykorzystałem tę moją klasę do implementacji strony i wszystko działa. Jakie są konsekwencje takiej implementacji. Są poważne zagrożenia dla bezpieczeństwa albo poprawnego działania?

Napisze jasno: Twoja klasa to nic innego, jak zwykły, przestarzały, prowizoryczny, amatorski itp. itd. etc. skrypt, który wyróżnia się tylko tym, że został upakowany w klamry poprzedzone słowem kluczowym "class" i podzielony na funkcje. Powiem więcej - klasa wręcz utrudnia utrzymanie takiego kodu! Lepszy logiczny kod strukturalny, niż obiektowa sieka bez ładu i składu!

Odpowiedz sobie na pytania:
- jaka jest przewaga/zaleta tego kodu w porównaniu z identycznym, lecz nie podzielonym na klasę i funkcje?
- czy zaimplementowanie tej klasy do obsługi artykułów w serwisie x.com będzie łatwe, czy trudne?
- albo inaczej, który kod jest lepszy:

[PHP] pobierz, plaintext 
//Twój:
function pokaz(){
echo '<p>Tekst</p>';
}
pokaz();
 
//Mój:
echo '<p>Tekst</p>';
[PHP] pobierz, plaintext 

Masz pisać aplikacje na warstwach:
1. Odbierasz dane i parametry z url, post, cfg
2. Przygotowujesz treść i meta (często używa się tu nazwy silnik - to zasadnicza część aplikacji, nie zawiera kodu html)
3. Upychasz to w kod html
4. Wysyłasz stronę do przeglądarki

Jaki sens pisać klasę czy funkcję, która drukuje coś na ekranie?? Na początek wywal z silnika chociaż echo i html, poczytaj trochę o obiektówce, wykorzystaj wzorce i dziedziczenie, konstruktory, metody dostępowe i wiele innych zalet obiektówki.

Czyli np. komunikaty robić:

1. wpisać do zmiennej

[PHP] pobierz, plaintext 
private function przyklad(){
// kod coś robiący
$this->info.='info błędu';
}
[PHP] pobierz, plaintext 

2. a potem dopiero w innej metodzie czy klasie zrobić wyświetlenie zmiennej $info

Jeżeli jakaś metoda nie może zakończyć się poprawnie powinna wyrzucić wyjątek.

ok. Musze poczytać o tych wyjątkach jeszcze. Ale czy teraz zacząłem lepiej kumać. W funkcji która jest silnikiem mam robić przypisania do zmiennych a potem dopiero w innych funkcjach korzystać z tych danych ?

Tak. I podzielić to funkcjonalnie:
1. Obróbka parametrów, generowanie meta-tagów, słów kluczowych etc.
2. Model danych - tu będą zapytania do bazy
3. Widok - w najprostszej postaci system szablonów via PHP:

[PHP] pobierz, plaintext 
<h4>Artykuły</h4>
<p>
<?php
echo $this->$tresc;
?>
</p><p>
<?php
echo $this->$data_dodania;
?>
[PHP] pobierz, plaintext 

- w ten sposób obsługa i walidacja parametrów/adresów/konfigów jest w jednym miejscu (czyli całe "wejście"), zapytania do bazy w drugim a kod html w trzecim, oczywiście łączy to kontroler, który na podstawie otrzymanych parametrów korzysta z określonych modeli danych, pakuje pobraną przy ich pomocy treść w zmienne i wysyła do widoku - w ten sposób powstaje przejrzysty kod łatwy w utrzymaniu i rozbudowie.

...czyli każdą warstwę mam podzielić na osobną klasę? Czy wystarczy jak podzielę na metody...?

Zależy jak to ma funkcjonować, integralna część jakiejś większej aplikacji czy tylko moduł do niej? Najlepiej każdą warstwę w oddzielnym pliku, a jeden plik = jedna klasa/funkcja (nazwa pliku = nazwa klasy/funkcji), dzięki czemu każdy od razu połapie się w kodzie, będzie to łatwe do rozbudowy/modyfikacji czy implementacji.

Ok ja tez się dorzucę. Jest to programowanie pseudo obiektowe. Ale bardzo dobrze, że próbujesz nie od razu Rzym zbudowano, z czasem Twój kod będzie dużo lepszy.

Twój kod niestety nie jest kodem obiektowym. I można to stwierdzić to zanim nawet człowiek spojrzy na pierwszą jego linijkę. Dobry obiekt ma bardzo wąska odpowiedzialnośc a Ty już na wstępie piszesz:



Za dużo odpowiedzialności, zresztą wyimaginowanej bo Twój obiekt w najlepszym razie odpowiada za dostęp do danych i za nic więcej. Jest to nieprzemyślana hybryda Active Record z wmieszanymi elementami prezentacji danych. Fuj!

Zainteresuj się MVC, a póki co, może zrób z tego co masz dwa obiekty niech jeden pobiera dane a drugi wyświetla dane które pobrał pierwszy.

I jeszcze uwaga:
Crozin:


To nie jest prawda... Prawdziwsze jest stwierdzenie:

Jeżeli wykonywanie jakiejś metody może zakończyć się błędem, przy czym jest to błąd, nad którym programista piszący dany kod nie ma kontroli (np. kod korzysta z zasobów zewnętrznych, które mogą być niedostępne) to należy wyrzucić wyjątek (obsłużony w innym miejscu programu).

Panie Crozin, warto chyba ubrać to co się chce powiedzieć w parę zdań tak aby to miało ręce i nogi, a nie rzucać sentencje w tylu "Jak się jest głodnym to trzeba zjeść"...

Tak na dobrą sprawę to to co napisałeś jest dokładnie tym samym co podałem ja.

A sentencja "głodny -> zjedz coś" jest przecież jak najbardziej poprawna. Nie ma sensu się rozpisywać co to znaczy być głodnym, ani co to znaczy zjeść coś. Jak odbiorca nie będzie czegoś pewny to sobie wygooglea albo poprosi o doprecyzowanie odpowiedzi.