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ą.

Czemu nie tworzysz konstruktora?

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

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

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 

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.

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 .

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.

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

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.

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