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 

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ć...

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

Nadal nie jest dobrze.


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


Małpy dopisałeś bezmyślnie?


Poczytać o funkcjach agregujących w SQL.


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


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

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

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