You are not logged in.

  • Login

PHP-Class-Problem

meme

Unregistered

1

Thursday, June 30th 2011, 2:26pm

PHP-Class-Problem

Hallo Community,

ich habe ein kleines Problem. Ich fange gerade erst so richtig mit PHP an. Ich bin dabei folgendes zu realisieren:
Eine Datei wird aufgerufen und diese startet eine Class. In dieser Class kann über c->addInstanz() eine neue (weitere) Klasse erzeugt werden. Hier der Code:

PHP Quellcode

 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26

class construct {
 
	var $debug = array();
 
	public function createInstanz($class){
		if(file_exists('libary/class/' . $class . '.php')){		
			self::addDebug($class.' included and started.');			
			$this->{$class} = new $class;				
		}else{
			self::addDebug($class.'-class not found.');
		}
	}
 
	private function addDebug($status){
		$this->debug[] = $status;
		var_dump($this->debug);
	}
 
	public function getDebug($format = 'Notice: <strong>%s</strong> <br />'){
		var_dump($this->debug);
		foreach($this->debug AS $message){
			printf($format, $message);
		}
	}
} 
$c = new construct;


Ich rufe also diese Class auf und weise diese an, eine neue zu erzeugen.

PHP Quellcode

 
1

$c->addInstanz('user');


Ich kann wunderbar auf die Objekte in der user-class zugreifen ($c->user->foobar() etc). Jedoch wenn ich innerhalb der user-class die Debug-Funktion nutzen will, klappt das nicht. Aufbau der user-class:

PHP Quellcode

 
1
2
3
4
5
6
7

class user extends construct{	
 
	 function foobar(){
		$this->addDebug('tet');
	}
 
}


Der Aufruf: $c->user->foobar() funktioniert, aber wenn ich $c->getDebug() ausführe, erhalte ich nur Debug-Nachrichten aus der construct-class.

Mein Problem ist also die Kommunikation der Classen untereinander. Da ich in Zukunft auch eine Datenbank-Class hinzufügen will, die eventuell auf Informationen aus der user-class zurückgreifen soll, weiß ich schon jetzt nicht, wie ich das anstelle, dass ich aus allen Classes in alle Classes kommunizieren kann.

Ich hatte folgende Idee:

PHP Quellcode

 
1
2
3
4
5
6
7
8
9
10
11

class construct{
 
	var $user = '';
	var $db = '';
 
	function addInstanz(&$class){
		// .. exists-check etc
		$this->user = &$class
	}
 
}

Dafür müsste ich aber schon jetzt alle Classes kennen, die ich irgendwann mal erzeugen würde. Und selbst dann bin ich mir über den Aufbau noch nicht so sicher, da ich nicht weiß, ob dann ein $this->db->foobar() in der user-class funktioniert.

Wenn jemand nen Tipp hat, wie ich das umsetzte, wäre ich sehr dankbar.

Danke & Grüße
meme

vince

Master

Posts: 1,105

  • Send private message

2

Thursday, June 30th 2011, 3:35pm

Hi,

vorweg ein paar Kleinigkeiten:
-Du solltest deine Klasse nicht "construct" nennen. Es ist kein reserviertes Wort, aber leicht mit "__construct" welches bei PHP für den Konstruktor steht, zu verwechseln.
-Dann ist es sinnvoll Klassennamen groß zu schreiben.
-Wie du in deiner Funktion schon richtig geschrieben hast instanziert man eine Klasse (nicht starten).
-Das Schlüsselwor "var" für Klassenvariablen gibt es in PHP5 nicht mehr. Dort kennzeichnest du die Klassenvariablen mit public,private oder protected.
-Wenn du eine Klasse instanzierst, dann solltest du dir angewöhnen die Klammern zu setzen. ("new Foo()")


Ohne es vll. zu wissen, hast du das Factory Pattern angewandt ;)


Jetzt zu deiner Problematik.

Quoted from "meme"

Der Aufruf: $c->user->foobar() funktioniert, aber wenn ich $c->getDebug() ausführe, erhalte ich nur Debug-Nachrichten aus der construct-class.

Wenn du $c->getDebug aufrufst, dann ist es klar, dass du nur die Debugnachricht der construct-Klasse erhälst, da diese Klasse nichts von Ihren Kindklassen weiß. Um das zu realisieren, musst du mit static arbeiten, so, dass die Debugvariable nicht immer für jede Klasse neu gesetzt wird, sondern alle die selbe Variable nutzen, dann funktioniert dein Vorhaben.


Was du da vorhast ist nicht sonderlich schön.

Exkurs:
Das Factory Pattern macht Sinn, wenn man z.B eine Klasse mit mehreren Adaptern besitzt.
Bei DB Klassen sieht man das häufig. Als Beispiel gibt es den Adapter "MySQL" der eine eigenständige Klasse darstellt. Dann kann man z.B so die DB Klasse mit dem Adapter MySQL instanzieren: $db = new DB("MySQL"); Wobei dann die Adapterklasse MySQL mittels dem Factorypattern instanziert wird.


Besser ist es, wenn du deine Klassen eigenständig instanzierst und alles was zur Debuglogik gehört auch in die Debuggerklasse packst, denn alle anderen Klassen brauchen davon nichts wissen und die interessiert das auch nicht.
Um das vernünftig umzusetzen brauchst du lediglich das Singleton Pattern einsetzen.


So könnte die Implementierung aussehen :

PHP Quellcode

 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72

<?php
 
class Debugger 
{
 
	private $debugMessages = array();
 
 
	static private $instance = null;
 
	public static function getInstance(){
		if(self::$instance === null){
	    	self::$instance = new Debugger();
	    }
	    return self::$instance;
	}
 
	private function __construct() {}
	private function __clone() {}
 
 
	public function addDebugMessage($message) {
		$this->debugMessages[] = $message;
	}
	public function getDebugMessages() {
		return $this->debugMessages;
	}
 
}
 
 
class User
{
	public function saveData($data = array()) {
		// la la la ... Programmlogik
		Debugger::getInstance()->addDebugMessage(get_class($this) . " says data saved");
	}
}
 
 
class DB
{
	public function connect($host) {}
	public function persist($object) {
		// Daten in die DB schreiben ....
		Debugger::getInstance()->addDebugMessage(get_class($this) . " says data saved");
	}
}
 
 
 
 
 
$db = new DB();
$db->connect("localhost");
 
$user = new User();
$user->saveData(array("username"=>"new-user-name"));
 
$db->persist($user);
 
 
 
$debugMessages = Debugger::getInstance()->getDebugMessages();
 
foreach($debugMessages as $m) {
	echo "Nachricht: " . $m . "<br />";
}
 
// AUSGABE
// Nachricht: User says data saved
// Nachricht: DB says data saved

meme

Unregistered

3

Thursday, June 30th 2011, 3:53pm

Hallo und vielen Dank für die Antwort.

Und eine Abfrage aus User nach DB wäre so:

PHP Quellcode

 
1

DB::getInstance()->query('BLA');

Vorher muss ich in der DB-Class auch diesen Abfangmechanismus einbauen:

PHP Quellcode

 
1

public static function getInstance(){ if(self::$instance === null){     self::$instance = new Debugger();     }     return self::$instance; }


Kann ich dann nicht das getInstance bei den aufrufen durch das __construct-Element ersetzten? Also dass der Aufruf so wäre: DB::query("Bla") und halt in __construct der Code steht, der die Anzahl der instanzen regelt?

meme

vince

Master

Posts: 1,105

  • Send private message

4

Thursday, June 30th 2011, 4:11pm

Hi,

Möchtest du deine DB Klasse auch als Singleton implementieren ?

Wenn ja, dann benötigt diese auch die getInstance() Methode und die statische Variable $instance. In der getInstance() Methode musst du natürlich die richtige Klasse eintragen :

PHP Quellcode

 
1
2
3

self::$instance = new DB();
// oder variabel für alle 
self::$instance = new self();



Zum Verständnis vom Singleton:

Quoted

Das Singleton gewährleistet, dass von einer Klasse nur ein Objekt erzeugt werden kann.
Quelle: http://www.phpbar.de/w/Singleton

Du hast keine Instanzen, sondern nur 1 Instanz !
Deshalb geht man den Weg über die getInstance-Methode, da diese Sicherstellt, das es nur 1 Instanz der Klasse gibt. Daher ist __clone und __construct auch privat so, dass von der Klasse niemals eine 2. Instanz existieren kann.


Quoted from "meme"

Und eine Abfrage aus User nach DB wäre so:
DB::getInstance()->query('BLA');

Was meinst du mit "einer Abfrage aus User zu DB" ?

meme

Unregistered

5

Friday, July 1st 2011, 8:38am

Hallo Vince,

hatte nu schnell Copy+Paste gemacht, darum stand da noch die alte Class. Mit der Abfrage meinte ich eine Anfrage/Abfrage aus der User-Class in die DB-Class (also ein Query der ausgeführt wird etc). Das Problem ist aber hinfällig bzw. gelöst.

Ich finde diesen Aufbau sehr schön. Ich frage mich aber nun noch, warum ich alles via getInstance steuern muss und nicht __construct nehmen kann. Weil es wäre, wenn ich darauf verzichten könnte, eine kürzere Schreibweise alá DB::query('FOO').

Eine letzte Frage, dann verschwinde ich wieder (kann mich auch mit der getInstance abfinden): Folgendes von phpbar:

PHP Quellcode

 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20

<?php 
class Singleton   
{       	
		static private $instances = array();          
 
		static public function getInstance($className) {           
			if (!isset(self::$instances[$className])) {               
				self::$instances[$className] = new $className();          
			}         
			 return self::$instances[$className];      
		}  
}  
 
require_once 'Girl.php';  
$girl = Singleton::getInstance('Girl'); 
 
 
 
require_once 'Boy.php';  
$boy = Singleton::getInstance('Boy');


Ich würde nach diesem Muster dann eine Function von (class) Boy ansteuern: Singleton::instances['Boy']->foobar(); ??

vince

Master

Posts: 1,105

  • Send private message

6

Friday, July 1st 2011, 9:28am

Hi,

nochmal zum Konstrukt & Singleton Pattern.

Der Konstruktor wird immer dann aufgerufen, wenn man eine Klasse instanziert.
Wenn du eine statische Methode "query" von der Klasse "DB" aufrufen würdest, dann würde der Konstruktor NICHT aufgerufen werden. Das geschieht NUR über das "new" Keyword. Also "new DB()". Mit dem Singleton möchte man aber erreichen, das es nur EINE Instanz gibt und man diese dann über die statische Methode "getInstance" erreichen kann.

Kleines Beispiel:

PHP Quellcode

 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20

<?php
 
$db = new DB();
$db->connect("localhost");
 
unset($db);
 
// die Verbindung ist nicht mehr verfügbar, da $db gelöscht wurde
 
 
////////////////////////////////////////////////////////
 
$db = DB::getInstance();
$db->connect("localhost");
 
unset($db);
 
// durch das Singleton Pattern, kann die Verbindung (Resource) über die statische getInstance-Methode immer wieder geholt werden 
 
$user = DB::getInstance()->query('SELECT * FROM User'); // query ist nicht static !


Du solltest zwischendurch ein bisschen "rumspielen" und testen. Schreib einfach eine Klasse mit mehreren Methoden und schaue, wann der Konstruktor überhaupt aufgerufen wird.
Meist klären sich dadurch viele Dinge von selbst.



Quoted from "meme"

Ich würde nach diesem Muster dann eine Function von (class) Boy ansteuern: Singleton::instances['Boy']->foobar(); ??


Die beiden Klassen sehen ja so aus:

PHP Quellcode

 
1
2

class Boy extends Singleton {}
class Girl extends Singleton {}


Jein, nicht du greifst im Endeffekt auf das $instances Array der Singleton Klasse zu, sondern der Interpreter von PHP.

Um das rumspielen von vorhin aufzugreifen.
Setz einfach mal das Instance Array auf public und schau dir den Inhalt an.

PHP Quellcode

 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25

<?php 
 
class Singleton   
{       	
		static public $instances = array();          
		[...]
} 
 
 
class Boy extends Singleton {}
class Girl extends Singleton {}
 
echo "Pre Girl-Boy";
var_dump(Singleton::$instances);
 
$girl = Singleton::getInstance('Girl'); 
 
 
echo "Post Girl";
var_dump(Singleton::$instances);
 
$boy = Singleton::getInstance('Boy');
 
echo "Post Girl-Boy";
var_dump(Singleton::$instances);





Quoted from "meme"

Eine letzte Frage, dann verschwinde ich wieder

du bist herzlich eingeladen dich zu registrieren :)

meme

Unregistered

7

Friday, July 1st 2011, 10:32am

Okay, Fragen sind alle beantwortet. Vielen Dank!

Similar threads

Tagging

Social bookmarks