Problem z klasą do mysql Opcje

[Kas]

Ostatnio mało bywałem na forum, ponieważ min. tworzyłem [usunąłem linka] alek.
Ale nie w tym problem. (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)
Stworzyłem klasę do obsługi bazy danych mysql.

[PHP] pobierz, plaintext 
<?php
class sql {
	  private $link;
	  private $result;
	  public $rows;
 
	  public function __construct {
 
		  $this -> $sql_host = 'localhost'; // Adres serwera bazy danych
		  $this -> $sql_name = 'forum'; // Nazwa bazy danych
		  $this -> $sql_user = 'root'; // Nazwa usera
		  $this -> $sql_password = ''; // Hasło
		  $this -> $sql_port = '3306'; // Port
 
		  $this -> link = mysql_connect($this -> $sql_host, $this -> $sql_user, $this -> $sql_password);
		  if (!is_resource($this -> link) {
			  die('Brak połączenia z bazą danych.' . mysql_error());
		  }
	  }
 
	  public function __destruct () {
		  if(is_resource($this -> link)) {
			  mysql_close($this -> link);
		  }
	  }
 
	  public function query($query) {
		 $this -> result = mysql_query($query, $this -> link);
	 }
	 
	  public function fetch_row() {
		 if($this -> rows = mysql_fetch_row($this -> result)){
			return 1;
		}
	}
}
?>
[PHP] pobierz, plaintext 

I nie działa. (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)
Cały kod wygląda tak.

[PHP] pobierz, plaintext 
<?php
 
class sql {
	  private $link;
	  private $result;
	  public $rows;
 
	  public function __construct {
 
		  $this -> $sql_host = 'localhost'; // Adres serwera bazy danych
		  $this -> $sql_name = 'forum'; // Nazwa bazy danych
		  $this -> $sql_user = 'root'; // Nazwa usera
		  $this -> $sql_password = ''; // Hasło
		  $this -> $sql_port = '3306'; // Port
 
		  $this -> link = mysql_connect($this -> $sql_host, $this -> $sql_user, $this -> $sql_password);
		  if (!is_resource($this -> link) {
			  die('Brak połączenia z bazą danych.' . mysql_error());
		  }
	  }
 
	  public function __destruct () {
		  if(is_resource($this -> link)) {
			  mysql_close($this -> link);
		  }
	  }
 
	  public function query($query) {
		 $this -> result = mysql_query($query, $this -> link);
	 }
	 
	  public function fetch_row() {
		 if($this -> rows = mysql_fetch_row($this -> result)){
			return 1;
		}
	}
}
 
$sql = new sql;
$sql -> $query('SELECT username FROM users');
 
while($sql -> fetch_row()){
	echo '<li>'.$sql -> rows[0].'</li>';
	}
echo '</ul>';
 
?>
[PHP] pobierz, plaintext 

Co jest źle?

[mike]

1. Po jakie licho wstawiłeś ten sam kod dwa razy? Nie można było wstawić kody klasy, a potem kodu, który z nij korzysta :?:
2. Pisząc "nie działa" napisz coś więcej. Nadworny Jasnowidz forum jest na urlopie i ludzie nie będą się domyślać. Jeżeli chcesz uzyskać odpowiedź, włóż w pytanie trocę więcej wysiłku.

[dr_bonzo]

Co to jest? $this -> $sql_host
Usun '$' sprzed nazw pol:
$this ->sql_host
a poza tym je zadeklaruj i wlacz wyswietlanie bledow
error_reporting( E_ALL );

[pies]

Ja bym jeszcze zrobił tak:

[PHP] pobierz, plaintext 
<?php
 
public function __construct 
{
		  $this -> sql_host = 'localhost'; // Adres serwera bazy danych
		  $this -> sql_name = 'forum'; // Nazwa bazy danych
		  $this -> sql_user = 'root'; // Nazwa usera
		  $this -> sql_password = ''; // Hasło
		  $this -> sql_port = '3306'; // Port
}
 
public function connect()
{
	$this -> link = mysql_connect($this->sql_host, $this->sql_user, $this->sql_password);
		  if (!is_resource($this -> link) {
			  die('Brak połączenia z bazą danych.' . mysql_error());
		  }
}
 
[..]		  
 
public function query($query) 
{
	if (!$this->link) $this->connect();
 
	$this -> result = mysql_query($query, $this -> link);
}
?>
[PHP] pobierz, plaintext 

Dzięki temu nie marnujesz czasu na łączenie się z bazą jeżeli akurat zdarzy się strona która z niej nie korzysta (lub na przykład skrypt się wysypie). Przy pisaniu skryptów pod www dobrze jest pamiętać, że nic nie robi się na zapas, bo wszystkie zapasy znikają razem z requestem.

Po drugie dobrze jest nie robić za dużo w konstruktorze klasy.

[Kas]

@mike_mech

1) Przepraszam za podwójne podanie kodu. Tak jakoś wyszło. (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

2) Napisałbym więcej na temat nie działania jakby więcej wiedział. Mam włączone błędy, mimo to pojawia się poprostu pusta strona.

3) Pozdrów ode mnie Nadwornego Jasnowidza. (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

@pies

1) Skrypt na 100% będzie potrzebował połączenie z bazą danych.

@all

Czy wogóle jest sens dawać konstruktor/destruktor?
Może lepiej dać wszystko w postaci zwykłych funkcji?

---

Trochę zmieniełem. Efekt ten sam.

[PHP] pobierz, plaintext 
<?php
class sql {
	  private $link;
	  private $result;
	  public $rows;
 
	  public function __construct() {
 
		  $this -> sql_host = 'localhost'; // Adres serwera bazy danych
		  $this -> sql_name = 'forum'; // Nazwa bazy danych
		  $this -> sql_user = 'root'; // Nazwa usera
		  $this -> sql_password = ''; // Hasło
		  $this -> sql_port = '3306'; // Port
 
		  $this -> link = mysql_connect($this -> sql_host, $this -> sql_user, $this -> sql_password);
		  if (!is_resource($this -> link) {
			  die('Brak połączenia z bazą danych.' . mysql_error());
		  }
	  }
 
	  public function __destruct() {
		  if(is_resource($this -> link)) {
			  mysql_close($this -> link);
		  }
	  }
 
	  public function query($query) {
		 $this -> result = mysql_query($query, $this -> link);
	 }
	 
	  public function fetch_row() {
		 if($this -> rows = mysql_fetch_row($this -> result)){
			return 1;
		}
	}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Kas 15.09.2005, 16:56:20

[nospor]

1) nie wybierasz bazy danych
2) zjadles nawias

tak ma byc:

[PHP] pobierz, plaintext 
<?php
 
//...
		  mysql_select_db($this -> sql_name);
		  if (!is_resource($this->link)) {
			  die('Brak połączenia z bazą danych.' . mysql_error());
		  }
//...
 
?>
[PHP] pobierz, plaintext 

pozaym jak robisz: $this -> sql_host
to zadeklaruj tą zmienną: private $sql_host; podobnie reszta

[Kas]

Działa cudowanie.

[PHP] pobierz, plaintext 
<?php
class sql {
	  private $link;
	  private $result;
	  public $rows;
 
	  public function __construct() {
 
		  $this -> sql_host = 'localhost'; // Adres serwera bazy danych
		  $this -> sql_name = 'forum'; // Nazwa bazy danych
		  $this -> sql_user = 'root'; // Nazwa usera
		  $this -> sql_password = ''; // Hasło
		  $this -> sql_port = '3306'; // Port
 
		  $this -> link = mysql_connect($this -> sql_host, $this -> sql_user, $this -> sql_password);
		  mysql_select_db($this -> sql_name);
		  if (!is_resource($this -> link)) {
			  die('Brak połączenia z bazą danych.' . mysql_error());
		  }
	  }
 
	  public function __destruct() {
		  if(is_resource($this -> link)) {
			  mysql_close($this -> link);
		  }
	  }
 
	  public function query($query) {
		 $this -> result = mysql_query($query, $this -> link);
	 }
	 
	  public function fetch_row() {
		 if($this -> rows = mysql_fetch_row($this -> result)){
			return 1;
		}
	}
}
?>
[PHP] pobierz, plaintext 

Będę jeszcze ulepszał. Następne pytania postaram się dać w tym topicu, więc proszę na razie nie usuwać

---

Wielkie podziękowania dla nospora! (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

---

Kiedy uruchamia się destruktor? Czy w momencie wykonania skryptu?

Natomiast co do destruktora... tego w ogóle nie wywoływaliśmy, zarówno jawnie, jak i niejawnie, a mimo to napis "Wywołanie destruktora" się pojawił. Co jest więc grane? Ano po prostu php w momencie kończenia pracy skryptu niszczył po kolei wszystkie obiekty, dla każdego z nich uruchamiając destruktor (o ile takowy istniał).

To oznacza, że skrypt z destruktorem będzie mi zawsze po wykonaniu się zrywał połączenie z bazą danych?

Ten post edytował Kas 15.09.2005, 17:19:05

[dr_bonzo]

To oznacza, że skrypt z destruktorem będzie mi zawsze po wykonaniu się zrywał połączenie z bazą danych?

Tak.

[Kas]

W takim razie muszę stworzyć zwykłą funkcją zamiast destruktora aby nie zamykać połączenia?

[dr_bonzo]

Ale po co ci otwarte polaczenie po zakonczeniu skryptu, szczegolnie ze uzywasz mysql_connect (a nie *_pconnect).

Destruktor uruchamia sie przed zniszczeniem obiektu (ktory posiada ten destruktor). Obiekty sa niszczone automatycznie tuz przed zakonczeniem wykonywania skryptu (mozna to tez recznie robic).

[Kas]

Czy jak mam skrypt, który pobiera dane z bazy to po pobraniu wszystkich danych, wykonaniu wszytskich poleceń oraz wsyświetleniu wszystkich danych połączenie będzie zrywane i przy wejściu w dowolną podstronę będzie tworzone ponownie?

---

Próbuję ulepszyć klasę. Co jeszcze należy zmienić?

[PHP] pobierz, plaintext 
<?php
class sql {
 
	  private $link;
	  private $result;
	  public $rows;
 
	  public function __construct {	
		  $this -> sql_host = 'localhost'; // Adres serwera bazy danych
		  $this -> sql_name = 'cms'; // Nazwa bazy danych
		  $this -> sql_user = 'root'; // Nazwa usera
		  $this -> sql_password = ''; // Hasło
		  $this -> sql_port = '3306'; // Port
	  }
 
	  public function connect() {
		  $this -> link = mysql_connect($this -> sql_host, $this -> sql_user, $this -> sql_password);
		  mysql_select_db($this -> sql_name);
		  if (!is_resource($this -> link)) {
			  die('Brak połączenia z bazą danych.' . mysql_error());
		  }
	  }
 
	  public function exit() {
		  mysql_close($this -> link);		  
	  }
 
	  public function query($query) {
		 $this -> result = mysql_query($query, $this -> link);
	 }
	 
	  public function fetch_row() {
		 if($this -> rows = mysql_fetch_row($this -> result)){
			return 1;
		}
	}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Kas 15.09.2005, 22:00:03

[eS...]

witam
dopiero zakupilem ksiazke o programowaniu oo w php5. I wlasnie skonczylem przerabiac rozdzial z polaczeniami (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) . W zwiazku z tym mam pytanko (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)
czy

[PHP] pobierz, plaintext 
<?php
 
 public function __construct {	
		  $this -> sql_host = 'localhost'; // Adres serwera bazy danych
		  $this -> sql_name = 'cms'; // Nazwa bazy danych
		  $this -> sql_user = 'root'; // Nazwa usera
		  $this -> sql_password = ''; // Hasło
		  $this -> sql_port = '3306'; // Port
	  }
?>
[PHP] pobierz, plaintext 

ten konstruktor jest potrzebny? czy nie lepiej trzymac ustwaienia w jakims pliku conf.php ? przeciez po require bedziemy miali dostep do tych ustawien. A i tak w konstruktorze nie wywolywane jest polaczenie. Moim zdaniem konstruktor jest zbedny (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ale moze dlatego ze nie zaszedlem daleko w tej ksiazce
oizdrawuan

[NuLL]

A dlaczego nie zestawiać połączenia w konstruktorze ? Gdzieś zakazali (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)
To czy dane połaczenia będą w osobnym pliku ( jak ja robię ), czy może w stałych a może na sztywno w konstruktorze to tylko kwestia upodobań programisty - nic innego. Przepraszam za lekki OT.

Nie robić OT dot połączeń z SQL-em bo zamkne