[php][mysql]Klasa obsługi bazy Opcje

[kuba749]

Napisałem klasę do obsługi mysql-a przez php-a i miałbym taką prośbę czy mógłby mi ją ktoś przetestować oraz podpowiedzieć co jeszcze dopisać.
Dzięki

[PHP] pobierz, plaintext 
<?php
 
	class mysql
	{	
		public $row;
 
		public function __construct(){
 
			$this->open = mysql_connect('localhost', 'root', 'root') or die("Nie udło się połczyć z serwerem " .mysql_error());
			$this->db = mysql_select_db('test', $this->open) or die("Nie udło się połczyć z bazą danych " .mysql_error());
 
		}
 
 
		function query($sql){
 
			mysql_query($sql) or die("Błąd: ".mysql_error());
 
		}
 
 
		function newtable($name, $pola){
 
			$this->query("CREATE TABLE $name ($pola)");
 
		}
 
		function insert($gdzie, $pola, $co){
 
			$this->query("INSERT INTO $gdzie ( $pola ) VALUES ( $co )");
 
		}
 
		function delete($skad, $co){
 
			$this->query("DELETE FROM $skad WHERE $co");
 
		}
 
		function update($co, $dane, $gdzie){
 
			$this->query("UPDATE $co SET $dane WHERE $gdzie");
 
		}
 
		function select($co, $tabela, $gdzie){
 
			$this->query("SELECT $co FROM $tabela WHERE $gdzie");
 
		}
 
		function fetch($co, $tabela, $gdzie){
			$this->row = mysql_fetch_row($this->select($co, $tabela, $gdzie));
		}
 
		public function __destruct(){
 
			mysql_close($this->open);
 
		}
	}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował kuba749 14.01.2008, 21:36:22

[nevt]

niestety, ta klasa nie będzie ci działać, więc nawet nie ma co testować... najpoważniesze błędy to:

1. metoda query() nie zapamiętuje ani nie zwraca wyników zapytania, tym samym metoda select() nie zadziała...
2. metoda fetch() nie będzie działać... a w zasadzie (jeśli wcześniej poprawisz metody query() i select() zawsze będzie zwracać pierwszy wiersz wyniku zapytania...
3. konstruktor powinien otrzymywać jawnie przekazywane parametry połączenia, albo nazwę pliku konfiguracyjnego...
4. nie zadeklarowałeś właściwości $open i $db a używasz ich w konstruktorze...

to tyle na początek...