[php][mysql] Problem z klasą Opcje

[Setesh]

Witam.

Mam do was prośbę - chcę wrzucić do klasy wszystkie ważniejsze odwołania do MySQL'a ale mam problem z samą funkcją połączenia się z bazą danych. Nie mogę coś napisać tego. O ile sama funkcja działa o tyle w klasie już coś nie chce. Możecie mi z tym pomóc ? Będę bardzo wdzięczny (IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
 
<?php
 
class BazaMySQL
{
 // Konstruktor
 function BazaMySQL()
 {
  $this->DB_Connect('Nazwa_bazy'); // Nie chce się odwołać do funkcji :(
 }
 
}
 
// Chciałem dodać rozszerzenie do klasy - jako funkcję używaną tylko przez klasę - niestety nie działa :(
 
class BazaMain extends BazaMySQL
{
 protected $HostAddr = '127.0.0.1';
 protected $UserName = 'baza_admin'; 
 protected $UserPassword = 'baza_password';
 protected $UserDefaultDataBase = 'Nazwa_bazy';
 
 protected $MYSQL_ERRNO = ''; 
 protected $MYSQL_ERROR = '';
 
 
public function DB_Connect($DataBaseName)
 {
  global $link_id;
   echo "wywolana z<br>";
  $link_id = mysql_connect($HostAddr, $UserName, $UserPassword);
  if(!$link_id) 
    {
     $MYSQL_ERRNO = 0;
     $MYSQL_ERROR = "Connection failed to the host $HostAddr.";
     return 0;
    }else if(empty($DataBaseName) && !mysql_select_db($UserDefaultDataBase)) 
	        {
             $MYSQL_ERRNO = mysql_errno();
             $MYSQL_ERROR = mysql_error();
             return 0;
            }else if(!empty($DataBaseName) && !mysql_select_db($DataBaseName)) 
			        {
                     $MYSQL_ERRNO = mysql_errno();
                     $MYSQL_ERROR = mysql_error();
                     return 0;
                    }else return $link_id; 
 }
 
}
?>
 
 
[PHP] pobierz, plaintext 

Ogólnie to chciałem aby stworzyć bezpieczne logowanie się do serwera mysql i tak aby nikt niepowołany nie mógł wywołać z zewnątrz klasy funkcji DB_Connect - niestety moje starania na nic się nie zdały pomimo iż rozszerzenie klasy jest napisane poprawnie (no może jakiś błąd w deklaracji funkcji w rozszerzeniu klasy)

Ewentualnie jeżeli możecie mi doradzić jak co zmienić aby było bezpieczniejsze i co źle robię w tym przykładzie (IMG:style_emoticons/default/sad.gif)

Ten post edytował Setesh 29.01.2011, 23:41:35

[darko]

sześć kwestii:
1. po co definiujesz metodę DB_Connect w klasie BazaMySQL i wywołujesz ją w konstruktorze skoro dalej w klasie potomnej ją nadpisujesz, czemu to ma służyć?
2. w konstruktorze klasy wyprowadzonej (którego nie zdefiniowałeś) wywołaj konstruktor klasy nadrzędnej (parent::__construct)
3. po co Ci dwie klasy do obsługi tej implementacji, którą tu uskuteczniasz, nie wystarczy jedna?
4. poczytaj o podstawach obiektowego programowania bo widzę, że się gubisz
5. nie używaj globali, przekazuj zmienne w parametrach metod lub konstruktorów
6. skorzystaj jednak z PDO nadpisując klasę bazową albo w ogóle jej nie nadpisuj, ale naucz się z niej korzystać i z pewnością wyjdzie Ci to na dobre.
Owocnej nauki.