Klasa w klasie Opcje

[Mephis]

Witam.

Mam problem... Przeszukałem google pod podobnymi hasłami jak w temacie, lecz nie znalazłem odpowiedniego, działającego rozwiązania. A musi jakieś istnieć - chyba, że to co chcę zrobić, jest nieodpowiednie, lub istnieje owiele prostrzy spodób...

Mianowicie, mam dwie klasy: SQL i UserData. Pierwsza z nich służy do obsługi bazy danych. Druga będzie miała za zadanie zbierać z tej bazy różnego rodzaju informacje i odpowiednio je łączyć.

Klasę SQL wywołuję w taki sposób:

[PHP] pobierz, plaintext 
require_once('inc/sql.class.php');
$SQL = new SQL($sqlhost, $sqluser, $sqlpassword, $sqldatabase);
[PHP] pobierz, plaintext 

Baza łączy się w konstruktorze.

Próbowałem dołączać klasę w argumencie, lecz to nic nie dało.

W klasie Data pragnę odwoływać się do klasy SQL, ponieważ będę tam przeprowadzać wszelkie operacje oparte o dane użytkownika.

[PHP] pobierz, plaintext 
class UserData {
	function getData() {
		$query = $SQL->query('zapytanie');
	}
}
[PHP] pobierz, plaintext 

Jak zrobić coś tego typu?

[Turson]

[PHP] pobierz, plaintext 
<?php
class UserData {
    function __construct(){
	require_once('inc/sql.class.php');
	$this->SQL = new SQL($sqlhost, $sqluser, $sqlpassword, $sqldatabase);
    }
    function getData() {
	$query = $this->SQL->query('zapytanie');
    }
}
[PHP] pobierz, plaintext 

W każdym bądź razie to nie najlepsze rozwiązanie.

Ten post edytował Turson 30.03.2014, 14:54:17

[trueblue]

A może do zainicjowanego obiektu klasy UserData przekazać obiekt klasy SQL?

[IProSoft]

Myślę, że dla Ciebie będzie odpowiednie hasło Singelton i metoda getInstance()

Dodasz w klasie SQL metodę, a następnie w innych klasach skorzystasz z SQL :

[PHP] pobierz, plaintext 
SQL::getInstance()->query('zapytanie');
[PHP] pobierz, plaintext 

[ber32]

Turson dlaczego

W każdym bądź razie to nie najlepsze rozwiązanie

też jestem ciekawy

[Mephis]

Jak połącze się w klasie z bazą, to strona nie chce mi się wyświetlić... Tzn. gdy mam złe parametry połączenia, to naturalnie wyświetla mi się komunikat o znaczącej treści, że dane są nieprawidłowe, lecz gdy sa prawidłowe, strona się nie wyświetla. Zna ktoś jakieś inne rozwiązanie?
Ostatecznie mogę potraktować to jako zwyky zbiór funkcji... Ewentualnie zamieścił te funkcje w klasie SQL.

Ten post edytował Mephis 30.03.2014, 15:03:40

[ctom]

też jestem ciekawy

bo każda klasa powinna być odpowiedzialna na "jedną rzecz" (?). Co klasę UserData obchodzi połączenie do DB.

[Crozin]

1. Ciebie nie interesuje wykorzystanie klasy w klasie, tylko obiektu w obiekcie.
2. Było już dosłownie dziesiątki razy na forum... zawsze kończyło się tym, że najlepszym rozwiązaniem, nie stwarzającym żadnych problemów i dającym jakieś korzyści z OOP jest IoC np. w realizacji przez dependency injection.

Ten post edytował Crozin 31.03.2014, 09:50:30

[Mephis]

Hmm... Jak dla mnie to zbyt skomplikowane, może kiedyś z tego skorzystam. Cóż, skoro nie ma prostego rozwiązania, będę musiął uczynić tak jak napisałem wcześniej. Istnieje jeszcze możliwość przekazania w argumencie zaptania SQL.

[in5ane]

Tutaj nie ma nic trudnego. Po prostu w swojej klasie Userdata zajmujesz się tylko tym, co ma ta klasa robić i zamiast tworzyć:

[PHP] pobierz, plaintext 
require_once('inc/sql.class.php');
$this->SQL = new SQL($sqlhost, $sqluser, $sqlpassword, $sqldatabase);
[PHP] pobierz, plaintext 

tworzysz w klasie SQL metodę

[PHP] pobierz, plaintext 
   public static function getInstance()
   {
      if (!self::$me)
      {
         self::$me = new Sql();
      }
 
      return self::$me;
   }
[PHP] pobierz, plaintext 

dodatkowo musisz utworzyć zmienną

[PHP] pobierz, plaintext 
private static $me = null;
[PHP] pobierz, plaintext 

i teraz w swojej klasie Userdata możesz się odwołać tak:

[PHP] pobierz, plaintext 
Sql::instance()->query('SELECT * FROM users');
[PHP] pobierz, plaintext 

I bez tworzenia obiektu klasy Sql możesz odwoływać się do jej metod.

Ten post edytował in5ane 31.03.2014, 10:09:01

[pyro]

Jak masz już tworzyć takie glamoty, to lepiej już w ogóle nie używaj klas i obiektów, bo zrobisz podwójnie źle. Oprzyj się w takim wypadku na funkcjach (powrót do PHP4).

To samo się tyczy @in5ane oraz @IProSoft

Ten post edytował pyro 31.03.2014, 11:24:05

[IProSoft]

@pyro - z tego co piszesz woskuję, że lepiej nie uczyć się wcale, niż uczyć na błędach ?
Dostał najwygodniejsze rozwiązania z jego punktu widzenia ( nie koniecznie najlepsze ), skoro uważasz, że są złe, podaj lepsze.

[in5ane]

A co jest złego w użytkowaniu singletona do połączenia z bazą danych jeśli na samym początku tworzenia aplikacji programista będzie wiedział, że połączenie będzie potrzebował zawsze jedno? Jak już zaczynasz coś pisać, to może rozwiń swoją myśl.

[pyro]

Jakbyś pomagał uczyć się na błędach to byłoby OK, zamiast tego jedynie pchasz użytkownika w dalsze błędy.

Lepsze rozwiązania zostały już podane i nie widzę sensu pisać tego samego.

A może do zainicjowanego obiektu klasy UserData przekazać obiekt klasy SQL?

bo każda klasa powinna być odpowiedzialna na "jedną rzecz" (?). Co klasę UserData obchodzi połączenie do DB.

1. Ciebie nie interesuje wykorzystanie klasy w klasie, tylko obiektu w obiekcie.
2. Było już dosłownie dziesiątki razy na forum... zawsze kończyło się tym, że najlepszym rozwiązaniem, nie stwarzającym żadnych problemów i dającym jakieś korzyści z OOP jest IoC np. w realizacji przez dependency injection.