[PHP] klasa do obsługi bazy danych - oop Opcje

[dwwa]

próbuję napisać klasę do obsługi bazy danych mysql, z oop u mnie marnie, dopiero zaczynam

mam takie coś:

[PHP] pobierz, plaintext 
<?php
 
	class SQL {
 
		private $db_host = DB_HOST;
		private $db_user = DB_USER;
		private $db_pass = DB_PASS;
		private $db_name = DB_NAME;
 
		public $connection;
		public $sql_error;
 
		public function sql_connect() {
			if ($connection = mysql_connect($this->db_host, $this->db_user, $this->db_pass)) {
				if (mysql_select_db($this->db_name)) {
					$this->connection = $connection;
					return true;
				}
				else {
					echo $this->sql_error = mysql_error();
					return false;
				}
			}
			else {
				echo $this->sql_error = mysql_error();
			}
		}
 
	}
 
?>
[PHP] pobierz, plaintext 

czy takie coś jest w miarę dobrze napisane?

[Mephistofeles]

Nie.
mysql_ to zamierzchła przeszłość. Dzisiaj mamy PDO, nie musisz pisać własnego opakowania.

[darko]

Do nauki podstaw chyba większość z nas opakowywała jakąś funkcjonalność bazodanową. Oczywiście z kodu żadnego użytku nie będzie, ale dla celów edukacyjnych takie rzeczy można pisać "do szuflady". To tak:
- metoda łącząca nas z bazą powinna przyjmować parametry: host, user, pass, db_name, opcjonalnie port U Ciebie bierze te dane z pól klasy i ... ok, ale nie ma tutaj możliwości podania tych parametrów, a powinno w takiej metodzie być miejsce na to;
- gdzie są metody CRUD ? Dopisz;
- nigdy przenigdy nie robimy echo, print itd w metodzie. Metoda powinna zwracać określoną wartość lub wartości, ale nigdy wyrzucać bezpośrednio na ekran jakichkolwiek danych, po prostu zamiast echo $this->sql_error = mysql_error(); wystarczy przypisać $this->sql_error = mysql_error(); i dopisać metodę getLastError() która zwróci $this->sql_error;
Osobiście poćwiczyłbym rozszerzając istniejącą klasę PDO, która i tak ma już większość tego, co jest potrzebne, w każdym razie powodzenia w nauce!

[dwwa]

ok dzięki za odp

- metoda łącząca nas z bazą powinna przyjmować parametry: host, user, pass, db_name, opcjonalnie port U Ciebie bierze te dane z pól klasy i ... ok, ale nie ma tutaj możliwości podania tych parametrów, a powinno w takiej metodzie być miejsce na to;

jak z pól? mam plik np. config.php a w nim:

[PHP] pobierz, plaintext 
DEFINE('DB_HOST', 'localhost')
[PHP] pobierz, plaintext 

czy nawet w index.php przed załadowaniem klasy można takie coś dać

jaką polecacie klasę do napisania na początek nauki oop?

Ten post edytował dwwa 20.10.2011, 21:44:12

[by_ikar]

Kod

jaką polecacie klasę do napisania na początek nauki oop?

Jaką kolwiek ja swoją pierwszą klasę jaką napisałem, to był pager, którego co prawda zdążyłem już kilkanaście razy zmodyfikować (ulepszyć), ale ogólny zarys działania klasy, pomijając drobne poprawki używam do dnia dzisiejszego i sprawuje się całkiem dobrze