witam
napisalem sobie prostą klasę do obsługi MySQL, nie zamieszczam tego jeszcze w dziale oceny ponieważ to nic wielkiego, chciałbym jeszcze nad nią tochę popracować, co radzicie, przeglądałem troche kody podobnych klas, czekam na Wasze opinie
myślę o jakiejś obsłudze błędów ponieważ narazie nie ma to żadnych zabezpieczeń
pzdr

[PHP] pobierz, plaintext 
<?
//show_source ('mysql.class.php');
error_reporting (E_ALL);
 
class mysql {
 
private $host;
private $user;
private $password;
private $database;
public $id_connection;
 
	   public function __construct ($host, $user, $password, $database) {
	   
	   $this->host = $host;
	   $this->user = $user;
	   $this->password = $password;
	   $this->database = $database;
 
						  if (! ($this->id_connection = mysql_connect ($this->host, $this->user, $this->password))) {
								throw new Exception ('Nie mogę połączyć się z serwerem bazy danych');
							 } 
						  if (! (mysql_select_db($this->database))) {
						  throw new Exception ('Nie mogę połączyć się bazą danych: '.$this->database);
						  } 
 
		}
 
		public function make_query($sql) {
		return new mysql_res (mysql_query($sql, $this->id_connection));
		}
 
		public function __destruct () {
		mysql_close ($this->id_connection);
		}
}
 
class mysql_res implements IteratorAggregate {
 
private $result;
 
		public function __construct($result) {
		$this->result = $result;		
		}
 
		public function getIterator() {
		return new mysql_res_iterator($this->result);
		}
 
 
}
 
class mysql_res_iterator implements Iterator {
 
private $result;
private $current = 0;
private $rows;
 
		public function __construct($result) {
		$this->result = $result;
		$this->rows = mysql_num_rows($this->result);
		}
 
		public function current() {
		return mysql_fetch_object($this->result);
		}
 
		public function key() {
		return $this->current;
		}
 
		public function rewind() {
		$this->current = 0;
		mysql_data_seek($this->result, $this->current);
		}
 
		public function next() {
		$this->current++;
		}
 
		public function valid() {
		return $this->current < $this->rows;
		}
 
}
?>
[PHP] pobierz, plaintext 

i użycie:

[PHP] pobierz, plaintext 
<?
 
function __autoload($classname) {
include_once $classname.'.class.php';
}
 
 
try {
$db = new mysql('localhost','root','krasnal', 'tomek');
$sql = 'SELECT * FROM ks_tel';
$dane = $db->make_query($sql);
 
foreach ($dane as $id ){
echo 'Imie: '.$id->imie.'<br>';
echo 'Nazwisko: '.$id->nazwisko.'<br>';
echo 'Numer: '.$id->numer.'<br><br>';
}
 
} catch (Exception $e) {
echo $e;
}
?>
[PHP] pobierz, plaintext 

Bardzo marna ta klasa. Brak obiektu wyniku, brak możliwośc zwrócenia wyniku jako iterator - sporo do zrobienia.

Bez obrazy ale wyglada to jak przepisanie strukturalnego mysql na obiekt i nic wiecej.
Null: jesli chdozi o wykonanie obiektu dla wyniku itd, nie ma sprawy, ale nie wszyscy tego wymagaja. Ja naprzyklad zrobilem swoj prosty obiekt dla wlasnego uzytku i miesci sie to w 2 klasach, gdzie ta druga to dodatkowy debugger

No ale wracajac do klasy. Dla ulatwienia zycia proponuje dopisac takie proste dosc metody jak getRow, getArray, getCol, getOne, insert, update bardzo przydatne przy wiekszych aplikacjach. Duzo szybciej sie pisze poznej cokolwiek.

W konstruktorze odradzal automatycznego laczenia sie z baza. Wiem ze moze to dziwne, ale ja przed polaczeniem wole ustawic sobie wszystko. wlaczenie logo, debuger, licznik zapytan i wiele innych rzeczy. pozniej dopiero wykonywane jest polaczenie.
To takie sugestie

Z mojej strony tylko tyle co już zaznaczył @Sh4adow. Klasa nie ma najmniejszego sensu, jest tylko i wyłącznie nadpisaniem metod już zaimplementowanych w php, więc po co jej uzywać? Co innego jeżeli wnosiłaby coś konkrentego, ale tak nie jest. Gdybym miał to oceniać to byłoby to 1/10.

proponuje ci skorzystac z gotowej klasy np. dostepnej na phpclasses.org. ja sobie bardzo chwale bodajze ezSql (ezMysql? jakos tak).

o wygoglalem http://php.justinvincent.com/

no chyba, że traktujesz to jako nauke i masz czas na zabawe z tym. wtedy to skorzystaj z rad kolegów wyżej. ja nie mam czasu wiec korzystam z tego co gotowe - o ile mi pasuje a akurat ten ezSql jest tym co sam chcialem napisac. dodalem do niego tylko zliczanie zapytan i czasow pytan.

dzięki za wyrażenie własnych opini, ta klasa to moja pierwsza klasa pisana pod PHP5 i wiem, że nie jest to nic wielkiego, mam zamiar nad nią popracować więc powinna się rozbudować, spróbuję dostosować się do rad kolegi Sh4dowa i dopisania tych metod, jeśli można by było to prosiłbym o dokłądniejsze skonkretyzowanie co wymienione przez Ciebie metody miały by dokładnie robić lub ewentualnie powiedzieć mi czy dobrze myślę:

getRow - jak myśle pobierałaby wiersz z bazy danych
getArray - hmm
getCol - kolumne
getOne - hmm

i jeszcze jedno czy warto dopisać metody typu insert, update, delate, które by poprostu dodawały, uaktualniały bądź usuwały dane z bazy czy poprostu zostawić to tak jak jest to w ten sposób i przypisywać to jakiejś zmiennej $sql, na której miała by działać metoda make_query()

i jeszcze jedno nie za bardzo zrozumiałem o co chodzi Tobie NuLL


jeśli mógłbyś wyjaśnić lub odesłać to jakiegoś przykładu byłbym wdzięczny

normanos: traktuję to jako formę nauki, ale dzięki za linka zapoznam się
pzdr

NuLL nie jesteś w dziale 'php Pro' więc miło by było gdybyś pisał by każdy zrozumial ;]

Co do klasy przypomina mi moja pierwsza klase do DB tyle ze pisana pod php4

Pare dni temu równierz pisałem klase pod php5 i bede musial ja pisać od nowa bo ona pokazuje tylko jak pisac nie wolno i jest bardzooo toporna

Co do klasy przedstawionej przez normanosa też nie jest super(ogladalem tylko wersja dla mysqla). Głównie przez to:

[PHP] pobierz, plaintext 
<?php
					 if ( preg_match("/^(insert|delete|update|replace)s+/i",$query) )
					 {
							$this->rows_affected = mysql_affected_rows();
 
							// Take note of the insert_id
							if ( preg_match("/^(insert|replace)s+/i",$query) )
							{
								   $this->insert_id = mysql_insert_id($this->dbh);	   
							}
 
							// Return number fo rows affected
							$return_val = $this->rows_affected;
					 }
?>
[PHP] pobierz, plaintext 

Można by to bez problemu zastąpić funkcjami str* bądź jednym pregiem

GetArray zazwyczaj pobiera wynik jako tablice asocjacyjna, gdzie indexy tablicy maja nazwy pol w tabeli. tablica jest oczywiscie wielo-wymiarowa. mozna zrobic modyfikacje pod tytulem ze zglowny index tablicy bedzie to powiedzmy klucz PRIMARY tablicy.

[PHP] pobierz, plaintext 
<?php
$array = $db->getArray("SELECT * FROM table");
// $array[0] = array('id' =>1, 'col1' => 'val1', 'col2' =>'val2');
// lub
// jesli kluczem tablicy jest naprzyklad ID
// $array[1] = array('col1' => 'val1', 'col2' =>'val2');
?>
[PHP] pobierz, plaintext 

Wsszystko zalezy od zastosowan jakie sa potrzebne
getOne - pobiera tylko jedna wartosc, zawsze pierwsza z brzegu. przydatne naprzyklad przy zliczaniu wierszy przy zapytaniach typu

[PHP] pobierz, plaintext 
<?php
$int = $db->getOne("SELECT COUNT(*) FROM table");
//zwroci liczbe
?>
[PHP] pobierz, plaintext 

Przy getCol moze zrobi generator zapytania naprzyklada podajac tylko nazwe kolumny i nazwe tablicy, ale za jednym zamachem utrudniamy limity, warunki itp. Wiec wykonujemy metode podobna do getOne, czyli ponieramy tylko jedna warosc z tablicy, zawsze pierwsza i wpladamy w polejne indexy. Tu mamy tablice jedno wymiarowa.

Metod jakie mozna wykonac przy takiej klasie jest naprawde duzo, mozna to wszystko laczyc razem z obiektem zajmujacym sie cache'em.

Klas do baz danych jest na tyle duzo ze oczywiscie mozna sobie cos wybrac. Ale jest to dobra rzecz do nauki i eksperymentow przy obiektach. Proste, przydatne chyba kazdemu. Ja osobiscie sie przekonalem ze uzywajac czyjejs klasy dla baz danych zawsze nadchodzi czas zeby ja chociaz troszeczke poprzerabiac

Zycze powodzenia. O tym to mozna by prawie arta napisac, ale ja sie nie podejmuje, nie mam czasu juz na nic.

witam
edytowałem troche swoją klasę, wprowadziłem drobną obsługę błędów, marna ale rozbuduje to i wprowadzilem funckje wykorzystujące Iterator, przyznam, że pierwszy raz się z tym spotkałem i większość kodu pochodzi w wiki.php.pl

teraz mam pytanie odnośnie tych klas, gdzie wprowadzać do której z klas pozostałe metody, które rozbudują możliwości tej klasy ?

i jak mam rozumiec pojęcie cacheowania i debugowania o których wspominaliście w komentarzach do pierwszej wersji tej klasy

pzdr

artykuł o cacheowaniu zapytań sql (do plików) jest na webcity, poszukaj. ogolnie szybciej bedzie keszowac cały template i ja po jakims czasie usunąłem mozliwośc keszu.

nie wiem co miał kolega na mysli pisząc debbugowanie -> byc może wypluwanie tabelek z rozbiciem zapytania sql na częsci pierwsze - pomaga zlokalizowac błąd, tudzież zoptymalizowac zapytanie.
przyklad tego masz w klasie, która wcześniej poleciłem.

zamieszczam kolejną wersję mojej skromnej klasy

[PHP] pobierz, plaintext 
<?
//show_source ('mysql.class.php');
error_reporting (E_ALL);
 
class mysql {
 
private $host;
private $user;
private $password;
private $database;
public $id_connection;
 
	   public function __construct ($host, $user, $password, $database) {
	   
	   $this->host = $host;
	   $this->user = $user;
	   $this->password = $password;
	   $this->database = $database;
 
						  if (! ($this->id_connection = mysql_connect ($this->host, $this->user, $this->password))) {
								throw new Exception ('Nie mogę połączyć się z serwerem bazy danych');
							 } 
						  if (! (mysql_select_db($this->database))) {
						  throw new Exception ('Nie mogę połączyć się bazą danych: '.$this->database);
						  } 
 
		}
 
		public function make_query($sql) {
		return new mysql_res (mysql_query($sql, $this->id_connection));
		}
 
		public function __destruct () {
		mysql_close ($this->id_connection);
		}
}
 
class mysql_res implements Iterator {
 
private $result;
private $current = 0;
private $rows;
 
		public function __construct($result) {
		$this->result = $result;	
		$this->rows = mysql_num_rows($this->result);	
		}
 
		public function getIterator() {
		return new mysql_res_iterator($this->result);
		}
 
		public function current() {
		return mysql_fetch_object($this->result);
		}
 
		public function key() {
		return $this->current;
		}
 
		public function rewind() {
		$this->current = 0;
		mysql_data_seek($this->result, $this->current);
		}
 
		public function next() {
		$this->current++;
		}
 
		public function valid() {
		return $this->current < $this->rows;
		}
 
 
}
 
 
?>
[PHP] pobierz, plaintext 

mam pytanie o metody wymuszone przez Iterator, w jakich sytuacjach ich najlepiej używać, zauważyłem również, że raczej nie potrzebna jest metoda typu mysql_fetch_array(), ponieważ można to zrobić inaczej poprawcie mnie jeśłi źle myślę oraz kolejne pytanie do której z klas dopisywać ewentualne dodatkowe metody, o których mowa był wcześniej?

pzdr