[MySQL][PHP]obiektowe połaczenie z bazą Opcje

[pablo_83]

witam,
jestem początkujący w obiektowym programowaniu i za bardzo nie chce mi to do głowy wejść.
Próbuję się połączyć obiektowo z bazą i napisałem poniższy kod

[PHP] pobierz, plaintext 
<?php
require_once ('hasla.php');
?>
 
<?php 
 
class DataAccess
{
    var $DataBase;
 
       function BaseConnect ($host, $user, $password, $db)
        {$this->db = mysql_connect ($host, $user, $password) or die ('Nie mogę nawiązać połączenia');
        mysql_select_db ($db,$this->db);
        mysql_query("Set Names 'utf8'");
        mysql_query("Set collation_connection = utf8_polish_ci");
        }    
 
        function BaseClose ()
        {$this->BaseConnect;
        $close = $this->db;
        mysql_close($close);
        }
}
 
 
$DataBase = new DataAccess(); 
/*if (!$DataBase) {echo 'niepołączony';} 
  else {echo 'połączony';}*/
?>
[PHP] pobierz, plaintext 

Moje pytanie brzmi, czy ten kod jest poprawny?
Jak w prosty sposób mogę sprawdzić połączenie z bazą ( czy mogę zrobić coś takiego jak w wykomentowaniu).
Proszę o pomoc w rozwiązaniu, chce zrozumieć w końcu tą obiektowość, czytałem sporo ale jakoś w teorii nie wchodzi mi to głowy
może na własnych błędach i swoim przykładzie uda mi się to zrozumieć.

Składnia kodu powinna być ok bo żadne błędy nie wyskakują.

[matiit]

Czemu nie tworzysz konstruktora?

Będziesz musiał samemu wykonywać DataAccess->BaseConnect($args) itd

[erix]

Składnia kodu powinna być ok bo żadne błędy nie wyskakują.

1. czemu nie podajesz kontekstu połączenia?
2. czemu korzystasz z var, które od PHP5 nie ma sensu?
3. czemu w baseClose wywołujesz zmienną, nie metodę?
4. skoro robisz klasę abstrakcji, to czemu nazewnictwo prefiksujesz zawsze base*?
5. czemu nie używasz wyjątków?
6. o konstruktorze już wspomniano

[pablo_83]

Poprawiłem trochę kod myślę że teraz będzie lepiej.
Jeśli nie to proszę o uwagi co źle i dlaczego. Co do var, raczej musi zostać na serwerze wywala mi błąd składni gdy zamieniam na public.
Hasła i dane do połączenia ładuje przez require_once - czy to w tym przypadku będzie działało?
podkreślam że raczkuje w tematyce OOP.

[PHP] pobierz, plaintext 
<?php
require_once ('hasla.php');
?>
 
<?php 
 
class DataAccess
{
    var $MyDataBase;
 
        function __construct ($host, $user, $password, $db)
        {$this->MyDataBase = mysql_connect ($host, $user, $password) or die ('Nie mogę nawiązać połączenia:'.mysql_error());
        mysql_select_db ($db) or die ('nie mogę wybrać bazy:'.mysql_error());
        mysql_query("Set Names 'utf8'");
        mysql_query("Set collation_connection = utf8_polish_ci");
        }    
 
        function __destruct()
        {mysql_close($this->MyDataBase);
        }
}
 
 
$MyDataBase = new DataAccess();
 
?>
[PHP] pobierz, plaintext 

[wookieb]

Jak w prosty sposób mogę sprawdzić połączenie z bazą ( czy mogę zrobić coś takiego jak w wykomentowaniu)

Pójść do serwerowni z ciasteczkami i marchewką i zapytać się serwera Mysql czy się połączył.
A bardziej serio (jakie komunikat w die napisałeś?)

W dodatku nazewnictwo zmiennych/właściwości
mysql_connect zwraca ci "Twoją bazę danych?". Jest to bardzo ważne dla innych osób, które będą czytały kod. W manualu pisze co zwraca funkcja mysql_connect. Zastosuj do tego odpowiednią nazwę.

Nazwa klasy. "DataAccess"? Dostęp do danych? A nie raczej "DatabaseConnection"? ablo DbConnection?

Uwierz. Potworzysz w projekcie parę takich klas z dziwacznymi nazwami to po roku nie załapiesz o co chodziło.

[230005]

Jak może działać, skoro konstruktor przyjmuje argumenty, a nie przekazujesz ich do niego? Var to przestarzała konstrukcja - jeśli na serwerze dostajesz błąd składni, to może serv śmiga na php4 (są takie jeszcze?) - zmień serwer. No i zmienne klasy deklaruje się raczej jako private niż public, chyba że jest ku temu sensowny powód. Swoją drogą, do nauki równie dobrze mógłbyś działać na localhoście, używając choćby tego: http://www.apachefriends.org/en/xampp.html - zero konfiguracji, instalujesz i śmiga .

[Rookie]

A nie powinno być zamiast:

[PHP] pobierz, plaintext 
var $MyDataBase;
[PHP] pobierz, plaintext 

to:

[PHP] pobierz, plaintext 
private $MyDataBase;
[PHP] pobierz, plaintext 

Żeby już pod php5 było.
Zamiast funkcji destrukcyjnej zrób funkcję np. zamknij().
I wtedy:

[PHP] pobierz, plaintext 
$MyDataBase = new DataAccess(localhost, uzytkownik, haslo, nazwa_bazy_danych);
//operacje na bazie
$MyDataBase->zamknij();
[PHP] pobierz, plaintext 

Nie sprawdzałem, ale powinno być ok.
PS. Sam zaczynam z OOP ale już siedzę w tym niecały tydzień
Pozdrawiam.

Ten post edytował Rookie 5.09.2009, 16:26:33

[viking]

Skoro chcesz obiektowo to zamiast cudować z własną klasą użyj mysqli albo PDO.

[pablo_83]

teraz kod wyświetla zawartość tabeli.
Mam pytanie czy jest on poprawny jeśli nie to co należałoby zmienić, mam pytanie dotyczące konstruktora jak to powinno wyglądać żeby najpierw zdefiniować konstruktor a potem metodę połączenia z bazą.
Nie mogę tego zczaić kompletnie proszę o rady.

[PHP] pobierz, plaintext 
<?php
//require_once ('hasla.php');
?>
 
<?php 
 
class Connecting
{
    private $baza;
 
 
      function __construct ($host, $user,$password, $baza)
        {$this->baza = mysql_connect (localhost, root, haslo, Serwis ) or die ('Nie mogę nawiązać połączenia:'.mysql_error());
        mysql_select_db (Serwis) or die ('nie mogę wybrać bazy:'.mysql_error());
        mysql_query("Set Names 'utf8'");
        mysql_query("Set collation_connection = utf8_polish_ci");
        }    
 
        private function DB_close()
        {$this->DB_connect();
        $close = $this->MyDataBase;
        mysql_close($close);
        }
}
 
 
$baza = new Connecting($host, $user, $password, $baza);
 
$zapytanie = ("SELECT * FROM Klient");
   $wynik = mysql_query($zapytanie) or die (mysql_error());
   echo '<form action="index.php"  method="post"><table >';
   echo '<tr><td><select name="id">';
   while ($row = mysql_fetch_array($wynik)){
 
   echo'<option value="'.$row['IdKlienta'].'">'.$row['Nazwisko']. '</option>';}
   echo  '</select></td></tr>';
   echo '<tr><td><input type="submit" value="Pokaż Klienta"/></td></tr>';
   echo '</table></form>'  ;
 
    if (!isset($_POST['id']))
    {}
    else {
 $nazwisko = $_POST['id'];
 
   $zapytanie = "SELECT * FROM Klient WHERE IdKlienta = '$nazwisko' ORDER By Nazwisko ASC";
   $wynik = mysql_query($zapytanie);
   $row = mysql_fetch_array($wynik);
 
   echo 
        '<table >
         <tr>
         <td>Nazwisko</td><td>-</td><td>'.$row['Nazwisko'].'</td></tr>
          <tr>
         <td>Imię</td><td>-</td><td>'.$row['Imie'].'</td></tr>
          <tr>
         <td>IdKlienta</td><td>-</td><td>'.$row['IdKlienta'].'</td></tr>
          <tr>
         <td>Telefon</td><td>-</td><td>'.$row['Telefon'].'</td></tr>
          </table>'; }
 
?>
[PHP] pobierz, plaintext 

Jak zrobić żeby hasło i loginy były ładowane z osobnego pliku, jak wykorzystać require lub include??
raczkuje w tematyce obiektowości więc proszę o wyrozumiałość nawet jeśli moje pytania mogą się wydawać głupie lub banalne.

[potreb]

Panie pablo, nie jestem biegły czy średni w obiektówce, ale twoja klasa byłą pisana tysiąc razy i więcej, zobacz sobie przykłady u wujka google jak powinien wyglądać model twojej klasy