Wywołanie klasy z funkcji - czy to możliwe?, Jak usprawnić pisanie aplikacji i mieć porządek w kodzie bez MVC. Opcje

[tikky]

Witam,

Projektuję pewną aplikację w PHP/MySQL.
Od razu dodam, że nie jestem ekspertem, robię to na własne potrzeby i do obiektówki jeszcze nie dojrzałem.
Ogólnie wszystko działa jak trzeba ale robi mi się trochę bałagan w kodzie - miesza się HTML z PHP.

Wiem, że dobrą metodą jest oddzielenie warstwy wyświetlania, od kodu PHP ale nie bardzo wiem jak to zrobić.

I teraz aby sobie troche posprzątać, kod chciałem pewne rzeczy pochować do poszczególnych funkcji.
Problem mam wtedy gdy dana funkcja ma się łączyć z MySQL i pobierać jakieś dane.
Do połącznia z bazą używam klasy EZ_SQL (http://justinvincent.com/ezsql) i pojawiają mi się błędy gdy chcę
w środku mojej funkcji łączyć się z bazą za pośrednictwem klasy EZ_SQL. Czy to w ogóle możliwe?
Jak mogę to przeskoczyć? Jak to ułatwić, posprzątać?

Wiem, że najlepiej byłoby zrobić całóść w modelu MVC, mieć kontrolery, szablony itp. ale tak jak wspomniałem nie jestem programistą i na razie nie mam możliwości przerabiać wszystkiego od nowa.

Co mógłbym zrobić z kodem aby ułatwić sobie pracę i mieć porządek na etapie już istniejącego kodu?

[nospor]

To moze bys pokazal jak probujesz sie laczyc oraz jakei to bledy..... wrozbita pietro wyzej.

[tikky]

Nospor, oczywiście już uściślam:

Poniżej przykład fragmentu kodu, który na stronie wyświetla dane z bazy.

[PHP] pobierz, plaintext 
$db->get_results("SET NAMES 'utf8'");
$produkcja = $db->get_results("SELECT * FROM tab_zamowienia WHERE id_statusu BETWEEN '20' AND '70' ORDER BY id_zamowienia");
	echo '<h3 class="content-subhead"><i class="fa fa-building-o"></i>  Zamówienia bieżące w produkcji</h3>
			<table>
			<thead>
				<tr>
					<th>ID <i class="fa fa-sort"></i></th>
					<th>Osoba <i class="fa fa-sort"></i></th>
					<th>Klient <i class="fa fa-sort"></i></th>
					<th>Status <i class="fa fa-sort"></i></th>
				</tr>
			</thead>';
	foreach ( $produkcja as $zamowienie )
	{
	echo    "<tr>
				<td>".$zamowienie->id_zamowienia."</td>
				<td>".$zamowienie->osoba."</td>
				<td>".$zamowienie->klient."</td>
				<td>".$zamowienie->status."</td>
			</tr>";
	}
	echo "</table>";
[PHP] pobierz, plaintext 

Ponieważ używam tej tabeli kilka razy chciałem przenieść ten fragment do funkcji typu:

[PHP] pobierz, plaintext 
function wyswietl_tabele(){
$db->get_results("SET NAMES 'utf8'");
$produkcja = $db->get_results("SELECT * FROM tab_zamowienia WHERE id_statusu BETWEEN '20' AND '70' ORDER BY id_zamowienia");
	echo '<h3 class="content-subhead"><i class="fa fa-building-o"></i>  Zamówienia bieżące w produkcji</h3>
			<table>
			<thead>
				<tr>
					<th>ID <i class="fa fa-sort"></i></th>
					<th>Osoba <i class="fa fa-sort"></i></th>
					<th>Klient <i class="fa fa-sort"></i></th>
					<th>Status <i class="fa fa-sort"></i></th>
				</tr>
			</thead>';
	foreach ( $produkcja as $zamowienie )
	{
	echo    "<tr>
				<td>".$zamowienie->id_zamowienia."</td>
				<td>".$zamowienie->osoba."</td>
				<td>".$zamowienie->klient."</td>
				<td>".$zamowienie->status."</td>
			</tr>";
	}
	echo "</table>";
}
[PHP] pobierz, plaintext 

... i na głównej stronie wywoływać tylko kilka razy tę samą funkcję.
Niestety ten sposób nie działa, bo nie mogę wywołać klasy EZ_AQL wewnątrz funkcji, czyli "$db->get_results"
Jeśli coś jeszcze nie jasne proszę o informacje.

[nospor]

Zmienna $db nie jest widoczna w funkcji....

1) Poczytaj w manualu o zasiegu zmiennych
2) ALbo przekazuj do funkcji jako parametr te $db
3) Albo przeczytaj to
http://forum.php.pl/index.php?showtopic=227036&hl=new+T1
tu masz napisane jak przekazywac obiekt bazy do klas. Posrednio niektore metody z tamtego tematu dotycza i ciebie

[tikky]

NOSPOR: dzięki za odpowiedzi i porady.

Stworzyłem coś mi działa.
Jako, że jest to mój pierwszy kod w obiektówce, mam prośbę abyś rzucił tylko okiem czy to co napisałem jest zgodne z zasadą.

[PHP] pobierz, plaintext 
<?php
class Test extends db {
    public $imie = 'imie jeszcze nie pobrane';
 
public function __construct($id)
    {
        echo 'Jestem konstruktorem</br>';
	$db = new db(EZSQL_DB_USER, EZSQL_DB_PASSWORD, EZSQL_DB_NAME, EZSQL_DB_HOST);
	$zapytanie = $db->get_row("SELECT imienazwisko FROM tab_uzytkownicy WHERE id_uzytkownika=$id");
	$this->imie = $zapytanie->imienazwisko;
    }
 
    function set_owner ($name) {
        $this->owner = $name;
	$zapytanie = $db->get_results("SELECT id_uzytkownika, imienazwisko FROM tab_uzytkownicy");
	$this->owner = $zapytanie->imienazwisko;
    }
}
 
 
//tworzymy nową instancję podając id intersującego nas użytkownika
$osobnik = new Test(30);
 
//wyświetlamy imię z bazy - pobrane w konstruktorze klasy
echo $osobnik->imie;
 
?>
[PHP] pobierz, plaintext 

[nospor]

zle. W linku co ci podalem masz jasno napisane ze:
1) Nie dziedziczy sie po klasie DB
2) Nie tworzy sie w klasie wlasnych obiektow DB

[tikky]

Rozumiem to co piszesz, ale nie bardzo wiem jak to przerobić żeby było zgodnie ze sztuką.

Czy to oznacza, że w moich klasach w ogóle nie powinienem używać EZ_SQL?
Do tej pory używałem go gdyż jest dla mnie bardzo wygodny i popularnie stosowany (np. WordPress go używa).

Czy mógłbyś mi jakiś przykład dać jak to powinienem zrobić, bez dziedziczenia db?
Chciałbym przejść z pisania strukturalnego na obiektowe, ale wolałbym wiedzieć jak zrobić porządnie chociaż proste
rzeczy - takie właśnie jak połączenie z bazą i wyciągnięcie danych.

[nospor]

Oczywiscie ze masz uzywac obiektu DB. ALe masz go do klasy przekazywac z zewnatrz.

[tikky]

Nospor: ok, chyba rozumiem.
Nie chcę iść na łatwiznę - próbuję coś sam zrobić ale idzie jak po gruzie.
Przsepisałem to mając na uwadze to co piszesz:

[PHP] pobierz, plaintext 
<?php
// dodaję klasę EZ_SQL
include_once "ez_sql.php";
 
//gdzieś tam w aplikacji tworzę sobie taki obiekt
$db = new db(EZSQL_DB_USER, EZSQL_DB_PASSWORD, EZSQL_DB_NAME, EZSQL_DB_HOST);
 
//dalej klasa bez dziedziczenia db
class Test2 {
    public $imie = 'imie jeszcze nie pobrane';
 
	public function __construct($db, $id)
    {
        echo 'Jestem konstruktorem klasy Test2</br>';
		$zapytanie = $db->get_row("SELECT imienazwisko FROM tab_uzytkownicy WHERE id_uzytkownika=$id");
		$this->imie = $zapytanie->imienazwisko;
    }
 }
 
 
//tworzymy nową instancję podając id intersującego nas użytkownika
$osobnik = new Test2($db, 30);
//wyświetlamy imię z bazy - pobrane w konstruktorze klasy
echo $osobnik->imie;
 
?>
[PHP] pobierz, plaintext 

Czy teraz dobrze? Lepiej?
Jeśli nie to prośba o pomoc, konkretny kod który by działał i był zgodny z zasadami.

[nospor]

Tak, teraz lepiej (IMG:style_emoticons/default/smile.gif)
Ale skoro przekazujesz $db do konstruktora, to dobrze by bylo bys to $db zapamietal jako wlasciwosc klasy i mogl przez $this sie do niej wszedzie odwolywac.

[tikky]

Nospor,

Wgryzam się powoli w temat OOP, ale idzie mi to opornie.

Masz racje, fajnie byłoby zapamiętać własciwości ale nie wiem jak to zrobić. Czy mogę prosić Cię o modyfikację mojego kodu?

[nospor]

Wlasciwosci klasy to sa podstawy klas i obiektow. W manualu php jest dział na temat klas i tam te wszystkie podstawy są wyjasnione.

[tikky]

Nospor: Wracam do tematu programowania obiektowego i pozwolę sobie wrócić do pytania.
Czy poniższy rodzaj programowania i łączenia z bazą jest ok:

[PHP] pobierz, plaintext 
//dołączenie pliku z klasą do połączenia z bazą - EZ_SQL
include 'ez_sql.php';
 
class Klienci
{
 
public function __construct()
{
  //tworzę jeden obiekt do połączenia z bazą
  $db = new db(EZSQL_DB_USER, EZSQL_DB_PASSWORD, EZSQL_DB_NAME, EZSQL_DB_HOST);
  $this->baza = $db;
}
 
 function podajImie($id)
 {
  //tu już nie tworzę nowe połączenia tylko odnoszę się do istniejącego
  $db = $this->baza;
  $zapytanie = $db->get_row("SELECT imienazwisko FROM tab_uzytkownicy WHERE id_uzytkownika=$id");
  if($zapytanie==TRUE) 
	$this->imie = $zapytanie->imienazwisko;
  else 
    echo 'Błąd: nie ma usera o tym ID -';
 }
 
  function podajFirme($id)
 {
  $db = $this->baza;
  $zapytanie = $db->get_row("SELECT id_firmy FROM tab_uzytkownicy WHERE id_uzytkownika=$id");
  echo $zapytanie->id_firmy;
 }
 
  function dodajEmail($adres_email, $id)
 {
  $db = $this->baza;
  $zapytanie = $db->query("UPDATE tab_uzytkownicy SET mail = '$adres_email' WHERE id_uzytkownika=$id");
  echo 'dodano email';
 }
 
  function ileKlientow()
 {
  $db = $this->baza;
  $ilosc = $db->get_var("SELECT count(*) FROM tab_uzytkownicy");
  echo "<br>W bazie jest: ".$ilosc." klientów";
 }
 
}
 
$osobnik = new Klienci;
print($osobnik->podajImie(99));
print($osobnik->podajFirme(30));
$osobnik->dodajEmail("adres@serwer.pl", 23);
$osobnik->ileKlientow();
[PHP] pobierz, plaintext 

[nospor]

1) Obiekt klasy bazy danych nie powinien byc tworzony w klasie klienta. Obiekt ten, ma byc tworzony oddzielnie i przekazywany do klasy klienta
2) Klasa nie powinna pluc ECHem wiadomosci. Klasa najlepiej powinna rzucac wyjątkami, gdy coś pojdzie nie tak
3) Bez sensu jest tworzyc metody podajImie, podajFirme... gdy bedziesz chcial pobrac dane danego usera, to bedziesz musial wywolan x takich metod, a kazda na dodatek leci z zapytaniem do bazy. Ma byc jedna metoda: podajKlienta i ona zwroci tablice danego usera
A juz w ogole najlepiej pownna byc klasa Klient, ktora by miala wlasciwosci takie jakie tabela Klienta.