Witam,

Przerobiłem troszkę materiału i proszę o poradę. Zanim się napiszę i potem wszystko będzie źle
Napisałem klasę do łączenia się z mysql i nawet działa. Pytanie mam takie czy jest to dobrze napisane ?
Czy na coś zwrócić uwagę i gdzieś mam jakieś błędy? które Wy widzicie ?

[PHP] pobierz, plaintext 
error_reporting(E_ALL);
ini_set('display_errors','1'); 
class mMySQL 
{
 
public $Serwer, $Baza, $Uzytkownik, $Haslo;
#-#############################################
# Nawiazywanie polaczenia z baza
function Polacz ($link=false) {
	$this->link_id=@mysql_connect($this->Serwer,$this->Uzytkownik,$this->Haslo,$link);
 
	if (!$this->link_id) {
		$this->blad("Nie mogę podłączyć się do serwera: <b>$this->Serwer</b>.");
		}
 
	if(!@mysql_select_db($this->Baza, $this->link_id)) {//no database
		$this->blad("Nie mogę wybrać bazy danych: <b>$this->Baza</b>.");
		}
	if(@mysql_select_db($this->Baza, $this->link_id)) {
		mysql_query("SET NAMES 'utf8'");
		mysql_query("SET CHARACTER_SET_CLIENT=utf8");
		mysql_query("SET CHARACTER_SET_RESULTS=utf8");
	}
}#-#Polacz()
#-#############################################
# zamykanie polaczenia z baza
function Zamknij() {
	if(!@mysql_close($this->link_id)){
		$this->blad("Nie można zamknąć połączenia z serwerem :(");
	}
}#-#zamknij()
#-#############################################
# komunikaty o błędach
function blad($msg='') {
	if($this->link_id>0){
		$this->error=mysql_error($this->link_id);
		$this->errno=mysql_errno($this->link_id);
	}
	else{
		$this->error=mysql_error();
		$this->errno=mysql_errno();
	}
	echo "<b>Błąd aplikacji</b><br />";
	echo "<hr>";
	echo "<b>Komunikat:</b>".$msg."<br />";
	echo "<b>MySQL błąd: </b>".$this->error."<br>";
	echo "<b>MySQL kod: </b>".$this->errno."<br>";
	echo "<br>";
}#-#blad()
 
}
## JAK TO DZIAŁA 
$SQL = new mMySQL();
$SQL->Serwer = 'localhost';
$SQL->Baza = 'mysql';
$SQL->Uzytkownik = 'root';
$SQL->Haslo = '';
$SQL->Polacz();
 
 
$SQL->Zamknij();
?>
[PHP] pobierz, plaintext 

Czy na przykład użyć protected Polacz; ?

No dla mnie to jest nieco dziwna klasa, czemu nie używasz konstruktorów/destruktorów i nie deklarujesz typów metod? Dodatkowo metody powinny zwracać jakieś wartości, a nie wyświetlać za pomocą echo czyli nie:

[PHP] pobierz, plaintext 
function metoda(){
echo "costam";
}
[PHP] pobierz, plaintext 

a

[PHP] pobierz, plaintext 
public function metoda(){
return $this->wartosc;
}
[PHP] pobierz, plaintext 

Dalej idą metody i zmienne do których nie powinno się mieć dostępu z zewnątrz (czyli login do bazy danych/hasło/itp.) powinny być ustawione na private albo protected (przy dziedziczeniu)


Sama klasa(prosta wersja) powinna wyglądać mniej więcej tak:

[PHP] pobierz, plaintext 
class Database{
    private $dbHandler;    
    public function  __construct($dbLogin, $dbPassword, $dbHost, $dbName) {
        $this->dbHandler = @mysql_connect($dbHost,$dbLogin,$dbPassword) or $this->sql_error(mysql_error());
        mysql_select_db($dbName,$this->dbHandler);
        mysql_query('SET NAMES utf8');
        return true;
    }
 
    public function query($query){
//tu kasowanie/update/dodawanie do sql jeżeli błąd to $this->sql_error(mysql_error());
        return true;
    }
 
    public function select($query){
//tu funkcja do wyciągania selecta z bazy danych
    }
 
    private function sql_error($error){
//tu funkcje raportowania błędów sql np. zapis do pliku nie powinno się raczej wyświetlać użytkownikom dokładnego błędu
die('Wystąpil problem z bazą danych.');
    }
   public function __destruct(){
//tu niszczenie połączenia bazy
}
 
}
a użycie:
 
$db = new Database($dbLogin, $dbPassword, $dbHost, $dbName); //tworzenie obiektu
$db->select($query); //zapytanie select do bazy
unset($db); //niszczenie obiektu
[PHP] pobierz, plaintext 

Tutaj masz poprawiony kod:

[PHP] pobierz, plaintext 
<?php
 
$db = new PDO('mysql:dbname=baza;host=localhost', 'root', 'pass');
[PHP] pobierz, plaintext 

Proste nie?

Hej,

Ja uparcie obiektowo, poprawiłem to na co zwróciliście mi uwagę i wygląda to tak
Proszę zobaczcie i powiedźcie czy jest dobrze lub czego brakuje

[PHP] pobierz, plaintext 
class mMySQL
{
private $polacz_db;	//połączenie do serwera
private $dbuzytkownik = "root1"; //uzytkownik nazy danych
private $dbhaslo = ""; // jego hasło
private $dbnazwa = "bz_txt"; //nazwa bazy
private $dbserwer = "localhost"; //nazwa serwera
private $dbwynik;	// wynik zapytania
 
public function __construct() {
	$this->polacz_db = @mysql_connect($this->dbserwer,$this->dbuzytkownik,$this->dbhaslo);
	if(!($this->polacz_db)) {
		return false;
	}
	else {
			mysql_query("SET NAMES 'utf8'");
			mysql_query("SET CHARACTER_SET_CLIENT=utf8");
			mysql_query("SET CHARACTER_SET_RESULTS=utf8");
			mysql_select_db($this->dbnazwa);
 
                        return true;
	}
}
 
public function __destruct() {
	@mysql_close($this->polacz_db);
}
 
public function query($sql) {
	$this->dbwynik = mysql_query($sql);
	return $this->dbwynik;
}
 
}
[PHP] pobierz, plaintext 

Będę wdzięczny za kolejne uwagi

Wszystkiego.
Ziomuś co my mamy oceniać? Wpakowanie na siłę w obiekt mysql_query i mysql_connect? Co mamy ocenić?
Pomysł? brak
Jakość? brak
Czy jesteś pewien, że wykonanie dwóch dodatkowych zapytań po "set names" jest potrzebne czy wyczytałeś tak "w necie" ?
Ludzie nie chwalcie się takimi rzeczami bo nie ma czym, a jedynie robicie śmietnik!!!

Zresztą PDO oferuje wystarczająco dużo, żeby takie klasy jak ta nie powstawały.
Ciekawostka przy 10 000 wywołaniach pdo przegrywa z mysql o jedna sekunde. A jak mysql opakujesz w klase to nie zrobi roznicy.

A to w odniesieniu do czego pisałeś? Bo Twój kod z obiektowym ma niewiele wspólnego.

Śmiejcie się śmiejcie. Nie chodziło mi o chwalenie swoim kodem.
Dziękuję za pomoc i zachętę.

To może inaczej.
Sporo jest tutaj tematów, które pytają "czy moja linijka kodu jest ok", "czy dobrze pisze" które są bez sensu. Wiesz dlaczego?
1) Ile programistów tyle porad i rozwiązań, więc co z tego, że dostaniesz porady, ale razem z tym większy mętlik w głowie niż teraz.
2) Jesteś programistą a nie kopiowaczem. To, że ja Ci powiem, zrób tak i tak a ty pójdziesz i to zrobisz to leżysz, bo słuchasz wszystkie co Ci mówię, a powinieneś sam myśleć. Sam sprawdzać, testować, poszukiwać lepszych rozwiązań.

Jeżeli chcesz wiedzieć czy jest dobrze to proszę bardzo.
Zassaj symfony 2, ZendFrameworka i spójrz jak tam to wygląda to bez tematu będziesz znał odpowiedź na swoje pytanie.

I jeszcze jedno. Ustal swój jeden standard, przemyśl dokładnie co i jak i trzymaj się go zawsze.
Przykład: wszędzie będę stosował method chaining.

Eee... no to to akurat powinno być zależne od tego, czy stosowanie tego ma sens czy nie, a nie od tego czy "zawsze będę stosować" czy nie.

No dobra faktycznie
No ale wiadomo o co chodzi