Forum PHP.pl

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

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

[PHP] pobierz, plaintext 
<?php
 
	class mysql
	{	
		public $row;
		public $open;
		public $db;
 
		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){
			return 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){
			return $this->query("SELECT $co FROM $tabela WHERE $gdzie");
		}
 
		function fetch($co, $tabela, $gdzie){
			while($this->row == mysql_fetch_row($this->select($co, $tabela, $gdzie))){
				$this->row[0];
				$this->row[1]; //to tylko przykład itd.
			}
		}
 
		public function __destruct(){
			mysql_close($this->open);
		}
	}
 
?>
[PHP] pobierz, plaintext 

Teraz już troche lepiej??

Ten post edytował kuba749 14.01.2008, 22:12:35

[PHP] pobierz, plaintext 
<?php
return mysql_query($sql) or die("Błąd: ".mysql_error());
?>
[PHP] pobierz, plaintext 

No kochany, jesli twoja klasa ma mi wywalac cala aplikacje bo w zapytaniu byl jakis blad to ja dziekuje...

Pozatym te select, update, delete itp. sa w zasadzie calkowicie bezuzyteczne, bo i tak trzeba im w zasadzie wszystko podac juz tak jak ma byc. Te funkcje same w sobie nic nie robia pod kątem sqlinjection itakie tam.

[PHP] pobierz, plaintext 
<?php
while($this->row == mysql_fetch_row($this->select($co, $tabela, $gdzie))){
				$this->row[0];
				$this->row[1]; //to tylko przykład itd.
			}
?>
[PHP] pobierz, plaintext 

oj, kiepsko poprawiles te fetch(). ty to wogole sam testowales? Jesli tak potestuj jeszcze raz a dopiero dawaj nam do teszczenia.

[PHP] pobierz, plaintext 
<?php
  class mysql{
 
	public $open;
	public $db;
	public $wynik;
	var $rows;
	var $bufor = array();
 
	function __construct(){
		$this->open = mysql_connect('localhost','root','')
		or die("Nie udało się połączyć z serwerem: ".mysql_error());
		$this->db = mysql_select_db('test',$this->open)
		or die("Nie udało sie połączyć z bazą danych: ".mysql_error());
	}
 
	function __destruct(){
		mysql_close($this->open);
	}
 
	function query($sql){
		$this->wynik = mysql_query($sql) or ($error = mysql_error());
		if($this->wynik) return ( $this->wynik );
		else{
			die ($error);
			exit;
		}
	}
 
	function fetch($sql, $typ){
		$typy = array(
		'1' => MYSQL_NUM,
		'2' => MYSQL_ASSOC,
		'3' => MYSQL_BOTH
		);
		while($rows = mysql_fetch_array($sql, $typy[$typ])){
			$this->bufor[] = $rows;
		}
		return ( $this->bufor );
	}
  }
?>
[PHP] pobierz, plaintext 

troche napoprawiałem i gdy np.

[PHP] pobierz, plaintext 
<?php
$sql = new mysql;
$query = $sql->query("SELECT * FROM users");
$row = $sql->fetch($query, 3);
echo $row[0];
?>
[PHP] pobierz, plaintext 

nie wyświetla tego rekordu tylko Array co zrobić??

Ten post edytował kuba749 15.01.2008, 14:25:17

tja, jasne. moją uwagę odnosnie die() to zmieniles jedynie kod, ktory defacto robi to samo co na poczatku - czyli wywala mi aplikacje gdy w zapytaniu jest blad.


nie:echo $row[0];
a: print_r($row[0])
Przeciez $row tablica wielowymiarowa.

a co byś chciał żeby robiło jak się pojawi błąd??

zeby nie wywalalo mi aplikacji. To chyba oczywiste.

Musisz zwrocic wtedy false. A bledy masz zapisywac w jakiejs tablicy, od biety mozesz rzucic trigger_error()

ale jak na przykład

[PHP] pobierz, plaintext 
<?php
$sql = new mysql;
$query = $sql->query(SELECT * FROM userrs); //błąd w nazwie tabeli
$row = $sql->fetch($query,2);
print_r ($row[0]);
?>
[PHP] pobierz, plaintext 

jak będzie taki błąd to i tak nie zadiała

Ten post edytował kuba749 15.01.2008, 14:48:25

Jesli pojawi sie blad, czyli query zwroci false, to dalszy kod ktory przetwarza dane ma sie nie wykonac. Slyszales o instrukcjach warunkowych?
if fajnie to fajnie
else niefajnie i kodu nie wykonam
?

w końcu chyba wiem już o ci dokładnie chodzi
a nie można by było tak:

[PHP] pobierz, plaintext 
<?php
function query($sql){
		$this->wynik = mysql_query($sql) or ($error = mysql_error());
		if($this->wynik) return ( $this->wynik );
		else{
			die ($error);
			exit(1);
		}
?>
[PHP] pobierz, plaintext 

może być tak:

[PHP] pobierz, plaintext 
<?php
function query($sql){
	$this->wynik = mysql_query($sql) or ($this->error .= mysql_error().'<br>');
	if($this->wynik) return ( $this->wynik );
	else return (false);
}
function fetch($sql, $typ){
	$typy = array(
	'1' => MYSQL_NUM,
	'2' => MYSQL_ASSOC,
	'3' => MYSQL_BOTH
	);
	if($this->query($sql) != false){
		while($rows = mysql_fetch_array($this->query($sql), $typy[$typ])){
			$this->bufor[] = $rows;
		}
		return ( $this->bufor );
	}
}
?>
[PHP] pobierz, plaintext 

Przykład

[PHP] pobierz, plaintext 
<?php
$sql = "SELECT * FROM users";
$mysql = new mysql;
$row = $mysql->fetch($sql,2);
print_r ($row[0][id]);
print $mysql->error;
?>
[PHP] pobierz, plaintext 

Ten post edytował kuba749 16.01.2008, 10:42:20