Forum PHP.pl

Ostatnio mało bywałem na forum, ponieważ min. tworzyłem [usunąłem linka] alek.
Ale nie w tym problem.
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.
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?

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.

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 );

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.

@mike_mech

1) Przepraszam za podwójne podanie kodu. Tak jakoś wyszło.

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.

@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

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

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!

---

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



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

Tak.

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

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

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

witam
dopiero zakupilem ksiazke o programowaniu oo w php5. I wlasnie skonczylem przerabiac rozdzial z polaczeniami . W zwiazku z tym mam pytanko
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 ale moze dlatego ze nie zaszedlem daleko w tej ksiazce
oizdrawuan

A dlaczego nie zestawiać połączenia w konstruktorze ? Gdzieś zakazali
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