PHP4 Obiektowo, Ocena kodu i prośba o wskazówki Opcje

[Fallout]

Napisałem takie coś (nieskończone of courz) co ma w założeniu być systemem oceny artykułów, mam na to w miarę sporo czasu ( na zaliczenie) więc probuje to pisać "obiektowo" w php4.. sam nie wiem czy to wogóle można nazwać obiektowym.. chętnie poznam wasze opinie i wskazówki co jest nie tak... (kod działa btw.)

plik artykuly_class.inc.php

[PHP] pobierz, plaintext 
<?php
/*
 
	@Projekt:		 System Ocen Artykułów
	@Plik:		  artykuly_class.inc.php
	@Opis:			Obłsuga i zarządzanie artykułami
	@Data:			 12 maj 2005
	@Autor:		 Marcin Bancerz
	@Inne:			Praca zaliczeniowa z przedmiotu P.S.I.
 
*/
	// klasa
	class artykuly
	{
 
		var $artykul;							 // zmienna do tablicy danych atrykulu
 
		// konstruktor klasy (PHP4)
		function artykuly()
		{
			$this->artykul['aid'] = 0;			// id artykulu
			$this->artykul['uid'] = 0;			// id uzytkownika
			$this->artykul['a_tytul'] = '';		// tytul artykulu
			$this->artykul['a_tresc'] = '';		// tresc artykulu
			$this->artykul['a_data'] = '';		// data i czas dodania artykulu
			$this->artykul['a_ocena'] = 0;		// laczna srednia ocen arykulu
		}
 
		// funkcja ustawia wartosc dla zmiennej
		function SetValue($zmienna,$wartosc)
		{
			$this->artykul[$zmienna] = $wartosc;
		}
 
		// funkcja pobiera wartosc dla zmiennej
		function GetValue($zmienna)
		{
			return $this->artykul[$zmienna];
		}
 
		// funkcja ustawia wartosci dla calego artykulu
		function SetArticle($aid = '',$uid,$a_tytul,$a_tresc,$a_data,$a_ocena)
		{
			$this->SetValue('aid',$aid);
			$this->SetValue('uid',$uid);
			$this->SetValue('a_tytul',$a_tytul);
			$this->SetValue('a_tresc',$a_tresc);
			$this->SetValue('a_data',$a_data);
			$this->SetValue('a_ocena',$a_ocena);
		}
 
		// funkcja laczy sie z baza danych, pobiera wskazany artykul i ustawia zmienne
		function GetArticle($aid)
		{
			// polaczenie z baza
			$db = new db();
 
			// pobiera zawartosc danego artykulu
			$db->DoSql(&#092;"SELECT * FROM artykuly WHERE a_id=$aid\");
			$res = $db->Result();
 
			// pobiera sume ocen komentarzy
			$db->DoSql(&#092;"SELECT SUM(a_ocena) FROM komentarze WHERE a_id='$aid'\");
			$res2 = $db->Result();
 
			// pobiera ilosc komentarzy odnoszacych sie do artykulu
			$db->DoSql(&#092;"SELECT k_id FROM komentarze WHERE a_id='$aid'\");
			$res3 = $db->Result();
 
			// wyliczanie sredniej
			$suma = $res2[0];
			$ilosc = $res3[0];
			if ((!$suma==0)or(!$ilosc==0)) $ocena = $suma/$ilosc; else $ocena = 0;
 
			   // ustawienie wartosci zmiennych
			   $this->SetArticle($res[0],$res[1],$res[2],$res[3],$res[4],$ocena);
 
			// usuniecie polaczenia z baza
			unset($db);
		}
 
		function ArtykulIstnieje($aid)
		{
			// sprawdza czy identyfikator artykulu zostal przydzielony
			if (!$aid=='')
			{
				$db = new db();
				$db->DoSql(&#092;"SELECT a_id FROM artykuly WHERE a_id='$aid'\");
				if ($db->DBError) echo &#092;"Nieudana pobranie ilości rekordów! Linia: \".__LINE__.\" w pliku \".__FILE__.\"<br>\";
				return ($db->NumOfRows()>0);
			}
			// jesli nie, zwraca falsz
			else return False;
		}
 
		// zapisuje biezace dane artykulu do bazy
		function SaveArticle()
		{
			$db = new db();
			if ($this->ArtykulIstnieje($this->GetValue('aid'))==TRUE)
			{
				   // jesli artykul nie istnieje zapisuje nowy
				$db->DoSql(&#092;"INSERT INTO artykuly VALUES('','$this->artykul['uid']')\");
				if ($db->DBError) echo &#092;"Nieudane dodanie rekordu! Linia: \".__LINE__.\" w pliku \".__FILE__.\"<br>\";
				$this->SetValue('aid',$db->AddedID());
				$this->GetArticle($this->GetValue('aid'));
			}
			else
			{
				// jesli istnieje aktualizuje go
				$n_uid = $this->GetValue('uid');
				$aid = $this->GetValue('aid');
				$db->DoSql(&#092;"UPDATE artykuly SET u_id='$n_uid' WHERE a_id=$aid\");
				if ($db->DBError) echo &#092;"Nieudana aktualizacja rekordu! Linia: \".__LINE__.\" w pliku \".__FILE__.\"<br>\";
			}
			unset($db);
		}
	}
 
?>
[PHP] pobierz, plaintext 

plik db_class.inc.php

[PHP] pobierz, plaintext 
<?php
/*
 
	@Projekt:		 System Ocen Artykułów
	@Plik:		  artykuly_class.inc.php
	@Opis:			Obłsuga i zarządzanie baza danych
	@Data:			 12 maj 2005
	@Autor:		 Marcin Bancerz
	@Inne:			Praca zaliczeniowa z przedmiotu P.S.I.
 
*/
 
	class db
	{
		var $dbhost;		// serwer bazy danych
		var $dbuser;		// uzytkownik bazy danych
		var $dbpass;		// haslo do bazy danych
		var $dbbase;		// nazwa bazy danych
		var $h_dblink;		// uchwyt do polaczenia
		var $h_dbbase;		// uchwyt do wybranej bazy
		var $result;		// wartosc wyniku zapytania
		var $DBError;		 // przechowuje stan bledu
 
		// konstruktor klasy
		function db()
		{
			$this->dbhost = &#092;"xxx\";
			$this->dbuser = &#092;"xxxt\";
			$this->dbpass = &#092;"xxx\";
			$this->dbbase = &#092;"xx\";
			$this->h_dblink = mysql_connect($this->dbhost,$this->dbuser,$this->dbpass) or $this->DBError = True;
			$this->h_dbbase = mysql_select_db($this->dbbase) or $this->DBError = True;
			if ($this->DBError)
			{
				echo &#092;"BŁĄD POŁĄCZENIA Z BAZĄ DANYCH!!!!\";
				exit;
			}
		}
 
		// wykonuje zapytanie SQL
		function DoSql($sql)
		{
			$this->result = mysql_query($sql,$this->h_dblink);// or $this->DBError = True;
		}
 
		// zwraca wiersz w postaci tablicy
		function Result()
		{
			return mysql_fetch_row($this->result);// or $this->DBError = True;
		}
 
		// zwraca liczbe wierszy z zapytania
		function NumOfRows()
		{
			return mysql_num_rows($this->result,$this->h_dblink);// or $this->DBError = True;
		}
 
		// zwraca id dodanego rekordu
		function AddedID()
		{
			return mysql_insert_id($this->h_dblink);
		}
	}
 
?>
[PHP] pobierz, plaintext 

plik index.php (do testow, zadnego interfejsu jeszcze nie ma):

[PHP] pobierz, plaintext 
<html>
 
<head>
  <meta http-equiv=\"content-type\" content=\"text/html; charset=iso-8859-2\">
  <title></title>
</head>
 
<body>
 
<?php
	//ini_set(\"display_errors\",0);
	//error_reporting(0);
	include 'db_class.inc.php';
	include 'artykuly_class.inc.php';
 
	$art = new artykuly();
	$art->GetArticle(1);
	$art->SetValue('uid',5);
	echo &#092;"Powinno<pre>\";
	print_r($art->artykul);
	echo &#092;"</pre>Jest\";
	$art->SaveArticle();
	$art->GetArticle(1);
	 echo &#092;"<pre>\";
	print_r($art->artykul);
	echo &#092;"</pre>\";
 
	unset($art);
 
?>
 
</body>
 
</html>
[PHP] pobierz, plaintext 

na koniec dodam strukture bazy:

[SQL] pobierz, plaintext 
/*
Created		2005-05-12
Modified		2005-05-12
Project		System ocen artykułów
Model		SOA
Company		FallNet Webdesign
Author		Marcin Bancerz
Version		1.0
Database		mySQL 4.0 
*/
 
DROP TABLE IF EXISTS komentarze;
DROP TABLE IF EXISTS artykuly;
DROP TABLE IF EXISTS uzytkownicy;
 
CREATE TABLE uzytkownicy (
	u_id Int NOT NULL AUTO_INCREMENT,
	u_nick Varchar(20),
	u_email Varchar(20),
 PRIMARY KEY (u_id)) TYPE = MyISAM
ROW_FORMAT = DEFAULT;
 
CREATE TABLE artykuly (
	a_id Int NOT NULL AUTO_INCREMENT,
	u_id Int NOT NULL,
	a_tytul Tinytext,
	a_tresc Text,
	a_data Timestamp(14),
 PRIMARY KEY (a_id)) TYPE = MyISAM
ROW_FORMAT = DEFAULT;
 
CREATE TABLE komentarze (
	k_id Int NOT NULL AUTO_INCREMENT,
	u_id Int NOT NULL,
	a_id Int NOT NULL,
	k_tytul Tinytext,
	k_tresc Text,
	a_ocena Tinyint,
	a_data Timestamp(14),
 PRIMARY KEY (k_id)) TYPE = MyISAM
ROW_FORMAT = DEFAULT;
 
ALTER TABLE artykuly ADD INDEX IX_napisal (u_id);
ALTER TABLE artykuly ADD FOREIGN KEY (u_id) REFERENCES uzytkownicy (u_id) ON DELETE  restrict ON UPDATE  restrict;
ALTER TABLE komentarze ADD INDEX IX_komentuje (u_id);
ALTER TABLE komentarze ADD FOREIGN KEY (u_id) REFERENCES uzytkownicy (u_id) ON DELETE  restrict ON UPDATE  restrict;
ALTER TABLE komentarze ADD INDEX IX_jest_komentowany (a_id);
ALTER TABLE komentarze ADD FOREIGN KEY (a_id) REFERENCES artykuly (a_id) ON DELETE  restrict ON UPDATE  restrict;
[SQL] pobierz, plaintext 

czy to co tutaj podalem ma chociaz cos wspolnego z php obiektowym? Co jest np niezgodne z tymi zasadami?

i mam maly problem z :

[PHP] pobierz, plaintext 
<?php
 
		function DoSql($sql)
		{
			$this->result = mysql_query($sql,$this->h_dblink) or $this->DBError = True;
		}
 
?>
[PHP] pobierz, plaintext 

gdy tak mam niestety mysql_query nie wykonuje sie.. (dlatego jest // w orginale), chcialem po prostu zastapic Die() wlasna "obsluga błędów" (prowizoryczną) lecz taki zaspis nie działa.. jak sprawdzić czy mysql_query nie zwraca błędu?


pozdrawiam

Ten post edytował Fallout 12.05.2005, 17:11:48

[Ociu]

var $artykuly; powinno był array chyba ?
daj w konstruktorze $this -> db = new db; i poczytaj o singletonie.

Zamiast

[PHP] pobierz, plaintext 
<?php
		// funkcja pobiera wartosc dla zmiennej
		function GetValue($zmienna)
		{
			return $this->artykul[$zmienna];
		}
?>
[PHP] pobierz, plaintext 

daj:

[PHP] pobierz, plaintext 
<?php
		// funkcja pobiera wartosc dla zmiennej
		function GetValue($zmienna)
		{
if(in_array($zmienna, $this -> artykul)) return $this->artykul[$zmienna];
else return false;
		}
?>
[PHP] pobierz, plaintext 

Ten post edytował Ociu 12.05.2005, 18:06:07

[Fallout]

daj w konstruktorze $this -> db = new db; i poczytaj o singletonie.

Witam,

dzieki za wskazowki, czytalem o tym singletonie i... nie kapuje (IMG:http://forum.php.pl/style_emoticons/default/aarambo.gif)
jako ze z programowanie obiektowym nie mialem nigdy do czynienia nie za bardzo potrafie sobie wyobrazic tego.. a konkretniej mowiac nie mam pojecia jak by to mialo u mnie w moim kodzie wygladac i jak dzialac i wogole po co (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

ogolnie zasada

[PHP] pobierz, plaintext 
<?php
$this->db = new $db();
?>
[PHP] pobierz, plaintext 

a potem w kazdej z funkcji odwolanie do tej zmiennej

[PHP] pobierz, plaintext 
<?php
$this->db->DoSql(&#092;"blabla\");
?>
[PHP] pobierz, plaintext 

to jest ten singleton? Bo ja to rozumiem ze to jest odwolanie (jedne) do klasy i przypisanie do obiektu na ktorum tylko sie operuje..

juz sam nie wiem co pisze.. moze daloby rade przyklad na tym moim biednym kodzie pokazac zasade tego singletonu?

pozdrawiam i sorki ze tak męczę (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Marcin

[bela]

Nie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Singleton wzraca Ci zawsze tą samą instancje klasy :]
Czyli robisz Klasa::getInstance(); i zawsze dostaniesz ten sam obiekt (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[sobstel]

mysle ze spokojnie tez mozna uzyc global $db

[Ociu]

http://wiki.php.pl/index.php/Singleton
Wiecej nie trzeba.

[Holyboy]

kwetstia natury nazweniczej - dobra praktyka jest nazywac klasy z duzych liter, a metody i wlasciwosci (oraz inne zmienne) z malych, generalnie przyjely sie dwie notacje:
Odstepy w zmiennych sygnowane znakiem podkreslenia dolnego:

[PHP] pobierz, plaintext 
<?php
 
class ExampleFoo extends Foo {
	var $foo_property;
 
	function ExampleFoo() {}
 
	function set_foo_property($new_value) {
		$this->foo_property = $new_value;
	}
}
 
?>
[PHP] pobierz, plaintext 

Odstepy w zmiennych sygnowane duza litera:

[PHP] pobierz, plaintext 
<?php
 
class ExampleFoo extends Foo {
	var $fooProperty;
 
	function ExampleFoo() {}
 
	function setFooProperty($newValue) {
		$this->fooProperty = $newValue;
	}
}
 
?>
[PHP] pobierz, plaintext 

doprawdy Twoj kod czyta sie nieswojo - na pierwszy rzut oka nie moglem doszukac sie konstruktora (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[bela]

kwetstia natury nazweniczej - dobra praktyka jest nazywac klasy z duzych liter, a metody i wlasciwosci (oraz inne zmienne) z malych, generalnie przyjely sie dwie notacje:

Nie do końca się zgodzę (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Wszystko zależy od języka i standartów w nim przyjętych w takim C# zwykło zaczynać się wszystko z dużej ;] Tudzież wypadało wspomnieć o notacji węgierskiej i stosowaniu przedrostków ( kto stosuje ten stosuje, ja niestety nie ;D )