Methode einer anderen Klasse aufrufen

Luka · 08.11.2008, 11:19

Hi,

ich programmiere gerade meine 2. Klasse, stehe also noch am Anfang. (Hoffe es gehört dennoch hier rein und nicht zu den Einsteigern)

Ich habe eine MySQL Klasse und eine AbstractSQL Klasse die von der MySQL Klasse erbt. Wenn ich also ein Query abschicke, geht das über diese AbstractSQL Klasse. Die beiden Klassen habe ich so im Internet gefunden und finde den Ansatz eigentlich auch ganz gut.

Wenn ich nun im Script auf die DB zugreifen möchte benutze ich folgenden Code:

PHP-Code:

  $abstract = new AbstractSQL ($dbhost,$dbuser,$dbpass,$dbname);

$query = "select * FROM user";

$res = $abstract -> checkQuery ($query);

Nun habe ich als nächstes eine Login Klasse geschrieben. Natürlich muss ich nun über die Login Klasse die AbstractSQL Klasse ansprechen, damit ich in der Datenbank die Userdaten überprüfen kann. Der Part oben fliegt also aus dem Script raus und muss ja irgendwie in der Login Klasse wieder auftauchen.

Also, ich rufe in meinem Script ja meine Klasse auf und instanziere das Objekt.

PHP-Code:

  $login = new LoginHandler();

In meiner Login Klasse komme ich dann irgendwann zu der Methode:

PHP-Code:

  function checkLoginData()

Wie benutze ich nun den oben genannten Code um auf die AbstractSQL Klasse zuzugreifen?
Ich bin mir sicher das Ihr Rat wisst, ich drehe mich hier leider im Kreis. Freue mich über jeden Tip.

Vielen Dank im Voraus. Gruss Luka

PS: Bin einen Schritt weitergekommen:

PHP-Code:

   $query = "select * FROM user";

         $res = AbstractSQL::checkQuery($query);

Er geht nun in die AbstractSQL Klasse, dort verursacht er jedoch ein Problem mit der Elterklasse. Er kann scheinbar nicht auf die Elterklasse zugreifen, also gibt es wohl Probleme mit der Vererbung. (Call to undefined method ) Hat jemand eine Idee?

tomtaz · 08.11.2008, 12:00

Versuch es mit Singleton.

Heist also, erstelle in der SQL Klasse eine statische funktion zb.: init

PHP-Code:

  
class AbstractSQL extends SQL
{
  static $init = NULL;

  static public function Init( $dbhost, $dbuser, $dbpass, $dbname )
  {
    if ( self::$init === NULL )
    {
      self::$init = new AbstractSQL ($dbhost,$dbuser,$dbpass,$dbname);
    }
   return self::$init;
  }
}

In der funktion kannst du dann via:

PHP-Code:

  function test( )
{
  $db = AbstractSQL::init();
  $db->Sendquery( ... );
}

auf deine SQL Klasse zugreifen

Luka · 08.11.2008, 12:11

Hi tomtaz,

habe ich gerade ausprobiert, aber das interessiert das script wohl wenig.

Ich habe mir mal etwa ausgeben lassen aus der Elternklasse im Konstruktor, das funktionierte. Also ist die Klasse richtig eingebunden.

Die Funktion ist protected, dachte das liegt daran, aber auch wenn ich es auf public setze geht es nicht.

Es soll also so aussehen

"LoginHandler" Klasse benutzt Methode von "AbstractSQL extends MySQL", diese ruft Methode von "MySQL" Klasse auf und in genau der letzten Phase hängt es.

dr.e. · 08.11.2008, 15:24

Hallo Luka,

da du noch am Anfang stehtst gleich der Hinweis, dass

Zitat:

AbstractSQL extends MySQL

wenig sinnvoll ist. Eine abgeleitete Klasse muss immer spezieller sein, sprich eben nicht mehr abstrakt.

Luka · 08.11.2008, 17:15

Hallo dr.e.,

ich habe die beiden Klassen von der Peter kropff Seite, er spricht dort zwar auch davon das die Abstrakt Klasse dort nicht wirklich Sinn macht, aber ich finde das es eigentlich super funktioniert. Du kannst ja mal einen Blick darauf werfen.

Peter Kropff - Tutorials - Objektorientierte Programmierung mit PHP5 - Anfängertutorial 3 - Die MySQL-Klasse Teil 1

Ich bin jetzt wirklich überfragt und völlig festgefahren. Wie bekomme ich das nur zum Laufen?

byFd · 08.11.2008, 17:26

Ich hab das Tutorial grad mal überflogen und ich glaube er erstellt diese "AbstractSQL" Klasse nur weil er Vererbung zeigen möchte.
Diese Klasse macht meiner Meinung nach absolut keinen Sinn und abstrakt ist sie erst recht nicht.
Diese checkQuery funktion braucht man ja auch nicht unbedingt.

Eine Singleton Implementierung könnte man hier machen, damit es immer nur ein SQL Objekt gibt, muss man aber nicht, Alternativ könntest du ein Objekt deiner SQL Klasse im Konstruktor deiner Login Klasse übergeben und dann dieses benutzen.

agrajag · 08.11.2008, 17:32

Mal ganz davon abgesehen, dass Dr.E natürlich recht hat, das ist wirklich kein gutes Beispiel für sinnvolle Vererbung und der Name "AbstractSQL" ist zudem noch suboptimal gewählt...:

Warum versuchst du plötzlich die checkQuery()-Methode als statische Methode aufzurufen?
Du hast im Prinzip zwei Möglichkeiten:
Entweder du übergibst deiner LoginHandler-Klasse ein AbstractSQL-Objekt:

PHP-Code:

  $abstract = new AbstractSQL(....);
$loginHandler = new LoginHandler($abstract);

Dazu musst du natürlich deinen Konstruktor entsprechend anpassen...

Oder du erzeugst eben in der LoginHandler-Klasse selbst ein neues AbstractSQL-Objekt...oder du holst es dir ähnlich wie tomtaz vorgeschlagen ...

Luka · 08.11.2008, 18:09

Mmh ok, also seid Ihr der Meinung, dass ich die Abstract Klasse weglassen soll, wenn ich das richtig verstehe. Schade eigentlich,das sah in meinen Augen ganz gut aus. Ich rufe einfach nur 1 Methode auf egal ob für einen SELECT oder ein INSERT, die Abstract SQL hat ja entschieden was es sein soll.

Kann ich das denn wirklich nicht einsetzen?

Ich habe nun mal mein script so geändert

Im Script erst:

PHP-Code:

  new MySQL ($dbhost,$dbuser,$dbpass,$dbname);

dann

PHP-Code:

  $login = new LoginHandler();

und in der Login Klasse

PHP-Code:

  $query = "select * FROM user";

$res = MySQL::doSelect($query);

So funktioniert es dann auch. Ist das dann nun sauber programmiert? Ich übergebe übrigens kein Object direkt an meine Login Klasse, auch im Konstruktor habe ich nichts verändert.

Natürlich musste ich die Methode on MySQL noch von protected auf public setzen, sonst hätte ich ja nicht darauf zugreifen können.

Ist das dann alles OK?

byFd · 08.11.2008, 18:17

du kannst die Klasse natürlich trotzdem benutzen, theoretisch könntest du die checkquery funktion auch in der ersten klasse implementieren...

Sauber ist das ganze insofern nicht, als das du die doSelect methode statisch aufrufst, obwohl sie nicht dafür vorgesehen ist. funktioniert auch nur weil PHP einfach die letzte geöffnete verbindung nimmt und du ja vorher new MYsql gemacht hast.

dr.e. · 08.11.2008, 18:20

Hallo Luka,

Zitat:

Ist das dann alles OK?

Das kommt auf den Betrachtungswinkel an. Für mich persönlich ist das weit von gutem Softwaredesign entfernt, da Aufgaben von Komponenten vermischt werden, keine klare Trennung zwischen Datenbank-Abstraktion und Datenschicht vollzogen wird, ...

Grundsätzlich funktioniert dein Code natürlich, du solltest jedoch darauf achten, die verschiedenen Aufgaben sauber in Komponenten / Klassen zu verpacken und diese allgemeingültig einsetzbar zu gestalten (Regel No. 4).
Da Komponenten in den meisten Fällen einen Status tragen, solltest du statische Addressierungen vermeiden. In der doSelect() ist es ratsam die Datenbank-Verbindung zur Verfügung zu haben um die Query ausführen zu können.

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
statische Eigenschaft einer unbekannten Klasse aufrufen	Zergling-new	PHP-Fortgeschrittene	30	05.01.2010 20:04
funnktion aus klasse aufrufen	nieselfriem	PHP Tipps 2008	3	05.01.2008 14:35
mehr als eine Klasse einbinden	Alpha Centauri	PHP-Fortgeschrittene	4	13.04.2006 20:56
Wert einer Variable in einer anderen Funtion übernehmen	seven-12	PHP Tipps 2005-2	7	14.06.2005 00:36
[Erledigt] PHP5 - Vererbung: Aufrufen einer Methode der Oberklasse		PHP-Fortgeschrittene	2	25.04.2005 13:01
Methode dynamisch (per Name) aufrufen		PHP Tipps 2005	6	10.03.2005 14:07
Methode aus einer anderen Methode aufrufen		PHP Tipps 2005	3	02.03.2005 10:07
[Erledigt] Problem mit Übergabe einer Klasse in PHP4		PHP-Fortgeschrittene	10	08.01.2005 21:00
[Erledigt] Instanz bilden innerhalb einer Klasse		PHP Tipps 2004	7	11.12.2004 12:58
Funktion einer Klasse über Link aufrufen?	MortakArtos	PHP Tipps 2004-2	4	18.11.2004 12:05
Klasse holt sich die Klasse		PHP-Fortgeschrittene	9	07.10.2004 11:53
[Erledigt] Brauche Hilfe bei meiner ersten Klasse		PHP-Fortgeschrittene	9	24.09.2004 17:09
Klassenobjet in anderer Klasse benutzen	inu	PHP Tipps 2004	6	19.09.2004 10:58
OOP: offizielle php funktion als methode einer klasse ...		PHP Tipps 2004	4	17.09.2004 04:24
Klasse ändern	UniQ	PHP Tipps 2004	5	24.08.2004 14:46

08.11.2008, 11:19
Luka Neuer Benutzer Registriert seit: 22.11.2007 Beiträge: 15	Methode einer anderen Klasse aufrufen Hi, ich programmiere gerade meine 2. Klasse, stehe also noch am Anfang. (Hoffe es gehört dennoch hier rein und nicht zu den Einsteigern) Ich habe eine MySQL Klasse und eine AbstractSQL Klasse die von der MySQL Klasse erbt. Wenn ich also ein Query abschicke, geht das über diese AbstractSQL Klasse. Die beiden Klassen habe ich so im Internet gefunden und finde den Ansatz eigentlich auch ganz gut. Wenn ich nun im Script auf die DB zugreifen möchte benutze ich folgenden Code: PHP-Code: `$abstract = new AbstractSQL ($dbhost,$dbuser,$dbpass,$dbname); $query = "select * FROM user"; $res = $abstract -> checkQuery ($query);` Nun habe ich als nächstes eine Login Klasse geschrieben. Natürlich muss ich nun über die Login Klasse die AbstractSQL Klasse ansprechen, damit ich in der Datenbank die Userdaten überprüfen kann. Der Part oben fliegt also aus dem Script raus und muss ja irgendwie in der Login Klasse wieder auftauchen. Also, ich rufe in meinem Script ja meine Klasse auf und instanziere das Objekt. PHP-Code: `$login = new LoginHandler();` In meiner Login Klasse komme ich dann irgendwann zu der Methode: PHP-Code: `function checkLoginData()` Wie benutze ich nun den oben genannten Code um auf die AbstractSQL Klasse zuzugreifen? Ich bin mir sicher das Ihr Rat wisst, ich drehe mich hier leider im Kreis. Freue mich über jeden Tip. Vielen Dank im Voraus. Gruss Luka PS: Bin einen Schritt weitergekommen: PHP-Code: `$query = "select * FROM user"; $res = AbstractSQL::checkQuery($query);` Er geht nun in die AbstractSQL Klasse, dort verursacht er jedoch ein Problem mit der Elterklasse. Er kann scheinbar nicht auf die Elterklasse zugreifen, also gibt es wohl Probleme mit der Vererbung. (Call to undefined method ) Hat jemand eine Idee? Geändert von Luka (08.11.2008 um 11:45 Uhr).


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

08.11.2008, 12:00
tomtaz Erfahrener Benutzer Registriert seit: 17.06.2008 Beiträge: 1.762 PHP-Kenntnisse: Fortgeschritten	Versuch es mit Singleton. Heist also, erstelle in der SQL Klasse eine statische funktion zb.: init PHP-Code: `class AbstractSQL extends SQL { static $init = NULL; static public function Init( $dbhost, $dbuser, $dbpass, $dbname ) { if ( self::$init === NULL ) { self::$init = new AbstractSQL ($dbhost,$dbuser,$dbpass,$dbname); } return self::$init; } }` In der funktion kannst du dann via: PHP-Code: `function test( ) { $db = AbstractSQL::init(); $db->Sendquery( ... ); } auf deine SQL Klasse zugreifen` __________________ Mfg Tomtaz "Es soll jetzt diese Erfindung geben.... Kugel oder so heißt die. Ist so eine Art Suchmaschine..."

08.11.2008, 12:11
Luka Neuer Benutzer Registriert seit: 22.11.2007 Beiträge: 15	Hi tomtaz, habe ich gerade ausprobiert, aber das interessiert das script wohl wenig. Ich habe mir mal etwa ausgeben lassen aus der Elternklasse im Konstruktor, das funktionierte. Also ist die Klasse richtig eingebunden. Die Funktion ist protected, dachte das liegt daran, aber auch wenn ich es auf public setze geht es nicht. Es soll also so aussehen "LoginHandler" Klasse benutzt Methode von "AbstractSQL extends MySQL", diese ruft Methode von "MySQL" Klasse auf und in genau der letzten Phase hängt es.

08.11.2008, 17:15
Luka Neuer Benutzer Registriert seit: 22.11.2007 Beiträge: 15	Hallo dr.e., ich habe die beiden Klassen von der Peter kropff Seite, er spricht dort zwar auch davon das die Abstrakt Klasse dort nicht wirklich Sinn macht, aber ich finde das es eigentlich super funktioniert. Du kannst ja mal einen Blick darauf werfen. Peter Kropff - Tutorials - Objektorientierte Programmierung mit PHP5 - Anfängertutorial 3 - Die MySQL-Klasse Teil 1 Ich bin jetzt wirklich überfragt und völlig festgefahren. Wie bekomme ich das nur zum Laufen?

08.11.2008, 17:26
byFd Erfahrener Benutzer Registriert seit: 06.11.2008 Beiträge: 154	Ich hab das Tutorial grad mal überflogen und ich glaube er erstellt diese "AbstractSQL" Klasse nur weil er Vererbung zeigen möchte. Diese Klasse macht meiner Meinung nach absolut keinen Sinn und abstrakt ist sie erst recht nicht. Diese checkQuery funktion braucht man ja auch nicht unbedingt. Eine Singleton Implementierung könnte man hier machen, damit es immer nur ein SQL Objekt gibt, muss man aber nicht, Alternativ könntest du ein Objekt deiner SQL Klasse im Konstruktor deiner Login Klasse übergeben und dann dieses benutzen.

08.11.2008, 17:32
agrajag Moderator Registriert seit: 02.10.2006 Beiträge: 3.820 PHP-Kenntnisse: Fortgeschritten	Mal ganz davon abgesehen, dass Dr.E natürlich recht hat, das ist wirklich kein gutes Beispiel für sinnvolle Vererbung und der Name "AbstractSQL" ist zudem noch suboptimal gewählt...: Warum versuchst du plötzlich die checkQuery()-Methode als statische Methode aufzurufen? Du hast im Prinzip zwei Möglichkeiten: Entweder du übergibst deiner LoginHandler-Klasse ein AbstractSQL-Objekt: PHP-Code: `$abstract = new AbstractSQL(....); $loginHandler = new LoginHandler($abstract);` Dazu musst du natürlich deinen Konstruktor entsprechend anpassen... Oder du erzeugst eben in der LoginHandler-Klasse selbst ein neues AbstractSQL-Objekt...oder du holst es dir ähnlich wie tomtaz vorgeschlagen ... __________________ Today you...Tomorrow me.

08.11.2008, 18:09
Luka Neuer Benutzer Registriert seit: 22.11.2007 Beiträge: 15	Mmh ok, also seid Ihr der Meinung, dass ich die Abstract Klasse weglassen soll, wenn ich das richtig verstehe. Schade eigentlich,das sah in meinen Augen ganz gut aus. Ich rufe einfach nur 1 Methode auf egal ob für einen SELECT oder ein INSERT, die Abstract SQL hat ja entschieden was es sein soll. Kann ich das denn wirklich nicht einsetzen? Ich habe nun mal mein script so geändert Im Script erst: PHP-Code: `new MySQL ($dbhost,$dbuser,$dbpass,$dbname);` dann PHP-Code: `$login = new LoginHandler();` und in der Login Klasse PHP-Code: `$query = "select * FROM user"; $res = MySQL::doSelect($query);` So funktioniert es dann auch. Ist das dann nun sauber programmiert? Ich übergebe übrigens kein Object direkt an meine Login Klasse, auch im Konstruktor habe ich nichts verändert. Natürlich musste ich die Methode on MySQL noch von protected auf public setzen, sonst hätte ich ja nicht darauf zugreifen können. Ist das dann alles OK?

08.11.2008, 18:17
byFd Erfahrener Benutzer Registriert seit: 06.11.2008 Beiträge: 154	du kannst die Klasse natürlich trotzdem benutzen, theoretisch könntest du die checkquery funktion auch in der ersten klasse implementieren... Sauber ist das ganze insofern nicht, als das du die doSelect methode statisch aufrufst, obwohl sie nicht dafür vorgesehen ist. funktioniert auch nur weil PHP einfach die letzte geöffnete verbindung nimmt und du ja vorher new MYsql gemacht hast.