[PHP]Programowanie obiektowe Opcje

[qba10]

Witam
Właśnie co uczę się na sucho programowania obietowego.
Mam o to taki kod:

[PHP] pobierz, plaintext 
<?php
class baza_danych
 {
  var $bazadanych_connect;
  var $bazadanych_connect_db;
  var $host; 
  var $user;
  var $password;
  var $dbname;
  var $rodzaj;
 
 public function  __construct($rodzaj)
 {
 $this->rodzaj = $rodzaj ;
 
 }
	   
 private function wybor_bazy_danych()
  {
  $konstrukcja_connect = '@'.$this->rodzaj.'_connect($this->host, $this->user, $this->password );';
  $konstrukcja_connect_db = '@'.$this->rodzaj.'_select_db($this->dbname);' ;
  $connect = $konstrukcja_connect;
  $connect_db = $konstrukcja_connect_db;
 
  if ($connect) { $this->bazadanych_connect = 1; } else { $this->bazadanych_connect = 0;}
  if ($connect_db) { $this->bazadanych_connect_db = 1; } else { $this->bazadanych_connect_db = 0;}
  }
	   
 
	   
  function polaczenie($host, $user, $password, $dbname )
  {
  $this->host = $host ;
  $this->user = $user ;
  $this->password = $password ;
  $this->dbname = $dbname ;
 
  if ($this->bazadanych_connect = 0)
  {
   echo'blad polaczenie z baza danych'.$this->rodzaj ;
  }
  if ($this->bazadanych_connect_db = 0)
  {
   echo'blad polaczenie z baza danych'.$this->rodzaj ;
  }
	   
  }
}
?>
[PHP] pobierz, plaintext 

I tu własnie moje pytanie. Czy ta klasa jest poprawna i czy przy wywołaniu jej w sposub pokazany ponizej otrzymam pozadany wynik?

[PHP] pobierz, plaintext 
<?php
$baza_danych = new baza_danych($mysql_lub_pg) ;
 echo $baza_danych->polaczenie($host, $user, $password, $dbname )  ;
?>
[PHP] pobierz, plaintext 

Ten post edytował qba10 7.06.2008, 19:26:02

[chlebik]

Re Up: Prosilbym adminow o usuniecie, cos mi automat do logowania nie zadzialal i wyszlo jak wyszlo.

Witam
Właśnie co uczę się na sucho programowania obietowego.

Na sucho to kiepski pomysl, sprobuj cos realnego napisac - najlepsza rzecz do tego - blog na uzytek domowy.

Mam o to taki kod:

[PHP] pobierz, plaintext 
<?php
class baza_danych
 {
  var $bazadanych_connect;
  var $bazadanych_connect_db;
  var $host; 
  var $user;
  var $password;
  var $dbname;
  var $rodzaj;
 
 public function  __construct($rodzaj)
 {
 $this->rodzaj = $rodzaj ;
 
 }
	   
 private function wybor_bazy_danych()
  {
  $konstrukcja_connect = '@'.$this->rodzaj.'_connect($this->host, $this->user, $this->password );';
  $konstrukcja_connect_db = '@'.$this->rodzaj.'_select_db($this->dbname);' ;
  $connect = $konstrukcja_connect;
  $connect_db = $konstrukcja_connect_db;
 
  if ($connect) { $this->bazadanych_connect = 1; } else { $this->bazadanych_connect = 0;}
  if ($connect_db) { $this->bazadanych_connect_db = 1; } else { $this->bazadanych_connect_db = 0;}
  }
	   
 
	   
  function polaczenie($host, $user, $password, $dbname )
  {
  $this->host = $host ;
  $this->user = $user ;
  $this->password = $password ;
  $this->dbname = $dbname ;
 
  if ($this->bazadanych_connect = 0)
  {
   echo'blad polaczenie z baza danych'.$this->rodzaj ;
  }
  if ($this->bazadanych_connect_db = 0)
  {
   echo'blad polaczenie z baza danych'.$this->rodzaj ;
  }
	   
  }
}
?>
[PHP] pobierz, plaintext 

I tu własnie moje pytanie. Czy ta klasa jest poprawna i czy przy wywołaniu jej w sposub pokazany ponizej otrzymam pozadany wynik?

[PHP] pobierz, plaintext 
<?php
$baza_danych = new baza_danych($mysql_lub_pg) ;
 echo $baza_danych->polaczenie($host, $user, $password, $dbname )  ;
?>
[PHP] pobierz, plaintext 

1. Wlasnosci egzemplarza definiuj jako public/private/protected - od razu widac zakresy wlasnosci.
2. Z gory zakladasz w konstruktorze, ze bedzie dostarczona poprawna nazwa. Na sucho i w prostej rzeczy to OK, ale dobrze byloby zrobic jakas walidacje i dac tez wartosc domyslna.
3. W metodzie 'wybor_bazy_danych' nie nalezy przypisywac uchwytu do innej zmiennej. Po co to? Zrob tak po prostu:

[PHP] pobierz, plaintext 
<?php
IF/CASE ($this->rodzaj == COSTAM) {
$connect = @mysql_connect($this->host, $this->user, $this->password );
}
 
ELSE IF($this->rodzaj == COSTAM_INNEGO)
{
$connect = @pg_connect($this->host, $this->user, $this->password );
}
ELSE 
JAKIES DOMYSLNE
 
if($connect) {
  $this->konstrukcja_connect = true;
  $konstrukcja_connect_db = '@'.$this->rodzaj.'_select_db($this->dbname);' ;
 
	IF($konstrukcja_connect_db) { $this->bazadanych_connect = true; } else { $this->bazadanych_connect = false; }
?>
[PHP] pobierz, plaintext 

4. Definiujesz zreszta te metode nie wywolujac jej w ogole (IMG:http://forum.php.pl/style_emoticons/default/exclamation.gif) !! W Ifach w metodzie polaczenie() zapisujesz wartosci do atrybutow obiektu, ale w ifie masz bledy, gdyz:
- $this->baza_danych_connect czy cos tam to cholera wie co w tym momencie zawiera (pewnikiem true albo NULL) zatem porownanie jest do chrzanu!!! Co wiecej operator porownania to '==', a nie '='. Ten drugi zapis zawsze daje TRUE lub FALSE w zaleznosci co tam niby porownujesz. (HINT: niektorzy purysci mowa by robic tak: IF(2 == $zmienna); )
- by zas byla okreslona przed ifami musisz wrzucic wywolanie metody wybor_bazy_danych. Zas najlepiej by bylo laczyc od razu w konstruktorze z danymi polaczenia jako parametrami (najlepiej przekazywanymi w tablicy, wygodniej to potem podpiac pod jakies pliki konfiguracyjne). Wiadomo przeciez, ze obiekt sluzy do polaczenia, a i jakiekolwiek jego metody tegoz polaczenia wymagaja.

Ten post edytował chlebik 8.06.2008, 00:33:56