[OO] Hasła w pliku końcowym, Programowanie obiektowe klasa BD Opcje

[primo]

Witam,

odkąd zacząłem czytać wszystko co mi wpadło w rękę odnośnie php to zawsze ucierało się przeświadczenie, że tworząc sobie plik z funckjami bazy to w nim wpisywało się hasło, bazę i dalej tylko icludowało się taki gotowiec i o haśle się nie myślało, bo było w jednym miejscu.
Teraz o stopień wyżej zacząłem czyli OO i tu mam następujący problem. Poniżej znajduje się klasa BD:

[PHP] pobierz, plaintext 
<?php
class BD
{
	// parametry połączeniowe
	var $wezel = '';
	var $uzytkownik = '';
	var $haslo = '';
	var $bazadanych= '';
	var $trwale = false;
 
	// identyfikator połączenia z bazą danych
	var $pol = NULL;
 
	// wynik zapytania
	var $wynik = false;
 
	function BD($wezel, $uzytkownik, $haslo, $bazadanych, $trwale = false)
	{
		$this->wezel = $wezel;
		$this->uzytkownik = $uzytkownik;
		$this->haslo = $haslo;
		$this->bazadanych = $bazadanych;
		$this->trwale = $trwale;
	}
 
	function otworz()
	{
		// wybierz właściwą funkcję połączeniową
		if ($this->trwale) {
			$funkcja = 'mysql_pconnect';
		} else {
			$funkcja = 'mysql_connect';
		}
 
		// nawiąż połączenie z serwerem MySQL
		$this->pol = $funkcja($this->wezel, $this->uzytkownik, $this->haslo);
		if (!$this->pol) {
			return false;
		}
 
		// ustaw wskazaną bazę danych jako bieżącą
		if (!@mysql_select_db($this->bazadanych, $this->pol)) {
			return false;
		}
 
		return true;
	}
 
	function zamknij()
	{
		return (@mysql_close($this->pol));
	}
 
	function blad()
	{
		return (mysql_error());
	}
 
	function zapytaj($zapytanie = '')
	{
		$this->wynik = @mysql_query($zapytanie, $this->pol);
 
		return ($this->wynik != false);
	}
 
	function liczbaZmienionychWierszy()
	{
		return (@mysql_affected_rows($this->pol));
	}
 
	function liczbaWierszy()
	{
		return (@mysql_num_rows($this->wynik));
	}
 
	function pobierzObiekt()
	{
		return (@mysql_fetch_object($this->wynik, MYSQL_ASSOC));
	}
 
	function pobierzTablice()
	{
		return (@mysql_fetch_array($this->wynik, MYSQL_NUM));
	}
 
	function pobierzTabliceAsocjacyjna()
	{
		return (@mysql_fetch_assoc($this->wynik));
	}
 
	function zwolnijZbiorWynikowy()
	{
		return (@mysql_free_result($this->wynik));
	}
}
?>
[PHP] pobierz, plaintext 

teraz jak wywołuję tą klasę w jakimś innym pliku:
załączam go przez indlude i później wywołuję:

[PHP] pobierz, plaintext 
<?php
$bd = new BD ('db.st.interia.pl', '11111', '11111', 'db11111');
?>
[PHP] pobierz, plaintext 

no i tu tkwi problem, że za każdym razem jak wywyłuję tą klasę to muszę to klepać we wszystkich osobnych plikach, znowu jak chciałem określić te parametry w ciele klasy to mi określa złego użytkownika.

pozdrawiam

[silent]

Przecież możesz includować sobie wszędzie plik np config.php w którym będzie:

[PHP] pobierz, plaintext 
<?php
$dbhost='localhost';
$dbuser='user';
$dbpass='pass';
$dbname='jakas_baza';
?>
[PHP] pobierz, plaintext 

a później:

[PHP] pobierz, plaintext 
<?php
 
include('config.php');
$db=new BD($dbhost,$dbuser,$dbpass,$dbname);
?>
[PHP] pobierz, plaintext 

[primo]

no myślałem o takim sposobie, ale w sumie to wolę mieć to rozwiązane w jeszcze prostszy sposób. Czy nie da rady określić tych argumentów już w samej klasie

[Jojo]

Hmm ja nie rozumiem w czym problem:

[PHP] pobierz, plaintext 
<?php
class BD
{
	// parametry połączeniowe
	var $wezel = 'db.st.interia.pl';
	var $uzytkownik = '11111';
	var $haslo = '11111';
	var $bazadanych= 'db11111';
	var $trwale = false;
 
	// (...)
 
}?>
[PHP] pobierz, plaintext 

O coś takiego Ci chodzi?

[primo]

dokładnie właśnie o coś takiego, z tym że ten sposób nie działa. Próbowałem przed zadaniem pytania

[silent]

Pokaż jak próbowałeś, bo nie może nie działać.

[primo]

no dokładknie jak Jojo pokazał oraz deklaracja:

[PHP] pobierz, plaintext 
<?php
$bd = new BD ()
?>
[PHP] pobierz, plaintext 

[silent]

Ok, a wywaliłeś wtedy funkcję BD() z ciała klasy?

[primo]

no, nie.

[Jojo]

Tak odbiegając nieco od tematu, to sądzę, że ta klasa jest kompletnie bezużyteczna. Poza wybieraniem rodzaju połączenia (connect/pconnect) nie robi praktycznie nic. Są tam tylko i wyłącznie aliasy dla wbudownych funkcji php. AFAIK może to bardzo spowolnić wykonywanie skryptu.
Dorobiłbyś do tego chociaż jakąś obsługę błędów albo pomiar czasu, aby to miało jakąkolwiek rację bytu.

[silent]

no, nie.

No to wywal, bo jeśli masz:

[PHP] pobierz, plaintext 
<?php
class BD
{
	// parametry połączeniowe
	var $wezel = 'localhost';
	var $uzytkownik = 'user';
	var $haslo = 'pass';
	var $bazadanych= 'database';
	var $trwale = false;
 
	// identyfikator połączenia z bazą danych
	var $pol = NULL;
 
	// wynik zapytania
	var $wynik = false;
 
	function BD($wezel, $uzytkownik, $haslo, $bazadanych, $trwale = false)
	{
		$this->wezel = $wezel;
		$this->uzytkownik = $uzytkownik;
		$this->haslo = $haslo;
		$this->bazadanych = $bazadanych;
		$this->trwale = $trwale;
	}
	//......
}
 
?>
[PHP] pobierz, plaintext 

To tworząc obiekt:

[PHP] pobierz, plaintext 
<?php
 
$bd=new BD();
 
?>
[PHP] pobierz, plaintext 

Wywołujesz konstruktor bez parametrów, więc nadpisuje te wpisane wartości zmiennych wartościami pustymi.

[primo]

odpowiadając koledze Jojo. W sumie to i masz rację, ale naukę obiektówki, tak sądzę, najlepiej zacząć od rzeczy łatwych, które zna się w miarę dobrze. Teraz będę próbował sił, dopisać funckję obsługującą prowadzenie witryny wypożyczalni a taki obiekt w postaci klasy dostępu do bazy będzie bez mała użyteczny. Bez powiązań z innymi obiektami faktycznie jest bezużyteczny, ale jak już dojdzie z kolejnych kilka klas to to się zmieni.

pozdrawiam

[MoD]

Jojo, przecież tę klasę można rozbudować jeszcze np. o obsługę PostgreSQL, żeby w skryptcie wszędzie nie pisać

[PHP] pobierz, plaintext 
<?php
//(...)
 
if ( $db_type = 'mysql' )
{
	mysql_query( $sql );
}
elseif ( $db_type = 'postgresql' )
{
	pg_query( $handle, $sql );
}
 
//(...)
?>
[PHP] pobierz, plaintext 

tylko po prostu wywołać konstruktor z typem bazy i klasa sama wywoła funckję. Tak więc ta klasa ma szansę bytu i nie jest całkowicie bez sensu.

[Jojo]

MoD: Zgoda, ale ja się wypowiadam na temat kodu, który widzę kilka postów wyżej, a nie ewentualnych możliwości jego rozbudowy. Tamten konkretny kod jest kompletnie bezwartościowy.
primo: Masz rację, że zaczynasz naukę od czegoś prostego. Obiekt w postaci klasy dostępu do bazy też Ci się przyda. Jest tylko jedno ale. Po co tworzyć taki obiekt jeśli nie oferuje on żadnej dodatkowej funkcjonalności ponad tą, którą oferuje nam samo php?