[PHP][MySQL]Klasa PHP do obsługi mySql Opcje

[Woytek950]

Witam. Napisałem klasę do obsługi bazy mysql i chciał bym zasięgnąć Waszej opini co poprawić, co zmienić i czy Waszym zdaniem taka klsa jest logiczna.
przykład użycia:

plik mysql.class.php

[PHP] pobierz, plaintext 
<?php
// Klasa do obsługi bazy danych mysql
// Nazwa klasy: MySql;
// Plik z klasą: mysql.class.php
 
	class MySql
	{
			public function MySql($host, $user, $pass, $db)
			{
				@mysql_connect($host,$user,$pass) or die("<div>Blad polaczenia MySql.</div><br/><i>".mysql_error()."</i>");
				@mysql_select_db($db) or die("<div>Blad wyboru bazy MySql.</div><br/><i>".mysql_error()."</i>");
 			}
 
			public function getRows($query)
			{
				$qu = mysql_query($query) or die("<div>Blad zapytania MySql.</div><br/><i>".mysql_error()."</i>");
				while($row = mysql_fetch_assoc($qu))
				{
					$arr[] = $row;
				}
 			}
 
			public function getNumRows($query)
			{
				$qu = mysql_query($query) or die("<div>Blad zapytania MySql.</div><br/><i>".mysql_error()."</i>");
				return mysql_num_rows($qu);
 			}
		// Koniec klasy
	}
 
?>
 
[PHP] pobierz, plaintext 

plik index.php

[PHP] pobierz, plaintext 
	$mysql = new MySql('localhost','root','krasnal','cms');
	$tpl->assign("jakaspetla", $mysql->getRows('SELECT * FROM menu'));
 
[PHP] pobierz, plaintext 

plik index.tpl

[PHP] pobierz, plaintext 
				{loop name="jakaspetla"}
				 <a href="{$value[alias]}">{$value[name]}</a><br />
				{/loop}
[PHP] pobierz, plaintext 

[erix]

1. getRows niczego nie zwraca
2. błędy wracają do gości
3. wyciszanie błędów zamiast logowania
4. zliczasz rekordy pamięciożerną metodą
5. brak wyjątków

Chyba nie ma sensu tego używać...

[Woytek950]

Poprawiłem troche. Chciał bym poinformować że to moja pierwsza klasa...


getRows niczego nie zwraca - było dobrze tylko nie skopiowalem.
błędy wracają do gości - w jaki sposób to z mienić?
wyciszanie błędów zamiast logowania - w jaki sposób to zmienić?
zliczasz rekordy pamięciożerną metodą - w jaki sposób to zmienić?
brak wyjątków - w jaki sposób to zmienić?

mam zamiar ją rozbudowac, to narazie początek

Ten post edytował Woytek950 4.06.2010, 15:56:46

[erix]

getRows niczego nie zwraca - było dobrze tylko nie skopiowalem.

Nadal nie jest dobrze.

błędy wracają do gości - w jaki sposób to z mienić?

Poczytaj o set_error_handler albo napisz odpowiednie metody logujące błędy.

wyciszanie błędów zamiast logowania - w jaki sposób to zmienić?

Małpy dopisałeś bezmyślnie?

zliczasz rekordy pamięciożerną metodą - w jaki sposób to zmienić?

Poczytać o funkcjach agregujących w SQL.

brak wyjątków - w jaki sposób to zmienić?

Poczytać i zrozumieć, po co są wyjątki.

mam zamiar ją rozbudowac, to narazie początek

Tylko czy jest sens pisać N-ty raz to samo...?

[Woytek950]

Tak, ponieważ się ucze i to moje pierwsze kroki w OPP.

[zend]

Ta klasa praktycznie nic nie wnosi do kodu, jedynie dodatkową niepotrzebną abstrakcję. Dorzuć do niej metody typu insert, update, delete, select, to już będzie lepiej